Skip to content
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

API rate limit exceeded #63

Closed
3 tasks done
ahrtr opened this issue Dec 26, 2022 · 1 comment
Closed
3 tasks done

API rate limit exceeded #63

ahrtr opened this issue Dec 26, 2022 · 1 comment
Assignees
@per1234
Labels
conclusion: invalid Issue/PR not valid topic: code Related to content of the project itself type: imperfection Perceived defect in any part of project

Comments

@ahrtr
Copy link

ahrtr commented Dec 26, 2022

Describe the problem

We are constantly getting this error recently,

Run arduino/setup-protoc@64c0c85d18e984422218383b81c52f8b077404d3
  with:
    version: 3.14.0
    include-pre-releases: false
Error: API rate limit exceeded for 40.84.170.113. (But here's the good news: Authenticated requests get a higher rate limit. Check out the documentation for more details.)

I see a similar issue actions/setup-go#16 , which I think you may refer to.

To reproduce

We see this error each time when we merge a PR,
https://github.com/etcd-io/etcd/blob/main/.github/workflows/static-analysis.yaml

Expected behavior

Successful

'arduino/setup-protoc' version

v1.1.2

Additional context

No response

Issue checklist

  • I searched for previous reports in the issue tracker
  • I verified the problem still occurs when using the latest version
  • My report contains all necessary details
@ahrtr ahrtr added the type: imperfection Perceived defect in any part of project label Dec 26, 2022
@per1234 per1234 self-assigned this Dec 28, 2022
@per1234
Copy link
Contributor

per1234 commented Dec 28, 2022

Hi @ahrtr. You must provide an access token via the action's repo-token input to get a higher GitHub API request allocation to avoid rate limiting.

You will find the automatically generated secrets.GITHUB_TOKEN token to be convenient for this purpose. See the readme for an example of that:

https://github.com/arduino/setup-protoc#:~:text=to%20avoid%20rate%20limiting

@per1234 per1234 closed this as not planned Won't fix, can't repro, duplicate, stale Dec 28, 2022
@per1234 per1234 added conclusion: invalid Issue/PR not valid topic: code Related to content of the project itself labels Dec 28, 2022
ahrtr added a commit to ahrtr/etcd that referenced this issue Dec 28, 2022
@ahrtr
Added 'secrets.GITHUB_TOKEN' for the static-analysis workflow
a60db11 
Refer to: arduino/setup-protoc#63

Signed-off-by: Benjamin Wang <wachao@vmware.com>
wafuwafu13 pushed a commit to wafuwafu13/etcd that referenced this issue Jan 5, 2023
refactor(client): remove duplicate processing
b9c4635 
Signed-off-by: wafuwafu13 <mariobaske@i.softbank.jp>

refactor(clientv3): add comment

Signed-off-by: wafuwafu13 <mariobaske@i.softbank.jp>

refactor: use assert

Signed-off-by: wafuwafu13 <mariobaske@i.softbank.jp>

test: add TestIsUnavailableErr

Signed-off-by: wafuwafu13 <mariobaske@i.softbank.jp>

fix(client): change error message

Signed-off-by: wafuwafu13 <mariobaske@i.softbank.jp>

fix: change error message

Signed-off-by: wafuwafu13 <mariobaske@i.softbank.jp>

clientv3: add protection code to prevent SnapshotWithVersion from panicking

Signed-off-by: Benjamin Wang <wachao@vmware.com>

changelog: update the release date of etcd v3.4.23

Signed-off-by: Benjamin Wang <wachao@vmware.com>

remove the dependency on busybox

Signed-off-by: Benjamin Wang <wachao@vmware.com>

update changelog to update base image to static-debian11 and removd dependency on busybox

Signed-off-by: Benjamin Wang <wachao@vmware.com>

update nsswitch.conf file

Signed-off-by: Benjamin Wang <wachao@vmware.com>

tests: Reproduce issue 14685

Signed-off-by: Marek Siarkowicz <siarkowicz@google.com>

clientv3/naming/endpoints: fix endpoints prefix bug
fixes bug with multiple endpoints with same prefix

Signed-off-by: Ramil Mirhasanov <ramil600@yahoo.com>

build(deps): bump ossf/scorecard-action from 2.1.0 to 2.1.2

Bumps [ossf/scorecard-action](https://github.com/ossf/scorecard-action) from 2.1.0 to 2.1.2.
- [Release notes](https://github.com/ossf/scorecard-action/releases)
- [Changelog](https://github.com/ossf/scorecard-action/blob/main/RELEASE.md)
- [Commits](ossf/scorecard-action@937ffa9...e38b190)

---
updated-dependencies:
- dependency-name: ossf/scorecard-action
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <support@github.com>

Added 'secrets.GITHUB_TOKEN' for the static-analysis workflow

Refer to: arduino/setup-protoc#63

Signed-off-by: Benjamin Wang <wachao@vmware.com>
@martin-g martin-g mentioned this issue Feb 10, 2023
Merged
@tzemanovic tzemanovic mentioned this issue Apr 28, 2023
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@per1234 per1234

Labels
conclusion: invalid Issue/PR not valid topic: code Related to content of the project itself type: imperfection Perceived defect in any part of project
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@per1234 @ahrtr