Skip to content

update gapic retry config #2390

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.

Sign up for GitHub

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

Jump to bottom
Merged
swcloud merged 2 commits into from
Jun 15, 2017
Merged

update gapic retry config #2390

swcloud merged 2 commits into from
Jun 15, 2017

Conversation

@swcloud
Copy link
Contributor

@swcloud swcloud commented Jun 14, 2017

No description provided.

@googlebot googlebot added the cla: yes This human has signed the Contributor License Agreement. label Jun 14, 2017
@stephenplusplus stephenplusplus added the api: spanner Issues related to the Spanner API. label Jun 14, 2017
@swcloud swcloud added api: datastore Issues related to the Datastore API. api: language Issues related to the Cloud Natural Language API API. api: logging Issues related to the Cloud Logging API. api: monitoring Issues related to the Cloud Monitoring API. api: pubsub Issues related to the Pub/Sub API. api: speech Issues related to the Speech-to-Text API. api: vision Issues related to the Cloud Vision API. api: dlp Issues related to the Sensitive Data Protection API. api: video-intelligence labels Jun 15, 2017
@swcloud
Copy link
Contributor Author

swcloud commented Jun 15, 2017

@stephenplusplus What's wrong with circleci?

$nvm ls
N/A: version "N/A -> N/A" is not yet installed.

You need to run "nvm install N/A" to install it before using it.
N/A
default -> 4.2.6 (-> N/A)
node -> stable (-> N/A) (default)
iojs -> N/A (default)

nvm ls returned exit code 3

Action failed: nvm ls

@coveralls
Copy link

coveralls commented Jun 15, 2017

Coverage Status

Changes Unknown when pulling 917d0f3 on swcloud:remove-retry-unavailable-non-idempotent into ** on GoogleCloudPlatform:master**.

@stephenplusplus
Copy link
Contributor

stephenplusplus commented Jun 15, 2017

Circle equips Linux builds with nvm by default, but when we were testing OS X, I had to add a step in the build to install it. I removed that line, restarted the build, and we're back to green.

@swcloud swcloud merged commit 3d98532 into googleapis:master Jun 15, 2017
This was referenced Jul 5, 2017
Merged
Merged
@snyk-bot snyk-bot mentioned this pull request Mar 11, 2020
Open
sofisl pushed a commit that referenced this pull request Sep 27, 2022
@swcloud
update gapic retry config (#2390)
621a8c5
@sofisl sofisl mentioned this pull request Sep 27, 2022
Closed
4 tasks
sofisl pushed a commit that referenced this pull request Oct 11, 2022
@swcloud
update gapic retry config (#2390)
bd3bba2
@sofisl sofisl mentioned this pull request Oct 11, 2022
Merged
4 tasks
sofisl pushed a commit that referenced this pull request Oct 13, 2022
@swcloud @sofisl
update gapic retry config (#2390)
0951713
This was referenced Nov 9, 2022
Closed
Merged
@sofisl sofisl mentioned this pull request Nov 11, 2022
Merged
4 tasks
sofisl pushed a commit that referenced this pull request Nov 11, 2022
@swcloud
update gapic retry config (#2390)
b656b12
@sofisl sofisl mentioned this pull request Jan 10, 2023
Merged
4 tasks
sofisl pushed a commit that referenced this pull request Jan 10, 2023
@swcloud
update gapic retry config (#2390)
eda15d9
@sofisl sofisl mentioned this pull request Jan 17, 2023
Merged
4 tasks
sofisl pushed a commit that referenced this pull request Jan 17, 2023
@swcloud
update gapic retry config (#2390)
aaf3280
sofisl pushed a commit that referenced this pull request Jan 24, 2023
@swcloud
update gapic retry config (#2390)
19732ed
sofisl pushed a commit that referenced this pull request Jan 25, 2023
@swcloud
update gapic retry config (#2390)
8b5073e
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@lukesneeringer lukesneeringer Awaiting requested review from lukesneeringer

1 more reviewer

@stephenplusplus stephenplusplus stephenplusplus approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

api: datastore Issues related to the Datastore API. api: dlp Issues related to the Sensitive Data Protection API. api: language Issues related to the Cloud Natural Language API API. api: logging Issues related to the Cloud Logging API. api: monitoring Issues related to the Cloud Monitoring API. api: pubsub Issues related to the Pub/Sub API. api: spanner Issues related to the Spanner API. api: speech Issues related to the Speech-to-Text API. api: video-intelligence api: vision Issues related to the Cloud Vision API. cla: yes This human has signed the Contributor License Agreement.

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@swcloud @coveralls @stephenplusplus @googlebot