Skip to content

Use pull-through cache on SMS for AIOs#2294

Merged
priteau merged 2 commits intostackhpc/2025.1from
aio-image-cache
May 7, 2026
Merged

Use pull-through cache on SMS for AIOs#2294
priteau merged 2 commits intostackhpc/2025.1from
aio-image-cache

Conversation

@Alex-Welsh
Copy link
Copy Markdown
Member

@Alex-Welsh Alex-Welsh commented Apr 30, 2026

After recent changes at leafcloud, we're regularly getting rate-limited by the RGW backend on Ark.

This change makes AIOs pull through a new authenticating caching proxy on SMS, which should reduce load on Ark, and maybe improve performance at the same time.

@Alex-Welsh Alex-Welsh force-pushed the aio-image-cache branch 2 times, most recently from 2762d6f to efe08c9 Compare April 30, 2026 07:46
Copy link
Copy Markdown
Contributor

@gemini-code-assist gemini-code-assist Bot left a comment

Choose a reason for hiding this comment

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

Code Review

This pull request updates the container registry configuration for the ci-aio environment to use a local registry cache and enables insecure registry access. A review comment suggests replacing the hardcoded IP address with a variable to improve configuration flexibility across different environments.

Comment thread etc/kayobe/environments/ci-aio/stackhpc-ci.yml Outdated
@Alex-Welsh Alex-Welsh force-pushed the aio-image-cache branch 2 times, most recently from 1bb11d9 to 5926075 Compare May 1, 2026 09:36
@Alex-Welsh Alex-Welsh changed the title WIP: Use pull-through registry cache on SMS Use pull-through cache on SMS for AIOs May 1, 2026
@Alex-Welsh Alex-Welsh marked this pull request as ready for review May 1, 2026 09:38
@Alex-Welsh Alex-Welsh requested a review from a team as a code owner May 1, 2026 09:38
@Alex-Welsh Alex-Welsh requested a review from priteau May 1, 2026 14:37
After recent changes at leafcloud, we're regularly getting rate-limited
by the RGW backend on Ark. This change makes AIOs pull through a new
authenticating caching proxy on SMS, which should reduce load on Ark
greatly, and maybe improve performance at the same time.
@Alex-Welsh
Copy link
Copy Markdown
Member Author

I think this will break the "Check container image tags" action, so don't merge it yet

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