-
Notifications
You must be signed in to change notification settings - Fork 24
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
allow github concourse contributor team to rerun builds #127
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It seems like --config-rbac
is pretty weird. I'd guess you have encountered the same confusion as @gowrisankar22 in concourse/concourse#5514.
@pivotal-jamie-klassen we realize giving the permission of So we are thinking moving WDYT? |
that just reminded me of RFC: Concourse k8s operator and the Pipeline CRD RFC - with a update: and without the use of concourse tokens, having client auth grant type being a thing, we could soon have our controller doing all of that stuff without being admin 👀 |
@cirocosta I know CRDs are the hot thing, but what if we had a Concourse terraform provider instead? Where you could define teams/pipelines using HCL, and when you spin up Concourse, you could have your initial teams/pipelines created as well all through Terraform? There may be other benefits in using CRD+operator would bring over a Terraform provider, though - what do you think? It's possible we'd be appealing to a wider demographic in K8s than Terraform, I suppose |
@cirocosta @aoldershaw rather than investing in a third-party plugin for team automation, I would vote for a core concourse feature, a la |
0e7e6fa
to
d849ae0
Compare
and check resources so they could retrigger failure build on PR pipeline and check pr resource in case wanted version is not available. Note: they could also rerun any builds and check any resources by this change. Signed-off-by: Rui Yang <[email protected]>
d849ae0
to
13d8872
Compare
Updated to config RBAC in CI deployment only. Added a team config file for manual set-team step for now. Also created |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good!
Signed-off-by: Rui Yang <[email protected]>
6a92026
to
ff5d44e
Compare
@pivotal-jamie-klassen I forgot to put concourse:pivotal as owner for team contributor. Seems we are doing this for all other teams on CI. |
Should be ok since concourse:pivotal are super admins |
once we have the pr bot working by concourse/concourse#5564
then we need to config concourse production CI to allow members of concourse:contributor team to rerun failure PR pipeline builds(due to flaky test) and check PR resource(if desired version is not fetched)