| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-09-14 | |||
| 17:32:26 | mriedem | and instance.availability_zone is None | |
| 17:32:28 | bauzas | sdague: nova doesn't honor AZs if you don't ask for | |
| 17:32:30 | mriedem | if the user didn't specify an AZ | |
| 17:32:37 | bauzas | but Cinder does, hence thr bug | |
| 17:32:51 | sdague | ah | |
| 17:32:56 | bauzas | at least, the opt is saying "please respect AZs" | |
| 17:33:03 | mriedem | i've always been told that AZs in cinder are half baked | |
| 17:33:08 | bauzas | yup | |
| 17:33:18 | bauzas | I need to think about a possible solution | |
| 17:33:40 | bauzas | I think I already express my opinion about cross_az_attach opt | |
| 17:33:52 | bauzas | I just feel it's promising something we really don't care | |
| 17:34:20 | openstackgerrit | Eric Berglund proposed openstack/nova master: PowerVM Driver: config drive https://review.openstack.org/409404 | |
| 17:36:37 | bauzas | mriedem: sdague: here is one thing: if the user didn't specify an AZ, then we feel we should not honor cross_az_attach and leave the instance be on a separate AZ than the Cinder AZ | |
| 17:36:52 | bauzas | because the instance could be migrated on a totally separate AZ | |
| 17:37:31 | bauzas | so I'd just relax the condition | |
| 17:37:48 | pooja | bauzas: On Newton, when scheduling 60 vms in parallel with heat stack, we see vms in scheduling state for about 30 sec | |
| 17:37:52 | bauzas | and make the conf opt pretty clear that's only enforced if you ask for an AZ | |
| 17:38:08 | pooja | My understanding is its because scheduler is processing them serially | |
| 17:38:18 | bauzas | pooja: that's a correct assumption | |
| 17:39:11 | bauzas | pooja: do you often request for such large requests ? | |
| 17:39:18 | bauzas | s/requests/numbers | |
| 17:40:11 | bauzas | pooja: given we really want to deprecate CachingScheduler, I'd rather be in favor of running multiple schedulers and increasing the conf opt responsible for the number of retries | |
| 17:40:50 | bauzas | pooja: that way, when you migrate to Ocata and eventually Pike, you wouldn't change your architecture and just leave multiple schedulers, since retries would seriously decrease | |
| 17:41:06 | bauzas | eventually, we will stop proposing reschedules | |
| 17:42:45 | openstackgerrit | Ed Leafe proposed openstack/nova master: Add Selection objects https://review.openstack.org/499239 | |
| 17:42:45 | openstackgerrit | Ed Leafe proposed openstack/nova master: Add alternate hosts https://review.openstack.org/486215 | |
| 17:45:09 | pooja | bauzas: Yeah, that makes sense.. We are using heat stacks at scale and the plan is go from 60 to 200 eventually | |
| 17:45:48 | pooja | So with Ocata, multiple schedulers feature is different from the Placement API you referred to earlier? | |
| 17:51:13 | openstackgerrit | Merged openstack/nova master: Remove deprecated keymgr code https://review.openstack.org/439855 | |
| 17:56:34 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Call terminate_connection when shelve_offloading https://review.openstack.org/257275 | |
| 17:58:14 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Call terminate_connection when shelve_offloading https://review.openstack.org/257275 | |
| 18:06:34 | mikal | Is opy35 broken by the testr change? | |
| 18:06:36 | mikal | http://logs.openstack.org/93/503893/1/check/gate-nova-python35/aed6ed0/console.html#_2017-09-14_15_17_45_846189 | |
| 18:07:49 | mikal | Ugh, ignore me | |
| 18:10:05 | mriedem | stvnoyes: http://logs.openstack.org/23/493323/13/gate/gate-tempest-dsvm-multinode-live-migration-ubuntu-xenial/71a6104/console.html#_2017-09-14_16_17_20_335891 | |
| 18:11:56 | mriedem | errors down here http://logs.openstack.org/23/493323/13/gate/gate-tempest-dsvm-multinode-live-migration-ubuntu-xenial/71a6104/logs/screen-n-cpu.txt.gz?level=TRACE | |
| 18:18:31 | mriedem | dansmith: https://review.openstack.org/#/c/503839/ is the bug fix for the cell targeting thing, will need to be backported | |
| 18:37:26 | dansmith | mriedem: got it | |
| 18:37:39 | mriedem | thanks | |
| 18:39:32 | dansmith | mriedem: you may like this trivial one too: https://review.openstack.org/#/c/499957/1 | |
| 18:39:50 | dansmith | gets rid of the need to update the service version every time in the notification samples | |
| 18:40:35 | mriedem | dansmith: i'm not sure i can approve that trivial change w/o mikal +1ing it first | |
| 18:41:09 | dansmith | mriedem: maybe not | |
| 18:48:15 | mriedem | i've got that patch from andrearosa rebased for the shelve offload terminate connection thing we talked about today https://review.openstack.org/#/c/257275/ | |
| 19:15:21 | mriedem1 | sean-k-mooney: do you have a link to the poc for vif negotation? | |
| 19:28:21 | openstackgerrit | jichenjc proposed openstack/nova master: fix race condition of instance host https://review.openstack.org/494458 | |
| 19:28:23 | bauzas | mriedem: can't find a ML thread but just the bug report for https://bugs.launchpad.net/nova/+bug/1497253 | |
| 19:28:24 | openstack | Launchpad bug 1497253 in OpenStack Compute (nova) "different availability zone for nova and cinder when AZ is not explicitly given" [Low,In progress] - Assigned to Roman Podoliaka (rpodolyaka) | |
| 19:28:38 | openstackgerrit | Matt Riedemann proposed openstack/nova stable/pike: Target context when setting instance to ERROR when over quota https://review.openstack.org/504178 | |
| 19:28:44 | bauzas | mriedem: except a very old thread where I was already saying all the good I think about the cinder/nova AZ relationship | |
| 19:29:04 | bauzas | mriedem: so I'm just going to release a new patch for that honestly | |
| 19:29:13 | bauzas | patch == revision I mean | |
| 19:29:59 | mriedem | bauzas: http://lists.openstack.org/pipermail/openstack-operators/2017-June/013676.html | |
| 19:30:19 | bauzas | mriedem: okay, I'll reply then, thanks | |
| 19:40:18 | mriedem | sean-k-mooney: nvm found them https://review.openstack.org/#/q/topic:bp/vif-port-profile,n,z | |
| 20:02:02 | tasker | when doing a live migration in a nova newton, in the log on the destination compute host, I get "TypeError: 'NoneType' object does not support item assignment" from nova/network/neutronv2/api.py#L289 -> port_profile[MIGRATING_ATTR] = host | |
| 20:04:38 | tasker | has anyone seen this or have a clue what the function is trying to do? what is port_profile? where is it set and why would it be None? | |
| 20:05:40 | openstackgerrit | Michael Still proposed openstack/nova master: Move nbd commands to privsep. https://review.openstack.org/500351 | |
| 20:05:41 | openstackgerrit | Michael Still proposed openstack/nova master: Move xend existence probes to privsep. https://review.openstack.org/495538 | |
| 20:05:41 | openstackgerrit | Michael Still proposed openstack/nova master: Move lvm handling to privsep. https://review.openstack.org/495516 | |
| 20:05:42 | openstackgerrit | Michael Still proposed openstack/nova master: Cleanup mount / umount and associated rmdir calls https://review.openstack.org/494423 | |
| 20:05:42 | openstackgerrit | Michael Still proposed openstack/nova master: Move shred to privsep. https://review.openstack.org/495537 | |
| 20:05:43 | openstackgerrit | Michael Still proposed openstack/nova master: Don't shell out to mkdir, use ensure_tree() https://review.openstack.org/492326 | |
| 20:05:43 | openstackgerrit | Michael Still proposed openstack/nova master: WIP / Aspirational: we don't need rootwrap any more. https://review.openstack.org/495542 | |
| 20:05:44 | openstackgerrit | Michael Still proposed openstack/nova master: Move loopback setup and removal to privsep. https://review.openstack.org/495664 | |
| 20:05:45 | openstackgerrit | Michael Still proposed openstack/nova master: Move ploop commands to privsep. https://review.openstack.org/492325 | |
| 20:05:47 | openstackgerrit | Michael Still proposed openstack/nova master: Move kpartx calls to privsep. https://review.openstack.org/500354 | |
| 20:05:47 | openstackgerrit | Michael Still proposed openstack/nova master: Move the idmapshift binary into privsep. https://review.openstack.org/495541 | |
| 20:05:48 | openstackgerrit | Michael Still proposed openstack/nova master: Move blkid calls to privsep. https://review.openstack.org/500398 | |
| 20:05:48 | openstackgerrit | Michael Still proposed openstack/nova master: Move libvirts dmcrypt support to privsep. https://review.openstack.org/490737 | |
| 20:05:49 | openstackgerrit | Michael Still proposed openstack/nova master: Move the dac_admin privsep code to a new location. https://review.openstack.org/504194 | |
| 20:05:49 | openstackgerrit | Michael Still proposed openstack/nova master: Use symbolic names for capabilities, expand sys_admin context. https://review.openstack.org/504193 | |
| 20:05:50 | openstackgerrit | Michael Still proposed openstack/nova master: Squash dac_admin privsep context. https://review.openstack.org/504195 | |
| 20:11:50 | mriedem | johnthetubaguy: should we revive this nova spec for the neutron port binding API changes for live migration? https://review.openstack.org/#/c/375580/ | |
| 20:12:54 | cdent | mriedem: when are we starting back up? | |
| 20:13:17 | cdent | and what’s next/ | |
| 20:13:34 | mriedem | neutron was supposed to be 1-3 | |
| 20:13:43 | mriedem | 3 was the generic device mgmt stuff | |
| 20:13:51 | cdent | woot 45 minute break!? | |
| 20:13:57 | mriedem | cburgess wants to talk about something | |
| 20:14:03 | cdent | no change then | |
| 20:14:05 | cburgess | Tomorrow is fine. | |
| 20:14:09 | cburgess | Ouch.... | |
| 20:14:16 | cdent | cburgess said with love | |
| 20:14:18 | cburgess | I'll remember that cdent | |
| 20:14:21 | cburgess | Yeah yeah. | |
| 20:14:21 | mriedem | we could get started on our friday pile of stuff | |
| 20:15:18 | mriedem | how about we just get started back up at 2:30 | |
| 20:15:32 | cdent | good plan | |
| 20:37:15 | tasker | mriedem: I'm inferring that spec might be related to the problems I'm seeing. is that right? | |
| 20:39:19 | openstackgerrit | Matt Riedemann proposed openstack/nova-specs master: Enable cold migration with target host https://review.openstack.org/489031 | |
| 20:39:32 | mriedem | tasker: what spec and what problem? | |
| 20:40:03 | mriedem | tasker: oh, um, no probably different issue | |
| 20:40:26 | mriedem | john's spec is for new functionality | |
| 20:40:38 | tasker | ah. the two were coincidentally close. | |
| 20:57:38 | openstackgerrit | Michael Still proposed openstack/nova master: Have one list of reboot task_states https://review.openstack.org/219981 | |
| 20:58:07 | tasker | in the line preceding the TypeError, port_profile = p.get(BINDING_PROFILE, {}), this returns None, which implies that p[BINDING_PROFILE] is None itself. | |
| 21:00:04 | tasker | and, occassioanlly, it doesn't have that value, and returns the empty dictionary {} defined in the .get() | |
| 21:01:54 | tasker | it returns {} when there is no "binding:profile" key in the port dictionary | |
| 21:02:11 | tasker | it returns None when there is a "binding:profile", but when it's set to None. | |