Skip to content

Remove all deprecated variables #652

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 4 commits into from
Apr 15, 2025
Merged

Remove all deprecated variables #652

merged 4 commits into from
Apr 15, 2025

Conversation

@lonegunmanb
Copy link
Member

@lonegunmanb lonegunmanb commented Apr 3, 2025

Describe your changes

Issue number

#643

Checklist before requesting a review

  • The pr title can be used to describe what this pr did in CHANGELOG.md file
  • I have executed pre-commit on my machine
  • I have passed pr-check on my machine

Thanks for your cooperation!

jiaweitao001
jiaweitao001 approved these changes Apr 15, 2025
View reviewed changes
Copy link
Collaborator

@jiaweitao001 jiaweitao001 left a comment

Choose a reason for hiding this comment

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

LGTM!

@lonegunmanb lonegunmanb merged commit 008a216 into main Apr 15, 2025
5 checks passed
@github-project-automation github-project-automation bot moved this from Todo to Done in Azure Module Kanban Apr 15, 2025
@lonegunmanb lonegunmanb deleted the e-643 branch April 15, 2025 05:27
@tobiasehlert tobiasehlert mentioned this pull request May 7, 2025
Closed
1 task
@anroots-by
Copy link

anroots-by commented Nov 11, 2025

This modifies how api_server_access_profile is evaluated, and removes previous behavior where specifying an allowed subnet for API server access would still create the block.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@jiaweitao001 jiaweitao001 jiaweitao001 approved these changes

Assignees

No one assigned

Labels

None yet

Projects

Azure Module Kanban
Status: Done

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@lonegunmanb @anroots-by @jiaweitao001