Skip to content
This repository has been archived by the owner on Oct 11, 2023. It is now read-only.

Support vSphere network protocol profiles #2490

Merged
merged 2 commits into from
Dec 19, 2018
Merged

Conversation

kinolaev
Copy link

@kinolaev kinolaev commented Sep 26, 2018

Part of rancher/rancher#15817

Added guestinfo params:

  • interface.<n>.ip.<m>.netmask
  • dns.servers
  • dns.domains

because vSphere doesn't provide IP address in CIDR format and have one field for DNS servers and one for DNS search paths.

@niusmallnan
Copy link
Contributor

Hi @kinolaev ,
We have supported these guestinfo.
Is there any special reason why these can't meet your needs?

@kinolaev
Copy link
Author

kinolaev commented Sep 27, 2018

Here is official documentation for vSphere vApp properties and for Network protocol profile. You can see that vSphere haven't type or expression for passing IP in CIDR format. Same for DNS properties.

@niusmallnan
Copy link
Contributor

It seems that this feature makes sense.

I have already recommended it to my colleague, looking forward to the feedback of other PRs.

@cjellick
Copy link
Contributor

@niusmallnan who are we waiting on for an LGTM?

@niusmallnan
Copy link
Contributor

@cjellick I can review this today and merge it if possible.

Copy link
Contributor

@niusmallnan niusmallnan left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@niusmallnan niusmallnan merged commit 4e0269e into rancher:master Dec 19, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants