Skip to content

Add new workflow for build/CD on PRs#1482

Merged
AndreasBurger merged 4 commits intomasterfrom
trust-me-bro
Sep 18, 2025
Merged

Add new workflow for build/CD on PRs#1482
AndreasBurger merged 4 commits intomasterfrom
trust-me-bro

Conversation

@AndreasBurger
Copy link
Copy Markdown
Member

also add trust-helper, a workflow that auto-labels PRs based on team-membership of the creator (as was set up via WHD previously)

also add trust-helper, a workflow that auto-labels PRs based
on team-membership of the creator (as was set up via
WHD previously)
@gardener-robot gardener-robot added needs/review Needs review size/S Denotes a PR that changes 10-29 lines, ignoring generated files. labels Sep 18, 2025
@gardener-robot-ci-1 gardener-robot-ci-1 added the reviewed/ok-to-test Has approval for testing (check PR in detail before setting this label because PR is run on CI/CD) label Sep 18, 2025
@github-actions github-actions Bot added needs/ok-to-test Needs approval for testing (check PR in detail before setting this label because PR is run on CI/CD) and removed reviewed/ok-to-test Has approval for testing (check PR in detail before setting this label because PR is run on CI/CD) labels Sep 18, 2025
@AndreasBurger AndreasBurger marked this pull request as ready for review September 18, 2025 12:46
@AndreasBurger AndreasBurger requested a review from a team as a code owner September 18, 2025 12:46
Copy link
Copy Markdown
Contributor

@kon-angelo kon-angelo left a comment

Choose a reason for hiding this comment

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

I think we have a "competing" script for the head-update that can probably be coalesced in this one

@@ -0,0 +1,36 @@
name: Build
on:
pull_request:
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Can we also run build on master and release branches push ?

@gardener-robot gardener-robot added the needs/changes Needs (more) changes label Sep 18, 2025
@gardener-robot-ci-2 gardener-robot-ci-2 added the reviewed/ok-to-test Has approval for testing (check PR in detail before setting this label because PR is run on CI/CD) label Sep 18, 2025
@gardener-robot gardener-robot added the size/XS Denotes a PR that changes 0-9 lines, ignoring generated files. label Sep 18, 2025
@github-actions github-actions Bot removed the reviewed/ok-to-test Has approval for testing (check PR in detail before setting this label because PR is run on CI/CD) label Sep 18, 2025
@gardener-robot gardener-robot removed the size/S Denotes a PR that changes 10-29 lines, ignoring generated files. label Sep 18, 2025
@gardener-robot-ci-2 gardener-robot-ci-2 added the reviewed/ok-to-test Has approval for testing (check PR in detail before setting this label because PR is run on CI/CD) label Sep 18, 2025
@github-actions github-actions Bot removed the reviewed/ok-to-test Has approval for testing (check PR in detail before setting this label because PR is run on CI/CD) label Sep 18, 2025
@github-actions
Copy link
Copy Markdown
Contributor

The overall change involves updates to GitHub workflows concerning automation of build processes and managing pull request trust. The modifications optimize workflow execution by adjusting triggers and permissions, while introducing a new helper workflow to manage trust for pull requests, thereby potentially streamlining the CI process and enhancing the security checks around pull requests.

Walkthrough

  • Refactor: Transition of the trigger from push to merge_group for build workflow, indicating a focus on optimizing CI triggers for specific branches.
  • New Feature: Introduction of pullrequest-trust-helper.yaml, which configures access controls for pull requests, ensuring automated trust checks based on defined teams, improving security and collaboration.
  • Chore: Reorganization and renaming of workflow configuration files to enhance clarity and maintainability.

Model: gpt-4o | Prompt Tokens: 464 | Completion Tokens: 157

@gardener-robot-ci-1 gardener-robot-ci-1 added the reviewed/ok-to-test Has approval for testing (check PR in detail before setting this label because PR is run on CI/CD) label Sep 18, 2025
@github-actions github-actions Bot removed the reviewed/ok-to-test Has approval for testing (check PR in detail before setting this label because PR is run on CI/CD) label Sep 18, 2025
Copy link
Copy Markdown
Contributor

@kon-angelo kon-angelo left a comment

Choose a reason for hiding this comment

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

/lgtm

@gardener-robot gardener-robot added reviewed/lgtm Has approval for merging and removed needs/changes Needs (more) changes needs/review Needs review labels Sep 18, 2025
@gardener-robot-ci-2 gardener-robot-ci-2 added the reviewed/ok-to-test Has approval for testing (check PR in detail before setting this label because PR is run on CI/CD) label Sep 18, 2025
@github-actions github-actions Bot removed the reviewed/ok-to-test Has approval for testing (check PR in detail before setting this label because PR is run on CI/CD) label Sep 18, 2025
@AndreasBurger AndreasBurger merged commit dd68f62 into master Sep 18, 2025
48 checks passed
@AndreasBurger AndreasBurger deleted the trust-me-bro branch September 18, 2025 14:39
@gardener-robot gardener-robot added the status/closed Issue is closed (either delivered or triaged) label Sep 18, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

needs/ok-to-test Needs approval for testing (check PR in detail before setting this label because PR is run on CI/CD) reviewed/lgtm Has approval for merging size/XS Denotes a PR that changes 0-9 lines, ignoring generated files. status/closed Issue is closed (either delivered or triaged)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants