-
Notifications
You must be signed in to change notification settings - Fork 4.9k
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
Document supported versions #1450
Comments
For apache I would suggest:
Tested with 2.4.20 @radoondas Does that make sense? Will it work with the 2.2.x versions? Logic behind this is supporting the releases from the last 6 month and the last version of the 2.2. release as I think it is still quite popular. |
For mysql I would suggest:
Tested with 5.7.10 I never tested with 5.5 and 5.6. I will do some manual testing to see if behaves the same. For MySQL the question that probably pops up is if it also works with MariaDB. I assume it does but I didn't test it so far. Should we mentioned MariaDB? |
@ruflin Apache 2.2 is still wildly used and it will take a long time for 2.4 to be the most used. The cycle between major releases is very long.
But we can also take the latest versions of apache as is now at the apache.org and define it always to the latest ones as you suggested. I noticed that there are new fields in 2.4 and most of the is already implemented. I do review metrics time to time to see if there are changes. Main fields don't change and stay as now. |
@radoondas Thanks for the details. There is a difference between "supported" and "should work". The version we mention above mean also that in case of a bug or crash, we fix it. For the others, it probably works, but we will not necessary fix all issues. That is why I try to keep the "official" supported list smaller the then list of actual versions it works with. You mentioned that there are more fields in 2.4. As we have these fields implemented, could it be that it breaks the 2.2 implementation? |
For redis Metricbeat I suggest:
Tested with Redis 3.0.7. This is probably the same for the output? @urso ? |
@ruflin No, it will not crash 2.2 implementation. Have tested it a lot. |
@radoondas Nice, we should probably put this on top of the |
For the output versions, I suggest:
@urso Does the above sound right? |
versions look good. For redis we might even support redis 1.0.0 . In redis 2.4 |
Lets only list redis 3.2, if it works with previous version that is great, but as long as we don't run the test on it, I would not list it. |
Closing it as the PR was merged. |
Supported versions for the outputs:
Supported versions for the Metricbeat modules:
The text was updated successfully, but these errors were encountered: