Skip to content

[Chore][Regression PR] Reuse Kibana regression PR - #10096

Merged
weronikaolejniczak merged 4 commits into
elastic:mainfrom
weronikaolejniczak:chore/reuse-kibana-regression-pr
Oct 1, 2026
Merged

weronikaolejniczak merged 4 commits into
elastic:mainfrom
weronikaolejniczak:chore/reuse-kibana-regression-pr

Conversation

@weronikaolejniczak

Copy link
Copy Markdown
Contributor

Summary

Adding the ci:regression-integration-test-kibana label again on the same EUI PR now updates the existing Kibana PR instead of opening a new one each time. This works the same way as the nightly.

Changes

  • Runs are triggered from an EUI PR push to update-dependencies/pr-<N>, force-pushed from latest Kibana main. Nightly (nightly/update-dependencies) and manual runs (update-dependencies/<timestamp>) are unchanged.
  • The open-PR workflow now looks for an existing open PR on every head branch, not only nightly. If it finds one, it updates the title and body and converts it back to draft.
  • release.yml, prepare and open-PR runs for the same EUI PR queue on the PR number. They go through the chain in trigger order, so an older snapshot can't force-push over a newer one.
  • For runs triggered from an EUI PR, a newer open-PR run cancels the older one while it waits on Kibana CI. The EUI PR comment always shows the result for the latest commits.

QA

Not testable now.

After the PR is merged:

  • add the label to an EUI PR - a Kibana draft PR opens from elastic/eui-kibana:update-dependencies/pr-<N>,
  • remove the label and add it again - the same Kibana PR gets new commits and the EUI PR comment updates,
  • manually dispatch update_kibana_dependencies.yml - it still opens a new PR,
  • the nightly still reuses nightly/update-dependencies.

@weronikaolejniczak weronikaolejniczak self-assigned this Sep 29, 2026
@weronikaolejniczak
weronikaolejniczak requested a review from a team as a code owner September 29, 2026 10:03
Copilot AI balanced review requested due to automatic review settings September 29, 2026 10:03

Copilot AI left a comment

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.

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@mgadewoll
mgadewoll self-requested a review September 30, 2026 06:52
concurrency:
group: ${{ inputs.nightly && 'kibana-nightly' || format('open-kibana-pr-{0}', github.run_id) }}
cancel-in-progress: false
group: ${{ inputs.nightly && 'kibana-nightly' || format('open-kibana-pr-{0}', inputs.source_pr_number && format('pr-{0}', inputs.source_pr_number) || github.run_id) }}

@mgadewoll mgadewoll Sep 30, 2026 •

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.

format('pr-{0}', source_pr_number) seems to produce duplication: pr-pr-N because of the outer prefix open-kibana-pr- . Maybe we should use open-kibana-{0} instead?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

I had cursor remove the prefix, good catch 馃憤馃徎 0e6ade7

@eokoneyo eokoneyo left a comment

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.

changes LGTM, didn't test locally

@weronikaolejniczak
weronikaolejniczak merged commit 4aa9846 into elastic:main Oct 1, 2026
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants