-
Notifications
You must be signed in to change notification settings - Fork 716
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
Updating kubeadm manifests #25
Comments
From @colemickens on October 7, 2016 8:5 @axsuul The |
From @axsuul on October 7, 2016 8:10 @colemickens I have edited it but what do I do after editing it? |
From @colemickens on October 7, 2016 8:14 I would've expected the If it didn't, you could try deleting the |
From @axsuul on October 7, 2016 19:21 When changing the Am I missing something here? |
From @colemickens on October 7, 2016 19:30 |
From @axsuul on October 7, 2016 19:36 Also want to point out that there's no events for |
From @colemickens on October 7, 2016 19:53 I wouldn't expect this ( |
From @axsuul on October 7, 2016 20:6 Sorry, what I meant was |
From @errordeveloper on October 13, 2016 9:24 I'd like to fix kubernetes/kubernetes#33638 ASAP, also after kubernetes/kubernetes#34501 lands we will be able to allow users to pass any custom flags to any of the control plane components. |
From @sandys on October 13, 2016 9:37 @errordeveloper does that mean that your official recommendation right now is to not touch kube-apiserver.json and instead use the flags from #33638 ? |
From @errordeveloper on October 13, 2016 11:10 @sandys unless you really have to, it seems like patching the static pod manifest results in this weird pending situation, and it's probably a separate bug, but I'm not entierely sure we need to look into it, given we are looking to get rid of static pods all together soon with proper kubelet API. |
From @sandys on October 13, 2016 12:23 @errordeveloper im not advocating for the manifest fix - im just asking what should we wait for ? Will this bug become WONTFIX and #33638 be the right way to do it ? |
From @errordeveloper on October 13, 2016 13:10 @sandys I've just opened #34719, hope we might be able to merge it and cut a snapshot release later today ;) |
From @errordeveloper on October 13, 2016 13:11 @sandys yes, this issues as it stands won't be fixed, as user's problem is only about setting the port. |
From @axsuul on October 13, 2016 18:44 @errordeveloper how would we change an existing kubeadm cluster's API ports with the flags? |
This was fixed in kubernetes/kubernetes#34719, and is in the latest version, closing... |
From @axsuul on October 7, 2016 7:57
I provisioned my kubernetes cluster using
kubeadm
but I would like to change the option to--secure-port=8081
for the API server. It seems like changing this manifest filewould give me what I want but how would I apply the changes? Running
doesn't appear to update it properly.
Copied from original issue: kubernetes/kubernetes#34307
The text was updated successfully, but these errors were encountered: