Skip to content

Conversation

@hari-bodicherla
Copy link
Contributor

Contributing to the Azure SDK

Please see our CONTRIBUTING.md if you are not familiar with contributing to this repository or have questions.

For specific information about pull request etiquette and best practices, see this section.

@ghost ghost added the Compute label Sep 1, 2022
@grizzlytheodore grizzlytheodore merged commit b3ed0f6 into Azure:feature/cplat-2022-08-01 Sep 20, 2022
grizzlytheodore added a commit that referenced this pull request Sep 21, 2022
* feature branch set up for new version

* set up branch

* regenerate sdk with proper swagger updates

* update recordings

* mitigate rolling upgrade test

* remove key

* last test recording

* last recordings

* updated sdk tests (#30870)

* Add property "priorityMixPolicy" of VM Scale Set to .NET SDK for Version 2022-08-01 (#30688)

* Run code generation from Swagger Specs

* Added custom constructor due to code generation changing the parameter order

* Added and verified successful run of new test for PriorityMixPolicy property

* more intuitive assertions in priorityMixPolicy test

* moved most recent test recording to correct folder

* Added parameter descriptions for Scale Set properties which did not have parameter descriptions in the file with custom VMSS constructors

* ci failures popped up again after adding new parameter descriptions

Co-authored-by: Alec Lawlor <[email protected]>

* regenerated

* update recording to resolve conflict

* SDK changes for VMAgent Versioning + ProtectedSettings  (#30395)

* init

* testing

Co-authored-by: Theodore Chang <[email protected]>

* newly generated

* generated

* commit new-recorded jsons

* add customization

Co-authored-by: Theodore Chang <[email protected]>
Co-authored-by: hari-bodicherla <[email protected]>
Co-authored-by: Alec Lawlor <[email protected]>
Co-authored-by: Alec Lawlor <[email protected]>
Co-authored-by: kamusta-msft <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants