Friday, 2018-09-21

zxiiroI'm flying to Amsterdam tomorrow and will be out for a ~3 weeks. gonna release JJB tonight01:12
zxiiroGuess I should review and see if there's patches to merge first.01:14
*** harlowja has quit IRC01:18
openstackgerritMerged openstack-infra/jenkins-job-builder master: email: Utilize convert_mapping_to_xml  https://review.openstack.org/60079501:25
openstackgerritMerged openstack-infra/jenkins-job-builder master: hue-light: Add commas to mapping tuple  https://review.openstack.org/60261701:26
openstackgerritMerged openstack-infra/jenkins-job-builder master: copyartifact-build-selector: Utilize convert_mapping_to_xml  https://review.openstack.org/59653201:29
openstackgerritMerged openstack-infra/jenkins-job-builder master: Fix options.update is not 'all' if no config  https://review.openstack.org/60380301:33
openstackgerritMerged openstack-infra/jenkins-job-builder master: build-keeper: Add support for RunCondition  https://review.openstack.org/59341901:43
*** hashar has joined #openstack-jjb06:15
openstackgerritJaime Melis proposed openstack-infra/jenkins-job-builder master: gitlab-merge-request: Add support for 2.x  https://review.openstack.org/60080408:51
*** ssbarnea|bkp2 has joined #openstack-jjb09:38
ssbarnea|bkp2zxiiro this seems important for at least 3 people, lets merge it unless you see a problem with it https://review.openstack.org/#/c/572179/09:38
ssbarnea|bkp2zxiiro also https://review.openstack.org/#/c/588154/ seems harmless.09:40
openstackgerritMerged openstack-infra/jenkins-job-builder master: config-file-provider: Add tests  https://review.openstack.org/59337211:38
openstackgerritAnil Belur proposed openstack-infra/jenkins-job-builder master: cucumber-reports: Add support for new options  https://review.openstack.org/59921812:28
*** hashar has quit IRC12:32
openstackgerritAnil Belur proposed openstack-infra/jenkins-job-builder master: jclouds: Utilize convert_mapping_to_xml  https://review.openstack.org/60261512:35
*** ssbarnea|bkp has joined #openstack-jjb12:51
openstackgerritAnil Belur proposed openstack-infra/jenkins-job-builder master: docker-container: Improve support for more options  https://review.openstack.org/60313512:52
*** ssbarnea|bkp2 has quit IRC12:54
*** hashar has joined #openstack-jjb13:03
openstackgerritAnil Belur proposed openstack-infra/jenkins-job-builder master: docker-custom-build-env: Add support for new options  https://review.openstack.org/60315013:09
zxiirossbarnea|bkp: approved the 2nd one. let me do a quick jenkins-jobs test against opendaylight with the first patch.13:39
openstackgerritMerged openstack-infra/jenkins-job-builder master: docker-custom-build-env: Add support for new options  https://review.openstack.org/60315013:43
ssbarnea|bkpzxiiro another one https://review.openstack.org/58893813:44
openstackgerritMerged openstack/python-jenkins master: Add folder credential support  https://review.openstack.org/58815413:46
openstackgerritMerged openstack-infra/jenkins-job-builder master: gitlab-merge-request: Add support for 2.x  https://review.openstack.org/60080413:49
*** openstackgerrit has quit IRC14:07
*** openstackgerrit has joined #openstack-jjb14:11
openstackgerritMerged openstack/python-jenkins master: Check for 'Location' header in the response  https://review.openstack.org/57217914:11
ssbarnea|bkpzxiiro: https://review.openstack.org/#/c/583535/ old git-review one.14:18
zxiirossbarnea|bkp: gotta head to the airport (heading to Amsterdam today) I'll take a look a bit later but looks like Darraugh and Tomas are still having a discussion there?14:19
zxiirooh yeah the openstack-infra people were talking about potentially adding us as core reviewers in git-review14:19
zxiiroI guess I should spend more time reviewing things over there14:20
zxiirossbarnea|bkp: gave you a +114:20
ssbarnea|bkpzxiiro: thanks, and have fun in AMS14:20
Liskni_siI don't think we're having any discussion any more in any commit related to tja-fast-get-jobs. I addressed everything Darragh pointed out and haven't heard from him ever since.14:22
Liskni_si(But I may have missed something. Asked him here a few days ago, didn't get an answer either.)14:22
zxiiroLiskni_si: sounds good. I'll try to find some time to review it. From my perspective if jenkins-jobs test passes against the opendaylight jjb repo I'm good.14:31
zxiiropython-jenkin's broke us a few times, so I'm being more cautious about releases of it now.14:32
Liskni_sizxiiro: you'd need a live jenkins instance to notice a breakage14:32
Liskni_si(just so you know what needs to be tested)14:33
zxiiroyeah, I was thinking of hooking our jenkins system into openstack's gerrit to report when a patch breaks us but haven't had time to investigate getting that done just yet.14:33
Liskni_siAnyway, thanks for taking the time to look at it. If I was travelling by an airplane, I'd probably be just stressed and do nothing. :-)14:34
zxiirossbarnea|bkp: that reminds me, I had to revert 2 of your python-jenkins patches https://review.openstack.org/596404 & https://review.openstack.org/59640314:35
zxiirossbarnea|bkp: you should probably resurrect them if you need them.14:35
zxiirobut those 2 patches broke us when it got released and deployed14:35
ssbarnea|bkpzxiiro unlikely that i will need them but I know where to search for them if needed.14:36
openstackgerritSorin Sbarnea proposed openstack-infra/jenkins-job-builder master: Blueocean configuration for multibranch project  https://review.openstack.org/56640415:11
*** hashar is now known as hasharAway15:18
*** ssbarnea|bkp has quit IRC17:18
*** harlowja has joined #openstack-jjb17:21
*** ktdreyer_ is now known as ktdreyer17:39
harlowjadoes the github scm option `ssh-checkout: true` also take a credential id?17:41
harlowja(if say i pass a string instead of true)17:42
harlowjaseems like not supported yet18:42
harlowja(durn)18:42
*** ssbarnea|bkp has joined #openstack-jjb18:42
*** ssbarnea|bkp has quit IRC19:31
*** hasharAway has quit IRC21:30
*** ssbarnea|bkp has joined #openstack-jjb21:52
*** ssbarnea|bkp2 has joined #openstack-jjb23:01
*** ssbarnea|bkp has quit IRC23:03
*** harlowja has quit IRC23:54

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