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

Fix dependabot commits not deploying. Closes #261 #340

Draft
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

juliangruber
Copy link
Member

Closes #261

Copy link
Member

@bajtos bajtos left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Quoting from dependabot/dependabot-core#3253 (comment) (Oct 2021):

The Actions team is also working on enabling Actions to fetch Dependabot secrets during Dependabot workloads, which will allow you to fetch secrets again. I'll link to the changelog post when it's shipped.

Have you tried adding FLY_API_TOKEN to Dependabot secrets? I suspect such a fix should solve the problem in a much simpler way.

@bajtos
Copy link
Member

bajtos commented Aug 30, 2024

See https://github.blog/changelog/2021-11-30-github-actions-workflows-triggered-by-dependabot-receive-dependabot-secrets/

GitHub Actions workflows triggered by Dependabot will now be sent the Dependabot secrets.

This change will enable you to pull from private package registries in your CI using the same secrets you have configured for Dependabot to use and will improve how Actions and Dependabot work together.

@juliangruber
Copy link
Member Author

If you look at https://github.com/filecoin-station/spark-evaluate/commits/main/, there's no CI being run on dependabot #main commits. It looks like unfortunately this is also the case for core-fly, where we thought we had fixed this issue: https://github.com/filecoin-station/core-fly/commits/main/ / https://github.com/filecoin-station/core-fly/actions/workflows/deploy.yaml.

This PR won't work then, unfortunately. I don't see how adding a dependabot secret will fix this either. I think we're back to the drawing board.

@juliangruber juliangruber marked this pull request as draft August 31, 2024 16:37
@bajtos
Copy link
Member

bajtos commented Sep 2, 2024

If you look at https://github.com/filecoin-station/spark-evaluate/commits/main/, there's no CI being run on dependabot #main commits. It looks like unfortunately this is also the case for core-fly, where we thought we had fixed this issue: https://github.com/filecoin-station/core-fly/commits/main/ / https://github.com/filecoin-station/core-fly/actions/workflows/deploy.yaml.

This PR won't work then, unfortunately. I don't see how adding a dependabot secret will fix this either. I think we're back to the drawing board.

😢

I think the first step is to find out why our CI workflow is not triggered for commits created by Dependabot that were pushed to the main branch. WDYT?

@bajtos
Copy link
Member

bajtos commented Sep 2, 2024

I don't see how adding a dependabot secret will fix this either.

Is it possible that GHA is smart enough to check if the Dependabot Secrets config provides values for all secrets used by the workflow, and if not, then it does not run the workflow at all?

@juliangruber
Copy link
Member Author

I don't see how adding a dependabot secret will fix this either.

Is it possible that GHA is smart enough to check if the Dependabot Secrets config provides values for all secrets used by the workflow, and if not, then it does not run the workflow at all?

If that were the case, then https://github.com/filecoin-station/core-fly/blob/main/.github/workflows/dependabot-deploy-check.yml should have worked as the fix though 🤔

@juliangruber
Copy link
Member Author

@juliangruber
Copy link
Member Author

Apparently it's because of auto-merge! dependabot/fetch-metadata#111

@juliangruber
Copy link
Member Author

So, we need to configure auto-merge using a personal access token. We can use the bot user https://github.com/orgs/filecoin-station/people/filecoin-station-robot for this. Do you have any concerns, @bajtos?

@bajtos
Copy link
Member

bajtos commented Sep 4, 2024

Apparently it's because of auto-merge! dependabot/fetch-metadata#111

Sight. Why are they creating so much friction. 😢

So, we need to configure auto-merge using a personal access token. We can use the bot user https://github.com/orgs/filecoin-station/people/filecoin-station-robot for this. Do you have any concerns, @bajtos?

None come to my mind. Let's give it a try!

@juliangruber
Copy link
Member Author

Let's see how this goes: filecoin-station/core-fly@d8fb90b

When it works after the next update, I'll port that here

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: No status
Development

Successfully merging this pull request may close these issues.

Fix dependabot not deploying on #main
2 participants