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

Fix vsphere #13314

Merged
merged 7 commits into from
Mar 13, 2017
Merged

Fix vsphere #13314

merged 7 commits into from
Mar 13, 2017

Conversation

gnufied
Copy link
Member

@gnufied gnufied commented Mar 8, 2017

Cherry pick for various vsphere patches to fix panic:
UPSTREAM: 39751: Changed default scsi controller type
UPSTREAM: 39752: Fix panic in vSphere cloud provider
UPSTREAM: 39754: Fix fsGroup to vSphere
UPSTREAM: 39757: Fix space in volumePath in vSphere
UPSTREAM: 40693: fix for vSphere DeleteVolume
UPSTREAM: 41217: Fix wrong VM name is retrieved by the vSphere Cloud Provider
UPSTREAM: 41226: Fix for detach volume when node is not present/ powered off

@erinboyd
Copy link

erinboyd commented Mar 8, 2017

LGTM

@childsb
Copy link
Contributor

childsb commented Mar 8, 2017

[test]

@childsb
Copy link
Contributor

childsb commented Mar 10, 2017

We have done a goodly amount of testing on this on very short turn around. As such there's things we haven't completely figured out or got working.

Talking with @eparis we feel this is better than whats available now even if its not perfect or fully tested. We are comfortable merging this as "tech preview" with a documented caveat that there still maybe issues we are sorting through and this isn't fully supported yet.

@childsb
Copy link
Contributor

childsb commented Mar 10, 2017

[merge]

@childsb
Copy link
Contributor

childsb commented Mar 12, 2017

[test]

@openshift-bot
Copy link
Contributor

Evaluated for origin test up to 4b51389

@eparis
Copy link
Member

eparis commented Mar 12, 2017

[merge] because the last thing was some device mapper flake

@openshift-bot
Copy link
Contributor

continuous-integration/openshift-jenkins/test SUCCESS (https://ci.openshift.redhat.com/jenkins/job/test_pull_request_origin/144/) (Base Commit: cdf2521)

@openshift-bot
Copy link
Contributor

openshift-bot commented Mar 12, 2017

continuous-integration/openshift-jenkins/merge SUCCESS (https://ci.openshift.redhat.com/jenkins/job/test_pull_request_origin/144/) (Base Commit: cdf2521) (Image: devenv-rhel7_6067)

@eparis
Copy link
Member

eparis commented Mar 13, 2017

[merge] flake on #13050

@openshift-bot
Copy link
Contributor

Evaluated for origin merge up to 4b51389

@openshift-bot openshift-bot merged commit 5022261 into openshift:release-1.5 Mar 13, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants