-
Notifications
You must be signed in to change notification settings - Fork 169
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
2023-11-22 stable API (porting 07-01-preview api) #3300
Conversation
f113f14
to
f68866b
Compare
pkg/util/azureclient/mgmt/redhatopenshift/2023-11-22/redhatopenshift/openshiftclusters.go
Dismissed
Show resolved
Hide resolved
pkg/util/azureclient/mgmt/redhatopenshift/2023-11-22/redhatopenshift/operations.go
Dismissed
Show resolved
Hide resolved
f68866b
to
7e64e09
Compare
7e64e09
to
8ac47de
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
To future reviewers - this PR includes both the generation of the skeleton API and adding the new fields this API supports. To view what is changing here, you can run:
diff -bur v20230904/ v20231122/
and to verify that everything related to multiple public IPs (not NSG/WorkerProfile stuff) made it over, you can run:
diff -bur v20230701preview/ v20231122/
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There appears to be a diff in one file compared to the preview API that's relevant.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
A few more diffs that need review.
LGTM, I went through the diffs it looks like everything is here. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
* adding 2023-11-22 stable --------- Co-authored-by: b-jhoreman <[email protected]>
* adding 2023-11-22 stable --------- Co-authored-by: b-jhoreman <[email protected]>
Which issue this PR addresses:
https://issues.redhat.com/browse/ARO-4613
What this PR does / why we need it:
This ports functionality from the latest preview API into a stable API definition.
Test plan for issue: