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

Change: Add base-image-label as input, control image-tags by base-image-label. #111

Merged
merged 22 commits into from
Sep 25, 2024

Conversation

robert-schardt
Copy link
Contributor

@robert-schardt robert-schardt commented Sep 23, 2024

What

Add image-type base-image-label as input for container-build-push-2nd-gen workflow.
The image-type base-image-label is an enum with the following possible values:

  • stable
  • oldstable
  • testing

According to the set value the image-tags are enabled.

Why

We wanted to merge the container.yml and push.yml workflows in gvm-libs.
The legacy workflow builds additional gvm-libs containers with oldstable and testing tags.
To represent this logic we need to parametrize the container-build-push-2nd-gen workflow somehow.

References

https://jira.greenbone.net/browse/DEVOPS-1230

This is the PR requiring the changes: greenbone/gvm-libs#838

Test is here:

https://github.com/greenbone/pipeline-experiments/actions/workflows/test-container-build-push-2nd-gen.yml

@robert-schardt robert-schardt requested a review from a team as a code owner September 23, 2024 13:04
@greenbonebot greenbonebot enabled auto-merge (squash) September 23, 2024 13:04
Copy link
Member

@y0urself y0urself left a comment

Choose a reason for hiding this comment

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

I would prefer a more clear naming of the input, what do you think?

.github/workflows/container-build-push-2nd-gen.yml Outdated Show resolved Hide resolved
.github/workflows/container-build-push-2nd-gen.yml Outdated Show resolved Hide resolved
robert-schardt and others added 3 commits September 23, 2024 15:28
Co-authored-by: Jaspar Stach <jaspar.stach@greenbone.net>
Co-authored-by: Jaspar Stach <jaspar.stach@greenbone.net>
@robert-schardt robert-schardt changed the title Change: Add image-type as input, control image-tags by image-type. Change: Add image-type as input, control image-tags by base-image-label. Sep 24, 2024
Move image-tags to docker-metadata action
Copy link
Member

@y0urself y0urself left a comment

Choose a reason for hiding this comment

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

Can you setup a test workflow and upload a test image somehow?

@robert-schardt
Copy link
Contributor Author

robert-schardt commented Sep 25, 2024

Can you setup a test workflow and upload a test image somehow?
Done, see top description.

@pascalholthaus pascalholthaus enabled auto-merge (squash) September 25, 2024 09:45
@robert-schardt robert-schardt changed the title Change: Add image-type as input, control image-tags by base-image-label. Change: Add base-image-label as input, control image-tags by base-image-label. Sep 25, 2024
@pascalholthaus pascalholthaus merged commit 56714bf into main Sep 25, 2024
1 check passed
@pascalholthaus pascalholthaus deleted the DEVOPS-1230_rschardt branch September 25, 2024 09:46
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.

3 participants