Skip to content
This repository has been archived by the owner on Nov 14, 2024. It is now read-only.

Description / doc changes related to spanner instances in Terraform. #171

Merged
merged 1 commit into from
Jan 22, 2019

Conversation

modular-magician
Copy link
Owner

def async_op_url(module, extra_data=None):
if extra_data is None:
extra_data = {}
url = "https://spanner.googleapis.com/v1/projects/{project}/global/operations/{op_id}"
Copy link
Collaborator

Choose a reason for hiding this comment

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

This is wrong, of course - but it's not obviously more wrong than the existing behavior, which doesn't wait at all. What should I do?

Copy link
Collaborator

Choose a reason for hiding this comment

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

Merge it in and file a bug.

Copy link
Collaborator

@rambleraptor rambleraptor left a comment

Choose a reason for hiding this comment

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

Please file a bug about Spanner async, but otherwise this will be fine (for now...)

@modular-magician modular-magician force-pushed the codegen-pr-1266 branch 2 times, most recently from e81d6d8 to 61430b8 Compare January 18, 2019 23:07
Signed-off-by: Modular Magician <magic-modules@google.com>
@nat-henderson
Copy link
Collaborator

@nat-henderson nat-henderson merged commit 8eb9167 into devel Jan 22, 2019
rambleraptor pushed a commit that referenced this pull request May 21, 2019
…171)

Signed-off-by: Modular Magician <magic-modules@google.com>
modular-magician added a commit that referenced this pull request Sep 5, 2019
…171)

Signed-off-by: Modular Magician <magic-modules@google.com>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants