*** hongbin has quit IRC | 00:12 | |
*** jpena|off has quit IRC | 00:30 | |
*** kragniz has quit IRC | 00:45 | |
*** kragniz has joined #openstack-requirements | 00:47 | |
*** jpena|off has joined #openstack-requirements | 01:01 | |
*** jpena|off has quit IRC | 01:19 | |
*** jpena|off has joined #openstack-requirements | 01:31 | |
tonyb | dirk: I *thought* worflow got dropped with a new revision | 05:11 |
---|---|---|
tonyb | dirk: either way I've removed my -1's but left my -W's to help me keep track of which changes need extra work | 05:12 |
openstackgerrit | Aman Kumar proposed openstack/requirements: Update the global requirement to use kombu<4.0.0 https://review.openstack.org/411778 | 05:26 |
*** armax has joined #openstack-requirements | 06:42 | |
*** openstackgerrit has quit IRC | 06:48 | |
*** openstackgerrit has joined #openstack-requirements | 07:23 | |
openstackgerrit | OpenStack Proposal Bot proposed openstack/requirements: Updated from generate-constraints https://review.openstack.org/413942 | 07:23 |
*** armax has quit IRC | 07:24 | |
*** ihrachys has joined #openstack-requirements | 07:47 | |
*** openstackgerrit has quit IRC | 07:48 | |
*** jpena|off is now known as jpena | 07:50 | |
*** jpena is now known as jpena|off | 08:01 | |
*** ihrachys has quit IRC | 08:07 | |
*** openstackgerrit has joined #openstack-requirements | 08:14 | |
openstackgerrit | Dirk Mueller proposed openstack/requirements: Updated from generate-constraints https://review.openstack.org/413942 | 08:14 |
*** jschlueter has quit IRC | 08:19 | |
*** jschlueter has joined #openstack-requirements | 08:20 | |
*** jschlueter has quit IRC | 08:42 | |
*** jschlueter has joined #openstack-requirements | 08:44 | |
*** jpena|off is now known as jpena | 09:50 | |
*** jpena is now known as jpena|brb | 09:51 | |
openstackgerrit | Dirk Mueller proposed openstack/requirements: Updated from generate-constraints https://review.openstack.org/413942 | 10:32 |
*** ihrachys has joined #openstack-requirements | 10:57 | |
*** jpena|brb is now known as jpena | 11:06 | |
dirk | tonyb: i find w-1 pretty hard to get rid of | 11:35 |
*** sdague has joined #openstack-requirements | 11:36 | |
dirk | prometheanfire: dib still not working, this is quite a tricky not problem.. looks like we need to create the dir the symlink is pointing to.. | 11:38 |
dirk | prometheanfire: why is this not using /tmp/somedir instead? Might be easier | 11:39 |
*** jpena is now known as jpena|lunch | 12:11 | |
*** jpena|lunch is now known as jpena | 12:56 | |
openstackgerrit | chenyingnan proposed openstack/requirements: Updated from global requirements https://review.openstack.org/414150 | 13:56 |
*** armax has joined #openstack-requirements | 14:03 | |
openstackgerrit | OpenStack Proposal Bot proposed openstack/requirements: update constraint for python-novaclient to new release 7.0.0 https://review.openstack.org/414170 | 14:30 |
openstackgerrit | OpenStack Proposal Bot proposed openstack/requirements: update constraint for pymod2pkg to new release 0.7.1 https://review.openstack.org/414172 | 14:33 |
openstackgerrit | OpenStack Proposal Bot proposed openstack/requirements: update constraint for reno to new release 2.0.1 https://review.openstack.org/414178 | 14:43 |
prometheanfire | dirk: portage is hardcoded to /dev/shm iirc | 15:34 |
*** hongbin has joined #openstack-requirements | 16:01 | |
sdague | folks are aware all the requirements jobs are failing on python 3.5? | 16:19 |
sdague | it looks like someone mistakenly made py35 unit test voting on requirements stable/newton | 16:40 |
sdague | so all the stable/newton approves are failing on gate | 16:41 |
dirk | sdague: looking | 17:11 |
dirk | sdague: hmm, looks like that was some mass-change in https://review.openstack.org/#/c/412574/ | 17:13 |
dirk | sdague: looks like it was shot down: https://review.openstack.org/#/c/411332/ | 17:15 |
dirk | tonyb: around by any chance? | 17:15 |
*** mugsie|alt has quit IRC | 17:22 | |
openstackgerrit | Ben Nemec proposed openstack/requirements: Add some missing tripleo projects to the sync list https://review.openstack.org/412636 | 17:31 |
*** mugsie|alt has joined #openstack-requirements | 17:35 | |
*** ihrachys has quit IRC | 17:56 | |
openstackgerrit | Merged openstack/requirements: Raise taskflow to >= 2.7.0 https://review.openstack.org/413624 | 18:45 |
openstackgerrit | Merged openstack/requirements: unblock PyMySQL-0.7.7 https://review.openstack.org/364541 | 18:45 |
*** jpena is now known as jpena|off | 18:45 | |
*** hongbin has quit IRC | 19:05 | |
*** r1chardj0n3s_afk is now known as r1chardj0n3s | 20:04 | |
openstackgerrit | OpenStack Proposal Bot proposed openstack/requirements: update constraint for reno to new release 2.0.2 https://review.openstack.org/414332 | 21:16 |
tonyb | dirk: I am now s'up? | 21:29 |
dirk | tonyb: https://review.openstack.org/#/c/411332/ | 21:35 |
dirk | tonyb: although in parallel a review was started to revert the py35 job addition on stable/ | 21:35 |
dirk | So i guess either one or both is fine | 21:36 |
tonyb | dirk: I like that infra fix better | 21:36 |
tonyb | dirk: as I said backporting that is a stable-policy violation | 21:36 |
dirk | tonyb: which part of the stable policy? | 21:39 |
dirk | tonyb: is it really a new feature? | 21:39 |
dirk | Any case a failing nonvoting job is also bad, and the change is relatively small, does not change an api and is in master for quite some time with no known regressions | 21:41 |
dirk | I think its fine | 21:41 |
dirk | It is a bit on the corner of the usual stable backporting, agreed | 21:42 |
tonyb | dirk: IMO it's a code refactor. I agree that it's a line ball call on backporting it. Even if it *is* a violation it's a minor one | 21:43 |
dirk | tonyb: true, but not for the sake of refactoring. Its a compatibility fix that is done by an isolated refactor | 21:47 |
tonyb | dirk: how about this as a compromise. I'll approve it after 1600 localtime if the project-config change hasn't merged | 21:48 |
tonyb | dirk: I still think it's the wrong solution | 21:48 |
dirk | Yeah. Hmm | 21:49 |
* dirk looks at the infra change again | 21:49 | |
dirk | Okay, so the infra change removes the check all together, rather than merely downgrading it to non-voting | 21:53 |
tonyb | dirk: Yeah. I messed up when I added that and made it global when that really wasn't the aim | 21:54 |
dirk | Right. I see why you believe its out of policy then. | 21:58 |
tonyb | dirk: My worklist for today: https://review.openstack.org/#/q/starredby:tonyb+is:open+topic:feature/add-constraints-support | 21:59 |
dirk | Anyway we don't have to fix it twice, ine fix is enough. Although i still prefer py3 compat, but maybe I'm overly biased | 21:59 |
tonyb | for 100+ patches from the 2 of us that's a pretty reasonable failure rate | 21:59 |
dirk | I am always confused on why a project with such a crazy test automation is so strict on stable .. | 22:00 |
tonyb | dirk: That's a complex issue but we basically responded to what consumers said they wanted. Also we've shown that anythign that isn't master doesn't get the same attention from developers and so is prone to bitrot | 22:02 |
dirk | Yeah, bitrot on stable is a serious issue, but also because it is exceptionally hard to get corner case fixes upstreamed in stable | 22:03 |
dirk | It's often better for your mental health to just patch it downstream.. which doesn't help in interop | 22:04 |
dirk | tonyb: remove thr pbr from your work queue, it is fixed, just forgot to rebase it on top if the latest revision | 22:05 |
tonyb | dirk: I'm interested to hear more about that. It shoudln't be that hard if the patch is appropriate (http://docs.openstack.org/project-team-guide/stable-branches.html#appropriate-fixes) or is the issue that appropriate doesn't cover what you'd like to see backported | 22:05 |
tonyb | dirk: \o/ | 22:06 |
dirk | tonyb: nova-docker is pretty darn deas so probably not important, the rest good luck | 22:07 |
tonyb | dirk: :) Yeah nova-docker is pretty low on the list I s'pose I could hjust abandon it | 22:08 |
tonyb | dirk: but I can be a little obsessive about this kind of thing | 22:08 |
*** sdague has quit IRC | 22:08 | |
dirk | tonyb: as an example i had issues already in backporting fixes for regressions introduced on stable in a.prevous stable backport | 22:15 |
tonyb | dirk: If only you knew the Stable PTL who could help you with that | 22:16 |
tonyb | dirk: that's pretty clearly a regression and shoudl be fixed | 22:16 |
dirk | The policy is very high level which is good but that leaves a lot of room for interpretation as well. | 22:16 |
tonyb | dirk: with some hand waving around timing | 22:16 |
dirk | Sure, but then people argue its not a relevantly user visible | 22:17 |
dirk | tonyb: right but it always requires discussion | 22:19 |
dirk | tonyb: in e.g. linux you have something like "fixes: githash" and tooling that automatically cherry picks that wherever the previous change was cherry picked | 22:20 |
dirk | No discussion whatsoever | 22:20 |
tonyb | dirk: Fair point there's lots of discussion | 22:20 |
dirk | Ad sometimes poking your eye and pulling hair out is just less pain ;-) | 22:21 |
tonyb | :) | 22:21 |
* dirk will go for some sleep | 22:24 | |
tonyb | dirk: rest well | 22:25 |
dirk | Bye, and enjoy your Christmas break in case we don't talk anymore! | 22:25 |
tonyb | Thanks! | 22:25 |
tonyb | dirk: I'm on leave but I'll probably duck in from time to time | 22:25 |
tonyb | dirk: You enjoy your Christmas | 22:25 |
dirk | I might occasionally click through a few reviews but won't do anything else. The week between Christmas and new year is the only real computer free time of the year ;-) | 22:27 |
dirk | Cya | 22:27 |
tonyb | cya | 22:27 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!