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

kubelet jobs prestart script does not work on vsphere with multiple datacenters #127

jostockley opened this issue Nov 16, 2017 · 4 comments


Copy link

@jostockley jostockley commented Nov 16, 2017

The kubelet job prestart script needs to add
export GOVC_DATACENTER="$(get_cloud_property "datacenter")"
otherwise on vcenter with more than one datacenter defined the kubelet job deployment fails. Examining the logs in worker/0 reveals:
/var/vcap/packages/govc/bin/govc: default datacenter resolves to multiple instances, please specify
I am using kubo-release build 0.9.0-dev.21 as provided by downloading kube-deployment-latest.tgz

Copy link

@cf-gitbot cf-gitbot commented Nov 16, 2017

We have created an issue in Pivotal Tracker to manage this:

The labels on this github issue will be updated when the story is started.

Copy link

@glestaris glestaris commented Nov 23, 2017

This is interesting. We can have a look. We would be open to a PR as well.

Copy link

@jostockley jostockley commented Nov 26, 2017

Trying to get contributor license agreement signed/submitted. If successful, I'll create a PR

Copy link

@bsnchan bsnchan commented Dec 18, 2017

Hey @jostockley - did you ever get around to signing the CLA? We'd love a PR for this.

tvs added a commit that referenced this issue Dec 21, 2017
@cf-gitbot cf-gitbot added in progress and removed unscheduled labels Dec 21, 2017
tvs added a commit that referenced this issue Dec 21, 2017

Issue #127

@cf-gitbot cf-gitbot added delivered and removed in progress labels Dec 21, 2017
@tvs tvs closed this Dec 21, 2017
@cf-gitbot cf-gitbot added accepted and removed delivered labels Dec 21, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
5 participants