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

redirect_circleci_artifacts.yml keeps failing #941

Closed
yarikoptic opened this issue Nov 29, 2021 · 2 comments · Fixed by #944
Closed

redirect_circleci_artifacts.yml keeps failing #941

yarikoptic opened this issue Nov 29, 2021 · 2 comments · Fixed by #944

Comments

@yarikoptic
Copy link
Collaborator

I keep receiving notifications that https://github.com/bids-standard/bids-specification/blob/master/.github/workflows/redirect_circleci_artifacts.yml is failing, and indeed looking at https://github.com/bids-standard/bids-specification/actions/workflows/redirect_circleci_artifacts.yml shows them all red and sample output is

2021-11-29T11:16:45.1868792Z ##[group]Run curl --fail  | grep $GITHUB_SHA
2021-11-29T11:16:45.1869624Z �[36;1mcurl --fail  | grep $GITHUB_SHA�[0m
2021-11-29T11:16:45.1915837Z shell: /usr/bin/bash -e {0}
2021-11-29T11:16:45.1916431Z ##[endgroup]
2021-11-29T11:16:45.2134994Z curl: no URL specified!
2021-11-29T11:16:45.2140224Z curl: try 'curl --help' or 'curl --manual' for more information
2021-11-29T11:16:45.2160362Z ##[error]Process completed with exit code 1.
2021-11-29T11:16:45.2188732Z Cleaning up orphan processes
@sappelhoff
Copy link
Member

sappelhoff commented Nov 29, 2021

I am also puzzled by this. It's almost as if the GitHub action is building old PRs piece by piece ... And it's at 590 something now. You can see that by checking the individual action runs...

@larsoner have you ever experienced this with the circleci artifact redirector? Or are you also puzzled? :-)

@larsoner
Copy link

This stuff:

      - name: Check the URL
        run: |
          curl --fail ${{ steps.step1.outputs.url }} | grep $GITHUB_SHA

shouldn't be necessary, this is more of a sanity check. You shouldn't need it

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
3 participants