-
Notifications
You must be signed in to change notification settings - Fork 16.8k
concourse: consume local-users from k8s secret #8099
Conversation
Wouldn't this also need to update #8094 has an example of what I would have expected to see |
@matthope I should have been more clear. |
@chrishiestand Cheers- thats clearer :) |
9fec4ff
to
f1e5723
Compare
Thanks @matthope, I didn't even know DCO was a standard thing. Fixed. |
/ok-to-test |
f1e5723
to
a98fd83
Compare
@chrishiestand please resolve conflicts / chart version bump and you could be next! |
/lgtm |
a98fd83
to
1163358
Compare
1163358
to
13bf6e5
Compare
@william-tran please review again after conflict resolution and bump |
instead of directly from helm chart. k8s secret will use the chart value as a default, so functionality is not lost Signed-off-by: Chris Hiestand <chrishiestand@gmail.com>
6a59513
to
8a3ebc2
Compare
/lgtm |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: chrishiestand, william-tran The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
instead of directly from helm chart. k8s secret will use the chart value as a default, so functionality is not lost Signed-off-by: Chris Hiestand <chrishiestand@gmail.com>
instead of directly from helm chart. k8s secret will use the chart value
as a default, so functionality is not lost
What this PR does / why we need it:
Which issue this PR fixes
(optional, in
fixes #<issue number>(, fixes #<issue_number>, ...)
format, will close that issue when PR gets merged)Special notes for your reviewer:
Checklist
[Place an '[x]' (no spaces) in all applicable fields. Please remove unrelated fields.]