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

Change warnings to info or debug #1785

Merged
merged 4 commits into from
Jan 11, 2024
Merged

Conversation

cherylking
Copy link
Member

Fixes #1782

@cherylking
Copy link
Member Author

cherylking commented Jan 11, 2024

Unrelated to my changes, a test case started failing on Linux (it is not run on Windows). I spent much of Wednesday looking into this test failure.

So I found that if I ran the failing test locally by itself (MultiModuleRunInstallEarTest in dev-it), I could recreate the problem. Basically, the test expects the ear to have been installed in .m2/repository. However, the run goal does not install it. The test was erroneously depending on some other part of dev-it that installs the ear. If I run all of dev-it locally, the test passes.

So I modified the test to handle the case where the ear is not already installed in .m2/repository.

@cherylking cherylking merged commit 4ee535e into OpenLiberty:main Jan 11, 2024
12 checks passed
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.

Maven plugin prints extraneous warnings
2 participants