Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix timestamp processing #3345

Closed
wants to merge 1 commit into from
Closed

Fix timestamp processing #3345

wants to merge 1 commit into from

Conversation

kkaarreell
Copy link
Collaborator

Fixes timestamp processing, especially in cases when the timestamp is missing in results.yaml.

Pull Request Checklist

  • implement the feature
  • write the documentation
  • extend the test coverage
  • update the specification
  • adjust plugin docstring
  • modify the json schema
  • mention the version
  • include a release note

@kkaarreell
Copy link
Collaborator Author

This is fixing issues encountered while processing results.yaml from
https://artifacts.osci.redhat.com/testing-farm/e3c4d1f2-a192-4d4f-b95d-47880ef7758c/
It would deserve serious testing though.

Fixes timestamp processing, especially in cases when the timestamp
is missing in results.yaml.
@seberm
Copy link
Collaborator

seberm commented Nov 12, 2024

As discussed on Slack, closing this PR in favor of #3356 which now integrates all of your changes. Thanks!

@seberm seberm closed this Nov 12, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
plugin | reportportal The reportportal report plugin
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants