Skip to content
This repository has been archived by the owner on Apr 17, 2019. It is now read-only.

Change namespace.yaml path to /opt #945

Closed
wants to merge 1 commit into from
Closed

Change namespace.yaml path to /opt #945

wants to merge 1 commit into from

Conversation

gouyang
Copy link
Contributor

@gouyang gouyang commented May 10, 2016

Fixes #944

@eparis
Copy link
Contributor

eparis commented May 10, 2016

I think the script looking in /opt needs to be fixed to look somewhere else. In atomic you can't even create /opt...

@gouyang
Copy link
Contributor Author

gouyang commented May 10, 2016

There is /opt exists on Atomic host.

$ ls -lsh / | grep opt
   0 lrwxrwxrwx.   1 root root    7 May  4 23:44 opt -> var/opt

It need to adjust on one side. Actually I don't see why it changes the path.

@gouyang
Copy link
Contributor Author

gouyang commented May 10, 2016

Open kubernetes/kubernetes#25440

@mikedanese
Copy link
Contributor

/opt is a path inside the kube-addon-manager container.

@gouyang
Copy link
Contributor Author

gouyang commented May 11, 2016

@mikedanese browsing files in https://github.com/kubernetes/kubernetes/blob/master/cluster/saltbase/salt/kube-addons/, /etc/kubernetes and /etc/kubernetes/addons is used there. is it /opt must be used for namespace.yaml?

@rh-kube-bot
Copy link

Can one of the admins verify this patch?

1 similar comment
@rh-kube-bot
Copy link

Can one of the admins verify this patch?

@jasonbrooks
Copy link
Contributor

#797 is how I'm fixing this issue in my fork of the scripts.

@eparis eparis closed this May 26, 2016
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

6 participants