Skip to content

Conversation

mashhurs
Copy link
Collaborator

@mashhurs mashhurs commented Dec 12, 2024

Description

This PR backports #200 (commit ID: 87c16ee1de04a8db5c251dbff400349939aaee32) into 8.17 and sets ES treeish to 8.17 after release.

CI failures

  • 8.17.1-SNAPSHOTs are not available yet, so this version jobs fail. However, since 8.17.0 just released, non-snapshot CIs are 🟢 , so it is safe to land the change.

mashhurs and others added 2 commits December 12, 2024 14:01
…t branch. (elastic#200)

* Sync up with ES main on `PipelineConfiguration` class changes. (elastic#190)

* Sync up with ES main that PipelineConfiguration class getConfigAsMap() method renamed to getConfig().

* Request modifiable donfig when creating a pipeline.

* Fixes PR CI jobs, adds build and E2E CI jobs.

* Fix the E2E test jobs.

* Generate CI steps based on defined test matrix.

* Apply suggestions from code review

- print target branch for debugging purpose
- add return type `None` to void method.

Co-authored-by: Cas Donoghue <[email protected]>

* Replace the matrix with origin URL (after landing).

---------

Co-authored-by: Cas Donoghue <[email protected]>
(cherry picked from commit 8e4c02a)
@elasticmachine
Copy link
Collaborator

elasticmachine commented Dec 12, 2024

@mashhurs mashhurs requested a review from donoghuc December 12, 2024 23:05
Copy link
Member

@donoghuc donoghuc left a comment

Choose a reason for hiding this comment

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

Backport looks clean. Only failures appear to be due to 8.17.1-SNAPSHOT builds not being available yet.

@mashhurs mashhurs merged commit 2cc7d7f into elastic:8.17 Dec 13, 2024
1 of 2 checks passed
@mashhurs mashhurs deleted the backport_199-estreeish-update branch December 13, 2024 19:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants