api
api copied to clipboard
openshift
CFE-1047: Add the field "CapacityReservationGroupID" to "AzureMachineProviderSpec" in openshift/api
This PR adds the support of capacity reservation group in AzureMachineProviderSpec. A new field, CapacityReservationGroupID, is added. This ID is used while creating the corresponding Azure VM and the VM is added to the capacity reservation group.
The capacity reservation group should already exist. Once the VM is created and associated to the capacity reservation group it cannot be removed from the group. Similarly, if a VM is created without any association to a capacity reservation group then it cannot be associated to one later on. Thus, CapacityReservationGroupID field is made immutable.
@anirudhAgniRedhat: This pull request references CFE-1046 which is a valid jira issue.
Warning: The referenced jira issue has an invalid target version for the target branch this PR targets: expected the spike to target the "4.16.0" version, but no target version was set.
In response to this:
This PR adds the support of capacity reservation group in
AzureMachineSpec. A new field,CapacityReservationGroupID, is added. This ID is used while creating the corresponding Azure VM and the VM is added to the capacity reservation group.The capacity reservation group should already exist. Once the VM is created and associated to the capacity reservation group it cannot be removed from the group. Similarly, if a VM is created without any association to a capacity reservation group then it cannot be associated to one later on. Thus,
CapacityReservationGroupIDfield is made immutable.
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the openshift-eng/jira-lifecycle-plugin repository.
Hello @anirudhAgniRedhat! Some important instructions when contributing to openshift/api: API design plays an important part in the user experience of OpenShift and as such API PRs are subject to a high level of scrutiny to ensure they follow our best practices. If you haven't already done so, please review the OpenShift API Conventions and ensure that your proposed changes are compliant. Following these conventions will help expedite the api review process for your PR.
![openshift-ci[bot] avatar](https://avatars.githubusercontent.com/in/91280?v=4 "Published by a pub.dev verified publisher"){kind=small}
@anirudhAgniRedhat: This pull request references CFE-1047 which is a valid jira issue.
Warning: The referenced jira issue has an invalid target version for the target branch this PR targets: expected the story to target the "4.16.0" version, but no target version was set.
In response to this:
This PR adds the support of capacity reservation group in
AzureMachineSpec. A new field,CapacityReservationGroupID, is added. This ID is used while creating the corresponding Azure VM and the VM is added to the capacity reservation group.The capacity reservation group should already exist. Once the VM is created and associated to the capacity reservation group it cannot be removed from the group. Similarly, if a VM is created without any association to a capacity reservation group then it cannot be associated to one later on. Thus,
CapacityReservationGroupIDfield is made immutable.
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the openshift-eng/jira-lifecycle-plugin repository.
This PR adds the support of capacity reservation group in
AzureMachineSpec.
AzureMachineSpec -> AzureMachineProviderSpec
@anirudhAgniRedhat: This pull request references CFE-1047 which is a valid jira issue.
Warning: The referenced jira issue has an invalid target version for the target branch this PR targets: expected the story to target the "4.16.0" version, but no target version was set.
In response to this:
This PR adds the support of capacity reservation group in
AzureMachineProviderSpec. A new field,CapacityReservationGroupID, is added. This ID is used while creating the corresponding Azure VM and the VM is added to the capacity reservation group.The capacity reservation group should already exist. Once the VM is created and associated to the capacity reservation group it cannot be removed from the group. Similarly, if a VM is created without any association to a capacity reservation group then it cannot be associated to one later on. Thus,
CapacityReservationGroupIDfield is made immutable.
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the openshift-eng/jira-lifecycle-plugin repository.
@anirudhAgniRedhat Do you have an implementation for this API that's ready to merge shortly after?
@JoelSpeed I will start working on its implementation from now onwards as we merge this PR.
Yeah lets get those added
Also, since we are close to the feature freeze date, I'd like to see the feature tested end to end by QE before we merge the API, else we risk shipping an API in one release with no implementation. Please link a WIP PR here once you have it
Yeah lets get those added
Also, since we are close to the feature freeze date, I'd like to see the feature tested end to end by QE before we merge the API, else we risk shipping an API in one release with no implementation. Please link a WIP PR here once you have it
Sure @JoelSpeed
Yeah lets get those added
Also, since we are close to the feature freeze date, I'd like to see the feature tested end to end by QE before we merge the API, else we risk shipping an API in one release with no implementation. Please link a WIP PR here once you have it
@JoelSpeed This is WIP PR for its implementation https://github.com/openshift/machine-api-provider-azure/pull/107
[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: anirudhAgniRedhat, JoelSpeed
The full list of commands accepted by this bot can be found here.
The pull request process is described here
- ~~OWNERS~~ [JoelSpeed]
Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment
@JoelSpeed The PR requireacknowledge-critical-fixes-only label to merge it before 4.16 release If you feel we Should merge it Could you please add the label?
Considering proximity to the release branch cut off, and the fact that TRT have had to introduce the critical fixes requirement. I'm inclined to say we should slip this feature to 4.17, do you see that causing an issue?
Considering proximity to the release branch cut off, and the fact that TRT have had to introduce the critical fixes requirement. I'm inclined to say we should slip this feature to 4.17, do you see that causing an issue?
As this feature is anyway planed for 4.17, we should be good.
@anirudhAgniRedhat: new pull request created: #1923
In response to this:
/cherrypick release-4.16
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository.
@anirudhAgniRedhat: #1866 failed to apply on top of branch "release-4.15":
Applying: Added new Field for CapacityReservationGroupID
Applying: Updated doc for CapacityReservationGroupID
Applying: Updated Doc
Applying: Updated Doc
Applying: Updated Doc
Applying: Added new Field for CapacityReservationGroupID
Using index info to reconstruct a base tree...
M openapi/generated_openapi/zz_generated.openapi.go
M openapi/openapi.json
Falling back to patching base and 3-way merge...
Auto-merging openapi/openapi.json
CONFLICT (content): Merge conflict in openapi/openapi.json
Auto-merging openapi/generated_openapi/zz_generated.openapi.go
CONFLICT (content): Merge conflict in openapi/generated_openapi/zz_generated.openapi.go
error: Failed to merge in the changes.
hint: Use 'git am --show-current-patch=diff' to see the failed patch
Patch failed at 0006 Added new Field for CapacityReservationGroupID
When you have resolved this problem, run "git am --continue".
If you prefer to skip this patch, run "git am --skip" instead.
To restore the original branch and stop patching, run "git am --abort".
In response to this:
/cherrypick release-4.15
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository.
@anirudhAgniRedhat: #1866 failed to apply on top of branch "release-4.14":
Applying: Added new Field for CapacityReservationGroupID
Applying: Updated doc for CapacityReservationGroupID
Applying: Updated Doc
Applying: Updated Doc
Applying: Updated Doc
Applying: Added new Field for CapacityReservationGroupID
Using index info to reconstruct a base tree...
M openapi/generated_openapi/zz_generated.openapi.go
M openapi/openapi.json
Falling back to patching base and 3-way merge...
Auto-merging openapi/openapi.json
CONFLICT (content): Merge conflict in openapi/openapi.json
Auto-merging openapi/generated_openapi/zz_generated.openapi.go
CONFLICT (content): Merge conflict in openapi/generated_openapi/zz_generated.openapi.go
error: Failed to merge in the changes.
hint: Use 'git am --show-current-patch=diff' to see the failed patch
Patch failed at 0006 Added new Field for CapacityReservationGroupID
When you have resolved this problem, run "git am --continue".
If you prefer to skip this patch, run "git am --skip" instead.
To restore the original branch and stop patching, run "git am --abort".
In response to this:
/cherrypick release-4.14
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository.