Tuesday, 2018-04-10

*** number80 has quit IRC00:21
*** cjloader has joined #openstack-requirements00:32
*** cjloader has quit IRC00:33
*** cjloader has joined #openstack-requirements00:34
*** odyssey4me has quit IRC00:34
*** odyssey4me has joined #openstack-requirements00:34
*** number80 has joined #openstack-requirements00:35
*** hongbin has joined #openstack-requirements01:33
*** andreas_s has joined #openstack-requirements02:32
*** andreas_s has quit IRC02:37
*** cjloader has quit IRC03:16
*** cjloader_ has joined #openstack-requirements03:16
*** cjloader_ has quit IRC03:19
*** cjloader has joined #openstack-requirements03:20
*** prashkre has joined #openstack-requirements03:24
*** prashkre has quit IRC03:49
*** hongbin has quit IRC03:57
*** prashkre has joined #openstack-requirements04:07
*** udesale has joined #openstack-requirements04:09
*** strigazi has quit IRC04:11
*** strigazi has joined #openstack-requirements04:14
*** prashkre has quit IRC04:26
*** cjloader has quit IRC04:54
*** strigazi_ has joined #openstack-requirements05:13
*** strigazi has quit IRC05:17
*** prashkre has joined #openstack-requirements06:15
*** cjloader has joined #openstack-requirements06:16
*** cjloader has quit IRC06:19
*** andreas_s has joined #openstack-requirements06:29
*** ralonsoh has joined #openstack-requirements06:53
*** jhesketh_ has joined #openstack-requirements07:31
*** jhesketh has quit IRC07:37
openstackgerritTony Breeds proposed openstack/requirements master: lower-requirements: Adds WMI and PyMI  https://review.openstack.org/55819607:41
*** florianf has joined #openstack-requirements07:49
*** prashkre has quit IRC07:57
*** jpich has joined #openstack-requirements08:04
*** prashkre has joined #openstack-requirements08:25
openstackgerritMerged openstack/requirements master: update constraint for paunch to new release 3.0.0  https://review.openstack.org/55977409:13
openstackgerritMerged openstack/requirements master: Remove XStatic-Magic-Search  https://review.openstack.org/55854409:13
openstackgerritMerged openstack/requirements master: Bump minimum osc-lib version  https://review.openstack.org/55911609:13
openstackgerritMerged openstack/requirements master: Update ryu minimum version to 4.24  https://review.openstack.org/55928809:13
openstackgerritMerged openstack/requirements master: update constraint for reno to new release 2.8.0  https://review.openstack.org/55973909:17
*** strigazi_ is now known as strigazi10:01
*** udesale has quit IRC10:45
*** strigazi is now known as strigaz_10:59
*** strigaz_ is now known as strigazi_10:59
*** strigazi_ is now known as strigazi10:59
*** odyssey4me has quit IRC12:00
*** odyssey4me has joined #openstack-requirements12:00
*** openstackgerrit has quit IRC12:04
*** edmondsw has joined #openstack-requirements12:17
*** edmondsw has quit IRC12:28
*** edmondsw has joined #openstack-requirements12:51
*** finucannot is now known as stephenfin13:05
*** udesale has joined #openstack-requirements13:06
*** udesale has quit IRC13:13
*** udesale has joined #openstack-requirements13:13
*** lbragstad has joined #openstack-requirements13:42
*** prashkre has quit IRC13:49
*** prashkre has joined #openstack-requirements13:52
*** prashkre has quit IRC14:09
*** prashkre has joined #openstack-requirements14:10
*** mnaser has quit IRC14:15
*** mnaser has joined #openstack-requirements14:16
*** samueldmq has quit IRC14:21
*** samueldmq has joined #openstack-requirements14:22
*** openstackgerrit has joined #openstack-requirements14:27
openstackgerritDoug Hellmann proposed openstack/requirements master: fix the logic for handling default for --reqs  https://review.openstack.org/56003514:27
*** prashkre has quit IRC14:29
dhellmannprometheanfire , tonyb , dirk : I don't understand why https://review.openstack.org/#/c/556255/3 is passing the requirements-check job. The branch-switching logic in project-requirements-change.py is confusing me.14:30
*** prashkre has joined #openstack-requirements14:31
*** coreycb has quit IRC14:34
*** coreycb has joined #openstack-requirements14:34
*** prashkre has quit IRC14:38
*** andreas_s has quit IRC14:47
*** andreas_s has joined #openstack-requirements14:47
*** andreas_s has quit IRC14:56
*** cjloader has joined #openstack-requirements14:57
dhellmannI think I'm not sure why we are concerned about checking the state of the branch receiving the patch instead of just comparing the contents of the repo with the patch against the requirements repo (and against itself)15:10
dhellmannrunning the check in --local mode gives me different results than the CI job gets, and I think CI is broken because --local mode reports the errors I expect to see15:10
dhellmannaha, the "ignore this line if it is the same as the branch" stuff is why that swift patch is passing15:13
openstackgerritDoug Hellmann proposed openstack/requirements master: be more explicit about the configuration being tested  https://review.openstack.org/56004915:20
openstackgerritDoug Hellmann proposed openstack/requirements master: remove optimization for values unchanged from the branch  https://review.openstack.org/56005015:20
*** cjloader has quit IRC15:20
*** cjloader has joined #openstack-requirements15:20
dhellmannprometheanfire , tonyb , dirk : ^^ should now cause https://review.openstack.org/556255 to fail the check job15:20
prometheanfiredhellmann: ok, I'll review soon, dealing with gentoo foundation stuff atm :(15:25
dhellmannprometheanfire : ack, thanks.15:25
*** udesale has quit IRC15:41
*** udesale has joined #openstack-requirements15:42
*** edmondsw has quit IRC16:00
*** edmondsw has joined #openstack-requirements16:01
*** jpich has quit IRC16:42
*** udesale has quit IRC16:49
*** andreas_s has joined #openstack-requirements17:03
*** andreas_s has quit IRC17:16
*** andreas_s has joined #openstack-requirements17:21
*** openstackgerrit has quit IRC17:34
*** ralonsoh has quit IRC17:34
*** andreas_s has quit IRC17:36
*** openstackgerrit has joined #openstack-requirements18:05
openstackgerritDoug Hellmann proposed openstack/requirements master: remove lower bounds from global requirements  https://review.openstack.org/56010818:05
openstackgerritDoug Hellmann proposed openstack/requirements master: add exclusion specifiers used by swift  https://review.openstack.org/56010918:05
prometheanfiredhellmann: good luck :D18:08
openstackgerritDoug Hellmann proposed openstack/requirements master: add exclusion specifiers used by swift  https://review.openstack.org/56010918:09
dhellmannprometheanfire : those changes should allow us to add the lower-constraints job to swift in https://review.openstack.org/55625518:13
prometheanfireyarp18:14
openstackgerritDoug Hellmann proposed openstack/requirements master: exclude cmd2 0.8.3 and update to 0.8.4  https://review.openstack.org/56012018:28
eanderssonWith the new siblin changes, lower-constratins may fail for horizon plugins, as it's pulling an older version of horizon19:02
prometheanfireeandersson: details?19:03
eanderssonhttp://logs.openstack.org/35/555835/2/check/openstack-tox-lower-constraints/bfc150f/job-output.txt.gz19:03
eanderssonhttps://review.openstack.org/#/c/555835/19:03
eanderssonnot quite sure why it's running the tests here, but basically it's a patch to remove mox319:03
*** openstackgerrit has quit IRC19:04
eanderssonand it's pulling in the queens build, and not horizon master19:05
eanderssonhttps://github.com/openstack/horizon/blob/master/openstack_dashboard/test/helpers.py#L4219:05
prometheanfireeandersson: you need to remove mox3 from lower-constraints.txt to fix one of the errors you will likely experience19:09
prometheanfirebut it's required by horizon19:17
prometheanfireso I guess it stays19:17
prometheanfireeandersson: sounds like horizon needs a release19:17
prometheanfiresmcginnis: ^ looks like horizon made a change (as a library) that a consumer (designate-dashboard) wants to consume19:18
prometheanfireand can't because it's missing a release19:18
prometheanfireeandersson: horizon did a bad thing and became both a library and 'end project'19:18
smcginniseandersson: Yeah, looks like horizon will need to request a new release so there's something out there newer than queens for you to pick up.19:24
eanderssonWould it not be possible to just enable the siblings extension on lower-constraints as well?19:25
prometheanfireprobably, can you make a patch for that to test?19:25
eanderssonsure19:28
*** andreas_s has joined #openstack-requirements19:48
*** andreas_s has quit IRC19:53
prometheanfiredirk: dhellmann: mind reviewing? https://review.openstack.org/55936720:05
*** openstackgerrit has joined #openstack-requirements20:11
openstackgerritDoug Hellmann proposed openstack/requirements master: exclude cmd2 0.8.3  https://review.openstack.org/56012020:11
*** florianf has quit IRC20:25
openstackgerritMerged openstack/requirements master: script to fix lower constraints entries that do not match local requirements  https://review.openstack.org/55861020:29
openstackgerritMerged openstack/requirements master: lower-requirements: Adds WMI and PyMI  https://review.openstack.org/55819620:30
openstackgerritMerged openstack/requirements master: fix the logic for handling default for --reqs  https://review.openstack.org/56003520:30
*** jroll has joined #openstack-requirements20:51
jrollhey folks - pycodestyle 2.4.0 broke ironic's pep8 jobs. clark mentioned that in the past hacking handled pins for these things, but maybe there was a mis-communication about moving that to u-c that didn't happen20:53
jrolland I'm not sure if this is going on with any other projects20:53
jrollso I guess I'm kind of wondering where a pin should go - ironic, hacking, or u-c?20:53
jrollit's a pretty large break for us: http://logs.openstack.org/20/531320/14/check/openstack-tox-pep8/c9d9d10/job-output.txt.gz#_2018-04-10_15_43_40_01473820:54
dhellmannprometheanfire : I don't understand your use of -W on other people's patches.21:05
prometheanfiredhellmann: am I not allowed to do that?21:05
dhellmannprometheanfire : I wouldn't myself. A -1 feels good enough.21:13
dhellmannI mean,  obviously the ACLs are there. It just feels heavy-handed21:13
dhellmannno one else can remove it, and so the patch is effectively blocked on you now21:14
*** jrist has quit IRC21:14
prometheanfireI don't mean for it to be, I just want to ensure discussion first, and -W is the only way to do so21:14
prometheanfiremaybe I'm overstepping, but I thought some projects require the PTL's ack on major changes21:15
dhellmannI guess. I feel like a -1 conveys that well enough. I don't have an issue with waiting for more discussion21:15
prometheanfiremaybe gentoo politics are making me paranoid21:15
dhellmannoh, I wasn't even thinking of it as a PTL thing, just another core reviewer21:15
dhellmannanyway, it's not a big deal, it just seemed different21:17
jrollfiled a bug wrt my question above, if someone wants to chime in :) https://storyboard.openstack.org/#!/story/200181921:18
prometheanfirejroll: oh, missed that lol21:18
jrollno worries, not a huge rush21:18
dhellmannjroll : sorry, I missed it, too21:18
*** cjloader has quit IRC21:19
dhellmannin the past hacking did deal with those pins. I think some of those projects are "blacklisted" so they aren't in constraints to allow projects to set their own pins now?21:19
prometheanfirejroll: the pin goes in hacking or ironic21:20
jrolloh!21:20
jrollI forgot about the blacklist :)21:20
prometheanfirepycodestyle is blacklisted so we don't track it21:20
jrollawesome, thanks dhellmann and prometheanfire :)21:21
*** jrist has joined #openstack-requirements21:41
*** edmondsw has quit IRC21:42
*** edmondsw has joined #openstack-requirements21:42
*** edmondsw has quit IRC21:43
openstackgerritMerged openstack/requirements master: uncap eventlet  https://review.openstack.org/55936721:44
*** edmondsw has joined #openstack-requirements22:09
*** edmondsw has quit IRC22:10
*** lbragstad has quit IRC22:42
*** lbragstad has joined #openstack-requirements23:09
*** cjloader has joined #openstack-requirements23:20
*** cjloader has quit IRC23:24

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