-
Notifications
You must be signed in to change notification settings - Fork 4.7k
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
Fix namespace initialization #5596
Fix namespace initialization #5596
Conversation
_, err = c.KubeClient().Namespaces().Create(namespace) | ||
if err != nil { | ||
glog.Errorf("Error creating namespace: %v due to %v\n", namespace, err) | ||
namespace, createErr := c.KubeClient().Namespaces().Create(&kapi.Namespace{ObjectMeta: kapi.ObjectMeta{Name: c.Options.PolicyConfig.OpenShiftSharedResourcesNamespace}}) |
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.
we needed to pass the created namespace to ensureNamespaceServiceAccountRoleBindings
or the namespace annotation update would fail with a missing resource version
@smarterclayton PTAL |
LGTM but want one more set of eyes |
You know that I'd want these set up with parallel structure. |
lgtm |
This structure on the refactor list overall to make it easier to deal with new service accounts, right? |
[merge] |
continuous-integration/openshift-jenkins/merge SUCCESS (https://ci.openshift.redhat.com/jenkins/job/merge_pull_requests_origin/3910/) (Image: devenv-rhel7_2630) |
Evaluated for origin merge up to 6223100 |
[Test]ing while waiting on the merge queue |
Evaluated for origin test up to 6223100 |
continuous-integration/openshift-jenkins/test FAILURE (https://ci.openshift.redhat.com/jenkins/job/test_pull_requests_origin/6650/) |
Fixes #5587