Monday, 2017-08-21

openstackgerritThanh Ha proposed openstack-infra/jenkins-job-builder master: Move macro expansion into YamlParser.  https://review.openstack.org/33307600:28
openstackgerritMerged openstack-infra/jenkins-job-builder master: Update cvs to use convert_mapping_to_xml()  https://review.openstack.org/47963200:31
*** Guest5481 has quit IRC01:33
*** larainema has quit IRC08:18
*** larainema has joined #openstack-jjb08:21
*** electrofelix has joined #openstack-jjb09:19
*** openstackgerrit has quit IRC11:17
*** yland has joined #openstack-jjb14:55
zxiirowaynr: so the jinja patch broke your macro expansion patch https://review.openstack.org/33307615:02
zxiirowaynr: you know how to fix it?15:02
zxiiroOdd_Bloke: ^15:02
zxiiroOdd_Bloke: it's one of the patches we need to get into JJB so we can release 2.0. So I'm trying to get it through quickly...15:03
Odd_Blokezxiiro: Yep, looking at it ATM.15:12
Odd_Blokezxiiro: There are a couple of issues; one is that jinja2.Template isn't deepcopy'able; we can work around that by generating the template at format-time instead of storing it against the class.15:13
zxiiroOdd_Bloke: cool thanks!15:13
Odd_Blokezxiiro: The other is that "!include-jinja2: {some_template}" returns a LazyLoader, which includes a reference to a LocalLoader, which includes a reference to an open file object (which can't be copied).15:13
Odd_BlokeFor that one, I think we can just copy the reference to the LocalLoader if it's finished parsing,15:14
*** openstackgerrit has joined #openstack-jjb15:14
openstackgerritYolande Amate proposed openstack-infra/jenkins-job-builder master: Add support for jira_changelog_trigger  https://review.openstack.org/49333615:14
openstackgerritYolande Amate proposed openstack-infra/jenkins-job-builder master: Add support for jira_comment_trigger  https://review.openstack.org/49226615:35
Odd_Blokezxiiro: Just sent email.15:39
zxiiroOdd_Bloke: thanks!15:40
*** yland has quit IRC15:53
*** yland has joined #openstack-jjb16:06
*** yland is now known as Guest3064016:06
openstackgerritYolande Amate proposed openstack-infra/jenkins-job-builder master: Update openstack to use convert_mapping_to_xml()  https://review.openstack.org/47999616:47
zxiiroGuest30640: ^ The 2 yaml inputs files should not even appear in the change list.17:02
*** Guest30640 has quit IRC17:31
*** Guest30640 has joined #openstack-jjb17:49
*** electrofelix has quit IRC18:13
*** Guest30640 is now known as yland18:21
openstackgerritYolande Amate proposed openstack-infra/jenkins-job-builder master: Update openstack to use convert_mapping_to_xml()  https://review.openstack.org/47999618:27
*** yland has quit IRC20:14
*** yland has joined #openstack-jjb20:41
openstackgerritYolande Amate proposed openstack-infra/jenkins-job-builder master: Update archive to use convert_mapping_to_xml()  https://review.openstack.org/47696022:32

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