Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2018-01-08
20:50:52 jaypipes efried: k, that matches the assumptions in the patch with the test case.
20:51:15 efried excellent.
20:52:53 efried jaypipes When you say, "check. test assertions (and commented out assertions) are correct and represent the bugs appropriately." -- are you asking for a code comment, or was this just acknowledging that you had looked at the test case and saw that it was good (in the Genesis sense)?
20:53:09 jaypipes efried: no, I'm saying it was good. ++
20:53:25 efried jaypipes Roger, thanks.
20:53:32 jaypipes efried: sometimes I mark reviews like that just to keep my place in the review.
20:53:49 efried suresure. Just making sure I understood correctly, since you *were* asking for comment updates elsewhere.
20:54:37 jaypipes ack
20:59:27 openstackgerrit Eric Berglund proposed openstack/nova master: WIP: PowerVM Driver: vSCSI https://review.openstack.org/526094
21:40:46 mriedem nicolasbock: you're with suse right? is this something you can confirm and if so, update the nova install guide for suse? https://bugs.launchpad.net/nova/+bug/1741329
21:40:47 openstack Launchpad bug 1741329 in OpenStack Compute (nova) "Install and configure controller node for openSUSE and SUSE Linux Enterprise in nova" [Undecided,New]
21:52:02 mriedem lyarwood: hemna: added some notes to https://bugs.launchpad.net/nova/+bug/1741364 - but yeah, spot on, doesn't do anything with getting the ephemeral disk config when building the new guest xml
21:52:03 openstack Launchpad bug 1741364 in OpenStack Compute (nova) "ceph ephemeral info not updated during live migrate" [Medium,Confirmed]
21:52:20 mriedem and cold migration would do that of course, or a hard reboot
21:52:28 mriedem but those involve downtime
21:54:12 mriedem would likely require something like https://github.com/openstack/nova/blob/74deea4d8f66a85e66ec79c72c9f257f562d5afd/nova/virt/libvirt/migration.py#L133 for non-volume disks
21:57:59 flwang1 mriedem: could you please help me understand the relationship between cell and sockets in cpu topo?
21:58:13 mriedem flwang1: i can't help you there
21:58:27 flwang1 mriedem: who is the right person I should ask?
21:58:30 mriedem probably need to talk to stephenfin or sahid when they are around
21:58:37 mriedem UK timezone
21:58:47 flwang1 mriedem: cool, thanks
21:58:54 mriedem or cfriesen (canada
21:59:26 flwang1 mriedem: great, and happy new year!
21:59:39 mriedem same to you
22:06:31 bauzas dansmith: still around ?
22:07:13 bauzas dansmith: I'll upload a new PS tomorrow for https://review.openstack.org/#/c/528832/ based on your's and jaypipes's comments
22:07:51 bauzas dansmith: but you had a main comment about how to recreate the existing mdevs with init_host()
22:20:24 bauzas dansmith: basically the main concern I have is that we don't know if an existing mdev is used by an instance unless by looking at the guest XML
22:21:13 dansmith bauzas: ...right, what's wrong with looking at the xml? that's what I was suggesting
22:21:37 bauzas dansmith: the main point is that it could be a long verification
22:21:47 dansmith bauzas: in what way?
22:22:04 dansmith we already iterate instances on startup
22:22:11 dansmith and do lots of db stuff for each
22:22:21 bauzas a-ha
22:22:43 bauzas in the libvirt init_host() or the compute service one ?
22:22:54 bauzas lemme see
22:23:12 dansmith we iterate instances in the compute manager init_host()
22:23:19 dansmith and call init_instance() (or something like that) for each
22:23:37 dansmith we could call into the driver in there, or just have the driver's init_host iterate instances again, tbh
22:23:52 dansmith just listing all instances from libvirt and re-creating mdevs would be fine and not heavy, IMHO
22:24:31 bauzas I see
22:25:01 dansmith dan@falcon:~$ time (for i in $(seq 0 100); do virsh dumpxml 10 >/dev/null; done)
22:25:01 dansmith real0m1.006s
22:25:06 dansmith on my quite slow machine
22:25:07 bauzas i just wonder how to verify the xml
22:25:10 dansmith I think it'll be fine :)
22:25:21 bauzas heh thanks
22:25:22 dansmith how to verify what/
22:25:47 bauzas how to verify the guest XML whether it has some mdevs
22:25:48 dansmith it was right when we shut down, we're in init/startup, why not just restore them to the way they were? we can't have processed any compute-side deletes yet
22:25:57 dansmith isn't it in the guest xml?
22:26:05 bauzas it will be
22:26:23 bauzas anyway, lemme see how to do that
22:26:35 dansmith okay, I'm confused about your confusion,
22:26:40 dansmith but if I'm missing something, slap me with it
22:28:03 bauzas apologies if I'm confusing
22:28:53 bauzas I'll work on that tomorrow morning so if I have better concerns, I'll tell you once you're up
22:28:57 bauzas thanks
22:29:04 dansmith okay
22:51:56 rybridges Have a question guys. Where do the logs for this file go? https://github.com/openstack/nova/blob/stable/ocata/nova/compute/manager.py
22:52:07 rybridges I do not see them in scheduler log, conductor log, or api log
22:52:24 mriedem compute
22:53:19 openstackgerrit Matt Riedemann proposed openstack/nova master: unquiesce instance on volume snapshot failure https://review.openstack.org/519464
22:53:19 openstackgerrit Matt Riedemann proposed openstack/nova master: Add the ability to get absolute limits from Cinder https://review.openstack.org/520122
22:53:20 openstackgerrit Matt Riedemann proposed openstack/nova master: Check quota before creating volume snapshots https://review.openstack.org/520158
22:54:08 rybridges oh its on the hypervisor
22:54:11 rybridges mriedem
22:54:14 rybridges not on the api node
22:54:33 mriedem well, nova/compute, "compute"
22:54:35 mriedem so yeah
22:54:45 rybridges ok
22:54:47 mriedem the directory structure is mostly laid out by service
22:54:52 mriedem except nova/virt is also in compute
#openstack-nova - 2018-01-09
00:13:59 openstackgerrit melanie witt proposed openstack/nova stable/pike: libvirt: Re-initialise volumes, encryptors, and vifs on hard reboot https://review.openstack.org/531407
00:45:01 openstackgerrit Takashi NATSUME proposed openstack/python-novaclient master: Add CLI to show instance usage audit logs https://review.openstack.org/409413
00:50:25 openstackgerrit Takashi NATSUME proposed openstack/nova master: List/show all server migration types (2/2) https://review.openstack.org/459483
01:11:56 openstackgerrit melanie witt proposed openstack/nova stable/ocata: libvirt: Re-initialise volumes, encryptors, and vifs on hard reboot https://review.openstack.org/531422
01:20:34 openstackgerrit Takashi NATSUME proposed openstack/python-novaclient master: Microversion 2.59 - List/Show all server migration types https://review.openstack.org/430839
01:26:11 openstackgerrit Zhenyu Zheng proposed openstack/nova master: Use neutron port_list when filtering instance by ip https://review.openstack.org/525505
02:13:37 openstackgerrit Matt Riedemann proposed openstack/nova master: Qualify the Placement 1.15 release note https://review.openstack.org/531963
02:22:59 openstackgerrit OpenStack Proposal Bot proposed openstack/nova master: Updated from global requirements https://review.openstack.org/531737
02:34:54 openstackgerrit Yikun Jiang (Kero) proposed openstack/nova master: Fix 500 in test_resize_server_negative_invalid_state https://review.openstack.org/531117
02:34:55 openstackgerrit Yikun Jiang (Kero) proposed openstack/nova master: Add index(instance_uuid, updated_at) on instance_actions table https://review.openstack.org/530429
02:46:54 openstackgerrit lei zhang proposed openstack/nova master: Fix the incorrect git.openstack.org source URL https://review.openstack.org/531979
03:36:03 openstackgerrit Merged openstack/nova master: Fix OpenStack capitalization https://review.openstack.org/531593
04:22:23 openstackgerrit Merged openstack/nova stable/ocata: Raise MarkerNotFound if BuildRequestList.get_by_filters doesn't find marker https://review.openstack.org/530981
04:24:21 openstackgerrit Merged openstack/nova master: Use ksa adapter for neutron client https://review.openstack.org/509892
04:44:23 openstackgerrit Matt Riedemann proposed openstack/nova master: [libvirt] Allow multiple volume attachments https://review.openstack.org/267587
04:44:24 openstackgerrit Matt Riedemann proposed openstack/nova master: Pass multiattach flag to reserve_block_device_name https://review.openstack.org/531244
04:44:24 openstackgerrit Matt Riedemann proposed openstack/nova master: WIP: [api] Allow multi-attach in compute api https://review.openstack.org/271047
04:45:03 mriedem ildikov: ^ the api support should be plumbed in now; required quite a bit of refactor which touched a billion tests, but not complicated. just needs the new paths covered with unit tests.
04:49:54 openstackgerrit Tetsuro Nakamura proposed openstack/nova master: Fix missing rps in allocation candidates https://review.openstack.org/522409
04:49:54 openstackgerrit Tetsuro Nakamura proposed openstack/nova master: placement: func tests for multiple shared RPs https://review.openstack.org/498737
04:52:14 openstackgerrit Takashi NATSUME proposed openstack/nova master: Add an additional description for 'token_ttl' https://review.openstack.org/532007
05:08:44 openstackgerrit Sriharsha Basavapatna proposed openstack/os-vif master: Fix VF-rep lookup routine to use parent PF number https://review.openstack.org/531674
05:33:41 takashin
06:05:38 sandanar hi.. does anyone know what is the reason behind CONF.libvirt.volume_use_path always returns the default value and not the one passed in [libvirt] section of /etc/nova/nova.conf - https://bugs.launchpad.net/nova/+bug/1723928 this is causing multipath failures on the cinder side
06:05:39 openstack Launchpad bug 1723928 in os-brick "In case of volume_use_multipath=True, Nova unable to fetch CONF.libvirt.volume_use_multipath value from nova.conf" [Undecided,New]
07:49:38 openstackgerrit chenpengzi proposed openstack/nova master: Add reference to policy sample https://review.openstack.org/531713

Earlier   Later