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

Maven plugin prints extraneous warnings #1782

Closed
lprimak opened this issue Dec 20, 2023 · 2 comments · Fixed by #1785
Closed

Maven plugin prints extraneous warnings #1782

lprimak opened this issue Dec 20, 2023 · 2 comments · Fixed by #1785
Assignees

Comments

@lprimak
Copy link

lprimak commented Dec 20, 2023

Hi, I have the following extraneous warning:

[WARNING] /Users/lprimak/dev/shiro/integration-tests/jaxrs/openliberty/target/liberty/wlp/usr/servers/shiro-its-jaxrs-openliberty/configDropins/overrides/liberty-plugin-variable-config.xml file deleted before processing plugin configuration.

Is there any way to shut it off?

Here is the pom: https://github.com/apache/shiro/blob/main/integration-tests/jaxrs/openliberty/pom.xml

Thank you

@scottkurz
Copy link
Member

scottkurz commented Dec 21, 2023

Hmm... I see your point. I don't see a way to shut this off currently.

Assuming we want to continue issuing this message, another idea would be downgrading it from WARNING to INFO message.

I'm realizing we issue the same WARNING for overwriting the jvm.options and bootstrap.properties files. I think in those cases the message is more important since it's more likely those files could have be written/copied by other means. But these configDropins are only coming from our plugin, so the message seems less important there.

We'll have to think about this. Thanks for the suggestion.

@lprimak
Copy link
Author

lprimak commented Dec 21, 2023

Thanks Scott,

I was able to work around this currently by using -Dorg.slf4j.simpleLogger.log.io.openliberty.tools.maven=error

However, I think this message should be downgraded to DEBUG level

Since the plugin is invoked in multiple executions during the maven integration test phase, this warning would still occur on the second and third pass if not quieted.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Done
Development

Successfully merging a pull request may close this issue.

3 participants