-
Notifications
You must be signed in to change notification settings - Fork 55
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.
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
VM, NIC and disk hydration #237
Open
neerajdixit-msft2
wants to merge
29
commits into
main
Choose a base branch
from
users/trailblazer/VmHydration
base: main
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Changes from 27 commits
Commits
Show all changes
29 commits
Select commit
Hold shift + click to select a range
2567747
protobuf change for node agent VM hydration
neerajdixit-msft2 f58a10d
add VM dicovery protobuf changes
neerajdixit-msft2 2c45b36
add VM discovery protobuf changes for cloudagent
katl-msft 22bb542
add VM discovery protobuf changes for cloudagent
katl-msft d22f44f
Revert "add VM discovery protobuf changes for cloudagent"
katl-msft 2e22ede
Merge pull request #256 from microsoft/users/katlynho/VmHydration
katl-msft ddd716a
hydrate vhd object changes
neerajdixit-msft2 2595b00
Fix crash in mocctl show, need to uncomment all VM structure members
neerajdixit-msft2 e0a276d
Merge pull request #261 from microsoft/users/ndixit/CloudAgentBugFixes
neerajdixit-msft2 644126d
protobuf change for node agent VM hydration
neerajdixit-msft2 8e5f774
add VM dicovery protobuf changes
neerajdixit-msft2 3874ecf
add VM discovery protobuf changes for cloudagent
katl-msft e7931c7
Revert "add VM discovery protobuf changes for cloudagent"
katl-msft c32f10a
add VM discovery protobuf changes for cloudagent
katl-msft 3f56ed6
hydrate vhd object changes
neerajdixit-msft2 9d385c0
Fix crash in mocctl show, need to uncomment all VM structure members
neerajdixit-msft2 f04e8d5
rebase conflict fixes
katl-msft 4881b8d
Merge branch 'users/trailblazer/VmHydration' into users/katlynho/mast…
katl-msft b5ac1e6
Merge pull request #263 from microsoft/users/katlynho/masterMerge
mduppre d715cb8
Update generated go files after catch-up merge with master
mduppre d200305
Merge branch 'main' into users/trailblazer/VmHydration
yingzhan-msft ed8ccfc
Remove discovery
mduppre de33cc4
address review comments
neerajdixit-msft2 62d02a7
add discover option in node agent
neerajdixit-msft2 bee5fc8
Merge pull request #286 from microsoft/users/ndixit/DiskNicIdinVm
neerajdixit-msft2 20b5ade
Go back to go.mod and go.sum from main
mduppre 6051be6
Merge branch 'main' into users/trailblazer/VmHydration
mduppre 25947e4
Add new hydrated state for bootstrapengine
mduppre bccf6b6
Merge pull request #297 from microsoft/users/mduppre/bug_fix
mduppre File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We are skipping (not adding into the map) if provisionStatus== nil? Is everyone aware of status map may not contains some statuses. or should we add the empty string for provisionStatus? Same approach should apply next 4 statuses.