-
Notifications
You must be signed in to change notification settings - Fork 40k
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
docs: Kubernetes on Azure with CoreOS and Weave #4483
Conversation
Documentation=https://github.com/GoogleCloudPlatform/kubernetes | ||
Requires=network-online.target | ||
[Service] | ||
Environment=KUBE_RELEASE_TARBALL=https://github.com/GoogleCloudPlatform/kubernetes/releases/download/v0.9.3/kubernetes.tar.gz |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I am using v0.9.3 due to #4414.
Pinging @kelseyhightower. |
0065db5
to
c7b4b18
Compare
@errordeveloper please take a look at |
@erictune I didn't realise there is that file, I do think it needs to be updated also. This was done in collaboration with Microsoft team - @squillace and @chanezon, who probably can be listed also. Unless you'd wish one contact there, otherwise we will have to decide who it shall be. |
multiple contacts is fine. |
@erictune cc @jeffmendoza yes I think it deserves a separate line. Jeff's docs refers to using salt to deploy, ubuntu and openvpn for networking. This one is about Weave for networking, on CoreOS, using cloud-init for deployment. |
feel free to add such a line in this PR. |
@erictune see 3a1dc50; I've added a networking column there as well, as now there are more options. |
80450b9
to
3a1dc50
Compare
7d232fc
to
1b4edc0
Compare
Thanks for updating the README. The new column is fine. |
@erictune I was wondering if the reviewer would be required to test this kind of pull-request manually at this point..? |
Works, http://kubernetes-service-cluster-43e4ff223570ba.cloudapp.net/ 1- script should create the output directory. If it doesn't exists yet, it stops. You need to remove createExternalLoadBalancer and add publicIPs with internal Azure IPs for minions. adding the Azure internal ips for the minion machines to the service json file, then creating an endpoint to these on the same port does the trick. } |
The ubuntu/salt Azure deployment uses openvpn for networking. |
@errordeveloper I think at this point, it isn't practical for reviewers to test these types of PRs, so we are trusting that you have tested it. Since Kelsey hasn't responded in a while, I'll assume he is busy and pick up this review. Looks like you need a rebase, so lets start there. |
I've tested it and it works, if @errordeveloper fixes the 3 issues I mention in the comment above #4483 (comment) |
Ok I am back on this now. Here is a todo list based on @chanezon's comment:
I can work on the two issues above, however there are some more...
These are actually working, if the examples that are bundled in the release are used... Pat, I think you have diverted from the instruction slightly and used examples that are on master branch now. However, I will need to test the latest release and see if the fix for #4414 had been merged into it as wells as if the changes to Guestbook PHP example has appeared there too. |
I can see there is 0.11.0 release candidate, it's probably best to wait for the release itself... |
I've now tested 0.11.0, and looks like it works, and, in fact, it's a no longer a release candidate 👍 I'll be pushing an update once confirm it's definitely working well. |
The issue I am seeing with Guestbook PHP example is this:
...which is documented, and indeed the Any ideas what could be wrong with this and where should I look? |
eec3c83
to
3999019
Compare
Oh, right. The kube-proxy flagname is |
@erictune I'll update proxy's arguments and retest tomorrow, also it'd be good if @chanezon or @squillace could test it once again before merging anyway. |
- improve the wording - suggest next step - remove unrelated code - minor fix to instructions - checkin empty dir
- add entry for CoreOS on Azure with Weave - add networking column
it turned out the issue was due to the domain lenght, so let's just keep it short.
@erictune I've updated the arguments passed to proxy and rebased. This is ready to merge, as far as I am concerned, unless Microsoft folks and/or @kelseyhightower have any input. |
|
||
To get started, you need to checkout the code: | ||
``` | ||
git clone https://github.com/GoogleCloudPlatform/kubernetes |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
can you change this to explicitly pull the v0.11.0 tag? That way, if upstream changes, your guide will keep working until you next update and test i.t
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Just thought this would a good idea at first, but then realised that I won't be able to pull-request into a tag... I could put the exact revision, but that's also impossible since revision is unknown until I commit it. I can see there are release branches, however the last one that exists is release-0.10
, so it seems like a release branches had been deprecated. In fact, this guide is pointing user to the JavaScript code that is part of the guide itself and Go code is not compiled as part of this or anything like that. I am considering it to be safe to keep the user on master here, as I am not expecting anyone to touch this Azure provisioning code much at all.
One minor request, and then good to merge. |
LGTM. |
docs: Kubernetes on Azure with CoreOS and Weave
Awesome to see this merged: as of today, this Weave/CoreOS tool and doc is the only way I was able to provision a Kubernetes cluster in Azure. I have not tested the Ubuntu/OpenVPN way but heard there were some issues with it in current Kubernetes version. I have tested the Flannel docs and after #4362 was fixed I ran into #4141 and stopped there. It may work, but needs further testing and documentation. |
Hi all, Curl to frontend gives valid HTML output The cluster is running on one master, 4 kube nodes and 3 etcd machines (one master and 2 nodes? I am not sure on etcd). This is deployed to Azure. Once the deployment fineshes all the 8 machines have the same Virtual IP and different private IPs (consistent with the subnet range they are in). I added the 8000 port endpoint on kube-01. But cannot hit the VIP. What could I be doing wrong. Thanks for any help |
Are you using the master branch? On Sat, 22 Aug 2015 16:54 rajnemani notifications@github.com wrote:
|
Hi Ilya, I am using the instructions at to set up Kubernetes on Azure/CoreOs/Weave. Per the instructions there I https://github.com/GoogleCloudPlatform/kubernetes. Actually after a bit of investigation and help, I can get the mapping in AZURE is 8000 (public) to 8000 (private). But the private type services (30000-32767). But I have a new problem now. The frontend PHP application cannot figured that it needs DNS setup. So started looking at https://github.com/kubernetes/kubernetes/blob/master/cluster/addons/dns/README.md Here I got couple of questions. Based on the information in the DNS with the following. ENABLE_CLUSTER_DNS="${KUBE_ENABLE_CLUSTER_DNS:-true}" Then I ran the https://github.com/kubernetes/kubernetes/tree/master/docs/getting-started-guides/coreos/azure/create-kubernetes-cluster.js which created the cluster on Azure. Then I ran the DNS server https://github.com/kubernetes/kubernetes/tree/master/docs/getting-started-guides/coreos/azure/addons on the Kubernetes master node. I ran the validation test to see if my document. Here is the output of the test core@kube-00 ~/addons $ kubectl exec busybox -- nslookup kubernetes Name: kubernetes The IPs above do not correspond to the DNS Server IP I setup at So my questions are
overall setup?
that. Also which 'set' of IP and Domain values do I use for starting mentioned in the DNS replication controller and service yaml files I know this is a long email but wanted to provide details so you can Can you provide some guidance? Is there an updated document that has all these steps outlined. Thanks in advance for your time and help. Raj On Mon, Aug 24, 2015 at 1:36 AM, Ilya Dmitrichenko <notifications@github.com
|
@rajnemani please look in master instead, and also see #12224 (comment) and #12907. |
Ok, I can see the updated doc with information on how to get the frontend port configuration. Can I ask if working from master (cloning from master) is also going to resolve the DNS issues between frontend and redis backend i.e I do not have to do all the DNS stuff manually? Thanks |
Yes. |
Hi, On Mon, Aug 24, 2015 at 9:36 AM, Ilya Dmitrichenko <notifications@github.com
|
No, please just follow the instruction as given, the provisioning scripts use released binaries and you don't need to build anything. |
Hi Ilya, I can get the app up but cannot post message to redis. It is the same Thank you for your helpFailed to load resource: the server responded with a status of 404 (Not angular.js:9435 TypeError: Cannot read property 'split' of undefined On Mon, Aug 24, 2015 at 1:03 PM, Ilya Dmitrichenko <notifications@github.com
|
Are you sure the DNS server is running and accessible? On Tue, 25 Aug 2015 00:05 rajnemani notifications@github.com wrote:
|
In this tutorial we will demonstrate how to deploy a Kubernetes cluster to Azure cloud. Weave makes networking of containers simple and secure, in a transparent, yet robust way. The focus of this tutorial is to provide an out-of-the-box production-ready implementation with dedicated Kubernetes master and etcd nodes. It will also show how to scale the cluster with ease.