Earlier  
Posted Nick Remark
#openstack-nova - 2017-11-23
08:52:19 openstackgerrit Naichuan Sun proposed openstack/nova master: xenapi: Get vgpu info from `allocations` https://review.openstack.org/521717
08:52:47 openstackgerrit Balazs Gibizer proposed openstack/nova master: Versioned notifications for service create and delete https://review.openstack.org/519588
09:07:57 openstackgerrit Li Xipeng proposed openstack/nova master: Fix bug as BDM failed when booting from volume https://review.openstack.org/522486
09:10:34 openstackgerrit Rong Han proposed openstack/nova master: Fix incorrect known vcpuset when CPUPinningUnknown raised https://review.openstack.org/517592
09:19:06 openstackgerrit Jianghua Wang proposed openstack/nova master: XenAPI: get vGPU stats from hypervisor https://review.openstack.org/512965
09:27:48 openstackgerrit Radoslav Gerganov proposed openstack/nova master: VMware: implement get_inventory() driver method https://review.openstack.org/506175
09:31:02 kalyan hi all, could someone please help me in launching containers with openstack devstack....magnum is only used for clusters? wat about if i want to launch a single container just like a vm using nova
09:31:09 kalyan thanks in advance :)
09:33:17 openstackgerrit Eli Qiao proposed openstack/nova master: Api-guide: Add concept for scheduler hints https://review.openstack.org/522106
09:40:11 openstackgerrit Yikun Jiang proposed openstack/nova master: Only query LIMIT instances when list in local cell https://review.openstack.org/522477
11:20:19 mdbooth lyarwood: johnthetubaguy remind me, in cinder's new world order, are we intending to no longer store (or at least use) connection_info on the compute host at all?
11:20:54 lyarwood mdbooth: yes AFAIK but I've not looked at the v3 patches for a while :)
11:21:07 mdbooth lyarwood: YES we are still going to store it?
11:21:40 lyarwood mdbooth: no, we don't intend to store it anymore
11:22:21 mdbooth lyarwood: That's what I thought I remembered. It's just that currently the only reason we can automatically reconnect volumes after a host reboot is that we store it, and therefore don't need to go back to cinder.
11:22:47 mdbooth The cinder credentials problem.
11:24:31 lyarwood mdbooth: yeah true, it would be the same as the encryption metadata issue we have at the moment right
11:24:48 mdbooth Indeed
11:24:50 lyarwood mdbooth: we don't cache it and can't fetch it at start up
11:25:14 mdbooth I think with encryption credentials we *mustn't* cache it, right?
11:26:07 lyarwood mdbooth: if we want to avoid the start up issue yeah, the rest of the time we are just fetching them from cinder with each encryptor attach/detach
11:26:18 lyarwood ah sorry
11:26:22 lyarwood mustn't
11:26:25 lyarwood didn't parse that
11:26:49 lyarwood yeah we don't want to store the key locally etc
11:26:55 mdbooth I mean, it seems dumb to isolate the security of your volume from Nova, then store the key material in Nova's db
11:27:01 lyarwood yeah
11:28:08 mdbooth The behaviour of libvirt's _get_guest_storage_config is extremely unfortunate
11:28:15 mdbooth It really shouldn't be connecting volumes
11:54:59 openstackgerrit Merged openstack/python-novaclient master: Add support for microversion 2.55 - flavor description https://review.openstack.org/520819
12:10:17 openstackgerrit Balazs Gibizer proposed openstack/nova master: Transform missing delete notifications https://review.openstack.org/410297
12:10:18 openstackgerrit Balazs Gibizer proposed openstack/nova master: Send soft_delete from context manager https://review.openstack.org/476459
12:18:34 openstackgerrit Merged openstack/nova master: factor out compute service start in ServerMovingTest https://review.openstack.org/503037
12:18:42 openstackgerrit Merged openstack/nova master: Test resource allocation during soft delete https://review.openstack.org/495159
12:22:02 kashyap mdbooth: Random aside -- Nova does use 'cache=none' by default; except for network storage backends (as you know).
12:22:13 kashyap libvirt/driver.py --> disk_cachemode() --> self._disk_cachemode = "none"
12:22:23 openstackgerrit Ildiko Vancsa proposed openstack/nova master: Implement new attach Cinder flow https://review.openstack.org/330285
12:22:24 openstackgerrit Ildiko Vancsa proposed openstack/nova master: libvirt: Allow multiple volume attachments https://review.openstack.org/267587
12:28:44 openstackgerrit Ildiko Vancsa proposed openstack/nova master: Implement new attach Cinder flow https://review.openstack.org/330285
12:28:45 openstackgerrit Ildiko Vancsa proposed openstack/nova master: libvirt: Allow multiple volume attachments https://review.openstack.org/267587
12:32:02 openstackgerrit sean mooney proposed openstack/nova master: WIP stub out changes for multi port bindings https://review.openstack.org/522537
12:33:16 openstackgerrit Ildiko Vancsa proposed openstack/nova master: libvirt: Allow multiple volume attachments https://review.openstack.org/267587
12:33:16 openstackgerrit Ildiko Vancsa proposed openstack/nova master: Implement new attach Cinder flow https://review.openstack.org/330285
12:43:09 openstackgerrit Merged openstack/nova master: Change live_migrate tests to use fakedriver https://review.openstack.org/505202
12:43:16 openstackgerrit Merged openstack/nova master: Use FakeLiveMigrateDriver in notification test https://review.openstack.org/519405
12:47:45 openstackgerrit Merged openstack/nova master: Transform instance-live_migration_pre notification https://review.openstack.org/482070
12:49:52 mdbooth kashyap: Right, that's what I thought from looking at the code. I think you showed me a CI log using writeback though, right?
12:50:03 kashyap mdbooth: Yes, that's for network storage (Ceph)
12:50:16 kashyap Which means something else, as you know.
12:50:34 mdbooth Ah, I thought the log you showed me was non-ceph
12:50:49 kashyap It was Ceph, sorry for misinforming. I sit corrected
12:50:51 mdbooth And I'd assumed we'd explicitly configured CI to prefer performance over data integrity
12:51:00 mdbooth Which wouldn't be unreasonable
12:51:04 kashyap Yeah
12:55:12 openstackgerrit Forest Romain proposed openstack/nova master: Fix list servers by hypervisor https://review.openstack.org/518699
12:56:03 kashyap mdbooth: Also - https://review.openstack.org/#/c/522067/1
13:06:20 openstackgerrit Merged openstack/nova master: Fix the usage of instance.snapshot notification sample https://review.openstack.org/520579
13:59:20 bauzas heh, I never saw the gate so empty :p
13:59:31 bauzas eat turkey, folks!
14:00:19 bauzas (at least, the ones wanting to eat turkey are just awakening and preparing the dinner :p )
14:20:57 mdbooth Can't build current requirements, as protobuf 3.5.0 doesn't seem to be a thing
14:21:24 mdbooth Although as it only seems to be in upper-requirement.txt, not entirely sure why it needs to be
14:21:28 mdbooth Anybody else hit this?
14:22:27 mdbooth probuf was bumped in Ie610db5304707e3db9f7025037405a3af79d2ce3, which landed on 17th (last Friday)
14:23:41 tssurya mdbooth : yes
14:24:03 openstack Launchpad bug 1734009 in OpenStack Global Requirements "No matching distribution found for protobuf===3.5.0" [Undecided,New]
14:24:03 tssurya mdbooth : https://bugs.launchpad.net/openstack-requirements/+bug/1734009
14:24:21 mdbooth tssurya: Thanks
14:27:11 kashyap mdbooth: It's not Friday yet, but check out this easter egg here :-)
14:27:16 kashyap https://unix.stackexchange.com/questions/405783/why-does-man-print-gimme-gimme-gimme-at-0030
14:27:32 kashyap Typing 'man' command after 00:30 prints "gimme gimme gimme"
14:27:39 kashyap An allusion to some pop culture song :P
14:29:06 kashyap The "bug" is fixed, though. The author admits "It was never meant to affect non-error cases."
14:29:21 mdbooth kashyap: lol
14:29:47 kashyap mdbooth: But I like it that they fixed it _while_ retaining the easter egg.
14:52:08 johnthetubaguy mdbooth: +1 what lyarwood said, hoping for connection_info fetched from cinder every time, the current patches don't quite get there yet, but should be possible.
14:52:47 mdbooth johnthetubaguy: Was pretty sure that was where we were heading, but noted that it would regress restart after host reboot
14:53:15 johnthetubaguy mdbooth: you would just pull from cinder I think, or would that break for some reason?
14:53:31 mdbooth johnthetubaguy: This currently works only because we cache it, as we can't fetch it from cinder without an auth context
14:53:53 johnthetubaguy oh right, there are other cases as well I think
14:54:05 johnthetubaguy auto unrescue I think
14:54:14 johnthetubaguy we need an admin token like we do for neutron
14:54:22 johnthetubaguy I had totally forgotten about that mess
14:54:29 johnthetubaguy just as well we don't drop the cache yet
14:54:47 johnthetubaguy ildikov: tl;dr you are correct to keep caching connection_info for now: ^
14:55:04 johnthetubaguy mdbooth: great reminder, complete forgot all that
14:56:24 mdbooth johnthetubaguy: lyarwood Not familiar with the admin token thing. Does that provide Nova with some special admin privs when calling a remote service without requiring user creds?
14:56:40 mdbooth And does it exist already for cinder, or is it currently a neutron only concept?
14:58:17 mdbooth johnthetubaguy: For additional context, btw, encrypted volumes are broken in the above situation because we *don't* cache credentials (and mustn't)
14:58:44 johnthetubaguy ah, right...
14:58:51 openstackgerrit Balazs Gibizer proposed openstack/nova master: Send soft_delete from context manager https://review.openstack.org/476459
14:58:51 openstackgerrit Balazs Gibizer proposed openstack/nova master: Transform missing delete notifications https://review.openstack.org/410297
14:59:10 mdbooth So we're already not doing this properly for instances with volumes
14:59:17 johnthetubaguy mdbooth: the admin token is really just something that access all projects, so you don't need to use the non-existent user token
14:59:43 johnthetubaguy creds are just in the nova conf
14:59:58 mdbooth johnthetubaguy: Would that allow me to fetch a volume's encryption credentials?
15:00:15 johnthetubaguy I presume it should, for better or worse
15:00:34 mdbooth If it does, we might as well just cache them
15:00:42 mdbooth In which case, we might as well not have them
15:01:21 johnthetubaguy there was a plan to fetch it from nova-conductor, assuming AMQP gets secured, but you can still get the key

Earlier   Later