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

Sync public forks for NCIPangea #10

Merged
merged 3 commits into from
Jan 2, 2024
Merged

Sync public forks for NCIPangea #10

merged 3 commits into from
Jan 2, 2024

Conversation

kelly-sovacool
Copy link
Member

@kelly-sovacool kelly-sovacool commented Jan 2, 2024

A new action to supersede the individual sync-fork actions we have in place for XAVIER and RENEE. This uses a matrix to run gh repo sync on all public forks we specify in this workflow's matrix.

@kopardev: I think you will need to go into this repo's settings and update the SYNC_FORK_TOKEN secret with a new token with repo & workflow scopes. This token will need to be configured to work for CCBR, NCIPangea, and any other organizations that will have forks synced automatically. I don't think my token will work for NCIPangea because the option didn't come up when I tried to configure SSO after creating the token. Edit: actually it looks like my token worked for NCIPangea! I'm unsure if it'll work for the ABCS org, but we can cross that bridge when we get there. I believe the creator of the token must have write access to all forks for it to work.

Resolves https://github.com/CCBR/kelly_log/issues/16

kelly-sovacool added a commit to CCBR/RENEE that referenced this pull request Jan 2, 2024
kelly-sovacool added a commit to CCBR/XAVIER that referenced this pull request Jan 2, 2024
not needed because forks can be manually updated with a button click from the fork's main page already
@kopardev
Copy link
Member

kopardev commented Jan 2, 2024

@kelly-sovacool ... can we also add new PRs for NCIPangea/RENEE and NCIPangea/XAVIER to remove their gh-actions which are getting replaced.

@kopardev kopardev merged commit d032b48 into main Jan 2, 2024
2 checks passed
@kopardev kopardev deleted the sync-forks branch January 2, 2024 19:09
@kopardev
Copy link
Member

kopardev commented Jan 2, 2024

@kelly-sovacool ... can you change the firing time to early morning ... I think you used 11:47 for testing purposes.

@kelly-sovacool
Copy link
Member Author

Yep can do! I picked a kinda random minute because they recommend not doing the beginning of the hour since many people do that and it overloads the runners.

@kelly-sovacool
Copy link
Member Author

Actually I just remembered these times are in UTC, which is 5 hours ahead of EST, so 11:47 UTC = 06:47 EST. Is this a good time @kopardev?

@kelly-sovacool
Copy link
Member Author

@kelly-sovacool ... can we also add new PRs for NCIPangea/RENEE and NCIPangea/XAVIER to remove their gh-actions which are getting replaced.

I already opened them:

(remember all changes should occur in the upstream repo)

Copy link
Member

kopardev commented Jan 2, 2024

Did not realize the UTC part... leave it as-is :)

Copy link
Member

kopardev commented Jan 2, 2024

Makes sense! Good to know.

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

Successfully merging this pull request may close these issues.

2 participants