Earlier  
Posted Nick Remark
#openstack-nova - 2017-09-14
22:55:05 openstackgerrit Michael Still proposed openstack/nova master: Cleanup mount / umount and associated rmdir calls https://review.openstack.org/494423
22:55:06 openstackgerrit Michael Still proposed openstack/nova master: Move loopback setup and removal to privsep. https://review.openstack.org/495664
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:07 openstackgerrit Michael Still proposed openstack/nova master: Move the idmapshift binary into privsep. https://review.openstack.org/495541
22:55:07 openstackgerrit Michael Still proposed openstack/nova master: Move ploop commands to privsep. https://review.openstack.org/492325
22:55:08 openstackgerrit Michael Still proposed openstack/nova master: Move libvirts dmcrypt support to privsep. https://review.openstack.org/490737
22:55:08 openstackgerrit Michael Still proposed openstack/nova master: Move kpartx calls to privsep. https://review.openstack.org/500354
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
15:08:13 dansmith and if you just do tox -epy27 foo it apparently runs everything without warning yiou
15:08:59 openstackgerrit Takashi NATSUME proposed openstack/nova master: Enable cold migration with target host(1/2) https://review.openstack.org/408955
15:09:02 mtreinish that's probably because tox isn't passing the foo to the command
15:09:22 openstackgerrit Evgeny Antyshev proposed openstack/nova master: Use privsep.dac_admin in create_ploop_image https://review.openstack.org/504429
15:09:25 dansmith we didn't have to do -- before, fwiw
15:09:27 mtreinish sigh, we need to add the single quotes back around posargs to get it to understand the regex on the cli withou --
15:10:08 openstackgerrit Stephen Finucane proposed openstack/nova-specs master: Websockify security proxy framework https://review.openstack.org/496160
15:10:29 mtreinish that will break other arguments being passed to stestr though
15:10:57 mtreinish or I think it will
15:11:51 mtreinish err, no I guess it works. (with -- for the extra args) I'll push a patch
15:13:28 melwitt okay, sorry I got confused I guess. 'tox -epy27 -- regex' works for me too
15:15:43 openstackgerrit Matthew Treinish proposed openstack/nova master: Add single quotes for posargs on jobs https://review.openstack.org/504432
15:15:45 mtreinish dansmith: ^^^ is that better for you?
15:18:55 dansmith mtreinish: I have to -r so I'll tell you in a sec
15:19:46 openstackgerrit Takashi NATSUME proposed openstack/nova master: Enable cold migration with target host(2/2) https://review.openstack.org/408964
15:20:21 openstackgerrit Stephen Finucane proposed openstack/nova master: Assume neutron auto_allocate extension's enabled https://review.openstack.org/480674
15:20:21 openstackgerrit Stephen Finucane proposed openstack/nova master: Add '_has_dns_extension' function https://review.openstack.org/480675
15:20:22 openstackgerrit Stephen Finucane proposed openstack/nova master: Add '_has_qos_queue_extension' function https://review.openstack.org/500010
15:44:59 openstackgerrit Michael Still proposed openstack/nova master: Move nbd commands to privsep. https://review.openstack.org/500351
15:45:00 openstackgerrit Michael Still proposed openstack/nova master: Move the dac_admin privsep code to a new location. https://review.openstack.org/504194
15:45:00 openstackgerrit Michael Still proposed openstack/nova master: Squash dac_admin privsep context. https://review.openstack.org/504195
15:45:01 openstackgerrit Michael Still proposed openstack/nova master: Use symbolic names for capabilities, expand sys_admin context. https://review.openstack.org/504193
15:45:01 openstackgerrit Michael Still proposed openstack/nova master: Move lvm handling to privsep. https://review.openstack.org/495516
15:45:02 openstackgerrit Michael Still proposed openstack/nova master: Move shred to privsep. https://review.openstack.org/495537
15:45:02 openstackgerrit Michael Still proposed openstack/nova master: Move xend existence probes to privsep. https://review.openstack.org/495538
15:45:03 openstackgerrit Michael Still proposed openstack/nova master: WIP / Aspirational: we don't need rootwrap any more. https://review.openstack.org/495542
15:45:03 openstackgerrit Michael Still proposed openstack/nova master: Cleanup mount / umount and associated rmdir calls https://review.openstack.org/494423
15:45:04 openstackgerrit Michael Still proposed openstack/nova master: Move loopback setup and removal to privsep. https://review.openstack.org/495664
15:45:04 openstackgerrit Michael Still proposed openstack/nova master: Don't shell out to mkdir, use ensure_tree() https://review.openstack.org/492326
15:45:05 openstackgerrit Michael Still proposed openstack/nova master: Move the idmapshift binary into privsep. https://review.openstack.org/495541
15:45:05 openstackgerrit Michael Still proposed openstack/nova master: Move ploop commands to privsep. https://review.openstack.org/492325
15:45:06 openstackgerrit Michael Still proposed openstack/nova master: Move libvirts dmcrypt support to privsep. https://review.openstack.org/490737
15:45:06 openstackgerrit Michael Still proposed openstack/nova master: Move kpartx calls to privsep. https://review.openstack.org/500354
15:45:07 openstackgerrit Michael Still proposed openstack/nova master: Move blkid calls to privsep. https://review.openstack.org/500398
15:48:26 efried melwitt Trivial doc review, cause you said you don't mind :) https://review.openstack.org/#/c/499748/
15:55:34 melwitt thanks tonyb. efried, I'm not core on stable branches :)
15:56:24 efried Sigh. Tribal knowledge. Thanks for looking. And thanks tonyb

Earlier   Later