-
Notifications
You must be signed in to change notification settings - Fork 414
Update vagrant to include kubeworkers and refator edge, worker loop #1365
Conversation
@@ -26,6 +30,36 @@ else | |||
config_hash = config_hash.merge(YAML.load(File.read(config_path))) | |||
end | |||
|
|||
def spin_up(config_hash:, config:, server_array:, hostvars:, hosts:, server_type:) |
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.
This function could use an explaining comment
This looks great, thanks @andreijs! I left just a few comments |
"ansible_ssh_host" => ip, | ||
"private_ipv4" => ip, | ||
"public_ipv4" => ip, | ||
"role" => server_type |
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.
server_type could just be renamed to "role" for consistency
We should also probably add a line for kubeworkers here: |
@siddharthist thanks for your feedback I had made the changes pointed out 👍 |
consul_package: consul-0.6.3 | ||
consul_ui_package: consul-ui-0.6.3 | ||
consul_package: consul-0.6.4 | ||
consul_ui_package: consul-ui-0.6.4 |
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 we put this in a separate PR? Doesn't seem directly related.
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.
Sorry I didn't realize I commited this
I got this error on
|
I get this one some times haven't seen the one you got @siddharthist
|
Everything works for me apart from Kube UI and nginx-consul starting on kubeworker ref #1346
|
I got another error this time:
|
@siddharthist can you give me your machine specs OS, ansible version etc ? |
@andreimc I have Vagrant 1.8.1 and Oracle VM VirtualBox Manager 5.0.16_OSE. My host's version of ansible doesn't/shouldn't affect anything, the VMs are provisioned from the control node. |
@siddharthist I really don't know why it fails, me and a co-worker both tried to spin it up and it worked fine, Vagrant file updates should not really cause ansible to fail ... maybe get someone else to try it. |
I'm also having a lot of trouble getting kubernetes to run on Vagrant. Provisioning fails intermittently with various errors. Here are a couple I have seen repeatedly:
Repeated provisioning attempts might ultimately complete but still seeing various problems with Kubernetes:
@BrianHicks @Zogg any ideas on this? |
I get the same issue, but if I re-run with a 'vagrant provision' it all springs to life. Looks like a timing issue to me (I know that the ansible scripts wait for hyperkube to be pulled, but do they start for it to actually be up and listening?). |
Ok, I tell a bit of a lie. Ansible finishes ok, hyperkube is running and I see a node in kubectl. However I can't actually get kubernetes to pull any images (the pod just sits there, no image pull events, and no sign on the kubeworker that any images are being pulled). |
With latest master merged in it fails to restart skydns, not sure what would be causing it It just hangs for a while then I get the following error message:
Not sure why. |
Docker fails on this
But this is due to the new docker implementation not creating a |
@siddharthist up to date. |
Had a successful build but I am back to
when trying to access the Kubernetes UI. 10.254.0.1 is the cluster ip for the
Logs from the
|
hey guys, I had some time span this up in vagrant, I still get 502 for kube ui. :(, asnible ran ok tho. |
I get the following on control-01 when i try to list pods: [vagrant@control-01 ~]$ kubectl --namespace kube-system get pods |
@andreimc @siddharthist curious how is file groups_var/all/kubernetes_vars.yml read in vagrant run? or is it required at all. |
@manishrajkarnikar |
@Zogg I don't see it being mentioned in the vagrant file. I added that in my vagrant file as raw parameter and I was able to get K8s multi node cluster up and running. I couldn't get single master and slave node going though probably because of bug reported in vagrant file. |
Opening new PR of a branch closing this |
Hey guys,
Here is the updated vagrant file for kubeworker.