Earlier  
Posted Nick Remark
#openstack-nova - 2018-06-06
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
21:46:37 mriedem dansmith: i know there was an iops one that i saw
21:46:41 mriedem that was the only one i saw i guess
21:47:49 dansmith mriedem: well, what do you think?
21:48:02 openstackgerrit Brianna Poulos proposed openstack/nova master: Add trusted_image_certificates to REST API https://review.openstack.org/486204
21:48:03 openstackgerrit Brianna Poulos proposed openstack/nova master: Add notification support for trusted_certs https://review.openstack.org/563269
21:48:04 openstackgerrit Brianna Poulos proposed openstack/nova master: Add certificate validation docs https://review.openstack.org/560158
21:48:26 openstackgerrit Merged openstack/nova master: Trivial: let internal use only func has _ prefix https://review.openstack.org/572304
21:50:19 mriedem sorry i got artom'ed
21:50:41 artom im in ur patch, artoming ur patch
21:50:42 dansmith ouch
21:50:54 mriedem dansmith: if negative makes more sense then that's fine by me
21:51:00 melwitt dansmith: if none of the other weighers auto-negate things, I would think it's more intuitive if it's specified as a negative number. with the thinking that if the normal pattern is "bigger numbers win" then specifying a larger negative number means weight a failure as far less likely to be selected
21:51:01 mriedem honestly the weight multipliers break my head
21:51:14 dansmith well, I dunno
21:51:14 melwitt am I thinking about that right?
21:51:32 dansmith melwitt: the way I think of all our weighers, the bigger the number, the more significance it has

Earlier   Later