prometheanfire | getting closer to caught up :D | 00:14 |
---|---|---|
prometheanfire | tonyb: I think it's the oslo.policy that messing things up | 00:18 |
openstackgerrit | Matthew Thode proposed openstack/requirements: Updated from generate-constraints https://review.openstack.org/349371 | 00:25 |
tonyb | prometheanfire: You mean WRT the silly large uploads? | 00:32 |
prometheanfire | ya | 00:46 |
openstackgerrit | Merged openstack/requirements: update constraint for oslo.service to new release 1.14.0 https://review.openstack.org/350201 | 01:41 |
openstackgerrit | Merged openstack/requirements: Bump to use min of glance_store 0.16.0 https://review.openstack.org/352510 | 01:41 |
openstackgerrit | Merged openstack/requirements: update constraint for oslo.vmware to new release 2.12.0 https://review.openstack.org/350203 | 01:41 |
openstackgerrit | Merged openstack/requirements: update constraint for keystonemiddleware to new release 4.8.0 https://review.openstack.org/350720 | 01:45 |
prometheanfire | woo | 01:46 |
openstackgerrit | Merged openstack/requirements: update constraint for python-keystoneclient to new release 3.4.0 https://review.openstack.org/350721 | 01:53 |
openstackgerrit | Merged openstack/requirements: update constraint for django_openstack_auth to new release 2.4.0 https://review.openstack.org/352469 | 02:02 |
prometheanfire | tonyb: grenade passed | 02:10 |
tonyb | prometheanfire: passed what? | 02:11 |
prometheanfire | Line 198: os-client-config===1.19.1 | 02:11 |
prometheanfire | This was reverted as it causes problems for grenade | 02:11 |
prometheanfire | but it also passed that too | 02:11 |
tonyb | prometheanfire: Sure requiremnts CI doens't catch it | 02:12 |
tonyb | prometheanfire: It was merged yesterday and then broke. See: https://review.openstack.org/#/c/352454/ | 02:12 |
patchbot | tonyb: patch 352454 - requirements - Revert "update constraint for os-client-config to ... (MERGED) | 02:12 |
prometheanfire | ya | 02:14 |
tonyb | patchbot: frankly I think we shoudl ignore https://review.openstack.org/349371 until we have all the new-release patches merged otherwise it's just a merge conflict wating to happen | 02:14 |
prometheanfire | still needs revert | 02:14 |
prometheanfire | ya | 02:15 |
prometheanfire | give zuul another hour or so, then I'll rebase | 02:15 |
*** coolsvap has joined #openstack-requirements | 02:19 | |
openstackgerrit | dane-fichter proposed openstack/requirements: Add cursive library https://review.openstack.org/352597 | 02:37 |
*** coolsvap is now known as coolsvap_ | 02:59 | |
openstackgerrit | Merged openstack/requirements: update constraint for oslo.context to new release 2.7.0 https://review.openstack.org/348204 | 03:48 |
openstackgerrit | Tony Breeds proposed openstack/requirements: update constraint for os-brick to new release 1.5.0 https://review.openstack.org/350866 | 03:58 |
*** jamespage has quit IRC | 04:02 | |
openstackgerrit | Matthew Thode proposed openstack/requirements: Updated from generate-constraints https://review.openstack.org/349371 | 04:03 |
prometheanfire | tonyb: don't think it needed +w | 04:08 |
prometheanfire | but thanks :D | 04:08 |
tonyb | prometheanfire: https://review.openstack.org/#/c/350866/ ? | 04:29 |
patchbot | tonyb: patch 350866 - requirements - update constraint for os-brick to new release 1.5.0 | 04:29 |
stevemar | prometheanfire: howd you get the keystone tests to not bloat? | 04:41 |
*** coolsvap_ is now known as coolsvap | 04:51 | |
openstackgerrit | Tony Breeds proposed openstack/requirements: Raise the minimum for olso.context to 2.6.0 https://review.openstack.org/352702 | 04:51 |
tonyb | stevemar: it *looks* like oslo.policy is to blame, based on the fact that https://review.openstack.org/#/c/350197/ is failing but the other u-c chnages are just going through | 05:05 |
patchbot | tonyb: patch 350197 - requirements - update constraint for oslo.policy to new release 1... | 05:05 |
tonyb | stevemar: right now I'm *guessing* it's Ie5704f444b22f32876156277595408424c0ac38e | 05:09 |
tonyb | stevemar: but I need some data to verify that ;P | 05:09 |
*** david-lyle_ has joined #openstack-requirements | 05:11 | |
*** david-lyle has quit IRC | 05:13 | |
prometheanfire | ya, haven't looked into which commit it was yet | 05:28 |
prometheanfire | also, sleepy time now | 05:29 |
prometheanfire | if anyone want's to +2+W :P https://review.openstack.org/349371 | 05:29 |
prometheanfire | keep in mind, I think tonyb has some concerns about it (os-brick being one) | 05:29 |
prometheanfire | and neutron things iirc | 05:29 |
tonyb | prometheanfire: It'll probably go to merge conflict before it can merge, I left a -1 with my concerns | 05:33 |
tonyb | prometheanfire: I'll try to identify the best Neutron folks to contact | 05:34 |
prometheanfire | k | 05:34 |
prometheanfire | nn | 05:34 |
prometheanfire | tonyb: feel free to prune it down then so I can +2+W when I wake | 05:35 |
tonyb | prometheanfire: okay | 05:36 |
openstackgerrit | Rabi Mishra proposed openstack/requirements: Bump oslo.context to 2.6.0 https://review.openstack.org/352711 | 05:44 |
stevemar | prometheanfire: tonyb ohhh that is somewhat good to hear (at least from my PoV :) ) | 05:49 |
tonyb | stevemar: and indeed 1.12.0 produces a 30Mb run vs 50Mb with 1.13.0 | 05:50 |
stevemar | tonyb: \o/ | 05:50 |
tonyb | stevemar: Yup that commit is the problem. | 05:54 |
tonyb | stevemar: I've CC'd you on the revert | 06:05 |
tonyb | coolsvap: why'd you +W the oslo bump? It doesn't have 2x+2 | 06:15 |
coolsvap | tonyb: prometheanfire pointed out the other day we dont need 2x+2 for g-r changes | 06:24 |
tonyb | coolsvap: what? that makes no sense | 06:24 |
tonyb | coolsvap: there is a fast path for u-c bumps for OpenStack projects but not g-r | 06:24 |
tonyb | coolsvap: and *certainly* not minimum bumps | 06:25 |
* tonyb adds an agenda item for the next meeting | 06:25 | |
coolsvap | tonyb: please, I removed +W | 06:26 |
coolsvap | tonyb: Changes to update the minimum version of a library developed by the OpenStack community can be approved by one reviewer, as long as the constraints are correct and the tests pass. | 06:27 |
coolsvap | https://github.com/openstack/requirements/blob/master/README.rst#for-upgrading-requirements-versions | 06:27 |
*** ttx has quit IRC | 06:27 | |
tonyb | coolsvap: link? | 06:27 |
*** ttx has joined #openstack-requirements | 06:28 | |
tonyb | coolsvap: Hmm okay I was wrong and I apologise. | 06:28 |
coolsvap | tonyb: np! | 06:29 |
coolsvap | I was also under the same impression when prometheanfire acked | 06:29 |
tonyb | coolsvap: in this case given the sensitivites around oslo.context I'd like some external review | 06:29 |
coolsvap | I also double checked | 06:29 |
coolsvap | tonyb: ack! | 06:29 |
coolsvap | removed +W feel free to +W anytime | 06:30 |
tonyb | We shoudl talk to dhellmann and dims about the rationale for that as I'm not certain it's a good idea | 06:30 |
coolsvap | I have had number of requests for oslo bump | 06:30 |
coolsvap | from heat folks | 06:30 |
coolsvap | tonyb: I think we can talk about it, but I am fine with the rationale | 06:32 |
tonyb | coolsvap: okay email sent. I guess I'm riosk adverse and g-r changes aren't trivial to revert | 06:39 |
coolsvap | tonyb: agree, I think we should not do any changes 2x+2+W | 06:40 |
coolsvap | requirements changes are not trivial in any case and we do not have sufficient coverage atm | 06:40 |
tonyb | coolsvap: I think u-c chnages for OpenStack projects are low enough risk *and* they're trivial to revert if needed but I don't think we shoudl touch g-r without 2x+2 | 06:45 |
tonyb | stevemar: Oh I didn't know you were oslo core. Thanks! | 06:46 |
coolsvap | tonyb: did you sent mail on -dev list? | 06:50 |
*** coolsvap is now known as coolsvap_ | 06:51 | |
* coolsvap_ brb lunch | 06:51 | |
tonyb | coolsvap: Yeah '[requirements] History lession pleas' | 06:51 |
openstackgerrit | OpenStack Proposal Bot proposed openstack/requirements: Updated from generate-constraints https://review.openstack.org/349371 | 07:00 |
openstackgerrit | Ihar Hrachyshka proposed openstack/requirements: Bump minimum for oslo.db to 4.10.0 https://review.openstack.org/351081 | 07:04 |
*** jpena|off is now known as jpena | 07:12 | |
*** david-lyle has joined #openstack-requirements | 07:43 | |
*** david-lyle_ has quit IRC | 07:46 | |
*** coolsvap_ is now known as coolsvap | 07:58 | |
stevemar | tonyb: only in oslo.policy and oslo.cache since those were keystone projects initially :) | 08:01 |
coolsvap | tonyb: i did not receive the msg :/ | 08:01 |
* coolsvap needs to check the mailing list options | 08:04 | |
*** amoralej|off is now known as amoralej | 08:23 | |
tonyb | coolsvap: http://lists.openstack.org/pipermail/openstack-dev/2016-August/101186.html | 08:34 |
tonyb | stevemar: Ahh makes sense | 08:34 |
coolsvap | tonyb: yeah saw that, probably I will wait for someone to repsond to see it my mailbox :) | 08:36 |
*** coolsvap is now known as coolsvap_ | 09:09 | |
openstackgerrit | Tony Breeds proposed openstack/requirements: Updated from generate-constraints https://review.openstack.org/349371 | 09:15 |
openstackgerrit | Merged openstack/requirements: update constraint for os-brick to new release 1.5.0 https://review.openstack.org/350866 | 10:22 |
*** coolsvap_ is now known as coolsvap | 11:39 | |
*** jpena is now known as jpena|lunch | 12:02 | |
*** amoralej is now known as amoralej|lunch | 12:12 | |
openstackgerrit | Monty Taylor proposed openstack/requirements: Re-bump os-client-config constraint https://review.openstack.org/352874 | 12:14 |
*** coolsvap is now known as _coolsvap_ | 12:58 | |
*** jpena|lunch is now known as jpena | 12:58 | |
openstackgerrit | Doug Hellmann proposed openstack/requirements: change django_openstack_auth to not use canonical name https://review.openstack.org/352534 | 13:05 |
openstackgerrit | dane-fichter proposed openstack/requirements: Add cursive library https://review.openstack.org/352597 | 13:17 |
openstackgerrit | Doug Hellmann proposed openstack/requirements: update constraint for python-muranoclient to new release 0.10.0 https://review.openstack.org/352896 | 13:20 |
*** amoralej|lunch is now known as amoralej | 13:25 | |
prometheanfire | the rationalle probably needs updating for gr-updates now that we have more reviewers | 13:37 |
prometheanfire | also | 13:37 |
prometheanfire | does someone want to +2+W the generate-constraints update? | 13:38 |
*** _coolsvap_ is now known as coolsvap | 14:00 | |
openstackgerrit | Doug Hellmann proposed openstack/requirements: update constraint for os_vif to new release 1.1.0 https://review.openstack.org/352926 | 14:12 |
openstackgerrit | Merged openstack/requirements: Updated from generate-constraints https://review.openstack.org/349371 | 15:46 |
*** amoralej is now known as amoralej|brb | 15:47 | |
openstackgerrit | Daniel Berrange proposed openstack/requirements: Require os-vif >= 1.1.0 https://review.openstack.org/352988 | 15:56 |
*** amoralej|brb is now known as amoralej | 16:20 | |
*** jpena is now known as jpena|off | 17:07 | |
*** amoralej is now known as amoralej|off | 17:11 | |
*** david-lyle has quit IRC | 18:41 | |
*** david-lyle has joined #openstack-requirements | 18:42 | |
*** david-lyle_ has joined #openstack-requirements | 19:41 | |
*** david-lyle has quit IRC | 19:43 | |
*** coolsvap has quit IRC | 19:51 | |
*** david-lyle_ is now known as david-lyle | 20:12 | |
*** zigo has quit IRC | 21:32 | |
*** zigo has joined #openstack-requirements | 21:35 | |
*** david-lyle has quit IRC | 23:20 | |
*** david-lyle has joined #openstack-requirements | 23:39 | |
*** rcarrillocruz has quit IRC | 23:58 | |
*** rcarrillocruz has joined #openstack-requirements | 23:59 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!