Skip to content

Conversation

mashhurs
Copy link
Collaborator

@mashhurs mashhurs commented Dec 14, 2024

Description

CI failures

8.17.1-SNAPSHOT is not available, related jobs will fail.

mashhurs and others added 2 commits December 13, 2024 16:43
…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)
(cherry picked from commit 87c16ee)
@mashhurs mashhurs force-pushed the backport-200-to-8.16 branch from 3eb8ac1 to 42fe41a Compare December 14, 2024 01:24
@elasticmachine
Copy link
Collaborator

elasticmachine commented Dec 14, 2024

💛 Build succeeded, but was flaky

Failed CI Steps

History

@mashhurs mashhurs requested a review from donoghuc December 14, 2024 01:45
@mashhurs mashhurs changed the title Backport 200 to 8.16 Backport PR-200 to 8.16 Dec 14, 2024
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.

Looks good!

@mashhurs mashhurs merged commit 65916db into elastic:8.16 Dec 16, 2024
2 checks passed
@mashhurs mashhurs deleted the backport-200-to-8.16 branch December 16, 2024 17:25
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