Skip to content

Support setting os_sku for default_node_pool#339

Merged
lonegunmanb merged 1 commit intoAzure:mainfrom
mjeco:main
Mar 29, 2023
Merged

Support setting os_sku for default_node_pool#339
lonegunmanb merged 1 commit intoAzure:mainfrom
mjeco:main

Conversation

@mjeco
Copy link
Copy Markdown
Contributor

@mjeco mjeco commented Mar 28, 2023

Describe your changes

Support setting of the os_sku parameter on the default_node_pool.

Issue number

#000

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!

@mjeco
Copy link
Copy Markdown
Contributor Author

mjeco commented Mar 28, 2023

@microsoft-github-policy-service agree

@github-advanced-security
Copy link
Copy Markdown

You have successfully added a new CodeQL configuration .github/workflows/pr-check.yaml:prepr-check. As part of the setup process, we have scanned this repository and found no existing alerts. In the future, you will see all code scanning alerts on the repository Security tab.

@mjeco mjeco temporarily deployed to acctests March 29, 2023 00:59 — with GitHub Actions Inactive
Copy link
Copy Markdown
Member

@lonegunmanb lonegunmanb left a comment

Choose a reason for hiding this comment

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

Thanks @mjeco, LGTM! 🚀

@lonegunmanb lonegunmanb merged commit 98bfc96 into Azure:main Mar 29, 2023
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