Skip to content

Send GH action trigger to EESSI/software_layer whenever EESSI/software-layer-scripts' main changes - #286

Closed
casparvl wants to merge 3 commits into
EESSI:mainfrom
casparvl:trigger_auto_update_software_layer_scripts_commit
Closed

casparvl wants to merge 3 commits into
EESSI:mainfrom
casparvl:trigger_auto_update_software_layer_scripts_commit

Conversation

@casparvl

Copy link
Copy Markdown
Contributor

Whenever the tip of EESSI/software-layer-scripts' main branch changes, this GH action is started. The only thing this GH action does is send a trigger to a GH action in EESSI/software-layer (added in EESSI/software-layer#1568 ), which in turn will update the bot/software_layer_scripts_commit in a feature branch, and create a PR.

This effectively provides an auto-update mechanism for bot/software_layer_scripts_commit with human oversight and minimal security impact.

@boegel

boegel commented Oct 28, 2026

Copy link
Copy Markdown
Contributor

@casparvl is this still relevant?

@casparvl

casparvl commented Nov 2, 2026

Copy link
Copy Markdown
Contributor Author

Meh, this would have allowed for event-based updating of the software_layer_scripts_commit has in EESSI/software-layer/bot/software_layer_scripts_commit, which would have been nice to have. Currently, it's updated by CI that runs on a 1-hour schedule.

Honestly, while I think the event-based approach is nicer, it may be a bit overkill here. And it would require a token that allows starting/stopping CI from software-layer. That's not a huge deal from a security perspective, but it's not nothing either. Since the current hourly runs work fine in practice:

  • no one is annoyed if the commit hash is outdated (unless they need some new feature from software-layer-scripts, but then they typically know and update the commit hash manually in their own PR)
  • the biggest latency is probably introduced by us having to merge the auto-generated commit-updates - not the PR being created on the hourly schedule

I'll close this.

@casparvl casparvl closed this Nov 2, 2026
@bedroge

bedroge commented Nov 2, 2026

Copy link
Copy Markdown
Contributor

Also note that you can start this workflow manually at https://github.com/EESSI/software-layer/actions/workflows/update_software_layer_scripts_commit.yml in case you really need it quickly.

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.

3 participants