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

fix: logging percents to cloudwatch #587

Closed

Conversation

vkrizan
Copy link
Contributor

@vkrizan vkrizan commented Oct 19, 2023

@vkrizan vkrizan requested a review from petrblaho October 19, 2023 08:46
@vkrizan
Copy link
Contributor Author

vkrizan commented Oct 19, 2023

@gwenneg @josejulio can you please help with the following error?

[ERROR] Failed to execute goal io.quarkus:quarkus-maven-plugin:2.13.3.Final:build (default) on project app: Execution default of goal io.quarkus:quarkus-maven-plugin:2.13.3.Final:build failed: A required class was missing while executing io.quarkus:quarkus-maven-plugin:2.13.3.Final:build: jakarta/enterprise/inject/spi/DeploymentException

Could it be the Quarkus platform version mismatch? If so, how can we still upgrade cloudwatch package?

Thank you.

@gwenneg
Copy link
Member

gwenneg commented Oct 19, 2023

I think you'll need to bump Quarkus to 3.x in this project if you want to use the latest quarkus-logging-cloudwatch releases.

Quarkus 2.x relied on javax. packages which were "recently" replaced with jakarta. packages in Quarkus 3 (that one of the major breaking changes of Quarkus 3) and the error message looks related to that.

@vkrizan
Copy link
Contributor Author

vkrizan commented Oct 19, 2023

@gwenneg we are planning to do that. Is there a way to overcome it? Maybe backporting the fix to version 5 of cloudwatch logging?

@vkrizan vkrizan marked this pull request as draft December 13, 2023 13:18
@vkrizan
Copy link
Contributor Author

vkrizan commented Jan 9, 2024

Superseded by #602

@vkrizan vkrizan closed this Jan 9, 2024
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