-
Notifications
You must be signed in to change notification settings - Fork 64
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
Typo - MUSKIE_MULTI_DC #17
Comments
Code review: https://cr.joyent.us/3191 Testing notes: I ran the updated "app_uuid" by hand on my test machine and it works correctly where the previous one didn't:
The change is |
joyent-automation
pushed a commit
that referenced
this issue
Jan 10, 2018
Reviewed by: Brian Bennett <brian.bennett@joyent.com> Reviewed by: Tim Kordas <tim.kordas@joyent.com> Approved by: Brian Bennett <brian.bennett@joyent.com>
davepacheco
pushed a commit
that referenced
this issue
Jan 10, 2018
The change is integrated and published. Thanks! |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
app_uuid="$(sdc-sapi /applications?name=manta | json -H uuid)" vs.
app_uuid="$(sdc-sapi /applications?name=manta | json -Ha uuid)"
The text was updated successfully, but these errors were encountered: