Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2018-06-06
17:38:34 tssurya thanks in advance
17:41:20 dansmith jaypipes: yeah I cc'd you on the above a couple weeks ago looking for your comment from a db perspective, so would appreciate a read from you
17:45:17 openstackgerrit Curt Moore proposed openstack/nova-specs master: Add spec for downloading images via RBD https://review.openstack.org/572805
17:46:04 jaypipes dansmith, tssurya: k, will do a review.
17:55:25 openstackgerrit Jan Gutter proposed openstack/nova-specs master: Spec to implement vRouter HW offloads https://review.openstack.org/567148
17:59:26 openstackgerrit Jeremy Freudberg proposed openstack/nova master: DNM Revert "Use ksa session for cinder microversion check" https://review.openstack.org/572853
18:32:51 openstackgerrit Merged openstack/nova master: Add granular placement policy rules for aggregates https://review.openstack.org/570621
18:56:30 jaypipes dansmith, tssurya: k, reviewed. sorry for the delay on that.
18:56:48 tssurya jaypipes: thanks,!
18:58:40 jaypipes tssurya: np. mriedem's heal_allocations patch is a really good example of doing that batch processing stuff.
18:59:08 tssurya absolutely! I will use similar logic then :)
18:59:45 jaypipes tssurya: cool. just remember the key to that is to ensure you pass a sort_key consistently so you get a stable paging system.
19:00:08 mriedem jaypipes: btw, i've got a +2 on heal_allocatoins now if you want to send it home
19:00:33 tssurya jaypipes: ack
19:00:41 jaypipes mriedem: well ahead of you :)
19:00:55 jaypipes mriedem: it's already off to the sarlacc pits.
19:01:14 jaypipes ok, now on to jangutter's spec.
19:02:06 jangutter jaypipes: that's not the sarlacc pits, that's R'lyeh.
19:02:14 jaypipes hehe
19:03:24 mriedem just aim a litter higher
19:03:55 mriedem dansmith: are you following all of this hot star wars action?
19:04:28 jaypipes mriedem: really? I mean, I'm in Florida and was able to feel dansmith's eyeroll over here.
19:04:39 jaypipes :)
19:05:13 dansmith um yeah
19:05:18 jaypipes hehe
19:05:19 dansmith not a huge star wars fan
19:24:09 openstackgerrit Eric Fried proposed openstack/nova-specs master: Spec: Handling Reshaped Provider Trees https://review.openstack.org/572583
19:24:43 openstackgerrit Eric Fried proposed openstack/nova-specs master: Spec: Handling Reshaped Provider Trees https://review.openstack.org/572583
19:28:07 openstackgerrit Doug Hellmann proposed openstack/osc-placement master: fix tox python3 overrides https://review.openstack.org/572917
19:28:17 openstackgerrit Doug Hellmann proposed openstack/os-traits master: fix tox python3 overrides https://review.openstack.org/572921
19:28:20 openstackgerrit Doug Hellmann proposed openstack/os-vif master: fix tox python3 overrides https://review.openstack.org/572922
19:30:09 mriedem dansmith: want to flip to +W now https://review.openstack.org/#/c/567682/
19:31:05 dansmith hell yes I do
19:31:20 mriedem sayonara fping?!
19:32:11 openstackgerrit Eric Fried proposed openstack/nova-specs master: Spec: Handling Reshaped Provider Trees https://review.openstack.org/572583
19:32:19 mriedem oolala
19:35:45 efried mriedem: working on that requirement now - does it need to be removed from test-requirements?
19:36:43 mriedem efried: you need to rebase
19:36:50 mriedem the sphinx thing should go in doc/requirements.txt
19:37:40 openstackgerrit Eric Fried proposed openstack/nova-specs master: Spec: Handling Reshaped Provider Trees https://review.openstack.org/572583
19:37:59 efried mriedem: Did the move ^. Rebase on what?
19:38:12 openstackgerrit Eric Fried proposed openstack/nova-specs master: Spec: Handling Reshaped Provider Trees https://review.openstack.org/572583
19:43:19 mriedem efried: the PTI change merged yesterday
19:43:32 efried mriedem: ack. I rebased. Thanks.
19:44:04 efried mriedem: actually, looks like I got on top of it the first time, by a whisker. Anyway...
19:45:25 mriedem le whew
19:45:56 mriedem mdbooth: so did you ever report a nova bug?
19:46:01 mriedem because i'm ready to push my fix
19:46:16 mriedem nvm https://bugs.launchpad.net/nova/+bug/1775418
19:46:18 openstack Launchpad bug 1775418 in OpenStack Compute (nova) "Swap volume of multiattached volume will corrupt data" [Undecided,New]
19:52:24 openstackgerrit Matt Riedemann proposed openstack/nova master: Block swap volume on volumes with >1 rw attachment https://review.openstack.org/572790
19:55:54 openstackgerrit Merged openstack/nova master: Add granular policy rules for traits in placement https://review.openstack.org/570625
19:57:35 openstackgerrit Matt Riedemann proposed openstack/nova master: Block swap volume on volumes with >1 rw attachment https://review.openstack.org/572790
20:29:04 openstackgerrit Surya Seetharaman proposed openstack/nova-specs master: Handling a down cell https://review.openstack.org/557369
20:32:16 openstackgerrit Brianna Poulos proposed openstack/nova master: Plumb trusted_certs through libvirt driver image paths https://review.openstack.org/561262
20:32:17 openstackgerrit Brianna Poulos proposed openstack/nova master: Add trusted_image_certificates to REST API https://review.openstack.org/486204
20:32:18 openstackgerrit Brianna Poulos proposed openstack/nova master: Add notification support for trusted_certs https://review.openstack.org/563269
20:32:19 openstackgerrit Brianna Poulos proposed openstack/nova master: Add certificate validation docs https://review.openstack.org/560158
20:32:20 openstackgerrit Merged openstack/nova master: Add nova-manage placement heal_allocations CLI https://review.openstack.org/565886
20:44:48 openstackgerrit Brianna Poulos proposed openstack/nova master: Implement certificate_utils https://review.openstack.org/479949
20:44:49 openstackgerrit Brianna Poulos proposed openstack/nova master: Plumb trusted_certs through libvirt driver image paths https://review.openstack.org/561262
20:44:50 openstackgerrit Brianna Poulos proposed openstack/nova master: Add trusted_image_certificates to REST API https://review.openstack.org/486204
20:44:51 openstackgerrit Brianna Poulos proposed openstack/nova master: Add notification support for trusted_certs https://review.openstack.org/563269
20:44:52 openstackgerrit Brianna Poulos proposed openstack/nova master: Add certificate validation docs https://review.openstack.org/560158
20:45:53 openstackgerrit Matt Riedemann proposed openstack/nova master: Remove support for /os-virtual-interfaces REST API https://review.openstack.org/569923
20:46:22 mriedem melwitt: https://review.openstack.org/#/c/565886/10/nova/tests/functional/test_nova_manage.py
20:46:52 melwitt ty
20:47:20 melwitt let's see where I'm going wrong here
21:00:59 openstackgerrit Merged openstack/nova master: Remove support for /os-fping REST API https://review.openstack.org/567682
21:01:05 openstackgerrit Merged openstack/nova master: Add contributor docs on deprecating and removing compute REST APIs https://review.openstack.org/567687
21:14:42 openstackgerrit Matt Riedemann proposed openstack/nova master: Remove support for /os-virtual-interfaces REST API https://review.openstack.org/569923
21:19:16 openstackgerrit Brianna Poulos proposed openstack/nova master: Implement certificate_utils https://review.openstack.org/479949
21:19:17 openstackgerrit Brianna Poulos proposed openstack/nova master: Plumb trusted_certs through libvirt driver image paths https://review.openstack.org/561262
21:19:18 openstackgerrit Brianna Poulos proposed openstack/nova master: Add trusted_image_certificates to REST API https://review.openstack.org/486204
21:19:19 openstackgerrit Brianna Poulos proposed openstack/nova master: Add notification support for trusted_certs https://review.openstack.org/563269
21:19:20 openstackgerrit Brianna Poulos proposed openstack/nova master: Add certificate validation docs https://review.openstack.org/560158
21:32:03 mriedem melwitt: not sure if anyone talked to you about this yet, but https://review.openstack.org/#/c/572583/ is likely going to be a major priority to get done in rocky since nested provider stuff is going to be held up on us handling the data migration properly
21:32:51 mriedem i think we might want to have a call or something to go over relative priorities for things we really need to get done in the 3rd milestone, because i have a feeling there are a few things floating around
21:33:05 mriedem this and handling a down cell are the 2 main ones i'm thinking of
21:33:54 melwitt I'm aware of the importance of the data migration stuff, for whatever reason I didn't realize the "reshape" spec was about that
21:34:29 melwitt yeah, sounds cool to me
21:35:03 mriedem i ignored the whole nrp upgrade thread in the ML, just jumped on the call yesterday that led to the spec
21:35:55 mriedem tl;dr there will be a new placement api to atomically change the inventory and allocations for a given set of providers/consumers, which we'll run on start of a compute when upgrading to rocky, and we'll also have an offline hook to do that also for FFU
21:36:37 melwitt right. I wasn't around in time for the call but I read through the etherpad afterward
21:41:22 melwitt my multi-cell func test fail is related to booting with a server group. boot without a group works fine. argh.
21:42:05 melwitt mriedem: a call sounds cool. are you thinking sometime tomorrow morning? since tomorrow is spec freeze
21:42:28 mriedem umm
21:42:38 mriedem maybe
21:42:51 mriedem i haven't gone through efried's spec in detail yet, and need to go through surya's latest revision
21:43:33 mriedem friday at least
21:43:38 mriedem once the spec freeze dust settles
21:43:44 mriedem and i'd consider these spec freeze exception candidates
21:43:52 melwitt okay, that's what I was about to ask
21:44:11 melwitt if we needed to do all the things by EOD tomorrow or if we have some time as exceptions
21:44:25 mriedem we can always make exceptions
21:45:18 melwitt alright
21:45:49 dansmith mriedem: reading the docs for the other weights we have makes me realize we have some that accept negative values
21:46:14 dansmith so maybe I should make the default -BIGNUMBER and require them to use negative values to properly weigh failures
21:46:15 artom mriedem, drive by review on https://review.openstack.org/#/c/572790/, will go to sleep soon (travelling back to my time zone)
21:46:25 dansmith I thought they were all zero/positive

Earlier   Later