Skip to content

Conversation

@msugakov
Copy link
Contributor

@msugakov msugakov commented Sep 15, 2025

See the related ticket.

Validation

See my comments below.

@msugakov msugakov added the konflux-build Run Konflux in PR. Push commit to trigger it. label Sep 15, 2025
@openshift-ci
Copy link

openshift-ci bot commented Sep 15, 2025

Skipping CI for Draft Pull Request.
If you want CI signal for your change, please convert it to an actual PR.
You can still manually trigger a test run with /test all

@msugakov msugakov changed the title build: Assign index image tag differently ROX-30898: Assign index image tag differently Sep 16, 2025
@msugakov
Copy link
Contributor Author

Looks like it executed as expected.
Task logs

scanner-slim-on-push-76mcx-apply-index-image-tags
place-scripts :-
2025/09/16 00:17:00 Decoded script /tekton/scripts/script-0-9ps6h
2025/09/16 00:17:00 Decoded script /tekton/scripts/script-1-76tqf
prepare :-
2025/09/16 00:16:59 Entrypoint initialization
step-apply-additional-tags-from-parameter :-
Applying tag konflux-971fc478725041993ab947bc5b039f305079117a
Getting image list signatures
Copying 0 images generated from 4 images in list
Writing manifest list to image destination
Storing list signatures
step-apply-additional-tags-from-image-labe :-
No additional tags specified in the image labels
$ skopeo inspect --format '{{.Digest}}' --no-tags docker://quay.io/rhacs-eng/release-scanner-slim:2.37.x-82-g971fc47872-fast
sha256:49b0bdf62fcbe82272239554bdc85ee04828568a7fa9b59ed06a0cea1893c698
$ skopeo inspect --format '{{.Digest}}' --no-tags docker://quay.io/rhacs-eng/release-scanner-slim:konflux-971fc478725041993ab947bc5b039f305079117a
sha256:49b0bdf62fcbe82272239554bdc85ee04828568a7fa9b59ed06a0cea1893c698

@msugakov
Copy link
Contributor Author

Now, the interesting thing is how the init task would behave if the image already exists. It shouldn't even call to skopeo but let's make it run to be sure.
https://github.com/konflux-ci/build-definitions/blob/8ddc2b52768ed7b8c0794b59ad5b3d944354b15e/task/init/0.2/init.yaml#L52

@msugakov
Copy link
Contributor Author

/retest scanner-slim-on-push

@stackrox stackrox deleted a comment from openshift-ci bot Sep 16, 2025
@msugakov
Copy link
Contributor Author

This is what succeeding init told us on re-run

step-init
Build Initialize: quay.io/rhacs-eng/release-scanner-slim:konflux-971fc478725041993ab947bc5b039f305079117a

Determine if Image Already Exists

@msugakov msugakov marked this pull request as ready for review September 16, 2025 08:20
@msugakov msugakov requested review from a team and rhacs-bot as code owners September 16, 2025 08:20
@msugakov
Copy link
Contributor Author

/retest

@msugakov
Copy link
Contributor Author

Builds feel fine. Merging.

@msugakov msugakov merged commit 93dd275 into master Sep 16, 2025
33 of 39 checks passed
@msugakov msugakov deleted the misha/try-apply-tags-on-index branch September 16, 2025 08:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

konflux-build Run Konflux in PR. Push commit to trigger it. ok-to-test

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants