Skip to content

fix(ci): add missing contents: read permission to integration review jobs#25067

Merged
thomasqueirozb merged 1 commit intomasterfrom
fix/ci-integration-review-permissions
Mar 30, 2026
Merged

fix(ci): add missing contents: read permission to integration review jobs#25067
thomasqueirozb merged 1 commit intomasterfrom
fix/ci-integration-review-permissions

Conversation

@thomasqueirozb
Copy link
Copy Markdown
Contributor

@thomasqueirozb thomasqueirozb commented Mar 30, 2026

Summary

Adds the missing contents: read permission to the build-test-runner and integration test runner jobs in the CI integration review workflow.

Vector configuration

NA

How did you test this PR?

/ci-run-integration-mqtt

Run

Change Type

  • Bug fix
  • New feature
  • Dependencies
  • Non-functional (chore, refactoring, docs)
  • Performance

Is this a breaking change?

  • Yes
  • No

Does this PR include user facing changes?

  • Yes. Please add a changelog fragment based on our guidelines.
  • No. A maintainer will apply the no-changelog label to this PR.

References

@thomasqueirozb thomasqueirozb added the no-changelog Changes in this PR do not need user-facing explanations in the release changelog label Mar 30, 2026
@github-actions github-actions bot added domain: ci Anything related to Vector's CI environment and removed no-changelog Changes in this PR do not need user-facing explanations in the release changelog labels Mar 30, 2026
@thomasqueirozb thomasqueirozb changed the title fix(ci): add missing contents:read permission to integration review jobs fix(ci): add missing contents: read permission to integration review jobs Mar 30, 2026
@thomasqueirozb thomasqueirozb marked this pull request as ready for review March 30, 2026 15:03
@thomasqueirozb thomasqueirozb requested a review from a team as a code owner March 30, 2026 15:03
Copy link
Copy Markdown
Contributor Author

@thomasqueirozb thomasqueirozb left a comment

Choose a reason for hiding this comment

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

/ci-run-integration-mqtt

@thomasqueirozb thomasqueirozb added the no-changelog Changes in this PR do not need user-facing explanations in the release changelog label Mar 30, 2026
@thomasqueirozb thomasqueirozb added this pull request to the merge queue Mar 30, 2026
Merged via the queue into master with commit 55be10e Mar 30, 2026
135 of 137 checks passed
@thomasqueirozb thomasqueirozb deleted the fix/ci-integration-review-permissions branch March 30, 2026 16:15
@github-actions github-actions bot locked and limited conversation to collaborators Mar 30, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

domain: ci Anything related to Vector's CI environment no-changelog Changes in this PR do not need user-facing explanations in the release changelog

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants