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(deps): update quay.io/redhat-appstudio/build-trusted-artifacts:latest docker digest #1433

Merged

Conversation

renovate[bot]
Copy link
Contributor

@renovate renovate bot commented Sep 17, 2024

This PR contains the following updates:

Package Type Update Change
quay.io/redhat-appstudio/build-trusted-artifacts tekton-step-image digest a390d28 -> c91de17

Warning

Some dependencies could not be looked up. Check the warning logs for more information.


Configuration

📅 Schedule: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).

🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.

Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.

👻 Immortal: This PR will be recreated if closed unmerged. Get config help if that's undesired.


  • If you want to rebase/retry this PR, check this box

This PR was generated by Mend Renovate. View the repository job log.

@renovate renovate bot force-pushed the renovate/quay.io-redhat-appstudio-build-trusted-artifacts-latest branch from 17236e3 to 7f2844c Compare September 18, 2024 07:52
@@ -65,7 +65,7 @@ spec:
name: varlibcontainers
steps:
- name: use-trusted-artifact
image: quay.io/redhat-appstudio/build-trusted-artifacts:latest@sha256:0379f5c2feec921a01300b386ce9414b10acfbc4499b187700d972b2360e1585
image: quay.io/redhat-appstudio/build-trusted-artifacts:latest@sha256:11d90835f4cc882715c37f16513db19876f346096ccba0f26d548b13bfd4a270
Copy link
Contributor

Choose a reason for hiding this comment

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

For some reason this one has a different digest from all the others 🤔

Copy link
Contributor

Choose a reason for hiding this comment

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

Ah, it's the amd64 manifest while the others are the multi-arch image index. That's probably fine

Copy link
Member

Choose a reason for hiding this comment

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

Good catch, I think we should change it to the c91de1..., for consistency and it'll be less confusing with that, I'll push a commit on this branch

@chmeliik chmeliik self-requested a review September 18, 2024 08:07
Let's not point to a specific architecture but use the image index, as
we do for the other steps that use TA.
Copy link
Contributor Author

renovate bot commented Sep 18, 2024

Edited/Blocked Notification

Renovate will not automatically rebase this PR, because it does not recognize the last commit author and assumes somebody else may have edited the PR.

You can manually request rebase by checking the rebase/retry box above.

⚠️ Warning: custom changes will be lost.

@zregvart zregvart added this pull request to the merge queue Sep 18, 2024
Merged via the queue into main with commit 4ebae9b Sep 18, 2024
13 checks passed
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.

2 participants