-
Notifications
You must be signed in to change notification settings - Fork 2
Test operator reports scraper #46
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
Conversation
3de8b89 to
92cb93e
Compare
Comletes the part of scrapping test operator logs by storing points into Qdrant DB. Started in #46
Comletes the part of scrapping test operator logs by storing points into Qdrant DB. Started in #46
Comletes the part of scrapping test operator logs by storing points into Qdrant DB. Started in #46
be41c32 to
7ab5eab
Compare
a8f8780 to
9037eca
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Apologies, I did not manage to review this entirely.
But I was able to run it locally and populate the vector database, though 🎉 (I limited the scraping only to 5 jobs). The generated data looked reasonable!:)
I would need a bit more time to merge this with clear consciousness (to investigate a bit the parsing and the rest of the code a bit more carefully). But if somebody wants to merge it, especially since it is a bit urgent, I won't object it. We can refactor and polish always later.
np. Meanwhile, I have added 2 additional pipelines and an additional argument to use this script from Zuul pipelines. |
- Connects to downstream Zuul server - Gets tempest reports for failed jobs in an openstack-uni-jobs-periodic-integration-rhoso18.0-rhel9 pipeline with default cutoff date of 14 days - Creates points in Qdrant DB for every failed test/traceback TODO (will be implemented as a separate PR): - Tobico tests parser - Add an argument to the scraper to skip the stage of connecting to the Zuul server. that will be used for storing test/traceback points directly from the Zuul jobs Co-authored-by: lpiwowar <lpiwowar@users.noreply.github.com>
… recreated Qdrant snapshots are stored separately from the collections and therefore peserved when the collection is deleted, enablling recovery if needed.
| return tempest_failures | ||
|
|
||
|
|
||
| # # pylint: disable=too-few-public-methods |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
why do we comment code?
EmilienM
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM for now, one minor comment but we can iterate later.
I agree. It's not perfect now but we'll iterate, as we have other stuffs to merge. |
Co-authored-by: lpiwowar lpiwowar@users.noreply.github.com