Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Fix handling of child vApp containers #941

Merged
merged 1 commit into from
Jan 15, 2020
Merged

Conversation

koikonom
Copy link
Contributor

  • Child vApp containers do not need folders
  • Child vApp containers do not have parent Folders set either, which unless
    checked will cause a panic when importing.
  • Add test case for importing child vApp containers.

Fixes #939

- Child vApp containers do not need folders
- Child vApp containers do not have parent Folders set either, which unless
checked will cause a panic when importing.
- Add test case for importing child vApp containers.

Fixes #939
@koikonom koikonom requested a review from a team January 14, 2020 14:23
@ghost ghost added the size/m Relative Sizing: Medium label Jan 14, 2020
Copy link
Contributor

@bill-rich bill-rich left a comment

Choose a reason for hiding this comment

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

LGTM

# for free to subscribe to this conversation on GitHub. Already have an account? #.
Labels
size/m Relative Sizing: Medium
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Crash after importing vapp_container
2 participants