-
Notifications
You must be signed in to change notification settings - Fork 76
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
add an option to enable jetty jmx #47
Conversation
IIUC it adds JMX Jetty when CLI option is added. It would be nice for sure. @olamy It would be great to have some user-facing documentation in README, e.g.:
If there are any related tickets in the bugtracker, it would be great to have them as well. Do you also plan to modify Extras Executable WAR or Jenkins codebases to enable/pre-configure the feature in Jenkins? Or is it a patch for another project using Winstone? Both are fine. @jenkinsci/code-reviewers FYI |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
needs documentation
@olamy Hi, any plans to finalize it soon? If no, I would propose to ship the current master branch state so that the new Jetty version gets adopted in weekly releases early |
Signed-off-by: olivier lamy <[email protected]>
Signed-off-by: olivier lamy <[email protected]>
@oleg-nenashev just adding a changelog entry and update documentation here https://wiki.jenkins.io/display/JENKINS/Starting+and+Accessing+Jenkins |
Signed-off-by: olivier lamy <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍
Signed-off-by: olivier lamy [email protected]