Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2017-09-14
21:56:03 sdague actualy never mind
21:56:17 sdague https://github.com/openstack/nova/blob/cfdec41eeec5fab220702efefdaafc45559aeb14/nova/db/sqlalchemy/api.py#L2312-L2351
21:56:26 sdague we are in the ELSE
21:59:04 bauzas Venn diagram FTW.
22:01:53 sdague I expect what is happening with nova show is that it does a nova list behind the scenes with the name filter and getting > 1 result goes funky
22:03:43 melwitt yeah, I was thinking the same
22:03:47 mriedem tasker: what's your email for a co-authored-by on this patch?
22:04:02 sdague https://github.com/openstack/python-novaclient/blob/57c397d822a77d2f832ce9ddae349e5d4073c271/novaclient/v2/shell.py#L2337-L2338
22:04:09 sdague it's supposed to give you an error back
22:08:03 sdague dansmith: the GET /server/{id} api does validate that to be a uuid or int, this is all novaclient logic
22:08:18 openstackgerrit Matt Riedemann proposed openstack/nova master: neutron: handle binding:profile=None during migration https://review.openstack.org/504260
22:08:21 dansmith sdague: yeah, figuring that out now
22:14:03 mriedem stephenfin: please remove the TODO you slipped into this https://review.openstack.org/#/c/480674/
22:15:01 tasker mriedem: should be linked to my account.
22:15:18 mriedem email was hidden
22:15:22 tasker oh.
22:16:59 openstackgerrit Matt Riedemann proposed openstack/nova master: neutron: handle binding:profile=None during migration https://review.openstack.org/504260
22:17:02 mriedem tasker: ^
22:17:52 tasker thanks!
22:20:57 openstackgerrit jichenjc proposed openstack/nova master: enhance api-ref for os-server-external-events https://review.openstack.org/504263
22:21:25 openstackgerrit Merged openstack/nova master: [placement] Correct a comment in _set_allocations https://review.openstack.org/502167
22:21:54 openstackgerrit Merged openstack/nova master: Add attachment_complete call to volume/cinder.py https://review.openstack.org/493323
22:23:18 openstackgerrit Merged openstack/nova master: Tweak connection_info translation for the new Cinder attach/detach API https://review.openstack.org/493324
22:33:25 openstackgerrit Merged openstack/nova master: Call terminate_connection when shelve_offloading https://review.openstack.org/257275
22:35:39 sdague stephenfin: https://review.openstack.org/#/c/496160 question on that, mostly I think we hit the minimums for testing now, right?
22:39:04 openstackgerrit Matt Riedemann proposed openstack/nova stable/pike: Call terminate_connection when shelve_offloading https://review.openstack.org/504270
22:44:00 openstackgerrit Ildiko Vancsa proposed openstack/nova master: Implement new attach Cinder flow https://review.openstack.org/330285
22:46:43 openstackgerrit Matt Riedemann proposed openstack/nova stable/ocata: Call terminate_connection when shelve_offloading https://review.openstack.org/504273
22:53:23 openstackgerrit Ed Leafe proposed openstack/nova-specs master: Return Alternate Hosts https://review.openstack.org/504275
22:55:01 openstackgerrit Michael Still proposed openstack/nova master: Move nbd commands to privsep. https://review.openstack.org/500351
22:55:02 openstackgerrit Michael Still proposed openstack/nova master: Squash dac_admin privsep context. https://review.openstack.org/504195
22:55:02 openstackgerrit Michael Still proposed openstack/nova master: Move the dac_admin privsep code to a new location. https://review.openstack.org/504194
22:55:03 openstackgerrit Michael Still proposed openstack/nova master: Move lvm handling to privsep. https://review.openstack.org/495516
22:55:03 openstackgerrit Michael Still proposed openstack/nova master: Use symbolic names for capabilities, expand sys_admin context. https://review.openstack.org/504193
22:55:04 openstackgerrit Michael Still proposed openstack/nova master: Move xend existence probes to privsep. https://review.openstack.org/495538
22:55:04 openstackgerrit Michael Still proposed openstack/nova master: Move shred to privsep. https://review.openstack.org/495537
22:55:05 openstackgerrit Michael Still proposed openstack/nova master: Cleanup mount / umount and associated rmdir calls https://review.openstack.org/494423
22:55:05 openstackgerrit Michael Still proposed openstack/nova master: WIP / Aspirational: we don't need rootwrap any more. https://review.openstack.org/495542
22:55:06 openstackgerrit Michael Still proposed openstack/nova master: Don't shell out to mkdir, use ensure_tree() https://review.openstack.org/492326
22:55:06 openstackgerrit Michael Still proposed openstack/nova master: Move loopback setup and removal to privsep. https://review.openstack.org/495664
22:55:07 openstackgerrit Michael Still proposed openstack/nova master: Move ploop commands to privsep. https://review.openstack.org/492325
22:55:07 openstackgerrit Michael Still proposed openstack/nova master: Move the idmapshift binary into privsep. https://review.openstack.org/495541
22:55:08 openstackgerrit Michael Still proposed openstack/nova master: Move kpartx calls to privsep. https://review.openstack.org/500354
22:55:08 openstackgerrit Michael Still proposed openstack/nova master: Move libvirts dmcrypt support to privsep. https://review.openstack.org/490737
22:55:09 openstackgerrit Michael Still proposed openstack/nova master: Move blkid calls to privsep. https://review.openstack.org/500398
22:56:47 bauzas question : when a compute is down, Nova is still saying instances are ACTIVE, right ? If so, I guess it's a design consensus about not having Nova checking the hypervisors for knowing the VMs status, right?
22:57:07 bauzas I remember some old discussion about that and how it's racing
22:58:24 bauzas IIRC also, we had some conf opt for saying nova-compute to verify the libvirt status, but AFAIK we stopped that, nope ?
22:58:34 bauzas because it was racy
22:59:25 dansmith bauzas: we have a host state exposed with the instance when you show it
22:59:35 dansmith bauzas: and if it's down, you know to ignore the instance tate
22:59:40 dansmith *state
23:00:15 bauzas dansmith: ah right, thanks!
23:29:42 tasker I've got that patch in place and the code moves along, but migrations are still not working.
23:30:08 tasker it attempts to "update_port" but neutron denies the request with "BadRequest: Bad port request: The interface name was not provided or empty."
23:30:51 tasker the request isn't even getting inside of neutron, it's being turned away at the gate, as analogy.
23:31:48 tasker so there's still something not working even with the catch-all case of {}
23:33:28 tasker the instance was made recently, but the network its attached to was made during mitaka
23:33:39 tasker I'm testing out in-place upgrades of mitaka to newton.
23:34:52 tasker correction: the network it is connected to was made at the same time as the instance.
23:35:33 tasker the other thing I'm working on bringing the compute host from ubuntu-trusty to ubuntu-xenial
23:40:19 jianghuaw bauzas, may you help to check if the new patch addressed your comments? https://review.openstack.org/#/c/450122
23:40:46 jianghuaw Thanks.
23:57:47 tasker and, for comparison, I can't get migrate an instance to compute-1 (newton-trusty) from compute-3 (newton-xenial) -- but this attempt gives no error, no log, doesn't even attempt to claim resources. it's like the request gets lost between the API container and the compute host.
23:58:10 tasker the port errors are caused when trying to migrate an instance from compute-1 to compute-3
#openstack-nova - 2017-09-15
00:15:48 tasker ignore my last two comments -- i need more information and a larger sample size.
01:50:29 openstackgerrit Merged openstack/nova master: Target context when setting instance to ERROR when over quota https://review.openstack.org/503839
04:01:58 mriedem mtreinish: did you check to see if passing a test to the functional tox target still works after stestr?
04:02:37 mriedem i can't run a single test, it just runs all functional, which takes quite awhile
04:03:04 openstackgerrit Merged openstack/nova master: Split out the core of the ironic flavor migration https://review.openstack.org/501024
04:30:26 openstackgerrit Matt Riedemann proposed openstack/nova master: Add nova-manage db command for ironic flavor migrations https://review.openstack.org/501025
04:36:16 gibi mriedem: I think the ability to run a single test was fixed in https://review.openstack.org/#/c/503811/
06:13:49 openstackgerrit Naichuan Sun proposed openstack/nova master: xenapi: Support live migration in pooled multi-nodes environment https://review.openstack.org/489451
06:27:52 openstackgerrit Naichuan Sun proposed openstack/nova master: xenapi: Support live migration in pooled multi-nodes environment https://review.openstack.org/489451
07:30:23 openstackgerrit johjuhyun proposed openstack/python-novaclient master: Remove TENANT_ID from flavor-access-list which is unnecessary https://review.openstack.org/504356
08:38:47 openstackgerrit Rodolfo Alonso Hernandez proposed openstack/os-vif master: Add Port Profile info to VIF objects Linux Bridge plugin https://review.openstack.org/490829
09:57:18 openstackgerrit Rodolfo Alonso Hernandez proposed openstack/os-vif master: Add VersionedObjectPrintable mixin https://review.openstack.org/493082
10:02:28 openstackgerrit Yikun Jiang proposed openstack/nova master: Add @targets_cell for live_migrate_instance method in conductor https://review.openstack.org/503601
12:15:03 openstackgerrit Rodolfo Alonso Hernandez proposed openstack/os-vif master: Add Port Profile info to VIF objects OVS plugin https://review.openstack.org/490819
12:21:36 openstackgerrit Rodolfo Alonso Hernandez proposed openstack/os-vif master: Add Port Profile info to VIF objects Linux Bridge plugin https://review.openstack.org/490829
12:22:33 openstackgerrit Rodolfo Alonso Hernandez proposed openstack/os-vif master: Add Port Profile info to VIF objects OVS plugin https://review.openstack.org/490819
14:07:29 openstackgerrit johjuhyun proposed openstack/python-novaclient master: Remove TENANT_ID from flavor-access-list which is unnecessary https://review.openstack.org/504356
14:22:16 openstackgerrit johjuhyun proposed openstack/python-novaclient master: Remove TENANT_ID from flavor-access-list which is unnecessary https://review.openstack.org/504418
14:45:31 mriedem lyarwood: can you take a pass through the open stable/pike review backlog? it's getting pretty big https://review.openstack.org/#/q/status:open+project:openstack/nova+branch:stable/pike
14:53:57 lyarwood mriedem: yeah I can do that today
14:55:07 mriedem thanks
15:02:09 dansmith mtreinish: are you around?
15:03:54 mtreinish dansmith: I am
15:04:15 dansmith mtreinish: do we have to pass --regex now? i thought you said we didn't
15:04:46 mtreinish dansmith: on which job? you shouldn't have to on the py27 job
15:05:04 dansmith mtreinish: I thought that's what you said but melwitt says we do, and mriedem apparently just confirmed
15:05:22 dansmith mtreinish: er, is it different for functional for some reason?
15:05:45 melwitt okay, well when I do 'tox -epy27 test_blah' it doesn't work that way anymore
15:05:59 melwitt have to do 'tox -epy27 -- --regex test_blah'
15:06:59 mtreinish melwitt: there is no --regex on stestr run
15:07:17 mtreinish dansmith: it should be tox -epy27 -- regex
15:07:39 mtreinish where regex is the test id you want
15:07:41 dansmith mtreinish: so we have to -- ?
15:07:50 mtreinish yeah that's a tox thing for posargs
15:07:58 dansmith but we didn't have to do that before

Earlier   Later