-
Notifications
You must be signed in to change notification settings - Fork 11
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
Keep workflows enabled #118
Changes from all commits
f9074c4
1583c9d
6d7ba19
e8f82f8
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change | ||||||||
---|---|---|---|---|---|---|---|---|---|---|
@@ -0,0 +1,58 @@ | ||||||||||
# This workflow is intended to keep scheduled GH Action workflows enabled | ||||||||||
# as a work-around for scheduled workflows being automatically disabled when | ||||||||||
# no repository activity has occurred in 60 days. | ||||||||||
name: Keep workflows enabled | ||||||||||
|
||||||||||
on: | ||||||||||
schedule: | ||||||||||
# Scheduled to run at 4pm UTC (8am PST) on the first day of the month | ||||||||||
- cron: '0 16 1 * *' | ||||||||||
|
||||||||||
workflow_dispatch: | ||||||||||
|
||||||||||
jobs: | ||||||||||
enable-workflow: | ||||||||||
permissions: | ||||||||||
actions: write | ||||||||||
env: | ||||||||||
GH_TOKEN: ${{ secrets.GH_TOKEN_NEXTSTRAIN_BOT_WORKFLOW_DISPATCH }} | ||||||||||
name: "${{ matrix.repo }}:${{ matrix.workflow }}" | ||||||||||
strategy: | ||||||||||
fail-fast: false | ||||||||||
matrix: | ||||||||||
include: | ||||||||||
- { repo: .github, workflow: keep-workflows-enabled.yaml } | ||||||||||
- { repo: augur, workflow: ci.yaml } | ||||||||||
- { repo: avian-flu, workflow: ingest-to-phylogenetic-ncbi.yaml } | ||||||||||
- { repo: cli, workflow: ci.yaml } | ||||||||||
- { repo: cli, workflow: standalone-installers.yaml } | ||||||||||
- { repo: conda-base, workflow: installation.yaml } | ||||||||||
- { repo: dengue, workflow: ingest-to-phylogenetic.yaml } | ||||||||||
- { repo: forecasts-ncov, workflow: update-ncov-case-counts.yaml } | ||||||||||
- { repo: lassa, workflow: ci.yaml } | ||||||||||
- { repo: lassa, workflow: ingest-to-phylogenetic.yaml } | ||||||||||
- { repo: measles, workflow: ingest-to-phylogenetic.yaml } | ||||||||||
- { repo: mpox, workflow: fetch-and-ingest.yaml } | ||||||||||
- { repo: ncov, workflow: rebuild-100k.yml } | ||||||||||
- { repo: ncov-ingest, workflow: fetch-and-ingest-genbank-master.yml } | ||||||||||
- { repo: ncov-ingest, workflow: fetch-and-ingest-gisaid-master.yml } | ||||||||||
- { repo: nextstrain.org, workflow: index-resources.yml } | ||||||||||
- { repo: nextstrain.org, workflow: remind-to-promote.yml } | ||||||||||
- { repo: nipah, workflow: ingest-to-phylogenetic.yaml } | ||||||||||
- { repo: oropouche, workflow: ingest-to-phylogenetic.yaml } | ||||||||||
- { repo: rabies, workflow: ingest-to-phylogenetic.yaml } | ||||||||||
- { repo: rsv, workflow: fetch-and-ingest.yaml } | ||||||||||
- { repo: rsv, workflow: rebuild.yaml } | ||||||||||
- { repo: seasonal-cov, workflow: ingest-to-phylogenetic.yaml } | ||||||||||
- { repo: status, workflow: ci.yaml } | ||||||||||
- { repo: WNV, workflow: ingest-to-phylogenetic.yaml } | ||||||||||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Not entirely clear to me why this failed for WNV:
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. OH, There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Updated There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. I think the error message is misleading and There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Ah you're right! I set There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. The access requirements are defined here, but it's not super clear. .github/.github/workflows/keep-workflows-enabled.yaml Lines 15 to 18 in 25cc821
Suggestion: summarize with a comment along the lines of "ensure nextstrain-bot has write access to all repos included in the matrix" so in the future we're less likely to see the misleading error message. |
||||||||||
- { repo: zika, workflow: ingest-to-phylogenetic.yaml } | ||||||||||
Comment on lines
+24
to
+49
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. I'd much prefer to query these (e.g. from GitHub's API) rather than hardcode them all here... (but we don't need to do that now if you aren't so inclined) There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Heh, it's a "feature" to allow explicit opt-in of workflows 😉 There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Will revisit when I have the appetite: #119 |
||||||||||
|
||||||||||
runs-on: ubuntu-latest | ||||||||||
steps: | ||||||||||
- run: | | ||||||||||
gh api \ | ||||||||||
--method PUT \ | ||||||||||
-H "Accept: application/vnd.github+json" \ | ||||||||||
-H "X-GitHub-Api-Version: 2022-11-28" \ | ||||||||||
/repos/nextstrain/${{matrix.repo}}/actions/workflows/${{matrix.workflow}}/enable |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.