Skip to content

expose REKOR and TUF in GH Actions and Jenkins#108

Merged
lcarva merged 2 commits intoredhat-appstudio:mainfrom
tnevrlka:load-rekor-tuf-secrets
Jan 22, 2025
Merged

expose REKOR and TUF in GH Actions and Jenkins#108
lcarva merged 2 commits intoredhat-appstudio:mainfrom
tnevrlka:load-rekor-tuf-secrets

Conversation

@tnevrlka
Copy link
Member

@tnevrlka tnevrlka commented Jan 17, 2025

REKOR_HOST and TUF_MIRROR are not exposed and can't be set by users in secrets/credentials in GH Actions and Jenkins

Expose them in GH Actions.
When getting an environment variable from secrets, GitHub always creates an env var, it just sets it to an empty string when secret is not set. This is going to be handled in env.sh in template definitions

Add them to Jenkins too, but leave them commented out and let users uncomment them when needed

Related redhat-appstudio/tssc-sample-templates#106

@tnevrlka tnevrlka force-pushed the load-rekor-tuf-secrets branch 2 times, most recently from 1b349b8 to 0d2fa8b Compare January 17, 2025 08:15
@tnevrlka tnevrlka marked this pull request as ready for review January 17, 2025 08:38
@tnevrlka tnevrlka force-pushed the load-rekor-tuf-secrets branch from 0d2fa8b to 4caf399 Compare January 17, 2025 20:30
@simonbaird
Copy link
Member

Not sure what the policy is currently, but I'm happy to merge it.

@lcarva
Copy link
Member

lcarva commented Jan 20, 2025

I'd like to make sure we're happy with the changes in redhat-appstudio/tssc-sample-templates#106 before merging them here.

REKOR_HOST and TUF_MIRROR are not exposed and can't be set by users in
secrets/credentials in GH Actions and Jenkins

Expose them in GH Actions.
When getting an environment variable from secrets, GitHub always creates
an env var, it just sets it to an empty string when secret is not set.
This is going to be handled in env.sh in template definitions

Add them to Jenkins too, but leave them commented out and let users
uncomment them when needed
The automatically generated GitHub Actions workflows include a step
which checks that exposed environment variables are actually set.

`REKOR_HOST` and `TUF_MIRROR` should not be required and there is
currently no way to easily mark the env vars as not required for the
required check step.
Comment them out and let users uncomment them when they want to use an
external instance
@tnevrlka tnevrlka force-pushed the load-rekor-tuf-secrets branch from 4caf399 to 541656b Compare January 22, 2025 15:26
@lcarva lcarva merged commit 0425a07 into redhat-appstudio:main Jan 22, 2025
4 checks passed
dperaza4dustbit pushed a commit to dperaza4dustbit/tssc-dev-multi-ci that referenced this pull request Dec 15, 2025
feat: split gitops resources in their own namespace
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