*** number80 has quit IRC | 00:21 | |
*** cjloader has joined #openstack-requirements | 00:32 | |
*** cjloader has quit IRC | 00:33 | |
*** cjloader has joined #openstack-requirements | 00:34 | |
*** odyssey4me has quit IRC | 00:34 | |
*** odyssey4me has joined #openstack-requirements | 00:34 | |
*** number80 has joined #openstack-requirements | 00:35 | |
*** hongbin has joined #openstack-requirements | 01:33 | |
*** andreas_s has joined #openstack-requirements | 02:32 | |
*** andreas_s has quit IRC | 02:37 | |
*** cjloader has quit IRC | 03:16 | |
*** cjloader_ has joined #openstack-requirements | 03:16 | |
*** cjloader_ has quit IRC | 03:19 | |
*** cjloader has joined #openstack-requirements | 03:20 | |
*** prashkre has joined #openstack-requirements | 03:24 | |
*** prashkre has quit IRC | 03:49 | |
*** hongbin has quit IRC | 03:57 | |
*** prashkre has joined #openstack-requirements | 04:07 | |
*** udesale has joined #openstack-requirements | 04:09 | |
*** strigazi has quit IRC | 04:11 | |
*** strigazi has joined #openstack-requirements | 04:14 | |
*** prashkre has quit IRC | 04:26 | |
*** cjloader has quit IRC | 04:54 | |
*** strigazi_ has joined #openstack-requirements | 05:13 | |
*** strigazi has quit IRC | 05:17 | |
*** prashkre has joined #openstack-requirements | 06:15 | |
*** cjloader has joined #openstack-requirements | 06:16 | |
*** cjloader has quit IRC | 06:19 | |
*** andreas_s has joined #openstack-requirements | 06:29 | |
*** ralonsoh has joined #openstack-requirements | 06:53 | |
*** jhesketh_ has joined #openstack-requirements | 07:31 | |
*** jhesketh has quit IRC | 07:37 | |
openstackgerrit | Tony Breeds proposed openstack/requirements master: lower-requirements: Adds WMI and PyMI https://review.openstack.org/558196 | 07:41 |
---|---|---|
*** florianf has joined #openstack-requirements | 07:49 | |
*** prashkre has quit IRC | 07:57 | |
*** jpich has joined #openstack-requirements | 08:04 | |
*** prashkre has joined #openstack-requirements | 08:25 | |
openstackgerrit | Merged openstack/requirements master: update constraint for paunch to new release 3.0.0 https://review.openstack.org/559774 | 09:13 |
openstackgerrit | Merged openstack/requirements master: Remove XStatic-Magic-Search https://review.openstack.org/558544 | 09:13 |
openstackgerrit | Merged openstack/requirements master: Bump minimum osc-lib version https://review.openstack.org/559116 | 09:13 |
openstackgerrit | Merged openstack/requirements master: Update ryu minimum version to 4.24 https://review.openstack.org/559288 | 09:13 |
openstackgerrit | Merged openstack/requirements master: update constraint for reno to new release 2.8.0 https://review.openstack.org/559739 | 09:17 |
*** strigazi_ is now known as strigazi | 10:01 | |
*** udesale has quit IRC | 10:45 | |
*** strigazi is now known as strigaz_ | 10:59 | |
*** strigaz_ is now known as strigazi_ | 10:59 | |
*** strigazi_ is now known as strigazi | 10:59 | |
*** odyssey4me has quit IRC | 12:00 | |
*** odyssey4me has joined #openstack-requirements | 12:00 | |
*** openstackgerrit has quit IRC | 12:04 | |
*** edmondsw has joined #openstack-requirements | 12:17 | |
*** edmondsw has quit IRC | 12:28 | |
*** edmondsw has joined #openstack-requirements | 12:51 | |
*** finucannot is now known as stephenfin | 13:05 | |
*** udesale has joined #openstack-requirements | 13:06 | |
*** udesale has quit IRC | 13:13 | |
*** udesale has joined #openstack-requirements | 13:13 | |
*** lbragstad has joined #openstack-requirements | 13:42 | |
*** prashkre has quit IRC | 13:49 | |
*** prashkre has joined #openstack-requirements | 13:52 | |
*** prashkre has quit IRC | 14:09 | |
*** prashkre has joined #openstack-requirements | 14:10 | |
*** mnaser has quit IRC | 14:15 | |
*** mnaser has joined #openstack-requirements | 14:16 | |
*** samueldmq has quit IRC | 14:21 | |
*** samueldmq has joined #openstack-requirements | 14:22 | |
*** openstackgerrit has joined #openstack-requirements | 14:27 | |
openstackgerrit | Doug Hellmann proposed openstack/requirements master: fix the logic for handling default for --reqs https://review.openstack.org/560035 | 14:27 |
*** prashkre has quit IRC | 14:29 | |
dhellmann | prometheanfire , 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-requirements | 14:31 | |
*** coreycb has quit IRC | 14:34 | |
*** coreycb has joined #openstack-requirements | 14:34 | |
*** prashkre has quit IRC | 14:38 | |
*** andreas_s has quit IRC | 14:47 | |
*** andreas_s has joined #openstack-requirements | 14:47 | |
*** andreas_s has quit IRC | 14:56 | |
*** cjloader has joined #openstack-requirements | 14:57 | |
dhellmann | I 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 |
dhellmann | running 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 see | 15:10 |
dhellmann | aha, the "ignore this line if it is the same as the branch" stuff is why that swift patch is passing | 15:13 |
openstackgerrit | Doug Hellmann proposed openstack/requirements master: be more explicit about the configuration being tested https://review.openstack.org/560049 | 15:20 |
openstackgerrit | Doug Hellmann proposed openstack/requirements master: remove optimization for values unchanged from the branch https://review.openstack.org/560050 | 15:20 |
*** cjloader has quit IRC | 15:20 | |
*** cjloader has joined #openstack-requirements | 15:20 | |
dhellmann | prometheanfire , tonyb , dirk : ^^ should now cause https://review.openstack.org/556255 to fail the check job | 15:20 |
prometheanfire | dhellmann: ok, I'll review soon, dealing with gentoo foundation stuff atm :( | 15:25 |
dhellmann | prometheanfire : ack, thanks. | 15:25 |
*** udesale has quit IRC | 15:41 | |
*** udesale has joined #openstack-requirements | 15:42 | |
*** edmondsw has quit IRC | 16:00 | |
*** edmondsw has joined #openstack-requirements | 16:01 | |
*** jpich has quit IRC | 16:42 | |
*** udesale has quit IRC | 16:49 | |
*** andreas_s has joined #openstack-requirements | 17:03 | |
*** andreas_s has quit IRC | 17:16 | |
*** andreas_s has joined #openstack-requirements | 17:21 | |
*** openstackgerrit has quit IRC | 17:34 | |
*** ralonsoh has quit IRC | 17:34 | |
*** andreas_s has quit IRC | 17:36 | |
*** openstackgerrit has joined #openstack-requirements | 18:05 | |
openstackgerrit | Doug Hellmann proposed openstack/requirements master: remove lower bounds from global requirements https://review.openstack.org/560108 | 18:05 |
openstackgerrit | Doug Hellmann proposed openstack/requirements master: add exclusion specifiers used by swift https://review.openstack.org/560109 | 18:05 |
prometheanfire | dhellmann: good luck :D | 18:08 |
openstackgerrit | Doug Hellmann proposed openstack/requirements master: add exclusion specifiers used by swift https://review.openstack.org/560109 | 18:09 |
dhellmann | prometheanfire : those changes should allow us to add the lower-constraints job to swift in https://review.openstack.org/556255 | 18:13 |
prometheanfire | yarp | 18:14 |
openstackgerrit | Doug Hellmann proposed openstack/requirements master: exclude cmd2 0.8.3 and update to 0.8.4 https://review.openstack.org/560120 | 18:28 |
eandersson | With the new siblin changes, lower-constratins may fail for horizon plugins, as it's pulling an older version of horizon | 19:02 |
prometheanfire | eandersson: details? | 19:03 |
eandersson | http://logs.openstack.org/35/555835/2/check/openstack-tox-lower-constraints/bfc150f/job-output.txt.gz | 19:03 |
eandersson | https://review.openstack.org/#/c/555835/ | 19:03 |
eandersson | not quite sure why it's running the tests here, but basically it's a patch to remove mox3 | 19:03 |
*** openstackgerrit has quit IRC | 19:04 | |
eandersson | and it's pulling in the queens build, and not horizon master | 19:05 |
eandersson | https://github.com/openstack/horizon/blob/master/openstack_dashboard/test/helpers.py#L42 | 19:05 |
prometheanfire | eandersson: you need to remove mox3 from lower-constraints.txt to fix one of the errors you will likely experience | 19:09 |
prometheanfire | but it's required by horizon | 19:17 |
prometheanfire | so I guess it stays | 19:17 |
prometheanfire | eandersson: sounds like horizon needs a release | 19:17 |
prometheanfire | smcginnis: ^ looks like horizon made a change (as a library) that a consumer (designate-dashboard) wants to consume | 19:18 |
prometheanfire | and can't because it's missing a release | 19:18 |
prometheanfire | eandersson: horizon did a bad thing and became both a library and 'end project' | 19:18 |
smcginnis | eandersson: 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 |
eandersson | Would it not be possible to just enable the siblings extension on lower-constraints as well? | 19:25 |
prometheanfire | probably, can you make a patch for that to test? | 19:25 |
eandersson | sure | 19:28 |
*** andreas_s has joined #openstack-requirements | 19:48 | |
*** andreas_s has quit IRC | 19:53 | |
prometheanfire | dirk: dhellmann: mind reviewing? https://review.openstack.org/559367 | 20:05 |
*** openstackgerrit has joined #openstack-requirements | 20:11 | |
openstackgerrit | Doug Hellmann proposed openstack/requirements master: exclude cmd2 0.8.3 https://review.openstack.org/560120 | 20:11 |
*** florianf has quit IRC | 20:25 | |
openstackgerrit | Merged openstack/requirements master: script to fix lower constraints entries that do not match local requirements https://review.openstack.org/558610 | 20:29 |
openstackgerrit | Merged openstack/requirements master: lower-requirements: Adds WMI and PyMI https://review.openstack.org/558196 | 20:30 |
openstackgerrit | Merged openstack/requirements master: fix the logic for handling default for --reqs https://review.openstack.org/560035 | 20:30 |
*** jroll has joined #openstack-requirements | 20:51 | |
jroll | hey 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 happen | 20:53 |
jroll | and I'm not sure if this is going on with any other projects | 20:53 |
jroll | so I guess I'm kind of wondering where a pin should go - ironic, hacking, or u-c? | 20:53 |
jroll | it'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_014738 | 20:54 |
dhellmann | prometheanfire : I don't understand your use of -W on other people's patches. | 21:05 |
prometheanfire | dhellmann: am I not allowed to do that? | 21:05 |
dhellmann | prometheanfire : I wouldn't myself. A -1 feels good enough. | 21:13 |
dhellmann | I mean, obviously the ACLs are there. It just feels heavy-handed | 21:13 |
dhellmann | no one else can remove it, and so the patch is effectively blocked on you now | 21:14 |
*** jrist has quit IRC | 21:14 | |
prometheanfire | I don't mean for it to be, I just want to ensure discussion first, and -W is the only way to do so | 21:14 |
prometheanfire | maybe I'm overstepping, but I thought some projects require the PTL's ack on major changes | 21:15 |
dhellmann | I guess. I feel like a -1 conveys that well enough. I don't have an issue with waiting for more discussion | 21:15 |
prometheanfire | maybe gentoo politics are making me paranoid | 21:15 |
dhellmann | oh, I wasn't even thinking of it as a PTL thing, just another core reviewer | 21:15 |
dhellmann | anyway, it's not a big deal, it just seemed different | 21:17 |
jroll | filed a bug wrt my question above, if someone wants to chime in :) https://storyboard.openstack.org/#!/story/2001819 | 21:18 |
prometheanfire | jroll: oh, missed that lol | 21:18 |
jroll | no worries, not a huge rush | 21:18 |
dhellmann | jroll : sorry, I missed it, too | 21:18 |
*** cjloader has quit IRC | 21:19 | |
dhellmann | in 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 |
prometheanfire | jroll: the pin goes in hacking or ironic | 21:20 |
jroll | oh! | 21:20 |
jroll | I forgot about the blacklist :) | 21:20 |
prometheanfire | pycodestyle is blacklisted so we don't track it | 21:20 |
jroll | awesome, thanks dhellmann and prometheanfire :) | 21:21 |
*** jrist has joined #openstack-requirements | 21:41 | |
*** edmondsw has quit IRC | 21:42 | |
*** edmondsw has joined #openstack-requirements | 21:42 | |
*** edmondsw has quit IRC | 21:43 | |
openstackgerrit | Merged openstack/requirements master: uncap eventlet https://review.openstack.org/559367 | 21:44 |
*** edmondsw has joined #openstack-requirements | 22:09 | |
*** edmondsw has quit IRC | 22:10 | |
*** lbragstad has quit IRC | 22:42 | |
*** lbragstad has joined #openstack-requirements | 23:09 | |
*** cjloader has joined #openstack-requirements | 23:20 | |
*** cjloader has quit IRC | 23:24 |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!