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

Changing an addon to be hosted mode does not delete old ManifestWork #145

Open
mprahl opened this issue Jan 26, 2023 · 3 comments
Open

Changing an addon to be hosted mode does not delete old ManifestWork #145

mprahl opened this issue Jan 26, 2023 · 3 comments

Comments

@mprahl
Copy link
Member

mprahl commented Jan 26, 2023

If I define a ManagedClusterAddOn like this:

apiVersion: addon.open-cluster-management.io/v1alpha1
kind: ManagedClusterAddOn
metadata:
  name: config-policy-controller
  namespace: cluster2
spec:
  installNamespace: cluster2-grc

Then change it to hosted mode:

apiVersion: addon.open-cluster-management.io/v1alpha1
kind: ManagedClusterAddOn
metadata:
  annotations:
    addon.open-cluster-management.io/hosting-cluster-name: cluster1
  name: config-policy-controller
  namespace: cluster2
spec:
  installNamespace: cluster2-grc

The ManifestWork for both deployment types remain:

❯ kubectl get manifestwork -A
NAMESPACE   NAME                                                       AGE
cluster1    addon-config-policy-controller-deploy-hosting-cluster2-0   3m18s
cluster2    addon-config-policy-controller-deploy-0                    10m
@qiujian16
Copy link
Member

@zhujian7 I think it should have been fixed?

@zhujian7
Copy link
Member

zhujian7 commented Apr 21, 2023

Hi @qiujian16 I don't think we currently support install mode switching, do we need to support it? if we do, I will investigate further.

@qiujian16
Copy link
Member

I see, yeah, we probably should doc that we do not support such case somewhere.

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

No branches or pull requests

3 participants