Skip to content

modifications for log chatter #160

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

Merged
merged 1 commit into from
Sep 11, 2018
Merged

Conversation

CarolynRountree
Copy link
Contributor

Please do not delete the repository or close the corresponding issue for this pull request until you have read the below:

I changed the logging properties to set the aliases and versions loggers to level FINE. This will reduce the ENTER, EXIT chatter. We can request the end user to change those logger levels if needed. However, the quality of logs in the aliases is not ideal. Informative loggers at the FINE level should be added to the aliases code. I have yet to find the information I need for an alias issue from the entering,exiting code. Entering,Exiting loggers is about all that aliases logs except for those at INFO and above.

If setting the version and alias loggers to level FINE is acceptable, we can close the issue and delete the repository. I suggest we open another issue for quality of logs in aliases if we need to spend our time on this. Else, we should add loggers as we find the need when troubleshooting issues.

I put loggers into the logging.properties for the more import logger names. This will be helpful to the user since the logger name is not included in the log stream. The user does not know the names or where to in the source to find the names.

The WDT logger names I did not include in the logger properties are as follows:
wlsdeploy.encrypt
wlsdeploy.validate
wlsdeploy.tool.util
wlsdeploy.json
wlsdeploy.archive
wlsdeploy.yaml
wlsdeploy.model
wlsdeploy.translator
wlsdeploy.variables

@CarolynRountree CarolynRountree removed the request for review from rakillen September 11, 2018 22:33
@CarolynRountree
Copy link
Contributor Author

Issue #122

@CarolynRountree CarolynRountree merged commit 17ffdcc into master Sep 11, 2018
@CarolynRountree CarolynRountree deleted the Issue#122-reduce-log-chatter branch September 11, 2018 22:35
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