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

Set the proper namespace #117

Merged
merged 2 commits into from
Aug 5, 2022
Merged

Set the proper namespace #117

merged 2 commits into from
Aug 5, 2022

Conversation

Itxaka
Copy link
Contributor

@Itxaka Itxaka commented Aug 5, 2022

We are trying to get the cattle-elemental-system namespace into rancher
as a system namespace, while we have a different namespace around,
including in the elemental examples.

This changes the wrong namespace into the proper one

Signed-off-by: Itxaka igarcia@suse.com

We are trying to get the cattle-elemental-system namespace into rancher
as a system namespace, while we have a different namespace around,
including in the elemental examples.

This changes the wrong namespace into the proper one

Signed-off-by: Itxaka <igarcia@suse.com>
@Itxaka Itxaka added the area/operator operator related changes label Aug 5, 2022
@Itxaka Itxaka requested review from kkaempf, mjura and fgiudici August 5, 2022 07:36
@codecov-commenter
Copy link

codecov-commenter commented Aug 5, 2022

Codecov Report

Merging #117 (1b13e00) into main (485ff21) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #117   +/-   ##
=======================================
  Coverage   29.16%   29.16%           
=======================================
  Files           5        5           
  Lines         360      360           
=======================================
  Hits          105      105           
  Misses        251      251           
  Partials        4        4           

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

@kkaempf
Copy link
Contributor

kkaempf commented Aug 5, 2022

This should probably be accompanied by (a pointer to) documentation about what the "proper" namespaces for Elemental are. 🤔

@Itxaka
Copy link
Contributor Author

Itxaka commented Aug 5, 2022

This should probably be accompanied by (a pointer to) documentation about what the "proper" namespaces for Elemental are. thinking

Well you can actually install it anywhere, but on the "official" install docs and chart defaults it should go to the one that rancher will mark as "system". I mean, you can install rancher to a different namespace, Im sure of that :D

@Itxaka Itxaka enabled auto-merge (squash) August 5, 2022 12:05
Copy link
Member

@fgiudici fgiudici left a comment

Choose a reason for hiding this comment

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

looks good here!

@Itxaka Itxaka merged commit 5b64a77 into main Aug 5, 2022
@Itxaka Itxaka deleted the proper-namespace branch August 5, 2022 14:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/operator operator related changes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants