Skip to content

Makes experimental logstash package work for 8.0.0 #2697

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

Merged
merged 1 commit into from
Feb 15, 2022

Conversation

jasonrhodes
Copy link
Member

This is required because for those who have installed it, upgrading to 8.0.0 stack can cause critical issues. There are no changes to the package itself beyond declaring that this package, as-is, is also compatible with ^8.0.0

@@ -1,4 +1,9 @@
# newer versions go on top
- version: "1.2.0"
Copy link
Contributor

Choose a reason for hiding this comment

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

this should probably be 1.1.0 to avoid skipping a minor

Copy link
Member Author

Choose a reason for hiding this comment

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

lol doh, that's what I meant thanks

@elasticmachine
Copy link

elasticmachine commented Feb 14, 2022

💚 Build Succeeded

the below badges are clickable and redirect to their specific view in the CI or DOCS
Pipeline View Test View Changes Artifacts preview preview

Expand to view the summary

Build stats

  • Start Time: 2022-02-14T23:18:24.978+0000

  • Duration: 22 min 57 sec

Test stats 🧪

Test Results
Failed 0
Passed 15
Skipped 0
Total 15

🤖 GitHub comments

To re-run your PR in the CI, just comment with:

  • /test : Re-trigger the build.

This is required because for those who have installed it, upgrading to 8.0.0 stack can cause critical issues. There are no changes to the package itself beyond declaring that this package, as-is, is also compatible with ^8.0.0

Also updates changelog for logstash v8 fix
@jasonrhodes
Copy link
Member Author

I fixed the version to be 1.1.0 in both places and rebased the 3 commits into 1 so we can pretend the wrong version thing never happened ... ;)

@jasonrhodes jasonrhodes merged commit 0c5b10e into main Feb 15, 2022
@hendry-lim
Copy link

Would it be possible to have the same for elasticsearch and kibana integrations please?

@jasonrhodes
Copy link
Member Author

@hendry-lim yes, once I see how this process works with one package, I will ask my team to do the same for the rest of the Stack Monitorong packages.

@jen-huang jen-huang deleted the update-logstash-8.0 branch February 15, 2022 18:56
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.

4 participants