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.

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

kit not downloading #10768

Closed
2 of 3 tasks
tico24 opened this issue Mar 28, 2023 · 14 comments · Fixed by #10841
Closed
2 of 3 tasks

kit not downloading #10768

tico24 opened this issue Mar 28, 2023 · 14 comments · Fixed by #10841
Assignees
Labels
area/build Build or GithubAction/CI issues P1 High priority. All bugs with >=5 thumbs up that aren’t P0, plus: Any other bugs deemed high priority type/bug

Comments

@tico24
Copy link
Member

tico24 commented Mar 28, 2023

Pre-requisites

  • I have double-checked my configuration
  • I can confirm the issues exists when I tested with :latest
  • I'd like to contribute the fix myself (see contributing guide)

What happened/what you expected to happen?

I submitted a Documentation enhancement PR #10767

The build repeatedly fails, not on the docs ci but on the main CI. Specifically it loops over and over again with

 + curl -fsL https://api.github.com/repos/kitproj/kit/releases/latest
+ jq -r .tag_name
+ tag=
+ [  =  ]
+ jq -r .tag_name
+ curl -fsL https://api.github.com/repos/kitproj/kit/releases/latest
+ tag=
+ [  =  ]
+ jq -r .tag_name
+ curl -fsL https://api.github.com/repos/kitproj/kit/releases/latest
+ tag=
+ [  =  ]
+ curl -fsL https://api.github.com/repos/kitproj/kit/releases/latest
+ jq -r .tag_name
+ tag=
+ [  =  ]
+ jq -r .tag_name
+ curl -fsL https://api.github.com/repos/kitproj/kit/releases/latest
+ tag=
+ [  =  ]
+ curl -fsL https://api.github.com/repos/kitproj/kit/releases/latest
+ jq -r .tag_name
+ tag=
+ [  =  ]
+ curl -fsL https://api.github.com/repos/kitproj/kit/releases/latest
+ jq -r .tag_name
+ tag=
+ [  =  ]
+ curl -fsL https://api.github.com/repos/kitproj/kit/releases/latest
+ jq -r .tag_name
+ tag=
+ [  =  ]
+ curl -fsL https://api.github.com/repos/kitproj/kit/releases/latest
+ jq -r .tag_name
+ tag=
+ [  =  ]
+ jq -r .tag_name
+ curl -fsL https://api.github.com/repos/kitproj/kit/releases/latest
+ tag=
+ [  =  ]
+ jq -r .tag_name
+ curl -fsL https://api.github.com/repos/kitproj/kit/releases/latest
+ tag=
+ [  =  ]
+ jq -r .tag_name
+ curl -fsL https://api.github.com/repos/kitproj/kit/releases/latest
+ tag=
+ [  =  ]
+ curl -fsL https://api.github.com/repos/kitproj/kit/releases/latest
+ jq -r .tag_name
+ tag=
+ [  =  ]
+ curl -fsL https://api.github.com/repos/kitproj/kit/releases/latest
+ jq -r .tag_name
+ tag=
+ [  =  ]
+ jq -r .tag_name
+ curl -fsL https://api.github.com/repos/kitproj/kit/releases/latest
+ tag=
+ [  =  ]
+ jq -r .tag_name
+ curl -fsL https://api.github.com/repos/kitproj/kit/releases/latest

I see little benefit on wasting CPU cycles testing the code that hasn't changed, and this blocks me getting my PR for review.

Version

latest

Paste a small workflow that reproduces the issue. We must be able to run the workflow; don't enter a workflows that uses private images.

n/a

Logs from the workflow controller

n/a

Logs from in your workflow's wait container

n/a
@terrytangyuan
Copy link
Member

It looks passing now

@tico24
Copy link
Member Author

tico24 commented Mar 29, 2023

@terrytangyuan
Copy link
Member

@alexec Could you take a look?

@alexec
Copy link
Contributor

alexec commented Mar 29, 2023

@terrytangyuan shall I add you as contributor to kit?

@alexec alexec changed the title Docs build failing kit now downloading Mar 29, 2023
@alexec alexec self-assigned this Mar 29, 2023
@alexec alexec changed the title kit now downloading kit not downloading Mar 29, 2023
@tico24
Copy link
Member Author

tico24 commented Mar 30, 2023

Failing for this PR too: #10778

@alexec
Copy link
Contributor

alexec commented Mar 30, 2023

If anyone can explain why a simple curl command fails, I’d love to know. I’m lost.

@tico24
Copy link
Member Author

tico24 commented Mar 30, 2023

Maybe drop the -s to see why it's failing?

The -f won't work if it's getting a 401 for example, and supressing the failure seems to be doing more harm than good anyway?

My guess besides "github actions suck" is its something to do with being inside the github ecosystem is making it require auth or something.

@alexec
Copy link
Contributor

alexec commented Mar 30, 2023

Good idea I’ll add -v and we can hopefully diagnose the issue.

@alexec
Copy link
Contributor

alexec commented Mar 30, 2023

Done.

@tico24
Copy link
Member Author

tico24 commented Mar 30, 2023

Looks like GitHub rate limits it's own access to the api.

So the only solutions I see are:

  • Don't host kit on a GitHub api endpoint.

  • Don't run ci on GitHub.

@terrytangyuan
Copy link
Member

terrytangyuan commented Mar 30, 2023

Also try authenticating the GitHub API requests to avoid rate-limiting?

@JPZ13 JPZ13 added the P1 High priority. All bugs with >=5 thumbs up that aren’t P0, plus: Any other bugs deemed high priority label Mar 30, 2023
@tico24
Copy link
Member Author

tico24 commented Apr 3, 2023

@terrytangyuan do you have access to secrets in this repo? Otherwise @alexec will have to make this change.

@terrytangyuan
Copy link
Member

Only @alexec and @sarabala1979

@tico24 tico24 mentioned this issue Apr 4, 2023
3 tasks
@terrytangyuan
Copy link
Member

@alexec We hit this quite frequently recently. Would you mind taking a look whenever you get a chance?

@alexec alexec linked a pull request Apr 6, 2023 that will close this issue
JPZ13 pushed a commit to pipekit/argo-workflows that referenced this issue Jul 4, 2023
@agilgur5 agilgur5 added the area/build Build or GithubAction/CI issues label Sep 3, 2023
dpadhiar pushed a commit to dpadhiar/argo-workflows that referenced this issue May 9, 2024
…#10841)

Signed-off-by: Dillen Padhiar <dillen_padhiar@intuit.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/build Build or GithubAction/CI issues P1 High priority. All bugs with >=5 thumbs up that aren’t P0, plus: Any other bugs deemed high priority type/bug
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants