it is much clear now, thank you very much, Jesse! it is true that most plugins such as timestamper-plugin,mask-passwords-plugin, logstash-plugin used build wrapper to decorate logger, and I am a little surprised that I was the first one to "abuse" the global ConsoleLogFilter <https://github.com/jenkinsci/splunk-devops-plugin/blob/master/splunk-devops/src/main/java/com/splunk/splunkjenkins/TeeConsoleLogFilter.java#L31> feature. Thanks, I will add a simple wrapper for WorkFlowRun job
On Wednesday, November 23, 2016 at 12:00:12 AM UTC+8, Jesse Glick wrote: > > On Tue, Nov 22, 2016 at 4:29 PM, Ted Xiao <[email protected] <javascript:>> > wrote: > > through the PR > > > https://github.com/jenkinsci/workflow-job-plugin/pull/27/files#diff-5ac278e2951f186bb7e2f76208e6ed05R76 > > > , there is a comment // TODO JENKINS-30777 decorate with > > ConsoleLogFilter.all() > > Yes—if and when a use case for applying a global > (`@Extension`-registered) `ConsoleLogFilter` arises. AFAIK there are > none; under JENKINS-38381, sending log results to an external system > uses a different API. (`ConsoleLogFilter` is involved in the > implementation, but these are locally created filters, not global > extensions.) See the downstream > https://github.com/jenkinsci/logstash-plugin/pull/18 for a (crude) > example. > > Please note that development on this system is currently on hold. It > is functionally complete and has basic test coverage, but is not > production ready. I have not yet written up a friendly design guide. > > > Does that mean I need to use explicit declare such as > > > > withContext(new MyConsoleLogFilter()) { > > sh 'process' > > } > > No, it is transparent to job definitions. > -- You received this message because you are subscribed to the Google Groups "Jenkins Developers" group. To unsubscribe from this group and stop receiving emails from it, send an email to [email protected]. To view this discussion on the web visit https://groups.google.com/d/msgid/jenkinsci-dev/b13dd559-9515-46d8-bb66-0e9962d9ceab%40googlegroups.com. For more options, visit https://groups.google.com/d/optout.
