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

suite: Improve package query caching #1903

Merged
merged 7 commits into from
Dec 20, 2023
Merged

suite: Improve package query caching #1903

merged 7 commits into from
Dec 20, 2023

Conversation

zmc
Copy link
Member

@zmc zmc commented Nov 22, 2023

No description provided.

@zmc zmc force-pushed the wip-package-queries branch 3 times, most recently from 77c5cc6 to 595fad1 Compare November 29, 2023 23:47
zmc added 6 commits December 5, 2023 14:01
We had our own "system" for caching, but it had the unfortunate characteristic
 of being a big bowl of spaghetti. While eating said pasta I also noticed we
had two competing "distro defaults" concepts - so that let me delete even more
code. Yum!

Signed-off-by: Zack Cerza <[email protected]>
Particularly in non-verbose mode.

Signed-off-by: Zack Cerza <[email protected]>
@zmc zmc force-pushed the wip-package-queries branch from 595fad1 to 82df7da Compare December 5, 2023 21:01
@zmc zmc force-pushed the wip-package-queries branch from 82df7da to 496225c Compare December 18, 2023 19:01
This takes a long time, and can time out. The mirror is updated every ten
minutes automatically.

Signed-off-by: Zack Cerza <[email protected]>
@zmc zmc force-pushed the wip-package-queries branch from 496225c to b1309ec Compare December 18, 2023 19:45
@zmc zmc marked this pull request as ready for review December 18, 2023 21:11
@zmc
Copy link
Member Author

zmc commented Dec 18, 2023

teuthology-suite -v --owner zmc -s orch:cephadm:smoke -m smithi -t wip-package-queries -n10 --dry-run on main takes ~34s, and on this branch takes ~18s.

I scheduled a run using this branch: https://pulpito-ng.ceph.com/runs/zack-2023-12-18_21:10:25-orch:cephadm:smoke-main-distro-default-smithi
All jobs passed except for two, which failed with reimage errors.

@zmc zmc requested a review from dmick December 18, 2023 23:29
Copy link
Member

@dmick dmick left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good.

@zmc zmc merged commit 9680d2a into main Dec 20, 2023
9 checks passed
@zmc zmc deleted the wip-package-queries branch December 20, 2023 22:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants