Conversation
| resources.LOOP.await_( | ||
| op.update(per_retry_timeout=resources.REQUEST_TIMEOUT, metadata=resources.REQUEST_MD) | ||
| ) | ||
| resources.LOOP.await_(op.update(per_retry_timeout=resources.REQUEST_TIMEOUT)) |
There was a problem hiding this comment.
@jvstme, op.update does not accept auth_options so not sure if it needs any. Do you know of any issues it may cause?
There was a problem hiding this comment.
not sure if it needs any
I think it does and it's worth reporting to the maintainer that the parameter is missing.
Operation.update calls OperationServiceClient.get, which does accept auth_options.
Do you know of any issues it may cause?
I'd expect the method call to block for 15 minutes if the credentials are invalid and auth_options is missing. For example, this can happen if the service account key expires right before we attempt the call
There was a problem hiding this comment.
Thanks for confirmation. Filed a nebius issue.
jvstme
left a comment
jvstme
left a comment
There was a problem hiding this comment.
It may be worth waiting until the issue from this comment is resolved in the SDK. They usually resolve issues very quickly
2 participants
Closes #3171
The PR updates nebius backend to use the new API from nebius sdk 0.3.