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

feat(ct): add labels with service dependency versions to images #9928 #9930

Merged
merged 1 commit into from
Sep 19, 2023

Conversation

poikilotherm
Copy link
Contributor

@poikilotherm poikilotherm commented Sep 18, 2023

What this PR does / why we need it:
Provide annotations/labels on Dataverse application container images carrying the information which PostgreSQL and Solr version is meant to be used with this image.

Which issue(s) this PR closes:

Special notes for your reviewer:
None. This is container metadata only, nothing else affected

Suggestions on how to test this:
Build the image, run docker inspect -f '{{ index .Config.Labels "org.dataverse.deps.solr.version"}}' gdcc/dataverse:unstable to see the label's content.

Does this PR introduce a user interface change? If mockups are available, please link/include them here:
Nope

Is there a release notes update needed for this change?:
Nope

Additional documentation:
None

@poikilotherm poikilotherm added the Component: Containers Anything related to cloudy Dataverse, shipped in containers. label Sep 18, 2023
@poikilotherm poikilotherm added the Size: 0.5 A percentage of a sprint. 0.35 hours label Sep 18, 2023
@pdurbin pdurbin self-assigned this Sep 18, 2023
Copy link
Member

@pdurbin pdurbin left a comment

Choose a reason for hiding this comment

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

I tested this and it works fine:

$ docker inspect -f '{{ index .Config.Labels "org.dataverse.deps.solr.version"}}' gdcc/dataverse:unstable
9.3.0
$ docker inspect -f '{{ index .Config.Labels "org.dataverse.deps.postgresql.version"}}' gdcc/dataverse:unstable
13

@kcondon this is very low risk. Zero impact on the core app.

@JR-1991
Copy link
Contributor

JR-1991 commented Sep 19, 2023

Can confirm!

❯ docker inspect -f '{{ index .Config.Labels "org.dataverse.deps.solr.version"}}' gdcc/dataverse:unstable
9.3.0

❯ docker inspect -f '{{ index .Config.Labels "org.dataverse.deps.postgresql.version"}}' gdcc/dataverse:unstable
13

@pdurbin pdurbin removed their assignment Sep 19, 2023
@kcondon kcondon self-assigned this Sep 19, 2023
@kcondon kcondon merged commit d674d66 into IQSS:develop Sep 19, 2023
4 checks passed
@pdurbin pdurbin added this to the 6.1 milestone Sep 19, 2023
@poikilotherm poikilotherm deleted the 9928-svc-labels branch October 2, 2024 06:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Component: Containers Anything related to cloudy Dataverse, shipped in containers. Size: 0.5 A percentage of a sprint. 0.35 hours
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Provide annotations about service deps in application image
4 participants