Skip to content

Comments

Remove limitation to pull latest images.#26086

Merged
potiuk merged 1 commit intoapache:mainfrom
potiuk:remove-pull-warning
Aug 31, 2022
Merged

Remove limitation to pull latest images.#26086
potiuk merged 1 commit intoapache:mainfrom
potiuk:remove-pull-warning

Conversation

@potiuk
Copy link
Member

@potiuk potiuk commented Aug 31, 2022

The #25380 restored regular pushing of "latest" images to ghcr.io
(after some period where we did not do it, because caching
is now solved differently - using separate "cache" images for
each of the architectures). We restored it because codespaces
were pulling the latest image rather than rebuilding the images using
cache. However we have not removed the protection against pulling
the latest image (because it could be old).

This PR removes the - now unnecessary - protection.


^ Add meaningful description above

Read the Pull Request Guidelines for more information.
In case of fundamental code changes, an Airflow Improvement Proposal (AIP) is needed.
In case of a new dependency, check compliance with the ASF 3rd Party License Policy.
In case of backwards incompatible changes please leave a note in a newsfragment file, named {pr_number}.significant.rst or {issue_number}.significant.rst, in newsfragments.

The apache#25380 restored regular pushing of "latest" images to ghcr.io
(after some period where we did not do it, because caching
is now solved differently - using separate "cache" images for
each of the architectures). We restored it because codespaces
were pulling the latest image rather than rebuilding the images using
cache. However we have not removed the protection against pulling
the latest image (because it could be old).

This PR removes the - now unnecessary - protection.
@potiuk potiuk merged commit 3a7e89d into apache:main Aug 31, 2022
@potiuk potiuk deleted the remove-pull-warning branch August 31, 2022 17:10
potiuk added a commit to potiuk/airflow that referenced this pull request Sep 1, 2022
The `pull` commands of Breeze ci-image and prod-image got the
possibility of pulling latest image in apache#26086 but the --image-tag
remained as required. This PR brings back the --image-tag to fall
back to 'latest' for pull commands.
potiuk added a commit that referenced this pull request Sep 1, 2022
The `pull` commands of Breeze ci-image and prod-image got the
possibility of pulling latest image in #26086 but the --image-tag
remained as required. This PR brings back the --image-tag to fall
back to 'latest' for pull commands.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants