-
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
Fixing ARO API to enable Terraform Provider for ARO #2983
Conversation
i think it'd be okay to summarize the outcome from design doc in the PR body. In particular, that table in page 6. |
...t/resource-manager/Microsoft.RedHatOpenShift/preview/2023-07-01-preview/redhatopenshift.json
Outdated
Show resolved
Hide resolved
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.
I finished my review.
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
Hey, I'm adding a |
Please rebase pull request. |
6bed9c1
to
220c6ff
Compare
These changes are moved to new PR #3053 |
Which issue this PR addresses:
Fixes https://issues.redhat.com/browse/ARO-2124
What this PR does / why we need it:
OpenShiftCluster GET API returns enriched worker profile data under a new field "workerProfilesStatus". This change is introduced only to new API. More details can be found on Design Doc
Any API call made to endpoints /openShiftClusters or /openShiftCluster with GET method using newer API versions (>= v2023-07-01-preview) will return the enriched worker profile data under a new field "workerProfilesStatus" in the response and "workerProfiles" filed will contain the last input that was passed in PUT/PATCH request body for cluster creation or update.
The response and behavior of any older APIs versions (< v2023-07-01-preview) remain unchanged i.e, only "workerProfiles" filed exists in the response.
Test plan for issue:
Existing unit tests and E2E test copied over from previous API.
Is there any documentation that needs to be updated for this PR?
N/A