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

Update the env var urls for Rekor and TUF #69

Merged
merged 1 commit into from
Aug 7, 2024

Conversation

simonbaird
Copy link
Contributor

I'm not confident about this change, but my reasoning for making this PR is that a similar change was made in
https://github.com/redhat-appstudio/tssc-sample-pipelines/pull/48/files for the Tekton pipelines.

FYI @Roming22 @jkopriva .

Ref: https://issues.redhat.com/browse/RHTAPBUGS-1280

I'm not confident about this change, but my reasoning for making
this PR is that a similar change was made in
https://github.com/redhat-appstudio/tssc-sample-pipelines/pull/48/files
for the Tekton pipelines.

Ref: https://issues.redhat.com/browse/RHTAPBUGS-1280
@Roming22
Copy link
Member

Roming22 commented Aug 6, 2024

I think the #internal, assumes jenkins is local openshift comment is key, and I agree it should reflect the new namespace.

@simonbaird
Copy link
Contributor Author

I think the #internal, assumes jenkins is local openshift comment is key, and I agree it should reflect the new namespace.

Oh yeah. So actually the Jenkins is not local and not in the cluster, hence this still won't work out of the box, even after this change is merged.

(I think @jkopriva understood that, I'm just catching up here.)

Bottom line: I still think this should be merged, but it doesn't really help with RHTAPBUGS-1280.

@flacatus flacatus merged commit 48d26df into redhat-appstudio:main Aug 7, 2024
3 checks passed
simonbaird added a commit to simonbaird/tssc-sample-templates that referenced this pull request Aug 7, 2024
@simonbaird
Copy link
Contributor Author

I forgot these are defined in two places, sorry about that. See #70.

rhopp pushed a commit that referenced this pull request Aug 9, 2024
This change should have been included in #69.
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.

4 participants