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

Allow vApp userConfigurable=false properties to be configured #1199

Merged
merged 1 commit into from
May 7, 2021

Conversation

adarobin
Copy link
Contributor

@adarobin adarobin commented Sep 2, 2020

Description

This adds a new option under ovf_deploy named enable_hidden_properties. This allows vApp properties that are marked with userConfigurable=false to be set. The most obvious use case for this functionality is directly deploying the vCenter Server Appliance OVA without using the installer. The setting defaults to false which preserves the existing behavior.

The name is based off the name for an advanced option to ovftool.

$ ovftool --help debug | grep -A3 enableHidden 
  --X:enableHiddenProperties          : Enable source properties that are 
                                        marked as ovf:userConfigurable=false, 
                                        OVF Tool will set the value to true. 
                                        The default is false

Acceptance tests

  • Have you added an acceptance test for the functionality being added?
  • Have you run the acceptance tests on this branch? (If so, please include the test log in a gist)

References

@ghost ghost added the size/s Relative Sizing: Small label Sep 2, 2020
@bill-rich
Copy link
Contributor

Hi @adarobin! Have you been successful using the changes to apply templates with userConfigurable=false? In the past when I've tried the same, it results in vCenter returning an error. I think ovftool may be altering the OVF on the fly when deploying to make this work.

@adarobin
Copy link
Contributor Author

Hi @bill-rich I have been using this to deploy vCenter Server Appliance consistently without using the installer in my homelab.

I have been thinking it might make more sense to move the new option under vApp instead of ovf_deploy.

@hashicorp-cla
Copy link

hashicorp-cla commented Nov 22, 2020

CLA assistant check
All committers have signed the CLA.

@tvishwanatharkin
Copy link

Any update on when this feature will be available?

Copy link
Contributor

@koikonom koikonom left a comment

Choose a reason for hiding this comment

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

LGTM

@koikonom koikonom merged commit 99eb07c into hashicorp:master May 7, 2021
@github-actions
Copy link

github-actions bot commented Jun 7, 2021

I'm going to lock this pull request because it has been closed for 30 days ⏳. This helps our maintainers find and focus on the active issues.
If you have found a problem that seems related to this change, please open a new issue and complete the issue template so we can capture all the details necessary to investigate further.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Jun 7, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
size/s Relative Sizing: Small
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants