Skip to content

task CML 🍬 Autogeneration of the name under the CI environment #390

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.

Sign up for GitHub

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

Jump to bottom
Closed
Tracked by #391
DavidGOrtega opened this issue Feb 9, 2022 · 2 comments · Fixed by #396
Closed
Tracked by #391

task CML 🍬 Autogeneration of the name under the CI environment #390

DavidGOrtega opened this issue Feb 9, 2022 · 2 comments · Fixed by #396
Assignees
@0x2b3bfa0
Labels
p2-nice-to-have Low priority resource-task iterative_task TF resource

Comments

@DavidGOrtega
Copy link
Contributor

Naming under CI environment should be automatically done based on the CI using the following conventions

Gitlab

tpi-$CI_PIPELINE_ID

Github

tpi-$GITHUB_RUN_ID

Bitbucket

tpi-$BITBUCKET_STEP_TRIGGERER_UUID
@DavidGOrtega DavidGOrtega mentioned this issue Feb 9, 2022
Closed
8 tasks
@0x2b3bfa0
Copy link
Member

Too much sugar to my taste, but... 🤷🏼‍♂️

Related to #379

@DavidGOrtega
Copy link
Contributor Author

Its a good compromise for UX, that way they can also change the SCM without hesitation, not having to know how it should be named. In any case they can setup the name as always.

@0x2b3bfa0 0x2b3bfa0 mentioned this issue Feb 15, 2022
Merged
0x2b3bfa0 added a commit that referenced this issue Feb 15, 2022
@0x2b3bfa0
Use randomly generated identifiers by default
0f52c4e 
Closes #379 and #390
@0x2b3bfa0 0x2b3bfa0 self-assigned this Feb 15, 2022
@0x2b3bfa0 0x2b3bfa0 added p2-nice-to-have Low priority resource-task iterative_task TF resource labels Feb 15, 2022
DavidGOrtega added a commit that referenced this issue Feb 17, 2022
@0x2b3bfa0 @DavidGOrtega
Use randomly generated identifiers by default (#396)
af2082e 
* Use randomly generated identifiers by default

Closes #379 and #390

* remove example in readme

Co-authored-by: DavidGOrtega <[email protected]>
@0x2b3bfa0 0x2b3bfa0 mentioned this issue Feb 17, 2022
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@0x2b3bfa0 0x2b3bfa0

Labels
p2-nice-to-have Low priority resource-task iterative_task TF resource
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Use randomly generated identifiers by default iterative/terraform-provider-iterative
2 participants
@DavidGOrtega @0x2b3bfa0