Skip to content

[chore][internal] Add build & read access to buildkite pipelines for members of integration-experience team#18856

Merged
Niceplace merged 5 commits into
mainfrom
chore/buildkite-config-add-team-access
May 7, 2026
Merged

[chore][internal] Add build & read access to buildkite pipelines for members of integration-experience team#18856
Niceplace merged 5 commits into
mainfrom
chore/buildkite-config-add-team-access

Conversation

@Niceplace
Copy link
Copy Markdown
Contributor

Proposed commit message

For members (i.e me) of the integration-experience Github team, pipeline access is limited i.e. read-only, as defined by this general setting. As advised by the platform engineering team, I'm adding build & read permissions to the integration-experience team so we can all read & execute builds.

The main impact here is that if a PR is submitted to a repository we steward, some of our team members would not have access to trigger builds from Buildkite's UI. It would be strictly readonly. This might be helpful in situations where flaky tests occur and we need to re-trigger things (although it would be preferable to fix the problem at the source instead)

Here is the platform engineering documentation reference: https://docs.elastic.dev/ci/getting-started-with-buildkite-at-elastic#2-adjust-your-pipeline-settings, more specifically the Assigning a Pipeline to Teams collapsible section (which I cannot provide a URL to unfortunately)

It's possible we have other repositories impacted by this but I'm seeing if this will fix it here and will open PRs in other repositories we steward afterwards.

How to test this PR locally

N/A

@Niceplace Niceplace requested a review from a team as a code owner May 6, 2026 21:11
@Niceplace Niceplace added the ci label May 6, 2026
Copy link
Copy Markdown
Collaborator

@mrodm mrodm left a comment

Choose a reason for hiding this comment

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

LGTM.
Just a minor comment about inserting this following the sorted order for BUILD_AND_READ permission.
TIA!!

Comment thread catalog-info.yaml Outdated
Comment on lines +108 to +109
integration-experience:
access_level: BUILD_AND_READ
Copy link
Copy Markdown
Collaborator

@mrodm mrodm May 7, 2026

Choose a reason for hiding this comment

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

Nit.
To try to keep all the teams using BUILD_AND_READ sorted, could you move this entry after the logstash team entry? Around line 80.

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.

before logstash probably

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.

@mrodm Thanks for pointing that out, I agree that it's good to keep things tidy. Will fix !

Copy link
Copy Markdown
Contributor

@bhapas bhapas left a comment

Choose a reason for hiding this comment

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

LGTM.. Just resolve the comment from Mario before push.

@Niceplace Niceplace merged commit a6f43a0 into main May 7, 2026
10 checks passed
@Niceplace Niceplace deleted the chore/buildkite-config-add-team-access branch May 7, 2026 13:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants