Wednesday, 2018-05-30

*** jplf_ has joined #openstack-jjb00:04
*** abelur_ has joined #openstack-jjb00:05
*** jplf has quit IRC00:06
*** jplf_ is now known as jplf00:06
*** abelur has quit IRC00:06
*** abelur_ is now known as abelur00:06
*** caphrim007_ has quit IRC00:30
*** alphadose[m] has quit IRC00:38
*** alphadose[m] has joined #openstack-jjb00:59
*** kien-ha has joined #openstack-jjb01:37
*** caphrim007 has joined #openstack-jjb02:21
*** caphrim007 has quit IRC02:26
*** caphrim007 has joined #openstack-jjb02:36
*** caphrim007 has quit IRC02:44
*** caphrim007 has joined #openstack-jjb02:45
*** kien-ha has quit IRC05:27
*** hashar has joined #openstack-jjb05:59
openstackgerritVeluri Mithun proposed openstack-infra/jenkins-job-builder master: Use convert_mapping_to_xml helper function  https://review.openstack.org/57109606:05
*** ssbarnea_ has joined #openstack-jjb06:58
*** hashar has quit IRC09:23
*** hashar has joined #openstack-jjb09:23
*** hashar is now known as hasharAway09:50
electrofelixssbarnea zxiiro: regarding https://review.openstack.org/#/c/567943/ I think either that should be reverted or completely remove kerberos support from python-jenkins, as removing the package when the test fails on some peoples system instead of fixing the cause of the failure probably means that kerberos support is broken10:02
ssbarneaelectrofelix: this is a tricky issue. currently we do not test kerberos as we mainly cannot.10:06
electrofelixright, but the error reported as the reason to remove the package from the tests suggests that something is likely broken for someone when they have an env for using kerberos but now we'll no longer spot it so we'll continue to break it further10:07
electrofelixIn fact that error also means that someone with the kerberos-requests package potentially even without using kerberos will find python-jenkins broken for them, there's no way to tell10:08
ssbarneatrue10:08
openstackgerritSorin Sbarnea proposed openstack/python-jenkins master: Revert "Do not require requests-kerberos for tests"  https://review.openstack.org/57116010:10
ssbarnearevert is simple, fixing it is not. mainly we should test both, with and without the presence of requests-kerberos packages10:11
ssbarneabut this means new tox target which is not something I enjoy doing, implementing new gate....10:12
ssbarneamaybe someone has a better idea for testing that it works with both cases.10:12
*** ssbarnea_ has quit IRC10:42
*** ssbarnea_ has joined #openstack-jjb11:19
electrofelixyeah, looks like it's going to be messy to fix with other dependencies having moved forward, I'll see what I can do over the next few days, want to catch up a bit12:21
Odd_BlokeI've just filed https://bugs.launchpad.net/python-cliff/+bug/1774176; looks like we'll need that fixed before any Python 2 gates will pass.12:29
openstackLaunchpad bug 1774176 in cliff "Uninstallable using Python 2 (due to cmd2 dropping Py2 support)" [Undecided,New]12:29
electrofelixstarting to look like there might have be another major release in 3-6 months where we drop python 2 rather than trying to continue to support12:32
openstackgerritVicky Chijwani proposed openstack-infra/jenkins-job-builder master: Add retain_anchors config option.  https://review.openstack.org/57119712:51
*** vickyc has joined #openstack-jjb12:59
*** caphrim007 has quit IRC13:06
openstackgerritVicky Chijwani proposed openstack-infra/jenkins-job-builder master: Add retain_anchors config option.  https://review.openstack.org/57119713:19
*** kien-ha has joined #openstack-jjb14:29
zxiirook I think I subscribed to the storyboard14:38
zxiiroelectrofelix: FYI I think we should give some attention to this issue https://storyboard.openstack.org/#!/story/2001508 as the guy has created a fork of JJB on pypi because of this.14:43
zxiirowe should probably resolve it and ask him to delete his fork to not confuse our users14:44
electrofelixagreed I guess this is what comes from not noticing issues14:50
zxiirokien-ha: can you help us clean up the storyboard for jjb? I suspect there's a bunch of easy ones that can be closed.14:51
kien-hasure I'll do what I can14:56
ssbarneaelectrofelix: regarding the SB,... I don't know where to start,... i was planning to write a long blog post about what is wrong with it, mainly is the *UX*. In the end you figured out how to subcribe... and you get few good years go software engineering ;). Should I add impossible to edit descriptions/comments, the project URL is a ... number, very "friendly". Impossible to configure pagination, always will reset,...15:33
electrofelixI guess there is a question, do we abandon it until it's fully working or do we take a philosophical view that we should support a fully open source solution and try to help point out it's defects until it's as good as all the other solutions15:35
Odd_BlokeLaunchpad is open source, if that's a concern.15:37
ssbarneai know for sure that i don't have bandwidth to fix it, i am already overloaded by minor contributions to other projects and fixing SB would need a full time job, if not more.15:37
ssbarneaOdd_Bloke: I know is open source, but maybe not as easy to use. not to mention that I only know one implementation. Last time I attempted to setup my own instance I failed miserably (8+ years ago), and realised that it was not really off the shelf ;)15:39
ssbarneaanyway, there is no pressure to make a decision15:39
Odd_Blokessbarnea: Sure, but you couldn't even try to do that with Github without forking over a large amount of cash.15:40
ssbarneathe cmd2 seems like a serious issue.15:40
Odd_BlokeAnd, yeah, it certainly isn't off-the-shelf.15:40
Odd_BlokeBut we wouldn't need to run an instance of it for JJB, we could just use launchpad.net (like much of Openstack).15:40
Odd_BlokeFull disclosure: I work for Canonical, so I already use Launchpad a _lot_ anyway.15:42
Odd_BlokeSo I'm going to be biased. :p15:42
*** caphrim007 has joined #openstack-jjb15:44
ssbarneacan launchpad browse code hosted on github?15:44
ssbarneaone plus for LP is the python-jenkins already have a project there.15:45
Odd_Blokessbarnea: I think we just wouldn't host any code on Launchpad and direct people to Github.15:46
Odd_Blokehttps://launchpad.net/nova does that in the project description, and there's just one random branch if you go to the Code tab.15:46
ssbarneaanother question would be if we can have an unified view for both projects jjb and python-jenkins.15:46
Odd_BlokeBugs can affect more than one project in Launchpad.15:47
ssbarneastill, the "code" tag is misleading users with last chage 2015.15:47
ssbarneasomehow I have the impression that nova received few... more recent contributions.15:47
Odd_BlokeRight, that's not an official branch.15:48
ssbarneai like GH because they have very good code browser and is very easy to put a link to the code using that, for thus I find a benefit to keep issue tracker and code on the same system.15:48
ssbarneasadly gerrit lacks on this area too15:49
Odd_BlokeWell, if we aren't using PRs, then some of the linking becomes less useful, right?15:49
ssbarneaon the other hand, going github could bring too much publicity, too easy to raise bugs.15:54
ssbarneai seen it in the past15:55
Odd_BlokeI would, honestly, be happy with either.15:56
zxiiroPersonally I'm fine with either but we need to make sure we can address user concerns in whatever system we use. I'm happy to give SB a try too now that I figured out how to subscribe15:57
zxiiroif we do decide to swtich to GitHub we need to update the instructions on how to raise a bug with us15:57
zxiirobecause it currently points people to SB15:57
electrofelixssbarnea: I wouldn't expect us to fix SB, just to report it's failings to those working on it15:58
zxiiroI'm confident the openstack-infra guys will allow us to move to GH if that's our final decisino but I'd like us all to fully agree on it before we make any major jump.15:58
ssbarneasure.15:58
zxiiroI do agree with electrofelix though in that if we use GH issues then our users will have the expectation that we will accept PRs which we don't. and there's a whole other mess with that route :)15:59
ssbarnealets fix urgent issues first, cmd215:59
ssbarneatrying to figure out what we can do.15:59
zxiirossbarnea: oh yeah, so we fixed that in OpenDyalight by pinning cmd2 to 0.8.515:59
zxiiromaybe we can push that into requirements.txt as a temporary workaround?15:59
ssbarnealink?16:00
Odd_BlokeThe OpenStack requirements project has a slightly better fix, I think.16:00
Odd_BlokePin it for Python 2, but not for Python 3.16:00
zxiiroOdd_Bloke: oh? what did they do?16:00
electrofelixlets try getting SB working for 1 month and then report on the deficiencies? can then decide16:00
ssbarneai was considering    cmd<=0.85;python....16:00
Odd_BlokeYeah, something like that.16:00
ssbarnealike pinning only for unsupported versions of python.16:00
zxiiroOpenDaylight just did a simple cmd==0.8.516:00
*** hasharAway is now known as hashar16:01
zxiirocmd2==0.8.516:01
electrofelixpin for py2 with env marker and that makes it clear when removing python2 support in the future that we can drop the pin16:02
Odd_BlokeI'd prefer cmd2<0.9.16:02
Odd_Blokecmd2 0.8.x is up to 0.8.7.16:02
zxiiro+1 I like the more specific pinning16:02
Odd_BlokeWhich one are you +1'ing?16:02
Odd_BlokeI'd say mine is less specific. :p16:03
zxiirocmd2<0.9 against py216:03
openstackgerritSorin Sbarnea proposed openstack/python-jenkins master: Freeze cmd2 library on python2  https://review.openstack.org/57124316:03
electrofelixis there an issue with the 0.8.3 version?16:04
ssbarneaelectrofelix: this is copy/paste from openstack, i bet there was a problem16:05
Odd_BlokeI believe that's following... yeah, that.16:05
electrofelixah16:05
ssbarneai am wondering if it would not make sense to point to openstack requirements directly, so we would delegate these issues to them ;)16:06
electrofelixdoes the check-requirements job take care of such issues?16:06
electrofelixmarked it CR+2 but left the workflow as is for the moment16:07
ssbarneayeah, me waiting for ci.16:08
*** electrofelix has quit IRC16:08
openstackgerritMerged openstack/python-jenkins master: Freeze cmd2 library on python2  https://review.openstack.org/57124316:37
*** caphrim007 has quit IRC16:40
openstackgerritSorin Sbarnea proposed openstack/python-jenkins master: Revert "Do not require requests-kerberos for tests"  https://review.openstack.org/57116016:51
*** hashar is now known as hasharAway17:00
*** ssbarnea_ has quit IRC17:27
openstackgerritVeluri Mithun proposed openstack-infra/jenkins-job-builder master: Use convert_mapping_to_xml helper function  https://review.openstack.org/57109617:30
openstackgerritVeluri Mithun proposed openstack-infra/jenkins-job-builder master: Remove trailing white space and convert tabs to spaces  https://review.openstack.org/57126217:30
*** ssbarnea_ has joined #openstack-jjb17:45
*** caphrim007 has joined #openstack-jjb17:45
*** caphrim007 has quit IRC17:47
*** caphrim007 has joined #openstack-jjb17:47
openstackgerritVeluri Mithun proposed openstack-infra/jenkins-job-builder master: Use convert_mapping_to_xml helper function  https://review.openstack.org/57127818:16
*** Mithun has joined #openstack-jjb18:21
openstackgerritVeluri Mithun proposed openstack-infra/jenkins-job-builder master: Use convert_mapping_to_xml helper function  https://review.openstack.org/57127818:24
*** ssbarnea_ has quit IRC18:31
*** Mithun has quit IRC18:45
*** ssbarnea_ has joined #openstack-jjb19:33
Odd_BlokeDoes someone want to throw the StoryBoard project link (https://storyboard.openstack.org/#!/project/723) in to the topic?19:40
Odd_BlokeNot sure how to do that.19:40
Odd_BlokeNot being able to triage importance in StoryBoard is annoying.19:46
Odd_BlokeAlso, I assume these were imported from somewhere; YAML is semi-valid MarkDown and none of the issues I've looked at thus far wrapped the YAML in ```s to make it code. ;.;19:47
*** kien-ha has quit IRC19:59
openstackgerritDaniel Watkins proposed openstack-infra/jenkins-job-builder master: Make the deprecated plugin config messaging clearer  https://review.openstack.org/57130020:27
*** hasharAway has quit IRC20:51
openstackgerritDaniel Watkins proposed openstack-infra/jenkins-job-builder master: Handle string skip-vote parameters in gerrit trigger  https://review.openstack.org/57131121:02
*** ssbarnea_ has quit IRC21:14
*** caphrim007_ has joined #openstack-jjb21:55
*** caphrim007 has quit IRC21:59
*** kien-ha has joined #openstack-jjb22:08
*** caphrim007_ has quit IRC23:47

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