-
Notifications
You must be signed in to change notification settings - Fork 501
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 0193.md #1251
base: master
Are you sure you want to change the base?
Update 0193.md #1251
Conversation
As part of the actionable errors initiative, I have modified the error message to be more descriptive and included a link to documentation. This is based on our recommended guidelines in the design process: https://docs.google.com/document/d/19z0AMtEqsG_d44sEVgNST0JN7NBWBHWjBAmWVA8yR38/edit#bookmark=id.rsxve02tqdl4 I have also changed the error reason to resource availability according to this guide: https://cloud.google.com/compute/docs/troubleshooting/troubleshooting-resource-availability#resolve_resource_errors These changes have been approved by Bryan Z. and Julie S., PM and IX lead of actionable errors. Please update the example on the AIP-193 website accordingly.
I understand there were some formatting issues, sorry about that. |
aip/general/0193.md
Outdated
@@ -47,19 +47,31 @@ following: | |||
"details": [ | |||
{ | |||
"@type": "type.googleapis.com/google.rpc.ErrorInfo", | |||
"reason": "STOCKOUT", | |||
"reason": "resource_availability", |
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.
should this be in all-caps?
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.
good point. I think it should be yes.
aip/general/0193.md
Outdated
} | ||
}, | ||
{ | ||
"@type": "type.googleapis.com/google.rpc.LocalizedMessage", | ||
"locale": "en-US", | ||
"message": "A e2-medium with local-ssd=3,nvidia-t4=2 is currently unavailable in the us-east1-a zone.\nCapacity changes frequently, so try your request in a different zone, with a different VM hardware configuration, or at a later time. For more information, see troubleshooting documentation." | ||
"message": An <e2-medium> VM instance with <local-ssd=3,nvidia-t4=2> is currently unavailable in the <us-east1-a> zone.\n |
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.
this field cannot contain line breaks, as it breaks json formatting. Please keep the content on a single line and use just escaped line breaks, '\n', instead.
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.
yes, thank you.
I fixed the formatting issues. let me know if there is anything else
As part of the actionable errors initiative, I have modified the error message to be more descriptive and included a link to documentation. This is based on our recommended guidelines in the design process: https://docs.google.com/document/d/19z0AMtEqsG_d44sEVgNST0JN7NBWBHWjBAmWVA8yR38/edit#bookmark=id.rsxve02tqdl4
I have also changed the error reason to resource availability according to this guide: https://cloud.google.com/compute/docs/troubleshooting/troubleshooting-resource-availability#resolve_resource_errors
These changes have been approved by Bryan Z. and Julie S., PM and IX lead of actionable errors. Please update the example on the AIP-193 website accordingly.