-
Notifications
You must be signed in to change notification settings - Fork 298
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
Update NEG reference in NEG CR #2604
base: master
Are you sure you want to change the base?
Conversation
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: sawsa307 The full list of commands accepted by this bot can be found here.
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
8449144
to
eefaff0
Compare
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.
Addressed all the comments, PTAL!
84e33cd
to
f343cbb
Compare
f6fa060
to
01cdea5
Compare
ba9f4e8
to
0de216d
Compare
0de216d
to
60f5b05
Compare
60f5b05
to
f2c2338
Compare
f2c2338
to
e074cca
Compare
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.
Updated, please take a look! @swetharepakula
3ddd5ea
to
c630d45
Compare
* Populate Subnet in NEG CR based on the subnet on the NEG. * Populate State as ACTIVE for new and existing NEGs. For NEGs in the zones which don't have any nodes, set their state as INACTIVE.
* Once a NEG has been deleted, remove its referece from NEG CR. This prevents NEG CR retaining reference of non-existing NEGs.
c630d45
to
581ba4d
Compare
/assign @swetharepakula