Wednesday, 2018-06-20

*** caphrim007 has joined #openstack-jjb01:03
*** caphrim007 has quit IRC01:07
*** caphrim007 has joined #openstack-jjb01:21
*** caphrim007 has quit IRC01:22
*** caphrim007 has joined #openstack-jjb01:44
*** caphrim007 has quit IRC01:49
*** caphrim007 has joined #openstack-jjb02:00
*** caphrim007 has quit IRC02:02
*** caphrim007 has joined #openstack-jjb02:02
openstackgerritqingszhao proposed openstack-infra/jenkins-job-builder master: change default python 3 env in tox to 3.5  https://review.openstack.org/57668002:30
*** caphrim007_ has joined #openstack-jjb02:50
*** caphrim007 has quit IRC02:54
*** r-mibu has quit IRC03:43
*** kien-ha has quit IRC03:44
*** kien-ha has joined #openstack-jjb03:46
*** r-mibu has joined #openstack-jjb03:46
*** larainema has quit IRC03:48
*** zxiiro has quit IRC03:49
*** spalger has quit IRC03:49
*** abelur has quit IRC03:50
*** jplf has quit IRC03:50
*** r-mibu has quit IRC03:51
*** kien-ha has quit IRC03:51
*** openstack has joined #openstack-jjb04:32
*** ChanServ sets mode: +o openstack04:32
*** kien-ha has joined #openstack-jjb05:21
*** abelur has joined #openstack-jjb05:25
*** zxiiro has joined #openstack-jjb05:25
*** spalger has joined #openstack-jjb05:29
*** larainema has joined #openstack-jjb05:31
*** kien-ha has quit IRC05:39
*** larainema has quit IRC05:48
*** spalger has quit IRC05:55
*** abelur has quit IRC05:55
*** zxiiro has quit IRC05:57
*** kien-ha has joined #openstack-jjb06:10
*** spalger has joined #openstack-jjb06:12
*** r-mibu has joined #openstack-jjb06:17
*** jplf has joined #openstack-jjb06:19
*** zxiiro has joined #openstack-jjb06:21
*** larainema has joined #openstack-jjb06:21
*** abelur has joined #openstack-jjb06:24
*** kbidarka_ has joined #openstack-jjb06:30
kbidarka_ssbarnea, testing job creation via jjb fails for python3.6 , https://pastebin.com/TkRjsAQT , related to this commit I guess , https://github.com/openstack-infra/jenkins-job-builder/commit/144ace119bc320927371f0bad7fc41fb566def1206:35
kbidarka_or may be i am missing something06:36
*** hashar has joined #openstack-jjb07:19
*** kbidarka_ has quit IRC08:07
openstackgerritKevin de Berk proposed openstack-infra/jenkins-job-builder master: multibranch: Configurable script-path  https://review.openstack.org/57612808:09
openstackgerritKevin de Berk proposed openstack-infra/jenkins-job-builder master: multibranch-bitbucket: discover strategies  https://review.openstack.org/57617908:54
*** kbidarka has joined #openstack-jjb09:28
*** electrofelix has joined #openstack-jjb09:34
ssbarneakbidarka: zxiiro electrofelix : this change is fixing py3 issue: https://review.openstack.org/#/c/576504/ -- needs to go in asap09:37
kbidarkassbarnea, yeah thanks09:41
ssbarneazxiiro: around here?12:09
*** caphrim007_ has quit IRC13:13
*** caphrim007 has joined #openstack-jjb13:19
*** caphrim007 has quit IRC13:24
*** hashar is now known as hasharAway13:39
zxiirossbarnea: merged13:54
ssbarneahurrah!13:54
zxiiroelectrofelix: opinion on https://review.openstack.org/576603 ? I know you didn't like that we added al lthe basepython so we reverted it with a compromise if it makes sense to you.13:54
ssbarneagood that we didn't release that one. also glad that we have enough people using master in order to spot problems.13:55
zxiiroI'm not sure who qingszhao is working for but seems like this contributor is blindly patching every openstack project the same way13:55
ssbarneamy oppinion on revert basepython, lets try to delay it, last merge should avoid big problems.13:56
ssbarneahaha! the big py3 migration :D13:56
ssbarneai was a bit worried that we do not have a gate for py3613:56
ssbarneai think we need at least one for py34 and one for py36. --- the py35 one is less important because is in between the other two.13:57
electrofelixzxiiro: it was more that the person was making changes without realising they could solve their problem without needing to patch anything14:01
zxiirossbarnea: agreed. Personally I'd recommend keeping at least whatever version the latest CentOS supports because RedHat / CentOS is the slowest distro heh14:03
openstackgerritMerged openstack-infra/jenkins-job-builder master: Allow jjb to be called as a module  https://review.openstack.org/57650414:03
openstackgerritSorin Sbarnea proposed openstack/python-jenkins master: Adopt use of pre-commit hooks  https://review.openstack.org/57687814:33
*** caphrim007 has joined #openstack-jjb15:32
*** caphrim007 has quit IRC15:32
*** caphrim007 has joined #openstack-jjb15:33
*** caphrim007 has quit IRC15:33
*** caphrim007 has joined #openstack-jjb15:33
openstackgerritKevin de Berk proposed openstack-infra/jenkins-job-builder master: multibranch-bitbucket: Configure server address  https://review.openstack.org/57614615:41
*** hasharAway is now known as hashar16:01
ssbarneazxiiro electrofelix : ready for review https://review.openstack.org/#/c/576878/16:23
zxiirossbarnea: I thought pbr managed the version for us16:25
ssbarneazxiiro: it does.16:26
ssbarneai am not sure I understand the question, as my change is unrelated to version16:27
zxiirossbarnea: your patch touched conf.py which is a docs change and you set version at the top16:27
ssbarneai only moved the import to the top of the file to make the linter happy16:27
zxiiroah I missed that16:27
zxiiroI thought it was being added16:28
zxiirossbarnea: yeah usually in projects I've worked on we ignore conf.py for sphinx in style checkers16:29
zxiirothat's why i was confused16:29
ssbarneai decided that less ignores makes it easier16:30
ssbarneai know, my approach so far was to ignore it too BUT this proved to break the docs multiple times.16:30
zxiirossbarnea: cool, I'll have to take a look at it later. I'm pretty busy today with internal stuff.16:32
zxiiroI want to make sure I understand how that pre-config hook things work16:32
ssbarneajust do a "pip install pre-commit"16:33
ssbarneathat should be enough to get them working.16:33
ssbarneato run the hooks manually do "pre-commit run" or "pre-commit --all" --- without all will check only last commit.16:33
ssbarneait is very fast because all repors are cached and shared, so only first time it will clone.16:34
ssbarneai really find the implementation beautiful16:34
ssbarneai plan to add support for our bashate tomorrow16:34
ssbarneathere are few other really handy hooks that I want to enable but not in the first round, read https://pre-commit.com/hooks.html16:35
ssbarneajust by reading their names you will figure out what they do. and is *FAST*, like doing linting in 2-3 seconds even with >10 hooks.16:36
*** electrofelix has quit IRC17:59
*** caphrim007_ has joined #openstack-jjb18:05
*** caphrim007 has quit IRC18:09
*** hashar is now known as hasharAway18:10
*** kbidarka_ has joined #openstack-jjb18:32
*** kbidarka has quit IRC18:36
*** kbidarka_ has quit IRC18:57
*** caphrim007_ has quit IRC19:05
*** caphrim007 has joined #openstack-jjb19:05
*** zaro__ has joined #openstack-jjb19:18
*** zaro__ has quit IRC19:25
*** zaro__ has joined #openstack-jjb19:25
openstackgerritDaniel Watkins proposed openstack-infra/jenkins-job-builder master: Refactor duplicated code in xml_config.py  https://review.openstack.org/57695019:41
*** hasharAway has quit IRC22:25
*** caphrim007_ has joined #openstack-jjb23:15
*** caphrim007 has quit IRC23:19
*** caphrim007_ has quit IRC23:19

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