Skip to content

Make the get-latest-components workflow conditional and authenticated#723

Open
kishansinghifs1 wants to merge 1 commit intoprecice:developfrom
kishansinghifs1:fix-api-github-clean
Open

Make the get-latest-components workflow conditional and authenticated#723
kishansinghifs1 wants to merge 1 commit intoprecice:developfrom
kishansinghifs1:fix-api-github-clean

Conversation

@kishansinghifs1
Copy link

Fix #682

  1. Add token: ${{ secrets.GITHUB_TOKEN }} to all nmbgeek/github-action-get-latest-commit steps to prevent GitHub API rate limit failures on shared runner IPs.
  2. Gate the gather-refs job in system-tests-pr.yml with the trigger-system-tests label check so the workflow skips entirely for unlabeled PRs.
  3. Both system-tests-latest-components.yml and system-tests-pr.yml are updated.

Checklist:

  • I added a summary of any user-facing changes (compared to the last release) in the changelog-entries/<PRnumber>.md.
  • I will remember to squash-and-merge, providing a useful summary of the changes of this PR.

@MakisH MakisH added the GSoC Contributed in the context of the Google Summer of Code label Feb 25, 2026
@precice-bot
Copy link
Collaborator

This pull request has been mentioned on preCICE Forum on Discourse. There might be relevant details there:

https://precice.discourse.group/t/gsoc-2026-kishan-singh-system-tests-improvements/2775/1

@MakisH MakisH changed the title Fix #682 : Fix api GitHub Make the get-latest-components workflow conditional and authenticated Feb 27, 2026
Copy link
Member

@MakisH MakisH left a comment

Choose a reason for hiding this comment

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

Thank you for the contribution.

The conditional check already worked in this PR.

I am not sure if passing the GITHUB_TOKEN would be the right approach here. In #682, the suggestion was for an app id, which we would need to set up.

Let's split this into two. Please remove the authentication aspects, and we see if that's enough. I think it will be.

Btw, I renamed the PR, please rename again after removing the authentication aspects.

@kishansinghifs1
Copy link
Author

Hiii @MakisH I was initially thinking of the same but now as you clarified it would be more helpful to me.I'll apply those changes and let you know soon. Thanks for the review!!!!!!!!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

GSoC Contributed in the context of the Google Summer of Code systemtests

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Latest components workflow hits API usage limits

3 participants