Skip to content

#39/#40 and ease zone param restriction if private-parent was passed#52

Merged
sgsullivan merged 4 commits intomasterfrom
cloud-public-ip-list-clarity
Oct 14, 2020
Merged

#39/#40 and ease zone param restriction if private-parent was passed#52
sgsullivan merged 4 commits intomasterfrom
cloud-public-ip-list-clarity

Conversation

@sgsullivan
Copy link
Contributor

implements:

#39
#47

Also now derives zone in derive private parent uniq id helper. Saves some time not having to always pass --zone on args to cloud server create for example if you gave a --private-parent already.

the api response doesn't help us a whole lot here, so parsing the range.
Realistically, this solution will be good enough for 99% of use cases.
Copy link
Contributor

@jvandellen jvandellen left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

@sgsullivan sgsullivan merged commit fba5b62 into master Oct 14, 2020
@sgsullivan sgsullivan deleted the cloud-public-ip-list-clarity branch October 14, 2020 15:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants