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

chore: add expiry label to images #1446

Merged
merged 1 commit into from
Mar 18, 2024
Merged

chore: add expiry label to images #1446

merged 1 commit into from
Mar 18, 2024

Conversation

RTann
Copy link
Collaborator

@RTann RTann commented Mar 15, 2024

See stackrox/stackrox#10385 for Scanner V4

Copy link
Contributor

@dcaravel dcaravel left a comment

Choose a reason for hiding this comment

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

Labels appear empty

docker inspect quay.io/stackrox-io/scanner-db-slim:2.33.x-13-gd0b3f1e261 | jq '.[].Config.Labels' | grep -E "version|release|quay"
  "io.buildah.version": "1.29.0",
  "quay.expires-after": "",
  "release": "",
  "version": ""

@RTann
Copy link
Collaborator Author

RTann commented Mar 15, 2024

Labels appear empty

docker inspect quay.io/stackrox-io/scanner-db-slim:2.33.x-13-gd0b3f1e261 | jq '.[].Config.Labels' | grep -E "version|release|quay"
  "io.buildah.version": "1.29.0",
  "quay.expires-after": "",
  "release": "",
  "version": ""

apparently we don't use the make targets when building in CI. I'll look into it

@RTann RTann force-pushed the ross/image-build-labels branch from d0b3f1e to 649f04c Compare March 15, 2024 16:06
@RTann RTann force-pushed the ross/image-build-labels branch from 649f04c to 948fb35 Compare March 15, 2024 16:10
Copy link

openshift-ci bot commented Mar 15, 2024

@RTann: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name Commit Details Required Rerun command
ci/prow/e2e-tests 948fb35 link false /test e2e-tests

Full PR test history. Your PR dashboard.

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@RTann
Copy link
Collaborator Author

RTann commented Mar 15, 2024

CI failure is unrelated

Copy link

@porridge porridge left a comment

Choose a reason for hiding this comment

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

This look OK, but I wonder why we're duplicating the docker build command in the workflow definition? 🤔

Comment on lines +106 to +108
.PHONY: image-build-args
image-build-args:
@echo $(IMAGE_BUILD_ARGS)

Choose a reason for hiding this comment

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

Rather than expose this here, and duplicate the docker build command in the GHA yaml file, why not just call make scanner-image... in the GH action in the first place?
cc @BradLugo

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

I'm going to merge as-is, as this repo is more-or-less in maintenance mode, so I think this is ok

@RTann RTann merged commit 6c47858 into master Mar 18, 2024
17 of 18 checks passed
@RTann RTann deleted the ross/image-build-labels branch March 18, 2024 17:45
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