Skip to content
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

[Availability Zone] VM Placement in a Zone integration #275

Merged
merged 37 commits into from
Oct 11, 2024
Merged

[Availability Zone] VM Placement in a Zone integration #275

merged 37 commits into from
Oct 11, 2024

Conversation

bgriddaluru
Copy link
Contributor

@bgriddaluru bgriddaluru commented Jul 31, 2024 •
edited
Loading

Design Doc: https://github.com/microsoft/wssdagent/pull/994 (VM_Placement_in_a_Zone.md)

Changes:

  1. Add proto buf related changes for VM association to Availability Zone
  2. Zone message used in VM proto on cloud agent is kept in common, so that it can be reused by Placement group.
  3. Placement Status is added to generic status. Since it is not applicable to all entities hiding it in output if field is not set.

mkatturu and others added 29 commits July 2, 2024 11:49
@mkatturu
added availability zone proto
f713194
@mkatturu
updated pb.go
f10fdb7
@bgriddaluru
VM Placement with Zone - Part 1
49851d6
@bgriddaluru
Rename zones to availability zones. Add strict affinity flag
f4fdaa0
@mkatturu
added status and virtualMachine reference in av zone
207a766
@mkatturu
added av zone reference in virtual machine proto
97f0383
@mkatturu
Merge branch 'users/mkatturu/AvailabilityZonesChanges' of github.com:…
fdfb8a1 
…mkatturu/moc into users/mkatturu/AvailabilityZonesChanges
@mkatturu
updated virtual machine reference number
bf22732
@mkatturu
updated avzone reference number
9d01a8d
@mkatturu
ran make generate
28c4989
@mkatturu
updated vitualMachineRef object
8968777
@mkatturu
updated avzonereference in vm
d976e70
@mkatturu
updated avzonereference in vm
2c35b47
@mkatturu
added groupName property
9a689ca
@mkatturu
sync with main branch
8ff814a
@mkatturu
after make generate
bf676cf
@bgriddaluru
Add Zone status in VM
ff7d141
@bgriddaluru
Move placement status under common status
3a45b25
@bgriddaluru
Fix naming after integration
726c02b
@mkatturu
removed some properties
1c19207
@mkatturu
moved availability zone to cloud
61f4d15
@mkatturu
moved the files to cloud
580db54
@mkatturu
added av zone update operation enum
919ecc8
@bgriddaluru
Merge remote-tracking branch 'manish-moc/users/mkatturu/AvailabilityZ…
589ddf2 
…onesChanges' into users/bgriddaluru/vm_placement_with_zones_integration
@mkatturu
adding roles for avzones
35fde8d
@mkatturu
fixed indentation
14d4136
@bgriddaluru
Merge remote-tracking branch 'manishi-moc/users/mkatturu/Availability…
ddce859 
…ZonesChanges' into users/bgriddaluru/vm_placement_with_zones_integration
@bgriddaluru
Merge branch 'main' into users/bgriddaluru/vm_placement_with_zones_in…
7b52d8e 
…tegration
@bgriddaluru
merget with main - 2
ab99fc8
@bgriddaluru bgriddaluru mentioned this pull request Sep 23, 2024
Closed
@bgriddaluru
Copy link
Contributor Author

/azp run

@azure-pipelines Azure Pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@bgriddaluru bgriddaluru merged commit d911efb into main Oct 11, 2024
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@piyushka17 piyushka17 piyushka17 approved these changes

@msftpgayam msftpgayam msftpgayam approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@bgriddaluru @piyushka17 @msftpgayam @mkatturu