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-171] [common-core] sync with CPI common-core (a1a149d80f32) for "Add Ovdc check when getting ovdc network" #399

Merged
merged 1 commit into from
Feb 24, 2023

Conversation

ymo24
Copy link
Contributor

@ymo24 ymo24 commented Feb 23, 2023

Description

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

  • sync with CPI common-core (a1a149d80f32) for "Add Ovdc check when getting ovdc network"

Checklist

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

API Changes

Are there API changes?

  • Yes
  • No

If yes, please fill in the below

  1. Updated conversions?
    • Yes
    • No
    • N/A
  2. Updated CRDs?
    • Yes
    • No
    • N/A
  3. Updated infrastructure-components.yaml?
    • Yes
    • No
    • N/A
  4. Updated ./examples/capi-quickstart.yaml?
    • Yes
    • No
    • N/A
  5. Updated necessary files under ./infrastructure-vcd/v1.0.0/?
    • Yes
    • No
    • N/A

Issue

If applicable, please reference the relevant issue

Fixes #


This change is Reviewable

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 5 of 5 files at r1, all commit messages.
Reviewable status: :shipit: complete! all files reviewed, all discussions resolved (waiting on @ymo24)

@ymo24 ymo24 requested a review from Anirudh9794 February 24, 2023 06:26
Copy link
Contributor

@Anirudh9794 Anirudh9794 left a comment

Choose a reason for hiding this comment

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

:lgtm:

Reviewed 1 of 5 files at r1.
Reviewable status: :shipit: complete! all files reviewed, all discussions resolved (waiting on @ymo24)

@ymo24 ymo24 merged commit 3c10715 into vmware:main Feb 24, 2023
@ymo24 ymo24 deleted the CAFV-171 branch February 24, 2023 19:51
ltimothy7 pushed a commit to ltimothy7/cluster-api-provider-cloud-director that referenced this pull request Apr 5, 2023
… "Add Ovdc check when getting ovdc network" (vmware#399)

Signed-off-by: ymo24 <[email protected]>
Anirudh9794 pushed a commit that referenced this pull request Apr 10, 2023
* [CAFV-171] [common-core] sync with CPI common-core (a1a149d80f32) for "Add Ovdc check when getting ovdc network" (#399)

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

* [CAFV-125] Update common core to commit bf93908 to prevent CAPVCD from crashing … (#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]>

* [CAFV-177] Ensure CAPVCD Controller accepts RDE 1.1.0 for cluster creation (#402)

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

* update to common core cpi 1.4

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

* calling tkg new vm function (#382)

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

* added tkg 1.6.1 crs test files (#388)

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

* added crs and antrea template (#390)

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

---------

Signed-off-by: ymo24 <[email protected]>
Signed-off-by: lzichong <[email protected]>
Signed-off-by: ltimothy7 <[email protected]>
Co-authored-by: ymo24 <[email protected]>
Co-authored-by: lzichong <[email protected]>
Anirudh9794 pushed a commit that referenced this pull request Apr 11, 2023
… "Add Ovdc check when getting ovdc network" (#399)

Signed-off-by: ymo24 <[email protected]>
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.

4 participants