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: add sign-image pipeline #192

Merged
merged 1 commit into from
Oct 1, 2024

Conversation

mmalina
Copy link
Contributor

@mmalina mmalina commented Oct 1, 2024

This pipeline yaml is a copy of the pipeline we currently use: hacbs-signing-pipeline

The goal here is to enable retry in the request-signature task, see here:
https://issues.redhat.com/browse/RELEASE-1194

For that, I needed to update the task reference. But that broke the task because the current version of the task does not have the signature_data result.

In order to test this, I am deploying this modified pipeline under a different name.

Once this is tested, we may actually switch to using this going forward - we can evaluate this afterwards.

This pipeline yaml is a copy of the pipeline we currently
use: hacbs-signing-pipeline

The goal here is to enable retry in the request-signature
task, see here:
https://issues.redhat.com/browse/RELEASE-1194

For that, I needed to update the task reference. But that
broke the task because the current version of the task
does not have the signature_data result.

In order to test this, I am deploying this modified
pipeline under a different name.

Once this is tested, we may actually switch to using this
going forward - we can evaluate this afterwards.

Signed-off-by: Martin Malina <[email protected]>
@mmalina mmalina merged commit 2d8ccb7 into hacbs-release:main Oct 1, 2024
4 checks passed
@mmalina mmalina deleted the signing-pipeline branch October 1, 2024 14:46
@johnbieren
Copy link
Contributor

We need a migration ticket for this like https://issues.redhat.com/browse/RELEASE-1040
The goal is to have less tekton resources in this repo (0 actually)

@mmalina
Copy link
Contributor Author

mmalina commented Oct 2, 2024

We need a migration ticket for this like https://issues.redhat.com/browse/RELEASE-1040 The goal is to have less tekton resources in this repo (0 actually)

I created https://issues.redhat.com/browse/RELEASE-1206 and also https://issues.redhat.com/browse/RELEASE-1207 to actually start using it.

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