| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-01-11 | |||
| 20:58:41 | mdrabe | mriedem: What if you're starting fresh on ocata? | |
| 20:58:47 | jaypipes | rp.generation = _increment... | |
| 20:59:03 | openstackgerrit | Lee Yarwood proposed openstack/nova: libvirt: utils.get_disk return the target dev of the root device https://review.openstack.org/402716 | |
| 20:59:05 | openstackgerrit | Lee Yarwood proposed openstack/nova: libvirt: Make guest.BlockDevice only use the target dev of a disk https://review.openstack.org/402163 | |
| 20:59:05 | openstackgerrit | Lee Yarwood proposed openstack/nova: libvirt: Use the mirror element to detect when to pivot https://review.openstack.org/399735 | |
| 20:59:07 | mriedem | mdrabe: same deal | |
| 20:59:17 | mriedem | mdrabe: you still have to have a cell0 cell mapping record in the api db | |
| 20:59:28 | mriedem | actually... | |
| 20:59:31 | dansmith | mriedem: the blocker doesn't enforce that it's done first, | |
| 20:59:34 | mriedem | i think the db schema migration ends early | |
| 20:59:41 | dansmith | but you still need that | |
| 20:59:43 | mriedem | if there are no flavors in the api db then it's a noop | |
| 21:00:19 | mriedem | mdrabe: we're also working on this nova-status upgrade check CLI for checking this stuff at any time prior to install/upgrade | |
| 21:00:27 | mriedem | just haven't integrated that yet into the grenade tooling we have | |
| 21:06:56 | mdrabe | mriedem: So say I didn't have any cells in my deployment down the road, things are certainly not going to work right? The cells requirement isn't just for want? | |
| 21:07:12 | openstackgerrit | Jay Pipes proposed openstack/nova: pci: add uuid field to PciDevice object https://review.openstack.org/414768 | |
| 21:07:13 | openstackgerrit | Jay Pipes proposed openstack/nova: placement: update client to set parent provider https://review.openstack.org/385693 | |
| 21:07:13 | openstackgerrit | Jay Pipes proposed openstack/nova: placement: SRIOV PF devices as child providers https://review.openstack.org/414769 | |
| 21:07:31 | dansmith | mdrabe: you can have a cell of one, of course, if that's what you mean | |
| 21:07:48 | dansmith | mdrabe: everyone will be transitioning to a single-cell deployment all the time | |
| 21:08:08 | mdrabe | dansmith: But if I had none like we could in newton that's not gonna fly? | |
| 21:08:48 | dansmith | mdrabe: I'm not sure what you mean.. "one" cell is just a single row in the database -- nothing else is different | |
| 21:09:58 | dansmith | mdrabe: meaning, moving from a newton deployment to an ocata one with just one cell is just adding a row to the database, which says your existing stuff is "cell 1" | |
| 21:10:26 | dansmith | mdrabe: no extra services or configuration like normal cells (i.e. cellsv1) requires | |
| 21:11:04 | mdrabe | dansmith: But if I didn't have a single cell could I still deploy? | |
| 21:11:20 | dansmith | mdrabe: no, there's no such thing as "no cells" anymore | |
| 21:12:53 | dansmith | mdrabe: well, you could deploy with no cells, but you could never create any instances.. :P | |
| 21:14:08 | mdrabe | because the instance has to be built to a specific cell | |
| 21:14:55 | dansmith | yes | |
| 21:16:25 | openstackgerrit | Ed Leafe proposed openstack/nova: Fix missing RP generation update https://review.openstack.org/419137 | |
| 21:16:44 | edleafe | jaypipes: The fix ^^ | |
| 21:17:21 | jaypipes | edleafe: cheers man | |
| 21:17:28 | jaypipes | mriedem: I'm good with https://review.openstack.org/#/c/140045/ now. | |
| 21:17:34 | jaypipes | feel free to wallaby. | |
| 21:19:00 | jaypipes | edleafe, reviewed. | |
| 21:20:23 | edleafe | jaypipes: the functional test that was failing was in the later patch that removed Allocation.create() | |
| 21:20:34 | openstackgerrit | Vasyl Saienko proposed openstack/nova: Support Ironic interface attach/detach in nova virt https://review.openstack.org/364413 | |
| 21:21:03 | edleafe | jaypipes: I'll see what I can add that would be close | |
| 21:21:51 | jaypipes | edleafe: cool, thanks Ed. | |
| 21:21:58 | cdent | more stuff borked? | |
| 21:22:38 | edleafe | cdent: one effect of removing Allocation.create() is that AllocationList.create_all() is much more stringent in its checking | |
| 21:23:15 | edleafe | cdent: and that uncovered a slew of ResourceProvider references that were stale as far as their generation was concerned | |
| 21:23:55 | cdent | in? | |
| 21:24:34 | edleafe | cdent: see https://review.openstack.org/419137 for what I fixed | |
| 21:24:41 | cdent | cool thanks | |
| 21:25:51 | edleafe | cdent: the tests in nova/tests/functional/db/test_resource_provider.py were creating allocations before there was any inventory | |
| 21:26:13 | edleafe | cdent: Allocation.create() allowed that. AllocationList.create_all() doesn't. | |
| 21:29:46 | cdent | gyres with gyres or something | |
| 21:34:32 | openstackgerrit | Jay Pipes proposed openstack/nova: pci: add uuid field to PciDevice object https://review.openstack.org/414768 | |
| 21:34:33 | openstackgerrit | Jay Pipes proposed openstack/nova: placement: SRIOV PF devices as child providers https://review.openstack.org/414769 | |
| 21:36:00 | openstackgerrit | Claudiu Belu proposed openstack/nova: Hyper-V: Adds vNUMA implementation https://review.openstack.org/282407 | |
| 21:42:47 | mriedem | claudiub|2: i was just about done commenting on that one :) | |
| 21:43:17 | dansmith | mriedem: melwitt: don't mind me, just bragging: https://review.openstack.org/#/c/418141/ | |
| 21:43:45 | claudiub|2 | ha, oups. :D | |
| 21:44:47 | openstackgerrit | Ed Leafe proposed openstack/nova: Fix missing RP generation update https://review.openstack.org/419137 | |
| 21:44:50 | mriedem | claudiub|2: comments posted | |
| 21:44:58 | mriedem | dansmith: nice | |
| 21:45:10 | mriedem | dansmith: i can see your nippies from here | |
| 21:45:11 | melwitt | hah, yay | |
| 21:45:29 | mriedem | i guess that should have been in a PM | |
| 21:45:42 | dansmith | haha | |
| 21:46:11 | openstackgerrit | Vasyl Saienko proposed openstack/nova: Support Ironic interface attach/detach in nova virt https://review.openstack.org/364413 | |
| 21:51:55 | bauzas | mriedem: jaypipes: could you maybe look at https://review.openstack.org/#/c/392569/ soon ? | |
| 21:52:33 | bauzas | given it's adding a microversion, I'd make sure that this change could be quickly merged | |
| 21:52:57 | bauzas | edleafe had some comments, but I could follow-up the change | |
| 21:54:23 | bauzas | mriedem: do you know if we have a problem with the Xen 3rd party CI? | |
| 21:54:26 | bauzas | wrt https://review.openstack.org/#/c/365505/ | |
| 21:54:42 | bauzas | I'm waiting to see it there | |
| 21:54:56 | mriedem | bauzas: it was good before | |
| 21:55:22 | mriedem | bauzas: i've rechecked https://review.openstack.org/#/c/416797/ which runs the attach/detach interfaces tests on that patch | |
| 21:55:47 | mriedem | bauzas: and yeah on the placement API change, i'm trying to wrap up some of the hyperv patches and then can move on | |
| 21:56:11 | mriedem | bauzas: we should also get alex_xu and sdague to help review https://review.openstack.org/#/c/392569/ | |
| 21:56:19 | bauzas | mriedem: cool by me | |
| 21:56:59 | bauzas | mriedem: last run of XenServer for the above change was -1 with http://dd6b71949550285df7dc-dda4e480e005aaa13ec303551d2d8155.r49.cf1.rackcdn.com/05/365505/22/check/dsvm-tempest-neutron-network/e2bfe3d/ | |
| 21:57:15 | bauzas | for PS22 | |
| 21:57:42 | mriedem | bauzas: that was some devstack setup issue http://dd6b71949550285df7dc-dda4e480e005aaa13ec303551d2d8155.r49.cf1.rackcdn.com/05/365505/22/check/dsvm-tempest-neutron-network/e2bfe3d/logs/devstacklog.txt.gz | |
| 21:57:59 | bauzas | oh right | |
| 21:58:06 | bauzas | meh | |
| 21:58:21 | mriedem | http://dd6b71949550285df7dc-dda4e480e005aaa13ec303551d2d8155.r49.cf1.rackcdn.com/97/416797/3/check/dsvm-tempest-neutron-network/d75b62d/run_tests.log was happy on 1/9 | |
| 21:58:53 | mriedem | bauzas: if you want to +2 and leave the +W to me that's fine too | |
| 21:59:08 | mriedem | given time of day i mean | |
| 21:59:13 | bauzas | mriedem: just looking at the XenServer change | |
| 21:59:26 | bauzas | I mean the tempest test removal | |
| 21:59:40 | bauzas | well, the exception removal rather | |
| 22:00:41 | bauzas | mriedem: how can we be sure that https://review.openstack.org/#/c/416797/ is run by XenServer CI ? | |
| 22:00:58 | bauzas | I'm verifying that | |
| 22:01:44 | bauzas | actually looking at the regex here http://dd6b71949550285df7dc-dda4e480e005aaa13ec303551d2d8155.r49.cf1.rackcdn.com/97/416797/3/check/dsvm-tempest-neutron-network/d75b62d/run_tests.log | |
| 22:02:19 | bauzas | awesome {2} tempest.api.compute.servers.test_attach_interfaces.AttachInterfacesTestJSON.test_create_list_show_delete_interfaces [349.753425s] ... ok | |
| 22:02:25 | bauzas | mriedem: okay, +Wing it | |
| 22:02:31 | bauzas | because of ^ | |
| 22:02:58 | mriedem | http://dd6b71949550285df7dc-dda4e480e005aaa13ec303551d2d8155.r49.cf1.rackcdn.com/97/416797/3/check/dsvm-tempest-neutron-network/d75b62d/logs/devstacklog.txt.gz | |
| 22:03:06 | mriedem | 2017-01-10 03:15:31.180 | + functions-common:git_clone:566 : git show --oneline 2017-01-10 03:15:31.194 | 823b053 xenapi: support the hotplug of a neutron port | |
| 22:03:10 | mriedem | tells you which nova change it's testing | |
| 22:03:17 | mriedem | which is our happy friend the vif hotplug patch | |
| 22:04:44 | mriedem | btw, easy tangential patch to the flavor notification series https://review.openstack.org/#/c/419086/ | |
| 22:04:53 | mriedem | but i'm going to mark that bp complete since the update patch is merged | |
| 22:11:06 | openstackgerrit | OpenStack Proposal Bot proposed openstack/nova: Updated from global requirements https://review.openstack.org/415978 | |
| 22:13:18 | mriedem | claudiub|2: tweak needed here :) https://review.openstack.org/#/c/140045/ | |
| 22:13:43 | mriedem | or i'll just do it if you're trying to end your day | |
| 22:14:42 | dansmith | "end your day" .. sounds ominous | |