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

Added options to configure dhcp lease times #47

Merged
merged 3 commits into from
Sep 27, 2017

Conversation

harmjanblok
Copy link
Contributor

Proposal to allow DHCP lease times to be configured, see UKCloud/govcloudair#8 for additional changes.

@bobbydeveaux
Copy link
Contributor

bobbydeveaux commented Aug 18, 2017

Great. Are you able to commit the changes to the vendor files by running govendor fetch, as I have now merged UKCloud/govcloudair#8 ?

Cheers

@harmjanblok
Copy link
Contributor Author

Done. UKCloud/govcloudair@0f21764 also includes some unrelated changes regarding vapp.go.

@bobbydeveaux
Copy link
Contributor

Awesome thanks. CI suite running this now

@bobbydeveaux
Copy link
Contributor

Status Test Duration Order#
OK TestAccVcdVApp_PowerOff 10m:41s,973ms 8
OK TestAccVcdVAppVm_Basic 7m:42s,446ms 9
OK TestAccVcdVAppRaw_Basic 3m:59s,604ms 7
OK TestAccVcdNetwork_Basic 1m:28s,837ms 5
OK TestAccVcdFirewallRules_basic 1m:03s,309ms 4
OK TestAccVcdVpn_Basic 57s,018ms 3
OK TestAccVcdDNAT_Basic 55s,741ms 1
OK TestAccVcdDNAT_tlate 55s,605ms 2
OK TestAccVcdSNAT_Basic 55s,465ms 6

@harmjanblok
Copy link
Contributor Author

@bobbydeveaux Currently I've some issues creating a VM when using this version:

Following errors occurred while updating network connections:
There are insufficient IP addresses to complete operation. You need to add IP addresses to the network that is associated with the object being created or deployed.

I'm still trying to find out what causes this issue.

@harmjanblok
Copy link
Contributor Author

The issues I had are unrelated to the changes in this pull request. I'll create a separate issue for that.

@bobbydeveaux bobbydeveaux merged commit 4ba23e3 into vmware:master Sep 27, 2017
@harmjanblok harmjanblok deleted the vcd_network-lease-time branch November 1, 2017 20:11
player-two pushed a commit to player-two/terraform-provider-vcd that referenced this pull request Nov 6, 2018
Updates Issue vmware#47.

The vendor directory includes our project direct dependencies (check.v1,
yaml.v2) and a dependency of yaml (pretty).

The vendor directory has been created using govendor
(https://github.com/kardianos/govendor)

The file vendor/vendor.json is the index. While it is not needed for
builds, it will simplify future changes to the vendor folder.

Signed-off-by: Giuseppe Maxia
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.

2 participants