Wednesday, 2017-09-27

waynrthat sounds like it could be similar to the problem Odd_Bloke reported00:00
zxiiroi think it also breaks using macros inside of other macros too00:01
waynrhmm no tests for macros inside of macros?00:02
zxiiroI guess not. but I use it extensively in my project00:02
zxiirolet me see if i can come up with a minimal test or something00:02
zxiirohttps://www.irccloud.com/pastebin/MKtJl60r/00:07
zxiirowaynr: ^ ok confirmed it's actually when you use a macro inside of a publisher that calls the postbuildscript. And when you call a macro from a maven project's prebuild and postbuild steps00:08
zxiirothey may be all the same situation....00:08
zxiiroI guess calling a macro from a different section or something.00:09
zxiirowaynr: I think we should revert? unless you know a solution? since this seems to break jjb in a pretty major way imo.00:10
waynroh i do know a solution, let me find the patch i submitted00:57
waynrhttps://review.openstack.org/50377100:58
waynrthat fixes a very similar issue in the conditional-step plugin00:59
waynrfixing it for additional plugins will require updating this method: https://review.openstack.org/#/c/503771/2/jenkins_jobs/registry.py@16501:01
waynrOdd_Bloke: had an alternative solution that is less complex but requires tangling the yaml parser up with the module registry again01:03
waynrhttp://paste.ubuntu.com/25540520/ <- Odd_Bloke's solution01:03
zxiiroah ok I'll  take a closer look at it in the morning01:23
*** abelur has quit IRC04:29
*** abelur has joined #openstack-jjb04:30
*** hashar has joined #openstack-jjb06:33
*** hashar has quit IRC07:58
*** hashar has joined #openstack-jjb08:04
*** electrofelix has joined #openstack-jjb09:15
*** electrofelix has quit IRC09:46
*** hashar is now known as hasharAway09:50
*** electrofelix has joined #openstack-jjb15:39
*** electrofelix has quit IRC17:47
*** hasharAway is now known as hashar19:09
*** hashar has quit IRC22:00

Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!