Commit254e151

committed

fix: add actionable errors for GCE long running operations

1 parent6acb3a9 commit254e151Copy full SHA for 254e151

File tree

+15

-2

lines changed

+15

-2

lines changed

Lines changed: 11 additions & 2 deletions

Original file line number	Diff line number	Diff line change
`@@ -142,10 +142,19 @@ def __init__(self, message, errors=(), details=(), response=None, error_info=Non`
`142`	`142`	`self._error_info=error_info`
`143`	`143`
`144`	`144`	`def__str__(self):`
	`145`	`+error_msg="".join([self.code,self.message])`
`145`	`146`	`ifself.details:`
`146`		`-return"{} {} {}".format(self.code,self.message,self.details)`
	`147`	`+error_msg=" ".join([error_msg,self.details])`
`147`	`148`	`else:`
`148`		`-return"{} {}".format(self.code,self.message)`
	`149`	`+ifself.errors:`
	`150`	`+errors= [`
	`151`	`+f"{error.code}:{error.message}"`
	`152`	`+forerrorinself.errors`
	`153`	`+ifhasattr(error,"code")andhasattr(error,"message")`
	`154`	`+ ]`
	`155`	`+iflen(errors)>0:`
	`156`	`+error_msg="".join(error_msg,"\n".join(errors))`
	`157`	`+returnerror_msg`
`149`	`158`
`150`	`159`	`@property`
`151`	`160`	`defreason(self):`

Lines changed: 4 additions & 0 deletions

Original file line number	Diff line number	Diff line change
`@@ -158,10 +158,14 @@ def _handle_refreshed_operation(self):`
`158`	`158`	`return`
`159`	`159`
`160`	`160`	`ifself.error_codeandself.error_message:`
	`161`	`+errors= []`
	`162`	`+ifhasattr(self,"error")andhasattr(self.error,"errors"):`
	`163`	`+errors=self.error.errors`
`161`	`164`	`exception=exceptions.from_http_status(`
`162`	`165`	`status_code=self.error_code,`
`163`	`166`	`message=self.error_message,`
`164`	`167`	`response=self._extended_operation,`
	`168`	`+errors=errors,`
`165`	`169`	`)`
`166`	`170`	`self.set_exception(exception)`
`167`	`171`	`elifself.error_codeorself.error_message:`

Comments

(0)