-
Notifications
You must be signed in to change notification settings - Fork 153
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
update guestbook example for 2.0 #130
base: master
Are you sure you want to change the base?
Conversation
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: cdoan1 The full list of commands accepted by this bot can be found here.
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
Did this code path work for you? I found that the policy was not setting the priority, which cause the Front End pods to fail? |
@jnpacker everything installed, but the frontend pods did not start--seems the scc did not apply and pods could not run as the user. |
technically you can edit the SCC on the managed cluster, set priority to 10 and quickly delete the pods and they will provision. But policy eventually switches priority back to null. Which is fine once the pods have started, but they won't restart |
Ok, let me try the workaround. Currently, I have this: |
Description of the change:
Motivation for the change:
NOTE this PR will only merge when 2.0 is available