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

Correct docker image for stac-populator optional component #483

Merged
merged 4 commits into from
Dec 3, 2024

Conversation

mishaschwartz
Copy link
Collaborator

Overview

This sets the docker image for the stac-populator component to a version that actually contains the code that is executed when stac-populator is called. The previous image no longer contained the relevant code.

Changes

Non-breaking changes

  • stac-populator image changed to version 0.0.0

Breaking changes
None

Related Issue / Discussion

Additional Information

CI Operations

birdhouse_daccs_configs_branch: master
birdhouse_skip_ci: false

@github-actions github-actions bot added component/STAC Features or components related to STAC documentation Improvements or additions to documentation labels Nov 27, 2024
Copy link
Collaborator

@tlvu tlvu left a comment

Choose a reason for hiding this comment

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

Thanks !!! I never liked using "evolving" tags like master or latest. It's just a timing bomb that causes surprise in the future.

Copy link
Collaborator

@fmigneault fmigneault left a comment

Choose a reason for hiding this comment

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

Just planing ahead, should we add ${STAC_POPULATOR_VERSION}, ${STAC_POPULATOR_DOCKER} and ${STAC_POPULATOR_IMAGE} variables as in the case of the other services?

@github-actions github-actions bot added the ci/operations Continuous Integration components label Dec 3, 2024
@mishaschwartz mishaschwartz merged commit 225f3c3 into master Dec 3, 2024
4 of 5 checks passed
@mishaschwartz mishaschwartz deleted the fix-stac-populator branch December 3, 2024 20:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ci/operations Continuous Integration components component/STAC Features or components related to STAC documentation Improvements or additions to documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants