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.
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
Add Proximity Placement Group Resources + Support #4020
Add Proximity Placement Group Resources + Support #4020
Changes from 3 commits
cace9a8
b915bf1
b7ede39
ffc1b69
3c6162d
25a6c87
332c8dc
83ccac5
6a55d35
f76e6fa
ac65ac2
0d24f02
fb8b294
f4f5b8b
ff19089
91ad7ec
fb392d3
b4a9e36
2aa393e
cf2cd97
843ce8a
File filter
Filter by extension
Conversations
Jump to
There are no files selected for viewing
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.
Could we add
tags.SchemaForDataSource
to the resource so this has an effect?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.
Could this have potential issues if there are capitables in other places? we should be more specific with the fix.
Also, has a bug report been files/opened anywhere? if not could we open on on the azure for go sdk and link it here with an addtional comment:
this can be removed when http://link.to.issue has been fixed
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.
See other comment the ID, this is in-line with how availability set ID are handled.
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.
Could we validate the name here?
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.
Validate how? I'm not going to pretend I'm not rather new to this. Much of this is taken directly from availability sets since the requirements were very similar.