Skip to content

Make pipeline self setting#681

Merged
markdboyd merged 2 commits intomainfrom
make-pipeline-self-setting
Feb 6, 2026
Merged

Make pipeline self setting#681
markdboyd merged 2 commits intomainfrom
make-pipeline-self-setting

Conversation

@markdboyd
Copy link
Copy Markdown
Contributor

Changes proposed in this pull request:

  • add job to make pipeline self-setting

security considerations

No direct security considerations of these changes. In general, having pipeline variables managed in Credhub is more maintainable and will reduce risk of inadvertent updates to infrastructure

@markdboyd markdboyd requested a review from a team as a code owner February 6, 2026 19:22
trigger: true
- get: bosh-config-old
resource: bosh-create-env-config
resource: bosh-config
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

The definition of bosh-config-old and bosh-config were both pointing at the bosh-create-env-config resource. And the bosh-create-env-config resource doesn't monitor to changes under ci/*, so I needed to use the bosh-config resource. And in order to block deploy-master-bosh on set-self, it also needed a trigger on the bosh-config resource. So I updated this block to get the bosh-config resource

Copy link
Copy Markdown
Contributor

@JasonTheMain JasonTheMain left a comment

Choose a reason for hiding this comment

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

LGTM

@markdboyd markdboyd merged commit dd85d1b into main Feb 6, 2026
4 checks passed
@markdboyd markdboyd deleted the make-pipeline-self-setting branch February 6, 2026 19:29
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