Skip to content
This repository was archived by the owner on Jan 20, 2026. It is now read-only.

fix: Sync the VCDMachine v1beta1 CRD with the go type - #514

Open
dlipovetsky wants to merge 1 commit into
vmware-archive:mainfrom
mesosphere:dlipovetsky/crd-sync-vcdmachine-machineaddress
Open

fix: Sync the VCDMachine v1beta1 CRD with the go type#514
dlipovetsky wants to merge 1 commit into
vmware-archive:mainfrom
mesosphere:dlipovetsky/crd-sync-vcdmachine-machineaddress

Conversation

@dlipovetsky

@dlipovetsky dlipovetsky commented Sep 20, 2023

Copy link
Copy Markdown
Contributor

Description

A recent PR1 updated the godoc comment that describes the VCDMachineSpec MachineAddress field, but the change was not reflected in the v1beta1 CRD.

This PR does not change the CRD schema, only a comment to a field, and therefore does not make any breaking changes.


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

A recent PR[1] updated the godoc comment that describes the
VCDMachineSpec MachineAddress field, but the change was not reflected in
the v1beta1 CRD.

This updates the comment in the CRD base, as well as
infrastructure-components.yaml

[1]: vmware-archive#445
@vmwclabot

Copy link
Copy Markdown

@dlipovetsky, you must sign our contributor license agreement before your changes are merged. Click here to sign the agreement. If you are a VMware employee, read this for further instruction.

@vmwclabot

Copy link
Copy Markdown

@dlipovetsky, we have received your signed contributor license agreement. The review is usually completed within a week, but may take longer under certain circumstances. Another comment will be added to the pull request to notify you when the merge can proceed.

@vmwclabot

Copy link
Copy Markdown

@dlipovetsky, VMware has approved your signed contributor license agreement.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants