Skip to content

AMBW-47325 [[Regression][Maven] Maven Goal parameters shows extra log… #749

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

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

lshivara-tibco
Copy link
Collaborator

@lshivara-tibco lshivara-tibco commented Apr 8, 2025

****What's this Pull request about?
Executiong of bwdesign with import, validate, gen_diagrams, generate_maifest_json commands were
generating extra logs compared to earlier releases.
The logs were generated by different eclipse internal plugins, which are not required in console.
The requirement is to maintain logs as in 6.8 release and remove or filterout the unwanted logs from console.

****Which Issue(s) this Pull Request will fix?
This will remove excess logs being introdueced for bwdesign commands import, validate, gen_diagrams, generate_maifest_json commands.
Maintain the logs output for the commands mentioned above to 6.80 release.

****Does this pull request maintain backward compatibility?
Yes, this is to maintain the log generation to 6.80 release.

****How this pull request has been tested?
The logs are generated by various external plugins. A system logger is added, which is used in the mojo class.
The logger will filter out logs generated by external plugins and also many of the parameters passed to the
bwdesing commands were also printed to console, cluttering the console. These are also filtered out.
Followed the test instructions given in the JIRA ticket.
Executed the commands mentioned in the ticket and compared the log generated to reference 6.80 logs.

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.

1 participant