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

Change some values from secrets to variables, for example PANTHEON_SITE_NAME #463

Closed
YesCT opened this issue Mar 4, 2024 · 3 comments · Fixed by #470
Closed

Change some values from secrets to variables, for example PANTHEON_SITE_NAME #463

YesCT opened this issue Mar 4, 2024 · 3 comments · Fixed by #470
Assignees
Labels
enhancement New feature or request lsm question Further information is requested

Comments

@YesCT
Copy link
Contributor

YesCT commented Mar 4, 2024

https://github.com/Lullabot/drainpipe?tab=readme-ov-file#pantheon

Change some values from secrets to variables

PANTHEON_SITE_NAME seems a candidate for a variable. Are there any reasons to have it be a secret?

TERMINUS_PLUGINS also?

any others?

Screenshot 2024-03-04 at 2 00 42 PM
@YesCT
Copy link
Contributor Author

YesCT commented Mar 4, 2024

Having a variable allows inspecting it's value. Easier for debugging.

@YesCT YesCT added enhancement New feature or request lsm labels Mar 4, 2024
@deviantintegral deviantintegral added the question Further information is requested label Mar 4, 2024
@davereid
Copy link
Member

davereid commented Mar 7, 2024

+1 for using a variable for PANTHEON_SITE_NAME since it's not a secret. I also have some code I could use from a prior project that makes setting that very optional based on reading from the Github repository name or the DDEV config file.

@justafish
Copy link
Member

justafish commented Mar 7, 2024

https://github.com/Lullabot/drainpipe/blob/main/scaffold/github/workflows/PantheonReviewAppsDDEV.yml#L45
apparently we can change this to e.g. ${{ vars.PANTHEON_SITE_NAME || secrets.PANTHEON_SITE_NAME }} to maintain backwards compatability

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request lsm question Further information is requested
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants