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

SUP-2777 - Allow configurable pipeline slugs #603

Merged

Conversation

petetomasik
Copy link
Contributor

@petetomasik petetomasik commented Dec 23, 2024

Changes slug from a read-only attribute to an optional configurable attribute. If not set, slug is derived from the pipeline name and returned from the API. The slug will be updated when the name of the pipeline changes. If the slug is defined, a second REST API call is made to update the slug for the pipeline. The slug can be set or excluded at any time in the pipeline's lifecycle and the value will be set based on the user-defined input or via the API's response.

Utilizes Terraform private state to track whether slug value in State was set via api or user. This extra value is required to correctly evaluate the transition of removing user-defined slug from an existing resource and re-trigger storing an updated slug from the API.

Addresses: #586

@petetomasik petetomasik requested a review from a team as a code owner December 23, 2024 20:05
@petetomasik petetomasik merged commit 0b9dbcc into buildkite:main Dec 24, 2024
1 check passed
@petetomasik petetomasik deleted the SUP-2777-decouple-pipeline-slug branch December 24, 2024 13:59
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