-
Notifications
You must be signed in to change notification settings - Fork 203
ci: make CI paths co-owned by Elastic Agent team #10048
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
ci: make CI paths co-owned by Elastic Agent team #10048
Conversation
|
Pinging @elastic/elastic-agent-control-plane (Team:Elastic-Agent-Control-Plane) |
|
This pull request does not have a backport label. Could you fix it @andrzej-stencel? 🙏
|
cmacknz
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, agree with Victor's suggestion that we take sole ownership of the update otel script
Co-authored-by: Victor Martinez <[email protected]>
|
@Mergifyio backport 8.18 8.19 9.0 9.1 |
✅ Backports have been created
|
* ci: make CI paths co-owned by Elastic Agent team * Apply suggestion from @v1v Co-authored-by: Victor Martinez <[email protected]> --------- Co-authored-by: Victor Martinez <[email protected]> (cherry picked from commit d09eb4c)
* ci: make CI paths co-owned by Elastic Agent team * Apply suggestion from @v1v Co-authored-by: Victor Martinez <[email protected]> --------- Co-authored-by: Victor Martinez <[email protected]> (cherry picked from commit d09eb4c)
* ci: make CI paths co-owned by Elastic Agent team * Apply suggestion from @v1v Co-authored-by: Victor Martinez <[email protected]> --------- Co-authored-by: Victor Martinez <[email protected]> (cherry picked from commit d09eb4c)
* ci: make CI paths co-owned by Elastic Agent team * Apply suggestion from @v1v Co-authored-by: Victor Martinez <[email protected]> --------- Co-authored-by: Victor Martinez <[email protected]> (cherry picked from commit d09eb4c)
* ci: make CI paths co-owned by Elastic Agent team * Apply suggestion from @v1v --------- (cherry picked from commit d09eb4c) Co-authored-by: Andrzej Stencel <[email protected]> Co-authored-by: Victor Martinez <[email protected]>
* ci: make CI paths co-owned by Elastic Agent team * Apply suggestion from @v1v --------- (cherry picked from commit d09eb4c) Co-authored-by: Andrzej Stencel <[email protected]> Co-authored-by: Victor Martinez <[email protected]>
* ci: make CI paths co-owned by Elastic Agent team * Apply suggestion from @v1v --------- (cherry picked from commit d09eb4c) Co-authored-by: Andrzej Stencel <[email protected]> Co-authored-by: Victor Martinez <[email protected]>
* ci: make CI paths co-owned by Elastic Agent team * Apply suggestion from @v1v --------- (cherry picked from commit d09eb4c) Co-authored-by: Andrzej Stencel <[email protected]> Co-authored-by: Victor Martinez <[email protected]>
* ci: make CI paths co-owned by Elastic Agent team * Apply suggestion from @v1v Co-authored-by: Victor Martinez <[email protected]> --------- Co-authored-by: Victor Martinez <[email protected]>
Adds the Elastic Agent Control Plane team as a code owner of CI directories like
.ci/and.buildkitebeside the ObservaBLT CI team.This should prevent pull requests that touch files in those directories from being blocked on the ObservaBLT CI team (example). The ObservaBLT CI team's help in reviewing these pull requests is still valuable and sometimes indispensable. Ultimately though, in my view the responsibility on the CI in this repository falls on the Elastic Agent Control Plane team, so it should be owned by them.
Happy to hear other's opinions on this and close this PR if the discussion results in another approach being more desirable.