| dims | dirk : see https://review.openstack.org/#/c/424906/1/neutron/tests/functional/test_server.py | 00:00 |
|---|---|---|
| dims | "return proc.ppid()" | 00:00 |
| * dims is confused | 00:02 | |
| *** sdague has joined #openstack-requirements | 00:02 | |
| dirk | dims: me too. | 00:03 |
| dirk | dims: the patch is basically a copy of https://git.openstack.org/cgit/openstack/oslo.reports/tree/oslo_reports/models/process.py#n43 | 00:03 |
| dirk | which is there for quite some time | 00:03 |
| dirk | and also the ppid -> ppid() change works for me | 00:03 |
| JayF | dims: with existing code, we work in psutil 1+2 | 00:04 |
| JayF | dims: new code 1/2/5 work, but rest doesn't break shit | 00:04 |
| dims | JayF : ok, so dasm gets to tell us the best way forward then | 00:05 |
| dims | dasm : revert or use dirk's patch your call. all of us would prefer to move forward for sure | 00:05 |
| dasm | yep. would be good to proceed with that without reverting anything. | 00:06 |
| dirk | dasm: I strongly prefer to update to a newer psutil, its been a pain | 00:06 |
| dasm | both patches are on gate | 00:06 |
| dasm | dirk: agree | 00:06 |
| dasm | i hope, both patches will pass, then it means that new psutil broke it, and it's straight forward to fix it | 00:07 |
| dims | :) | 00:09 |
| dasm | all: we have pretty good success rate for new psutil: project:"openstack/neutron" AND build_status:"SUCCESS" AND message:"psutil==5.0.1" (1759) count per 1h | (1759 hits) | 00:09 |
| JayF | dims: https://review.openstack.org/#/c/424895/ is the patch to IPA; it's pretty straightforward how it's busted | 00:10 |
| JayF | dims: also it's only breaking IPA gate; ironic + dependencies are fine, so it's not even a major issue fwiw | 00:10 |
| dims | JayF : cool that's good news | 00:10 |
| JayF | we try really hard to isolate the ironic-proper jobs from failures :) if everything that broke ipa broke ironic we'd be in a lot of hurt :) | 00:11 |
| dims | understood :) | 00:12 |
| JayF | dims: one related question/request: can we tighten up the lower end of that? | 00:12 |
| JayF | dims: as it stands we'll have to support 5 major versions of psutil, which is crazy :/ | 00:12 |
| dirk | JayF: I'd love to propose that.. as well | 00:13 |
| dirk | but time is getting really short, so its not going to get in for ocata I guess | 00:13 |
| dirk | JayF: its a nightmare to support ,< 2 really.. the rest of the changes are really minor | 00:13 |
| dims | dirk : we might as well | 00:13 |
| dims | right | 00:13 |
| JayF | dirk: well we def had an API change in one of the only things IPA uses it for from 2->5 | 00:14 |
| dirk | anything beyond 2.0 were tiny changes and bugfixes | 00:14 |
| dirk | JayF: yeah, I see that to my surprise | 00:14 |
| JayF | it's more just, ugly to have so many statements that switch based on major version number | 00:14 |
| dirk | JayF: to be honest most usages of psutil that I could find with codesearch where not that advanced | 00:14 |
| dasm | dims: FYI. I see that depends-on patch failed for functional tests. | 00:15 |
| JayF | We could maybe even deprecate that value at some point, tbh, the physical_mb we calculate with dmidecode now is more accurate/closer to what most deployers expect | 00:15 |
| dasm | Let's wait for dirk's one | 00:15 |
| dasm | (which i believe, should work :) ) | 00:15 |
| dasm | dirk: dims looks like functional is green | 00:16 |
| dasm | dirk: thanks man | 00:16 |
| JayF | I'm getting back off IRC for the evening :) if there's anything else you all need, my email address for the list (jay at jvf dot cc) will get you an answer tonight if you need one :) | 00:16 |
| dims | dasm : so what's the verdict? | 00:16 |
| dasm | dims: no revert. we need to merge dirk's change asap | 00:16 |
| dims | thanks JayF | 00:16 |
| dims | ack thanks dasm | 00:17 |
| dims | thanks everyone, back in a bit to check | 00:17 |
| *** hongbin has quit IRC | 00:22 | |
| dirk | dasm: thanks for the support | 00:24 |
| dasm | dirk: thanks for quick reaction. without you, i wouldn't even know where to look :) | 00:24 |
| *** hoangcx has joined #openstack-requirements | 00:39 | |
| openstackgerrit | Merged openstack/requirements: add check-python2-support command https://review.openstack.org/424363 | 00:44 |
| prometheanfire | dirk: so... mind +2+W https://review.openstack.org/404878 ? | 00:58 |
| *** sdague has quit IRC | 01:01 | |
| prometheanfire | no w? | 01:01 |
| dirk | prometheanfire: it's too late for oslo.messaging now, isn't it? | 01:01 |
| prometheanfire | maybe? | 01:01 |
| prometheanfire | even if it is doesn't hurt to update on our side | 01:02 |
| openstackgerrit | Merged openstack/requirements: update constraint for python-tackerclient to new release 0.9.0 https://review.openstack.org/424887 | 01:02 |
| dirk | prometheanfire: yeah.. still debating with myself on the W+1 wording | 01:06 |
| dirk | I see it as a failure that we didn't get both sides to agree on the same library | 01:09 |
| dirk | This is a bit outside of the goal of g-r | 01:09 |
| dirk | But apparently it was too hard this time | 01:09 |
| dirk | It's yet another bullet in the list of things that are messy in OpenStack dependencies | 01:10 |
| prometheanfire | :D | 01:15 |
| openstackgerrit | Merged openstack/requirements: update constraint for python-neutronclient to new release 6.1.0 https://review.openstack.org/424769 | 01:19 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/requirements: update constraint for os-brick to new release 1.11.0 https://review.openstack.org/424941 | 02:24 |
| *** armax has quit IRC | 02:31 | |
| *** ihrachys has joined #openstack-requirements | 03:23 | |
| *** udesale has joined #openstack-requirements | 03:33 | |
| *** ihrachys has quit IRC | 03:37 | |
| prometheanfire | tonyb: I assume you aren't doing the meeting? | 03:43 |
| openstackgerrit | Michael Carden proposed openstack/requirements: Update sphinx requirement https://review.openstack.org/418772 | 04:00 |
| *** armax has joined #openstack-requirements | 04:10 | |
| *** armax has quit IRC | 04:30 | |
| *** armax has joined #openstack-requirements | 04:55 | |
| *** armax has quit IRC | 04:56 | |
| stevemar | prometheanfire: hehe you typo'd the subject headr | 05:10 |
| stevemar | prometheanfire: s/swift/sphinx | 05:10 |
| stevemar | ah you fixed it | 05:10 |
| stevemar | boourns | 05:10 |
| prometheanfire | :P | 05:11 |
| *** udesale__ has joined #openstack-requirements | 05:17 | |
| *** udesale has quit IRC | 05:17 | |
| openstackgerrit | Steve Martinelli proposed openstack/requirements: bump django_openstack_auth to 3.1.0 https://review.openstack.org/424981 | 05:19 |
| *** udesale has joined #openstack-requirements | 05:19 | |
| *** udesale__ has quit IRC | 05:21 | |
| openstackgerrit | Merged openstack/requirements: update constraint for os-brick to new release 1.11.0 https://review.openstack.org/424941 | 06:03 |
| *** armax has joined #openstack-requirements | 06:13 | |
| *** armax has quit IRC | 06:16 | |
| prometheanfire | dirk: I see you sleep as much as I do | 06:20 |
| dirk | prometheanfire: today, yes ;-( | 06:22 |
| * dirk had way too muvh caffeine yesterday | 06:22 | |
| prometheanfire | heh | 06:23 |
| prometheanfire | just think 3.5 more hours til the meeting | 06:23 |
| prometheanfire | which will probably last 15 minutes | 06:24 |
| prometheanfire | wooo time usage | 06:24 |
| *** udesale__ has joined #openstack-requirements | 06:56 | |
| *** hoangcx_ has joined #openstack-requirements | 06:57 | |
| *** hoangcx has quit IRC | 07:00 | |
| dirk | prometheanfire: if it's better for you we can do it earlier | 07:08 |
| * dirk doesn't remember the normal body timezone of prometheanfire | 07:09 | |
| dirk | looks like we can not approve the Kafka bump with one -2 | 07:10 |
| prometheanfire | too bad we don't have willikins in here | 07:10 |
| prometheanfire | !time prometheanfire | 07:10 |
| openstack | prometheanfire: Error: "time" is not a valid command. | 07:10 |
| prometheanfire | heh | 07:10 |
| prometheanfire | !help | 07:10 |
| openstack | prometheanfire: (help [<plugin>] [<command>]) -- This command gives a useful description of what <command> does. <plugin> is only necessary if the command is in more than one plugin. | 07:10 |
| prometheanfire | anyway, I'm US-Central | 07:10 |
| dirk | So changing the change id it is | 07:10 |
| prometheanfire | ya, that kafka thing is kinda annoying | 07:10 |
| prometheanfire | we could just make a new review | 07:11 |
| prometheanfire | we've had to do that in the past | 07:11 |
| prometheanfire | the only thing I wanted to bring up was the sphinx bump, for which I started a thread on -dev | 07:11 |
| prometheanfire | and that I'd be -2'ing everything at 20:00 UTC thursday | 07:12 |
| dirk | Ok, i was wondering about that as well | 07:14 |
| prometheanfire | but that's it | 07:16 |
| prometheanfire | you think we should submit a new review for the kafka thing to unblock it? | 07:16 |
| openstackgerrit | Dirk Mueller proposed openstack/requirements: requires python-kafka>=1.3.1 https://review.openstack.org/425014 | 07:16 |
| dirk | prometheanfire: sorry, already did ;-) | 07:18 |
| prometheanfire | :D | 07:18 |
| prometheanfire | good job | 07:18 |
| dirk | prometheanfire: are you sure about the time+date ? I'm fine with it but I didn't check with release mgmt or so | 07:18 |
| dirk | the only other thing I'd like to get in is the discover removal | 07:18 |
| dirk | but thats depending on tempest-lib which fails and is abandoned | 07:18 |
| dirk | I'm tempted to remove that depends-on and merge it | 07:19 |
| dirk | it really should be a no brainer imho .. and I'd like to have that before the stable branch is cut | 07:19 |
| dirk | when is that happening? and who does it when tony is out? | 07:19 |
| prometheanfire | yes, ran it past doug | 07:19 |
| prometheanfire | ya, since the discover thing has deps we probably won't be able to get it in | 07:20 |
| prometheanfire | but since it's a removal it's not horrible to stick around a release | 07:20 |
| prometheanfire | just a unsightly blemish | 07:20 |
| dirk | yeah, not horrible, but it allows more usages to creep in rather than less | 07:22 |
| dirk | it is also the oldest review we have | 07:23 |
| * dirk is in spring cleaning mood | 07:23 | |
| prometheanfire | indeed | 07:24 |
| openstackgerrit | Dirk Mueller proposed openstack/requirements: Remove discover from g-r and u-c https://review.openstack.org/325875 | 07:26 |
| prometheanfire | looks like the sphinx change is a good one | 07:27 |
| prometheanfire | https://review.openstack.org/418772 | 07:27 |
| prometheanfire | so, have at it | 07:27 |
| prometheanfire | I'm not sure I agree with removing tempest-lib | 07:31 |
| prometheanfire | and why won't it ever be merged? | 07:31 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/requirements: Updated from generate-constraints https://review.openstack.org/425025 | 07:34 |
| *** hoangcx has joined #openstack-requirements | 07:46 | |
| prometheanfire | anyway, dirk, I'm going to go to bed, no meeting for me today, I don't think I was suposed to run this meeting | 07:48 |
| prometheanfire | just last one... | 07:48 |
| *** hoangcx_ has quit IRC | 07:48 | |
| dirk | prometheanfire: thanks, have a good sleep | 08:04 |
| dirk | I'm still a bit concerned with the sphinx bump, so i spent the last half hour on codesearch | 08:05 |
| dirk | And linux.conf.au videos, damn you procastination | 08:05 |
| *** jpich has joined #openstack-requirements | 08:55 | |
| *** hoangcx has quit IRC | 09:00 | |
| *** udesale__ has quit IRC | 09:47 | |
| *** udesale has quit IRC | 09:47 | |
| *** strigazi_AFK is now known as strigazi | 09:50 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/requirements: update constraint for python-keystoneclient to new release 3.10.0 https://review.openstack.org/425093 | 09:55 |
| *** openstackgerrit has quit IRC | 10:17 | |
| *** jpich has quit IRC | 10:35 | |
| *** openstackgerrit has joined #openstack-requirements | 11:57 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/requirements: update constraint for python-heatclient to new release 1.8.0 https://review.openstack.org/425147 | 11:57 |
| -openstackstatus- NOTICE: Gerrit is going to be restarted due to slow performance | 12:47 | |
| -openstackstatus- NOTICE: Gerrit has been successfully restarted | 12:50 | |
| *** udesale has joined #openstack-requirements | 13:33 | |
| *** ihrachys has joined #openstack-requirements | 13:56 | |
| *** sdague has joined #openstack-requirements | 14:15 | |
| *** udesale has quit IRC | 14:36 | |
| *** hongbin has joined #openstack-requirements | 14:55 | |
| *** udesale has joined #openstack-requirements | 15:00 | |
| *** sdague has quit IRC | 15:12 | |
| *** sdague has joined #openstack-requirements | 16:03 | |
| *** armax has joined #openstack-requirements | 16:18 | |
| *** udesale has quit IRC | 16:26 | |
| dtroyer | Good day req folk! Just wanted to mention that stevemar has removed -W from https://review.openstack.org/#/c/424847/ and it should be ready to move. Thanks | 17:06 |
| stevemar | prometheanfire: dims dirk ^ | 17:06 |
| prometheanfire | thanks | 17:07 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/requirements: update constraint for python-manilaclient to new release 1.13.0 https://review.openstack.org/425310 | 17:32 |
| dims | dtroyer : stevemar : give me a sec :) | 17:50 |
| dims | looks like prometheanfire already too care of it :) | 17:50 |
| prometheanfire | :D | 17:51 |
| dtroyer | thanks guys | 17:51 |
| *** sdague has quit IRC | 18:03 | |
| *** karthik__ has joined #openstack-requirements | 18:23 | |
| *** karthik__ has quit IRC | 18:26 | |
| stevemar | prometheanfire: question, who has stable core permission? https://review.openstack.org/#/c/424865/ | 18:28 |
| stevemar | prometheanfire: nvm, dhellmann and fungi have it :) | 18:28 |
| dhellmann | stevemar : should we raise the minimum version in master as well as updating that constraint? | 18:30 |
| stevemar | dhellmann: hmm, i thought i saw it raised in master | 18:30 |
| dhellmann | ok, I hadn't checked | 18:30 |
| prometheanfire | stevemar: yep | 18:31 |
| fungi | stevemar: more specifically, stable-maint-core does: https://review.openstack.org/#/admin/groups/stable-maint-core (infra-core is there as a backup more for emergencies and infrastructure-related breakages) | 18:32 |
| fungi | also the Release Managers group are implicit members of stable-maint-core just in case they need it | 18:32 |
| stevemar | dhellmann: pyparsing>=2.0.7 in master and 2.1.8 (proposed bump is to 2.1.10) in newton :) | 18:34 |
| stevemar | dhellmann: err wait, confused g-r and u-c | 18:34 |
| dhellmann | stevemar : we should probably raise the min in g-r to 2.1.10 if we have projects that depend on that because of setuptools | 18:35 |
| stevemar | dhellmann: master: g-r >= 2.0.7 u-c 2.1.10 | 18:35 |
| dhellmann | if you propose the change and link it to the same bug, prometheanfire and I can approve it | 18:35 |
| stevemar | dhellmann: newton: g-r >= 2.0.1 u-c 2.1.8 | 18:36 |
| dhellmann | I don't think we can change the lower bound for newton, unfortunately | 18:36 |
| stevemar | oh | 18:36 |
| stevemar | but upper is OK | 18:36 |
| prometheanfire | dhellmann: for stable? | 18:37 |
| dhellmann | we're seeing the issue because we're forcing the newer setuptools in our test environments, but stable deployments won't do that | 18:37 |
| stevemar | ok, i can do a patch for master too, set minimum to 2.1.10 | 18:37 |
| prometheanfire | I don't have stable access | 18:37 |
| dhellmann | we can change the u-c in newton and g-r and u-c in master | 18:37 |
| dhellmann | prometheanfire : master | 18:37 |
| prometheanfire | ah, k | 18:37 |
| dhellmann | the min there is no longer accurate if we need 2.1.10 in order for things to actually work | 18:37 |
| stevemar | u-c master is already at the right value | 18:37 |
| dhellmann | ok, so just the g-r list needs the update then | 18:38 |
| prometheanfire | then you only have to edit one file :P | 18:38 |
| * prometheanfire goes to read backlog | 18:38 | |
| openstackgerrit | Steve Martinelli proposed openstack/requirements: bump pyparsing minimum to 2.1.10 https://review.openstack.org/425336 | 18:43 |
| stevemar | dhellmann: done ^ | 18:43 |
| dhellmann | stevemar : could you add the bug reference to the commit message? | 18:44 |
| stevemar | ack | 18:45 |
| openstackgerrit | Steve Martinelli proposed openstack/requirements: bump pyparsing minimum to 2.1.10 https://review.openstack.org/425336 | 18:45 |
| prometheanfire | done | 18:46 |
| *** sdague has joined #openstack-requirements | 19:00 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/requirements: update constraint for python-muranoclient to new release 0.12.0 https://review.openstack.org/425349 | 19:03 |
| *** karthik__ has joined #openstack-requirements | 19:20 | |
| *** karthik__ has quit IRC | 19:24 | |
| *** sdague has quit IRC | 19:26 | |
| *** sdague has joined #openstack-requirements | 19:27 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/requirements: update constraint for python-ceilometerclient to new release 2.8.0 https://review.openstack.org/425361 | 19:34 |
| *** karthik__ has joined #openstack-requirements | 19:52 | |
| *** sdague has quit IRC | 20:06 | |
| openstackgerrit | Merged openstack/requirements: bump django_openstack_auth to 3.1.0 https://review.openstack.org/424981 | 20:12 |
| *** sdague has joined #openstack-requirements | 20:18 | |
| *** karthik__ has quit IRC | 20:33 | |
| *** sdague has quit IRC | 20:53 | |
| *** sdague has joined #openstack-requirements | 20:54 | |
| * sigmavirus is hoping some requirements cores could review https://review.openstack.org/#/c/424758/ (besides prometheanfire who already has) | 21:31 | |
| prometheanfire | yourwelcome | 21:31 |
| sigmavirus | Once it's merged, Glance should be able to complete rolling upgrades work | 21:31 |
| sigmavirus | (or come really darn close) | 21:31 |
| sigmavirus | prometheanfire: I agree with you though that it is awfully late in the cycle | 21:32 |
| sigmavirus | I'd argue you should have the release calendar updated to have a new-requirements-freeze a week or two earlier than requirements hard-freeze | 21:32 |
| prometheanfire | that's a good idea | 21:33 |
| sigmavirus | #helping | 21:34 |
| prometheanfire | your a helper | 21:37 |
| sigmavirus | your? | 21:37 |
| prometheanfire | hey, I'm not on the docs team | 21:39 |
| sigmavirus | prometheanfire: go figure, I aren't either | 21:40 |
| *** sdague has quit IRC | 22:02 | |
| *** ihrachys has quit IRC | 22:20 | |
| openstackgerrit | Dirk Mueller proposed openstack/requirements: Raise psutil min version to 3.0.1 https://review.openstack.org/425455 | 23:13 |
| openstackgerrit | Dirk Mueller proposed openstack/requirements: remove upper cap of kombu/amqp https://review.openstack.org/410176 | 23:15 |
| openstackgerrit | Merged openstack/requirements: update constraint for python-heatclient to new release 1.8.0 https://review.openstack.org/425147 | 23:18 |
| *** karthik__ has joined #openstack-requirements | 23:39 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!