Skip to content

adding in site config reinstallation#145

Open
johnharmon wants to merge 5 commits into
mainfrom
feature/site-config-reinstall
Open

adding in site config reinstallation#145
johnharmon wants to merge 5 commits into
mainfrom
feature/site-config-reinstall

Conversation

@johnharmon
Copy link
Copy Markdown
Contributor

No description provided.

metadata:
name: siteconfig-operator-configuration
namespace: open-cluster-management
data:
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

this seems like more of a label that needs to be applied to the hub instead of a managed cluster name resource since it is global for all clusters installed

name: siteconfig-operator-configuration
namespace: open-cluster-management
data:
{{ "{{hub-" }} if eq ($siteConfig).enableReinstall true {{ "hub}}" }}
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

This should probably just be a label next to the rest of the acm labels.

@@ -0,0 +1,70 @@
{{- $policyNamespace := $.Values.policy_namespace }}
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

This is now in the wrong folder now

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants