Skip to content
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

Always include operator namespace in cached k8s client namespaces #2566

Merged
merged 7 commits into from
Feb 14, 2020

Conversation

pebrc
Copy link
Collaborator

@pebrc pebrc commented Feb 14, 2020

The trial license e2e test was failing since #2530 because we are now running the e2e with a operator restricted to the managed namespaces via --namespaces, but we did not include the operator namespace itself in the list of namespaces.

As the list of namespaces controls which resources will be cached by the k8s client the operator was not able to access the license related secrets in the operator namespace and trial mode could not be activated.

This PR makes sure the operator namespace is always included in the client cache even when running in namespace restricted mode.

@pebrc pebrc added >test Related to unit/integration/e2e tests v1.1.0 labels Feb 14, 2020
@thbkrkr

This comment has been minimized.

ECK keeps internal resources in the operator namespace that would be
otherwise inaccessible.
@pebrc pebrc requested a review from thbkrkr February 14, 2020 10:45
@pebrc pebrc changed the title Fix license e2e tests and namespace operator manifest Always include operator namespace in cached k8s client namespaces Feb 14, 2020
Copy link
Contributor

@thbkrkr thbkrkr left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM.

@pebrc pebrc merged commit f63396e into elastic:master Feb 14, 2020
@anyasabo
Copy link
Contributor

I was wondering if we needed to update https://github.com/elastic/cloud-on-k8s/blob/master/docs/operator-config.asciidoc indicating that no matter what, the operator always needs to be monitoring its own namespace. After thinking about it though I don't think this will be surprising to people, even if they do not know about the licensing configmap.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
>test Related to unit/integration/e2e tests v1.1.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants