Skip to content

feat: move from Poetry to UV for maintenance prep.#188

Open
tarilabs wants to merge 3 commits into
containers:mainfrom
tarilabs:tarilabs-20260629-migrateFROMpoetryTOuv
Open

feat: move from Poetry to UV for maintenance prep.#188
tarilabs wants to merge 3 commits into
containers:mainfrom
tarilabs:tarilabs-20260629-migrateFROMpoetryTOuv

Conversation

@tarilabs

Copy link
Copy Markdown
Member

No description provided.

tarilabs and others added 3 commits June 29, 2026 11:22
Co-Authored-By: Claude <noreply@anthropic.com>
Signed-off-by: tarilabs <matteo.mortari@gmail.com>
Co-Authored-By: Claude <noreply@anthropic.com>
Signed-off-by: tarilabs <matteo.mortari@gmail.com>
Homebrew now ships a skopeo version that rejects the v1 registries.conf
format still present on GitHub Actions runners. Overwrite it with a
minimal v2 format config after installation.

Co-Authored-By: Claude <noreply@anthropic.com>
Signed-off-by: tarilabs <matteo.mortari@gmail.com>

@jonburdo jonburdo left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

lgtm - works locally in my testing!

Comment on lines +13 to +17
- name: Fix registries.conf to v2 format (newer skopeo rejects the v1 format shipped on GHA runners)
shell: bash
run: |
sudo mkdir -p /etc/containers
echo 'unqualified-search-registries = ["docker.io"]' | sudo tee /etc/containers/registries.conf

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Looks good as is for our purposes imo, but fwiw I was curious what the default is and noticed quay.io is included: https://github.com/actions/runner-images/blob/84d512e310aacbead6836732a08c9031f1d397a8/images/ubuntu/scripts/build/install-container-tools.sh#L31

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