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

Possiblility to configure logging for domain and adminserver #305

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

Conversation

rmestrum
Copy link
Contributor

@rmestrum rmestrum commented Feb 8, 2016

It is now possible to configure the following logging:

  • Domain logging
  • Adminserver logging
  • Adminserver http logging

- Domain logging
- Adminserver logging
- Adminserver http logging
@biemond
Copy link
Owner

biemond commented Feb 8, 2016

Hi Robert,

why don't you use wls_server and wls_domain types because with these types you can already set almost all the log parameters, if I allow this then the domain.pp will be massive and is probably never finished.

@rmestrum
Copy link
Contributor Author

rmestrum commented Feb 9, 2016

In order to use the customtypes wls_server and wls_domain the adminserver must be started.

When the domain is created and the adminserver is started for the first time the logging settings
do not meet the preferred settings.

When the log settings of the adminserver and domain are changed using the customtype wls_server and wls_domain the adminserver must be restarted in order to use the new settings.
Now there's a possibility that the changes you make using the customtypes are not logged using the right setting you prefer.

My opinion is that the logsetting must part of the domain creation because you want every change you made logged in the logfile that's in the correct place with the right setting.

@biemond
Copy link
Owner

biemond commented Feb 9, 2016

I like the webserver part because then you have all the logs on the right location from the beginning that is most important thing in this part but not all the log detail properties.

The rest should be defined by the wls types and subscribe to these changes and it will automatically restart the adminserver. So domain.pp should just do the minimal and wls types are for the full configuration.

@rmestrum
Copy link
Contributor Author

rmestrum commented Feb 9, 2016

Oke. I can live with that. I'll make the necessary changes

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