Skip to content

Conversation

@donoghuc
Copy link
Contributor

@donoghuc donoghuc commented Aug 6, 2025

Previously logstash versions were tracked in the logstash repo and curled during
CI steps. This commit moves the source of truth to the .ci repo where the
consumer of that information lives. This also changes the format from json to
yaml to allow commenting in the file.

Previously logstash versions were tracked in the logstash repo and curled during
CI steps. This commit moves the source of truth to the `.ci` repo where the
consumer of that information lives. This also changes the format from json to
yaml to allow commenting in the file.
Temporarily include logstash-versions until logstash-plugins/.ci#77
is merged.
@donoghuc
Copy link
Contributor Author

donoghuc commented Aug 6, 2025

logstash-plugins/.ci#79 should fix CI

Copy link
Contributor

@mashhurs mashhurs left a comment

Choose a reason for hiding this comment

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

LGTM!

@donoghuc donoghuc closed this Aug 7, 2025
@donoghuc donoghuc reopened this Aug 7, 2025
Copy link
Contributor

@mashhurs mashhurs left a comment

Choose a reason for hiding this comment

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

LGTM!
ELASTIC_STACK_VERSION=8.current CI failure seems transient, retry should be okay.

@donoghuc
Copy link
Contributor Author

donoghuc commented Aug 7, 2025

yep, took a couple retries but we got there!

@donoghuc donoghuc merged commit 7d3e313 into logstash-plugins:main Aug 7, 2025
3 checks passed
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.

2 participants