Skip to content

ci: fix pull_request_target add schedule dependabotbot PR#2688

Merged
kubicar merged 3 commits into
mainfrom
kubicar/ci-fix-pull-request-target
Jun 3, 2026
Merged

ci: fix pull_request_target add schedule dependabotbot PR#2688
kubicar merged 3 commits into
mainfrom
kubicar/ci-fix-pull-request-target

Conversation

@kubicar

@kubicar kubicar commented Jun 2, 2026

Copy link
Copy Markdown
Contributor

Description

Changes proposed in this pull request:

  • ...

Pre-Merge Checklist

Consider all the following items. If your contribution violates any of them, or you are not sure about it, add a comment to the PR.

  • The code coverage is acceptable.
  • Release notes for the introduced changes are created.
  • If Kubebuilder changes were made, you ran make manifests and committed the changes before the merge.
  • Pre-existing managed resources are correctly handled.
  • The change works on all hyperscalers supported by SAP BTP, Kyma runtime.
  • There is no upgrade downtime.
  • For infrastructure changes, you checked if the changes affect the hyperscaler's costs.
  • RBAC settings are as restrictive as possible.
  • If any new libraries are added, you verified license compliance and maintainability, and made a comment in the PR with details. We only allow stable releases to be included in the project.
  • You checked if this change should be cherry-picked to active release branches.
  • The configuration does not introduce any additional latency.
  • You checked if Busola updates are needed.

Related issues

Copilot AI review requested due to automatic review settings June 2, 2026 11:53
@kubicar
kubicar requested a review from a team as a code owner June 2, 2026 11:53
@hyperspace-pr-bot

Copy link
Copy Markdown
Contributor

Control Panel

Hi, I'm an AI-powered Review Bot that helps you with summarizing and reviewing pull requests.
To interact with me, just use the following actions:

  • 📝 Summarize PR
  • 🔍 Review
  • 🗑️ Delete all bot comments and reviews

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

This PR removes pull_request_target-based automation for bot PR project triage and replaces it with a scheduled workflow that periodically discovers open bot PRs (Dependabot + kyma-goat-bot) and adds them to a GitHub Project with predefined field values.

Changes:

  • Add a scheduled/manual workflow that queries open bot PRs and adds them to the org Project (including setting Status=Review and Ready=Sprint).
  • Remove the previous pull_request_target workflow that added bot PRs to the Project on PR events.
  • Remove the pull_request_target trigger from the external image sync workflow (now runs only on push to main).

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 6 comments.

File Description
.github/workflows/schedule-add-bot-prs-to-project.yaml New scheduled workflow to find bot PRs, avoid pull_request_target, and add/update GitHub Project items.
.github/workflows/pull-add-bot-prs-to-project.yaml Deletes the pull_request_target workflow that previously added bot PRs on PR events.
.github/workflows/post-sync-external-images.yaml Removes pull_request_target trigger, leaving only a push-to-main trigger for syncing external images.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread .github/workflows/schedule-add-bot-prs-to-project.yaml
Comment thread .github/workflows/schedule-add-bot-prs-to-project.yaml
Comment thread .github/workflows/schedule-add-bot-prs-to-project.yaml
Comment thread .github/workflows/schedule-add-bot-prs-to-project.yaml
Comment thread .github/workflows/schedule-add-bot-prs-to-project.yaml
Comment thread .github/workflows/schedule-add-bot-prs-to-project.yaml
@kubicar
kubicar enabled auto-merge (squash) June 3, 2026 06:43
@kubicar
kubicar merged commit 1a4b7e3 into main Jun 3, 2026
28 checks passed
@kubicar
kubicar deleted the kubicar/ci-fix-pull-request-target branch June 3, 2026 06:44
kubicar added a commit that referenced this pull request Jun 10, 2026
* ci: change pull_request_target to pull_request, don't sync images from fork

* ci: move adding PRs from dependabot to scheduled job

* fix: cr apply suggested changes

(cherry picked from commit 1a4b7e3)
kubicar added a commit that referenced this pull request Jun 10, 2026
* ci: fix pull_request_target, add schedule dependabotbot PR (#2688)

* ci: change pull_request_target to pull_request, don't sync images from fork

* ci: move adding PRs from dependabot to scheduled job

* fix: cr apply suggested changes

(cherry picked from commit 1a4b7e3)

* ci: hardening pipelines (#2697)

* ci: add environments to scope secrets and variables

(cherry picked from commit 41945ae)

* ci: move not secret information to vars from secrets (#2698)

* ci: add environments to scope secrets and variables

* ci: add environments to scope secrets and variables

* ci: move not secret information to vars from secrets (not needed rotation)

(cherry picked from commit 1843359)
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