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

[CAFV-217] Support TKG 1.6.1 vm creation with CPI 1.4 common core #429

Merged
merged 7 commits into from
Apr 11, 2023

Conversation

Anirudh9794
Copy link
Contributor

@Anirudh9794 Anirudh9794 commented Apr 11, 2023

Description

Please provide a brief description of the changes proposed in this Pull Request

Checklist

  • tested locally
  • updated any relevant dependencies
  • updated any relevant documentation or examples

API Changes

Are there API changes?

  • Yes
  • No

Issue

If applicable, please reference the relevant issue

Fixes #


This change is Reviewable

ymo24 and others added 7 commits April 5, 2023 19:00
… "Add Ovdc check when getting ovdc network" (vmware#399)

Signed-off-by: ymo24 <[email protected]>
…m crashing … (vmware#372)

* Update common core to commit bf93908 to prevent CAPVCD from crashing when using IP Spaces

Signed-off-by: lzichong <[email protected]>

* Update externalIP param to take empty string as vcdmachine will not update VS IP

Signed-off-by: lzichong <[email protected]>

* Concluded on externalIp param input, added additional comments

Signed-off-by: lzichong <[email protected]>

* Update ticket to CAPVCD Board

Signed-off-by: lzichong <[email protected]>

* Include CAFV ticket in other TODO comment

Signed-off-by: lzichong <[email protected]>

---------

Signed-off-by: lzichong <[email protected]>
Copy link
Contributor

@lzichong lzichong left a comment

Choose a reason for hiding this comment

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

Reviewed 1 of 3 files at r3, 8 of 923 files at r4, 5 of 5 files at r5, all commit messages.
Reviewable status: 14 of 930 files reviewed, all discussions resolved

Copy link
Contributor

@ymo24 ymo24 left a comment

Choose a reason for hiding this comment

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

Reviewed all commit messages.
Reviewable status: 14 of 930 files reviewed, all discussions resolved

@Anirudh9794 Anirudh9794 merged commit 6f8b0b9 into vmware:1.0.z Apr 11, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants