Earlier  
Posted Nick Remark
#openstack-nova - 2018-06-27
19:13:40 mriedem because in that case we should have actually deleted the instance in the cell
19:13:54 melwitt right
19:14:08 mriedem and then queued_for_delete is only read while listing/showing over down cells
19:14:17 mriedem and never goes from True back to False
19:14:31 melwitt ++
19:17:39 melwitt I'm gonna try to do a check with devstack to verify what is returned for a delete when the database is down
19:33:20 mriedem i need a palate cleanser
19:40:24 dansmith mriedem: before I go write examples in the docs that don't satisfy what you're looking for, can you review my response here first? https://review.openstack.org/#/c/578163/2/doc/source/user/cells.rst
19:44:51 mriedem never satisfied
19:44:53 mriedem replied
19:45:02 mriedem i've heard that snickers really satisfies
19:54:53 openstackgerrit Dan Smith proposed openstack/nova master: Allow templated cell_mapping URLs https://review.openstack.org/578163
20:02:38 SpamapS mriedem: we are very far from needing multi-cell yet. Our only Pike clouds are greenfield new deployments. We'll move our bigger clouds last, and will likely be on Rocky+ by then.
20:05:21 mriedem ok
20:06:23 openstackgerrit Matt Riedemann proposed openstack/nova master: Use consumer generation in _heal_allocations_for_instance https://review.openstack.org/577905
20:15:06 openstackgerrit Matt Riedemann proposed openstack/nova master: Wait for vif plugging during live migration job https://review.openstack.org/578551
20:36:27 openstackgerrit Rick Bartra proposed openstack/nova master: Add granularity to os-services API https://review.openstack.org/578553
21:21:43 Nova Hi Eric, please ping me when you can
21:22:33 Sundar efried: Please ping me when you have a moment
21:28:16 openstackgerrit Matt Riedemann proposed openstack/nova master: Make nova-lvm run in check on libvirt changes and compute API tests https://review.openstack.org/569149
21:30:54 openstackgerrit Matt Riedemann proposed openstack/osc-placement master: Usages per project and user (v1.8, v1.9) https://review.openstack.org/514646
21:30:55 openstackgerrit Matt Riedemann proposed openstack/osc-placement master: CLI allocation candidates (v1.10) https://review.openstack.org/514647
21:30:56 openstackgerrit Matt Riedemann proposed openstack/osc-placement master: New dict format of allocations (v1.11, v1.12) https://review.openstack.org/542819
21:30:57 openstackgerrit Matt Riedemann proposed openstack/osc-placement master: Transactionally update allocations (v1.13) https://review.openstack.org/546674
21:30:58 openstackgerrit Matt Riedemann proposed openstack/osc-placement master: Add nested resource providers (v1.14) https://review.openstack.org/546675
21:30:59 openstackgerrit Matt Riedemann proposed openstack/osc-placement master: Limit allocation candidates (v1.15, v1.16) https://review.openstack.org/548043
21:31:00 openstackgerrit Matt Riedemann proposed openstack/osc-placement master: Allocation candidates parameter: required (v1.17) https://review.openstack.org/548326
21:31:15 efried Sundar: Hello
21:32:28 Sundar efried: Can you please try out https://drive.google.com/open?id=1ICdNeG3bbX4l-Qkj1Q8IzF8QmLsn4U7K ?
21:32:54 Sundar This patch adds the sequence diag
21:33:52 efried Sundar: Okay. We're still busted on the doc build in general, but I can try to build it locally. At a glance, I believe you might need semicolons at the end of each line.
21:43:27 openstackgerrit Merged openstack/nova master: libvirt: remove old rbd snapshot removal error handling https://review.openstack.org/566369
21:57:21 openstackgerrit Takashi NATSUME proposed openstack/nova master: Remove mox in unit/network/test_neutronv2.py (1) https://review.openstack.org/572018
21:57:30 openstackgerrit Takashi NATSUME proposed openstack/nova master: Remove mox in unit/network/test_neutronv2.py (2) https://review.openstack.org/573481
21:57:37 openstackgerrit Takashi NATSUME proposed openstack/nova master: Remove mox in unit/network/test_neutronv2.py (3) https://review.openstack.org/574104
21:57:46 openstackgerrit Takashi NATSUME proposed openstack/nova master: Remove mox in unit/network/test_neutronv2.py (4) https://review.openstack.org/574106
21:57:53 openstackgerrit Takashi NATSUME proposed openstack/nova master: Remove mox in unit/network/test_neutronv2.py (5) https://review.openstack.org/574110
22:06:24 openstackgerrit Merged openstack/nova master: Handle network-changed event for a specific port https://review.openstack.org/465787
22:06:32 openstackgerrit Merged openstack/nova master: Refactor _build_device_metadata https://review.openstack.org/533804
22:06:42 openstackgerrit Merged openstack/nova master: Consider hostdev devices when building metadata https://review.openstack.org/533805
22:06:50 openstackgerrit Merged openstack/nova master: Add information of deprecation nova-network in system-admin.rst https://review.openstack.org/576698
22:06:58 openstackgerrit Merged openstack/nova master: ironic: bugfix: ensure a host is set for volume connectors https://review.openstack.org/571982
22:26:57 efried Sundar: Okay, I worked with infra a bit and should have the doc build working properly now. I also incorporated your seqdiag (which builds fine, btw) and reproposed your patch.
22:30:13 Sundar efried: Thanks a lot! I didn't mean to impose that on you. Had it worked on your local environment, I would have pushed it myself.
22:30:36 Sundar s/on your/in your/
22:30:58 openstackgerrit Takashi NATSUME proposed openstack/nova master: Remove mox in unit/network/test_neutronv2.py (6) https://review.openstack.org/574113
22:31:11 efried Sundar: No, I wanted to get the specs rendering and built properly by zuul. I had a POC patch on there anyway, so this just allowed me to replace it with the real thing.
22:31:59 efried Sundar: Do you have the background on why we're not putting specs into cyborg-specs yet?
22:48:28 Sundar efried: In the Cyborg meetings, it was discussed that this is the direction to go but we can do it in the next release, or after the specs are merged
23:18:03 openstackgerrit Matt Riedemann proposed openstack/osc-placement master: Usages per project and user (v1.8, v1.9) https://review.openstack.org/514646
23:18:04 openstackgerrit Matt Riedemann proposed openstack/osc-placement master: CLI allocation candidates (v1.10) https://review.openstack.org/514647
23:18:05 openstackgerrit Matt Riedemann proposed openstack/osc-placement master: New dict format of allocations (v1.11, v1.12) https://review.openstack.org/542819
23:18:06 openstackgerrit Matt Riedemann proposed openstack/osc-placement master: Transactionally update allocations (v1.13) https://review.openstack.org/546674
23:18:07 openstackgerrit Matt Riedemann proposed openstack/osc-placement master: Add nested resource providers (v1.14) https://review.openstack.org/546675
23:18:08 openstackgerrit Matt Riedemann proposed openstack/osc-placement master: Limit allocation candidates (v1.15, v1.16) https://review.openstack.org/548043
23:18:09 openstackgerrit Matt Riedemann proposed openstack/osc-placement master: Allocation candidates parameter: required (v1.17) https://review.openstack.org/548326
23:19:20 openstackgerrit Takashi NATSUME proposed openstack/nova master: Remove mox in unit/network/test_neutronv2.py (7) https://review.openstack.org/574974
23:19:22 openstackgerrit Matt Riedemann proposed openstack/osc-placement master: Usages per project and user (v1.8, v1.9) https://review.openstack.org/514646
23:19:23 openstackgerrit Matt Riedemann proposed openstack/osc-placement master: CLI allocation candidates (v1.10) https://review.openstack.org/514647
23:19:24 openstackgerrit Matt Riedemann proposed openstack/osc-placement master: New dict format of allocations (v1.11, v1.12) https://review.openstack.org/542819
23:19:25 openstackgerrit Matt Riedemann proposed openstack/osc-placement master: Transactionally update allocations (v1.13) https://review.openstack.org/546674
23:19:26 openstackgerrit Matt Riedemann proposed openstack/osc-placement master: Add nested resource providers (v1.14) https://review.openstack.org/546675
23:19:27 openstackgerrit Matt Riedemann proposed openstack/osc-placement master: Limit allocation candidates (v1.15, v1.16) https://review.openstack.org/548043
23:19:28 openstackgerrit Matt Riedemann proposed openstack/osc-placement master: Allocation candidates parameter: required (v1.17) https://review.openstack.org/548326
23:19:54 openstackgerrit Takashi NATSUME proposed openstack/nova master: Remove mox in unit/network/test_neutronv2.py (8) https://review.openstack.org/575311
23:20:16 openstackgerrit Takashi NATSUME proposed openstack/nova master: Remove mox in unit/network/test_neutronv2.py (9) https://review.openstack.org/575581
23:20:37 openstackgerrit Takashi NATSUME proposed openstack/nova master: Remove mox in unit/network/test_neutronv2.py (10) https://review.openstack.org/576017
23:20:52 openstackgerrit Takashi NATSUME proposed openstack/nova master: Remove mox in unit/network/test_neutronv2.py (11) https://review.openstack.org/576018
23:21:34 openstackgerrit Takashi NATSUME proposed openstack/nova master: Remove mox in unit/network/test_neutronv2.py (12) https://review.openstack.org/576019
23:27:17 efried Do virt drivers call into os-vif or is all of that interaction done outside of virt?
23:36:40 melwitt efried: happens in the driver, example https://github.com/openstack/nova/blob/master/nova/virt/libvirt/vif.py
23:37:25 efried melwitt: Thank you
23:50:52 openstackgerrit Takashi NATSUME proposed openstack/nova master: Remove mox in unit/network/test_neutronv2.py (13) https://review.openstack.org/576020
23:51:19 openstackgerrit Takashi NATSUME proposed openstack/nova master: Remove mox in unit/network/test_neutronv2.py (14) https://review.openstack.org/576027
23:51:35 openstackgerrit Takashi NATSUME proposed openstack/nova master: Remove mox in unit/network/test_neutronv2.py (15) https://review.openstack.org/576031
23:52:02 openstackgerrit Takashi NATSUME proposed openstack/nova master: Remove mox in unit/network/test_neutronv2.py (16) https://review.openstack.org/576299
23:52:33 openstackgerrit Takashi NATSUME proposed openstack/nova master: Remove mox in unit/network/test_neutronv2.py (17) https://review.openstack.org/576344
23:52:46 openstackgerrit Takashi NATSUME proposed openstack/nova master: Remove mox in unit/network/test_neutronv2.py (18) https://review.openstack.org/576673
23:53:01 openstackgerrit Takashi NATSUME proposed openstack/nova master: Remove mox in unit/network/test_neutronv2.py (19) https://review.openstack.org/576676
#openstack-nova - 2018-06-28
00:16:34 openstackgerrit Matt Riedemann proposed openstack/nova stable/queens: Handle network-changed event for a specific port https://review.openstack.org/578612
00:21:15 openstackgerrit Matt Riedemann proposed openstack/nova master: Mention server status in api-ref when rebuild https://review.openstack.org/576438
00:24:05 openstackgerrit Takashi NATSUME proposed openstack/nova master: Remove mox in unit/network/test_neutronv2.py (20) https://review.openstack.org/576689
00:24:55 openstackgerrit Takashi NATSUME proposed openstack/nova master: Remove mox in unit/network/test_neutronv2.py (21) https://review.openstack.org/576709
00:26:57 mnaser please forigve me but
00:27:14 mnaser where are 'flavors' stored that are tied to the instance
00:27:34 mnaser bad idea™ but customer who did not have qos quotas needs to have them applied on existing instance
00:27:43 mnaser and well, changing the flavor isnt tenough
00:29:36 efried mnaser: It's possible you need to do a "resize" to make that happen.
00:29:57 mnaser efried: resize to same flavor, wonder if i can do that, that might be a clean way of doing it
00:30:18 efried mnaser: You said "changing the flavor" - you mean you edited the existing flavor?
00:30:22 mnaser yes
00:30:41 efried That's probably a bad idea™ :)
00:31:06 mnaser even worse i was thinking of using the nova code base to write a python script that modifies the objects
00:31:17 mnaser so it goes through the actual codeflow to make the change in the db layer
00:31:25 efried mnaser: So, assuming resize-to-same-flavor isn't a thing nova will let you do (it may be, but I wouldn't be surprised if it bounced) you could probably just create a new flavor identical to the old (edited) one.
00:31:48 efried mnaser: Oh, that's *definitely* a rathole you don't want to go down.
00:32:00 mnaser haha, i thought it would have been slightly cleaner
00:32:07 mnaser inspired by nova-manage
00:32:17 efried Not knowing the details, I would expect that there's more to it than just db data
00:32:31 mnaser there is instance_extra.

Earlier   Later