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

Support NSX Logical Switch created by NSX-T #97

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

jessehu
Copy link

@jessehu jessehu commented Feb 20, 2019

This enhancement is for recognizing NSX Logical Switch created by NSX-T.
Otherwise when a NSX Logical Switch is specified as the network name, it won't find it and create a new Logical Switch with the same name. However, VMs connected to this new Logical Switch cannot get DHCP IP.

@ticobotbot
Copy link
Collaborator

Can one of the admins verify this patch?

@jjlimepoint
Copy link

This one looks good to me!

@jessehu
Copy link
Author

jessehu commented Mar 15, 2019

@jjlimepoint can this be merged?

@jjlimepoint
Copy link

I can't commit myself, and i think this being in chef-boneyard now means no-one can/will? - I'm chasing up to see what to do from here to get it merged somewhere :).

@jessehu
Copy link
Author

jessehu commented Mar 18, 2019

Thanks @jjlimepoint ;)
BTW, can chef-provisioning-fog (using fog::vsphere) replace chef-provisioning-vsphere?

@jjlimepoint
Copy link

chef-provisioning-fog has long been unmaintained, and is believed to not work - I don't use that myself, ymmv!

I maintain a fork of c-p-vsphere at https://github.com/jjlimepoint/chef-provisioning-vsphere (which we use with our customers and integration test daily), and was advised by chef people that this one will never recieve commit again and that we should just run with that fork - if you're cool, i'm going to merge you're two branches into that, as well as any other PR's you might have, and then I'll try and see how we can get a release out from that forked version (I only have maintainer myself on chef-provisioning and chef-provisioning-aws, sadly)

@jessehu
Copy link
Author

jessehu commented Mar 20, 2019

@jjlimepoint Sounds like a plan! Let's figure out get a release out from that forked version :)

@jessehu
Copy link
Author

jessehu commented Mar 20, 2019

It's a pitty that chef-provisioning-fog has long been unmaintained. How about the future plan for chef-provisioning?

@jjlimepoint
Copy link

jjlimepoint commented Mar 20, 2019 via email

@jessehu
Copy link
Author

jessehu commented Mar 20, 2019

That's nice!! I really like chef-provisioning. It's so quick to create a deployment service prototype with chef-provisioning.

@jessehu
Copy link
Author

jessehu commented Mar 20, 2019

Loop in @jjasghar. Please help. We have problem no merging PR:

I can't commit myself, and i think this being in chef-boneyard now means no-one can/will? - I'm chasing up to see what to do from here to get it merged somewhere :).

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.

3 participants