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-443] Consume latest common-core from CPI 1.6.0 to enable IP space support #599

Merged
merged 2 commits into from
Feb 20, 2024

Conversation

rocknes
Copy link
Collaborator

@rocknes rocknes commented Feb 15, 2024

Description

Consume latest common-core from CPI 1.6.0 to enable IP space support
CPI commit : af504c145ad5ff8485682b0fd67d12a126836606

Updated references of gateway.CreateLoadBalancer, gateway.DeleteLoadBalancer to consume IP space support in those methods

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
    • [X 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

Aritra Sen added 2 commits February 14, 2024 15:22
…hanges made to them in CPI for IP space support

Signed-off-by: Aritra Sen <[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.

:lgtm:

Reviewed 7 of 7 files at r1, all commit messages.
Reviewable status: :shipit: complete! all files reviewed, all discussions resolved (waiting on @Anirudh9794, @arunmk, and @sahithi)

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:

Reviewable status: :shipit: complete! all files reviewed, all discussions resolved (waiting on @arunmk and @sahithi)

@rocknes rocknes merged commit ba3f236 into vmware:1.3.z Feb 20, 2024
2 checks passed
@rocknes rocknes deleted the topic/ip-space branch February 20, 2024 19:51
rocknes added a commit that referenced this pull request Feb 20, 2024
…le IP space support (#599) (#602)

Refresh common core from CPI main branch.
Update gateway code to consume IP space related changes in common core. (commit : ba3f236)

---------

Signed-off-by: Aritra Sen <[email protected]>
Co-authored-by: Aritra Sen <[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.

5 participants