-
Notifications
You must be signed in to change notification settings - Fork 5.2k
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
fleet autoupgrade GA - add autoupgradeprofilestatus #32634
base: release-aks-fleet-2025-03-01
Are you sure you want to change the base?
fleet autoupgrade GA - add autoupgradeprofilestatus #32634
Conversation
Next Steps to MergeNext steps that must be taken to merge this PR:
|
Generated ApiView
|
API change check APIView has identified API level changes in this PR and created following API reviews. |
@@ -2249,6 +2278,36 @@ | |||
}, | |||
"readOnly": true | |||
}, | |||
"AutoUpgradeProfileStatus": { |
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.
"type": "object", | ||
"description": "AutoUpgradeProfileStatus is the status of an auto upgrade profile.", | ||
"properties": { | ||
"lastTriggeredAt": { |
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.
"readOnly": true | ||
}, | ||
"lastTriggerError": { | ||
"$ref": "../../../../../../common-types/resource-management/v3/types.json#/definitions/ErrorDetail", |
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.
"description": "The error details of the last trigger.", | ||
"readOnly": true | ||
}, | ||
"lastTriggerUpgradeVersions": { |
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 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.
no, there will be a large amount / undefined variations of version types
@@ -2212,6 +2237,10 @@ | |||
"disabled": { | |||
"type": "boolean", | |||
"description": "If set to False: the auto upgrade has effect - target managed clusters will be upgraded on schedule.\nIf set to True: the auto upgrade has no effect - no upgrade will be run on the target managed clusters.\nThis is a boolean and not an enum because enabled/disabled are all available states of the auto upgrade profile.\nBy default, this is set to False." | |||
}, | |||
"autoUpgradeProfileStatus": { |
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.
Please fill in this too. |
|
||
@doc("AutoUpgradeLastTriggerStatus is the status of the last AutoUpgrade trigger (attempt to automatically create and start UpdateRun when there are new released versions) of an auto upgrade profile.") | ||
@added(Versions.v2025_03_01) | ||
union AutoUpgradeLastTriggerStatus { |
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.
why union and not enum?
Choose a PR Template
Switch to "Preview" on this description then select one of the choices below.
Click here to open a PR for a Data Plane API.
Click here to open a PR for a Control Plane (ARM) API.
Click here to open a PR for only SDK configuration.