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

check if nic is already available on update and get the same nic #889

Merged
merged 2 commits into from Jun 27, 2017

Conversation

Projects
None yet
2 participants
@shantanukande
Collaborator

shantanukande commented Jun 22, 2017

No description provided.

@dandagopi dandagopi changed the title from check if nic is already available on update and get the same nic to Don't Merge Shantanu still testing - check if nic is already available on update and get the same nic Jun 22, 2017

@shantanukande

This comment has been minimized.

Show comment
Hide comment
@shantanukande

shantanukande Jun 23, 2017

Collaborator

to take care of subnet full error, checking subnet for available ip and using same. Take care of 'NIC deletion from azure side', catching exception and displaying message.

Collaborator

shantanukande commented Jun 23, 2017

to take care of subnet full error, checking subnet for available ip and using same. Take care of 'NIC deletion from azure side', catching exception and displaying message.

@shantanukande shantanukande changed the title from Don't Merge Shantanu still testing - check if nic is already available on update and get the same nic to check if nic is already available on update and get the same nic Jun 23, 2017

@shantanukande shantanukande changed the title from check if nic is already available on update and get the same nic to Don't merge : check if nic is already available on update and get the same nic Jun 23, 2017

@shantanukande shantanukande changed the title from Don't merge : check if nic is already available on update and get the same nic to check if nic is already available on update and get the same nic Jun 23, 2017

@sleepyorion sleepyorion merged commit ebc3cf9 into oneops:master Jun 27, 2017

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment