Skip to content

[main] added aks 1.7 support#601

Open
Abhishek-Valaboju wants to merge 3 commits intorancher:mainfrom
Abhishek-Valaboju:aks-1.7
Open

[main] added aks 1.7 support#601
Abhishek-Valaboju wants to merge 3 commits intorancher:mainfrom
Abhishek-Valaboju:aks-1.7

Conversation

@Abhishek-Valaboju
Copy link
Contributor

issue: rancher/rancher#53866

added aks 1.7 support

@Abhishek-Valaboju Abhishek-Valaboju force-pushed the aks-1.7 branch 2 times, most recently from 4e9beea to 2fd5212 Compare February 23, 2026 08:26
@Abhishek-Valaboju Abhishek-Valaboju marked this pull request as ready for review February 24, 2026 06:29
@Abhishek-Valaboju Abhishek-Valaboju requested review from a team as code owners February 24, 2026 06:29
mitulshah-suse
mitulshah-suse previously approved these changes Feb 24, 2026
Copy link
Contributor

@andypitcher andypitcher left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, just added a small suggestion regarding the configmap for aks profiles, if you could adapt it.

"aks-1.0":
- "node"
"aks-1.7":
- "node"
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
- "node"
- "master"
- "node"
- "controlplane"
- "policies"
- "managedservices"

As per upstream's configmap it seems that we're missing targets (even for aks-1.0).

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

added

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@andypitcher node is the only target that we keep, since AKS control plane is managed by Azure and the scanner does not have access to verify those settings.

Co-authored-by: Andy Pitcher <andy.pitcher@suse.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants