Earlier  
Posted Nick Remark
#openstack-nova - 2017-11-30
00:32:58 mriedem in close proximity
00:33:05 dansmith if I let go
00:33:07 dansmith that's the idea
00:33:26 mgagne can those details be added to the etherpad? like you should be running global placement. it will return all UUIDs of all compute nodes but scheduler will filter them out. And there are plans to optimize that.
00:33:46 mriedem mgagne: i added something along those lines in the pros/cons sections for global vs local placement
00:33:51 mriedem might not be coherent at this point
00:33:59 mgagne ok, I didn't fully understand what it meant ^^'
00:34:01 mriedem dansmith: i just like the idea of saying we should do that and watching heads explode
00:35:00 mriedem mgagne: probably need some kind of summary section
00:35:10 mgagne yea
00:35:44 mgagne perfect
00:36:51 mgagne I think we should make it clear which use case is addressed: placement with cellsv1 vs cellsv2
00:38:02 mgagne I think the summary section is good. does it apply to newton too?
00:38:23 mriedem newton scheduler didn't talk to placement
00:38:28 mriedem i think, have to check
00:38:32 mgagne oooooh
00:38:34 mgagne so...
00:39:14 mgagne placement in newton, compute reports resources but nothing is consuming from placement?
00:39:32 mriedem mgagne: correct
00:39:44 mgagne holy cow
00:39:44 mriedem https://docs.openstack.org/releasenotes/nova/ocata.html
00:39:50 mriedem "The Nova FilterScheduler driver is now able to make scheduling decisions based on the new Placement RESTful API endpoint that becomes mandatory in Ocata. "
00:40:10 mriedem and in ocata, the scheduler only starts asking placement once all compute are upgraded
00:40:16 mriedem knowing they are reporting in to placement
00:40:25 mgagne well, if i'm upgrading to newton, I might not read ocata release notes (yet) =)
00:40:27 mriedem mgagne: it's done that way for rolling upgrades
00:40:38 mriedem that's why we do a min service version check
00:40:48 mgagne awesome news then
00:41:00 mriedem dansmith: yeah she poked her head in here about 2 minutes after i said htat
00:41:06 mgagne finally understood something today :D
00:41:30 dansmith mriedem: mmhmm, I can hear her from oregon
00:41:34 mriedem mgagne: https://github.com/openstack/nova/blob/stable/ocata/nova/scheduler/filter_scheduler.py#L169
00:42:04 mriedem she sent our daughter in to yell at me
00:42:10 mriedem so ttyl
00:42:40 mgagne I'm off for today too, have a good one. thanks for all the explanations!
01:04:25 openstackgerrit Jackie Truong proposed openstack/python-novaclient master: Microversion 2.56 - Add trusted_image_certificates https://review.openstack.org/500396
01:08:16 openstackgerrit Jackie Truong proposed openstack/python-novaclient master: Microversion 2.56 - Add trusted_image_certificates https://review.openstack.org/500396
01:30:30 Kevin_Zheng mriedem, Hi, what is the -1 for in https://review.openstack.org/#/c/523653/ ? I saw that you are suggesting to make change on manager.py in a follow up patch
01:30:41 mriedem not a follow up patch,
01:30:44 mriedem a preceeding patch
01:30:53 mriedem your patch introduces a bug
01:32:13 Kevin_Zheng Ah, OK, got it.
01:37:06 openstackgerrit Matt Riedemann proposed openstack/nova master: Assert that we restrict cold migrations to the same cell https://review.openstack.org/524035
01:38:17 mriedem takashin: i split this test out into it's own patch https://review.openstack.org/#/c/524027/
01:38:24 mriedem and added this https://review.openstack.org/#/c/524035/1
01:44:36 openstackgerrit Matt Riedemann proposed openstack/nova master: Enable cold migration with target host(2/2) https://review.openstack.org/408964
01:44:37 openstackgerrit Matt Riedemann proposed openstack/nova master: Add multi-cell negative test for cold migration with target host https://review.openstack.org/524027
01:44:37 openstackgerrit Matt Riedemann proposed openstack/nova master: Assert that we restrict cold migrations to the same cell https://review.openstack.org/524035
01:45:07 openstackgerrit Naichuan Sun proposed openstack/nova master: xenapi(N-R-P):Get vgpu info from `allocations` https://review.openstack.org/521717
01:56:04 openstackgerrit Matt Riedemann proposed openstack/nova master: Enable cold migration with target host(2/2) https://review.openstack.org/408964
01:56:05 openstackgerrit Matt Riedemann proposed openstack/nova master: Add multi-cell negative test for cold migration with target host https://review.openstack.org/524027
01:56:05 openstackgerrit Matt Riedemann proposed openstack/nova master: Assert that we restrict cold migrations to the same cell https://review.openstack.org/524035
02:00:57 openstackgerrit Andreas Karis proposed openstack/nova master: Fixes 'Not enough available memory' log message https://review.openstack.org/524038
02:01:39 Kevin_Zheng mriedem: for your comment on https://review.openstack.org/#/c/523653/ , we have a instance uuid lock here, if directly call _shelve_offload_instance might be a problem ? https://github.com/openstack/nova/blob/master/nova/compute/manager.py#L4637
02:03:27 mriedem Kevin_Zheng: that was added for a race tempest was hitting https://github.com/openstack/nova/commit/e285eb1a382e6d3ce1cc596eeb5cecb3b165a228
02:03:37 mriedem Kevin_Zheng: you could do the same instance lock in the periodic task when calling _shelve_offload_instance
02:04:04 Kevin_Zheng Ah, right, I'm so stupid...
02:16:25 openstackgerrit Takashi NATSUME proposed openstack/nova master: [placement] Fix foreign key constraint error https://review.openstack.org/516880
02:19:14 openstackgerrit Naichuan Sun proposed openstack/nova master: xenapi(N-R-P):Get vgpu info from `allocations` https://review.openstack.org/521717
02:25:15 openstackgerrit Huan Xie proposed openstack/nova master: XenAPI: Don't use nicira-iface-id for XenServer VIF https://review.openstack.org/451641
02:46:16 openstackgerrit Zhenyu Zheng proposed openstack/nova master: Modify _poll_shelved_instances periodic task call _shelve_offload_instance() https://review.openstack.org/524047
03:26:22 openstackgerrit Zhenyu Zheng proposed openstack/nova master: Modify _poll_shelved_instances periodic task call _shelve_offload_instance() https://review.openstack.org/524047
05:23:37 openstackgerrit Naichuan Sun proposed openstack/nova master: xenapi(N-R-P):Get vgpu info from `allocations` https://review.openstack.org/521717
06:36:35 openstackgerrit Zhenyu Zheng proposed openstack/nova master: Modify _poll_shelved_instances periodic task call _shelve_offload_instance() https://review.openstack.org/524047
07:03:26 Guest72929 >>> youtube.com/watch?v=X8qL6cAI0zQ
07:34:35 openstackgerrit Takashi NATSUME proposed openstack/nova master: [placement] Fix an error message in API validation https://review.openstack.org/524088
07:38:05 openstackgerrit Merged openstack/nova master: Enable cold migration with target host(2/2) https://review.openstack.org/408964
08:32:48 openstackgerrit Takashi NATSUME proposed openstack/nova master: [placement] Add functional tests for traits API https://review.openstack.org/524094
08:34:56 openstackgerrit Takashi NATSUME proposed openstack/nova master: Fix 500 error while passing 4-byte unicode data https://review.openstack.org/407514
08:35:15 openstackgerrit Takashi NATSUME proposed openstack/nova master: [cellv2] Improve getting BDMs in multiple cells https://review.openstack.org/521400
08:35:57 openstackgerrit Takashi NATSUME proposed openstack/nova master: Fix missing marker functions https://review.openstack.org/514579
08:36:22 openstackgerrit Takashi NATSUME proposed openstack/nova master: Add a warning in 'nova-manage cell_v2 delete_cell' https://review.openstack.org/513771
08:36:40 openstackgerrit Takashi NATSUME proposed openstack/nova master: [placement] Fix format in placement API ref https://review.openstack.org/521483
08:37:19 openstackgerrit Takashi NATSUME proposed openstack/python-novaclient master: Microversion 2.56 - Enable cold migration with target host https://review.openstack.org/406707
08:38:42 openstackgerrit Takashi NATSUME proposed openstack/nova master: List/show all server migration types (1/2) https://review.openstack.org/430608
08:42:35 openstackgerrit Yikun Jiang (Kero) proposed openstack/nova master: Add migration db and object pagination support. https://review.openstack.org/514904
08:42:36 openstackgerrit Yikun Jiang (Kero) proposed openstack/nova master: Implement query param schema for migration index https://review.openstack.org/518644
08:42:36 openstackgerrit Yikun Jiang (Kero) proposed openstack/nova master: Extract SortContext/SortWrapper of instance_list into utils https://review.openstack.org/518871
08:42:37 openstackgerrit Yikun Jiang (Kero) proposed openstack/nova master: Add cross cell sort support for get_migrations https://review.openstack.org/517273
09:21:41 openstackgerrit Yikun Jiang (Kero) proposed openstack/nova master: Add pagination and Changes-since filter support for os-migrations. https://review.openstack.org/330406
09:36:50 openstackgerrit Chen Hanxiao proposed openstack/nova master: libvirt: throw NotImplementedError if qga is not responsive when setting password https://review.openstack.org/523828
09:46:59 openstackgerrit Merged openstack/nova master: XenAPI: resolve VBD unplug failure with VM_MISSING_PV_DRIVERS error https://review.openstack.org/514902
09:54:38 ebbex nova-compute[5505]: 2017-11-30 09:14:28.222 5505 WARNING os_brick.initiator.connectors.iscsi [req-e7903a5a-325e-4a69-9bd7-f6ea60d02c36 a4401f306bfc4d4eaa35fa8a61037268 9fd9ce5c36694a99824c50806496b581 - default default] Could not find the iSCSI Initiator File /etc/iscsi/initiatorname.iscsi: ProcessExecutionError: Unexpect
09:54:43 ebbex ed error while running command.
09:54:52 ebbex Command: cat /etc/iscsi/initiatorname.iscsi
09:55:38 ebbex What is that warning/error, and why is it even running?
10:00:14 openstackgerrit Yikun Jiang (Kero) proposed openstack/nova master: Update Instance action's updated_at when action event updated. https://review.openstack.org/507473
10:00:15 openstackgerrit Yikun Jiang (Kero) proposed openstack/nova master: Add pagination and changes-since for instance-actions https://review.openstack.org/326326
10:11:59 stephenfin bauzas: Could you sanity check this for me? I really don't understand the problem they're trying to solve https://review.openstack.org/#/c/523828/
10:12:10 stephenfin Just a quick gawk is fine ;)
10:21:08 bauzas stephenfin: roger, man
10:21:41 bauzas stephenfin: FWIW, "gawk" has a very different meaning for me given I was an operator before :p
10:28:52 openstackgerrit Yikun Jiang (Kero) proposed openstack/nova master: Add pagination and changes-since for instance-actions https://review.openstack.org/326326
10:32:57 openstackgerrit Chris Dent proposed openstack/nova master: [placement] Object changes to support last-modified headers https://review.openstack.org/521639
10:32:57 openstackgerrit Chris Dent proposed openstack/nova master: [placement] Add cache headers to placement api requests https://review.openstack.org/521640
10:34:29 openstackgerrit Matthew Booth proposed openstack/nova master: Remove unused global variables https://review.openstack.org/524126
10:34:30 openstackgerrit Matthew Booth proposed openstack/nova master: Add uuid field to BlockDeviceMapping https://review.openstack.org/524127
10:38:02 stephenfin bauzas: And, dare I say, what would that other meaning be? :D
10:38:06 stephenfin *dare I ask
10:40:56 stephenfin mostly...

Earlier   Later