Skip to content

github-actions: support id-token for the bk flaky test reporter#49826

Merged
v1v merged 1 commit intomainfrom
v1v-patch-1
Apr 1, 2026
Merged

github-actions: support id-token for the bk flaky test reporter#49826
v1v merged 1 commit intomainfrom
v1v-patch-1

Conversation

@v1v
Copy link
Copy Markdown
Member

@v1v v1v commented Apr 1, 2026

Proposed commit message

Fix permissions and remove the checkout, it's not needed

Checklist

  • My code follows the style guidelines of this project
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation
  • I have made corresponding change to the default configuration files
  • I have added tests that prove my fix is effective or that my feature works. Where relevant, I have used the stresstest.sh script to run them under stress conditions and race detector to verify their stability.
  • I have added an entry in ./changelog/fragments using the changelog tool.

Disruptive User Impact

Author's Checklist

  • [ ]

How to test this PR locally

NA

Only in the CI, see https://github.com/elastic/beats/actions/runs/23837259262/job/69483908576 that produced:

Related issues

Use cases

Screenshots

Logs

@botelastic botelastic bot added the needs_team Indicates that the issue/PR needs a Team:* label label Apr 1, 2026
@botelastic
Copy link
Copy Markdown

botelastic bot commented Apr 1, 2026

This pull request doesn't have a Team:<team> label.

@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Apr 1, 2026

🤖 GitHub comments

Just comment with:

  • run docs-build : Re-trigger the docs validation. (use unformatted text in the comment!)

@v1v v1v added backport-skip Skip notification from the automated backport with mergify skip-changelog labels Apr 1, 2026
@v1v v1v marked this pull request as ready for review April 1, 2026 07:32
@v1v v1v requested a review from a team as a code owner April 1, 2026 07:32
@mergify mergify bot assigned v1v Apr 1, 2026
@mergify
Copy link
Copy Markdown
Contributor

mergify bot commented Apr 1, 2026

This pull request does not have a backport label.
If this is a bug or security fix, could you label this PR @v1v? 🙏.
For such, you'll need to label your PR with:

  • The upcoming major version of the Elastic Stack
  • The upcoming minor version of the Elastic Stack (if you're not pushing a breaking change)

To fixup this pull request, you need to add the backport labels for the needed
branches, such as:

  • backport-8./d is the label to automatically backport to the 8./d branch. /d is the digit
  • backport-active-all is the label that automatically backports to all active branches.
  • backport-active-8 is the label that automatically backports to all active minor branches for the 8 major.
  • backport-active-9 is the label that automatically backports to all active minor branches for the 9 major.

@coderabbitai
Copy link
Copy Markdown

coderabbitai bot commented Apr 1, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: f009ef36-3b34-4a1e-92bf-53bb4d44c65d

📥 Commits

Reviewing files that changed from the base of the PR and between ca355b1 and 257ec8b.

📒 Files selected for processing (1)
  • .github/workflows/buildkite-test-flaky.yml

📝 Walkthrough

Walkthrough

The buildkite-test-flaky workflow job configuration was updated with two changes: the job-level permissions now include id-token: write to support token-based operations, and the actions/checkout@v6 step was removed from the job steps. The remaining steps that handle token creation and flaky test reporting continue unchanged. No additional checkout step was added as a replacement.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch v1v-patch-1
  • 🛠️ Update Documentation: Commit on current branch
  • 🛠️ Update Documentation: Create PR

Comment @coderabbitai help to get the list of available commands and usage tips.

@v1v v1v enabled auto-merge (squash) April 1, 2026 10:42
@v1v v1v merged commit ed27f61 into main Apr 1, 2026
23 of 25 checks passed
@v1v v1v deleted the v1v-patch-1 branch April 1, 2026 10:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport-skip Skip notification from the automated backport with mergify needs_team Indicates that the issue/PR needs a Team:* label skip-changelog

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants