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

PR from a fork fails CI #16

Open
pgarrison opened this issue Apr 12, 2024 · 4 comments
Open

PR from a fork fails CI #16

pgarrison opened this issue Apr 12, 2024 · 4 comments
Labels
bug Something isn't working

Comments

@pgarrison
Copy link
Collaborator

I opened a PR and hoped that even though I don't have permissions to download the test data, maybe the CI job would be able to run the tests for me. Instead, the github action also had an authentication issue.

Run aws-actions/configure-aws-credentials@v4
  with:
    role-to-assume: arn:aws:iam::978220035532:role/bioio_github
    role-session-name: bioio-lif-5a500dfce0a2661d3141369df3de968efc769829
    aws-region: us-west-2
    audience: sts.amazonaws.com
  env:
    BUCKET_NAME: bioio-dev-test-resources
    AWS_REGION: us-west-2
It looks like you might be trying to authenticate with OIDC. Did you mean to set the `id-token` permission? If you are not trying to authenticate with OIDC and the action is working successfully, you can ignore this message.
Error: Credentials could not be loaded, please check your action inputs: Could not load credentials from any providers
@pgarrison pgarrison added the bug Something isn't working label Apr 12, 2024
@toloudis
Copy link
Contributor

Another data point:
Previously run actions worked (4 days prior) with OIDC authentication:

Assuming role with OIDC
Authenticated as assumedRoleId AROA6HQTVFXGGP4HAE5W3:bioio-lif-964a301b04e07b8d88957df46a836b8dc792ca66

@toloudis
Copy link
Contributor

(bioio-lif looks fishy there, as it looks like a name from a different repo)

@pgarrison pgarrison changed the title Tests for PR failing in CI due to authentication issue PR from a fork fails CI Apr 24, 2024
@pgarrison
Copy link
Collaborator Author

Update: resolved by getting myself added to bioio-devs and making the PR not from a fork. This will continue to be an issue for external contributors.

@toloudis
Copy link
Contributor

We might have to meet up to discuss what to do about this, so that we can enable third party contributions.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants