Skip to content
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

Update MpTelemetry 2.0 for Java 8 Compatibility #7792

Open
yasmin-aumeeruddy opened this issue Jan 15, 2025 · 4 comments
Open

Update MpTelemetry 2.0 for Java 8 Compatibility #7792

yasmin-aumeeruddy opened this issue Jan 15, 2025 · 4 comments
Assignees
Milestone

Comments

@yasmin-aumeeruddy
Copy link
Member

MpTelemetry 2.0 is compatible with Java 8 which is not currently obvious from the documentation.

Pages to update:
https://openliberty.io/docs/latest/reference/diff/mp-61-70-diff.html#tm
https://openliberty.io/docs/latest/reference/feature/mpTelemetry-2.0.html#_supported_java_versions

@yasmin-aumeeruddy yasmin-aumeeruddy self-assigned this Jan 15, 2025
@dmuelle
Copy link
Member

dmuelle commented Jan 15, 2025

Hi Yasmin- The "Supported Java versions" section of the MPTelemetry 2.0 feature page is autogenerated from the feature metatype, so the fix for that will have to come from the development side. I noticed the MPT 1.0 and 1.1 also claim the same Java levels (11+).

For the differences page- is Java 8 support new with version 2.0?

@yasmin-aumeeruddy
Copy link
Member Author

yasmin-aumeeruddy commented Jan 16, 2025

@dmuelle It is new with MpTelemtry 2.0 as versions 1.1 and 1.0 were not Java 8 compatible. MicroProfile 7.0, 6.1 and 6.0 are not Java 8 compatible though so the feature is only compatible on Java 8 with MicroProfie 5.0 (EE9), MicroProfile 4.1 (EE8) and MicroProfile 1.4 (EE7) . This is why it's not listed in the feature metatype. I'm not yet sure how we can make that clear in the differences page.

@dmuelle dmuelle added this to the 25.0.0.1 milestone Jan 16, 2025
@dmuelle dmuelle self-assigned this Jan 16, 2025
dmuelle added a commit that referenced this issue Jan 16, 2025
This was referenced Jan 16, 2025
dmuelle added a commit that referenced this issue Jan 16, 2025
@yasmin-aumeeruddy
Copy link
Member Author

@dmuelle That looks good and works as described. Thank you!

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

No branches or pull requests

2 participants