Skip to content

feat: add a check on the owner of the repo#38

Merged
BaptisteCentreon merged 1 commit intomainfrom
feat/workflows-add-owner-condition
Mar 9, 2026
Merged

feat: add a check on the owner of the repo#38
BaptisteCentreon merged 1 commit intomainfrom
feat/workflows-add-owner-condition

Conversation

@kabreu
Copy link
Contributor

@kabreu kabreu commented Mar 9, 2026

Add a check on the owner of the repo for the reusable workflows.

Previously, if you have a private repository in another organization, it would call the centreon-security runners.
Those runners are only registered under the centreon organization. Meaning it would try to spin up the centreon-security runners in an organization that doesn't have access to them.

This PR adds a check on the repo owner. If the repo owner is not the centreon organization it uses the ubuntu-24.04 runners.

This is useful for this repo in the centreon-website organization :
https://github.com/centreon-website/website
Currently since this repo is private it tries to uses the centreon-security runners but can't and gets stuck waiting.
With this PR it should use the public ubuntu-24.04 runners.

Summary by Aikido

Security Issues: 0 Quality Issues: 0 Resolved Issues: 0

⚡ Enhancements

  • Added repository owner check to choose appropriate GitHub runner

More info

@kabreu kabreu self-assigned this Mar 9, 2026
@kabreu kabreu requested a review from a team as a code owner March 9, 2026 13:35
@kabreu kabreu requested a review from BaptisteCentreon March 9, 2026 13:35
@github-actions
Copy link

github-actions bot commented Mar 9, 2026

Logo
Checkmarx One – Scan Summary & Details609a2c94-5f95-499b-8797-fd7ea3f6fba5

Great job! No new security vulnerabilities introduced in this pull request

@BaptisteCentreon BaptisteCentreon merged commit 905122f into main Mar 9, 2026
5 checks passed
@BaptisteCentreon BaptisteCentreon deleted the feat/workflows-add-owner-condition branch March 9, 2026 13:41
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