Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2019-10-23
19:46:27 efried mriedem, dansmith: ^
19:50:16 openstackgerrit Matt Riedemann proposed openstack/nova stable/queens: Fix listing deleted servers with a marker https://review.opendev.org/690725
19:50:16 openstack bug 1849409 in OpenStack Compute (nova) train "openstack server list --deleted --limit -1 hangs" [High,In progress] https://launchpad.net/bugs/1849409 - Assigned to Matt Riedemann (mriedem)
19:50:16 openstackgerrit Matt Riedemann proposed openstack/nova stable/queens: Add functional regression test for bug 1849409 https://review.opendev.org/690724
19:57:45 openstackgerrit Elod Illes proposed openstack/nova stable/rocky: cleanup evacuated instances not on hypervisor https://review.opendev.org/687550
20:02:15 elod mriedem: I've updated the commit message ^^^ (forgot to put me '_off', as today is national holiday here, but tomorrow gibi will be back)
20:02:29 mriedem elod: i saw that, thanks
20:10:04 openstackgerrit Matt Riedemann proposed openstack/nova stable/rocky: cleanup evacuated instances not on hypervisor https://review.opendev.org/687550
20:10:46 openstackgerrit Matt Riedemann proposed openstack/nova stable/rocky: Add functional test for resize crash compute restart revert https://review.opendev.org/687563
20:10:53 openstackgerrit Matt Riedemann proposed openstack/nova stable/rocky: Prevent init_host test to interfere with other tests https://review.opendev.org/687862
20:11:14 openstack bug 1833581 in OpenStack Compute (nova) rocky "instance stuck in BUILD state if nova-compute is restarted" [Low,In progress] https://launchpad.net/bugs/1833581 - Assigned to Balazs Gibizer (balazs-gibizer)
20:11:14 openstackgerrit Matt Riedemann proposed openstack/nova stable/rocky: Functional reproduce for bug 1833581 https://review.opendev.org/687564
20:11:21 openstackgerrit Matt Riedemann proposed openstack/nova stable/rocky: Error out interrupted builds https://review.opendev.org/687565
20:11:53 openstackgerrit Eric Fried proposed openstack/nova master: Consolidate [image_cache] conf options https://review.opendev.org/690723
20:16:07 efried dansmith: support matrix, you want just one section for image caching (where maybe without precaching the support is "partial") or two?
20:16:44 dansmith doesn't matter to me
20:16:51 mriedem i vote for one
20:17:36 efried ack. dansmith, I haven't looked, which drivers does precaching work for today?
20:17:46 dansmith libvirt
20:18:14 efried ight
20:23:13 KeithMnemonic mriedem , melwitt if you use the nova cli --deleted --marker work as expected ;-)
20:24:08 mriedem nova cli A #1
20:24:11 efried do lxc and xen fall under the same libvirt umbrella?
20:24:20 efried I know they inherit some stuff...
20:25:16 mriedem for image caching....don't know, but i'd think yes?
20:25:26 mriedem at least for xen, not really sure about lxc and how it uses the image cache
20:26:02 efried looks like xen explicitly says nay
20:26:13 efried but there's not a separate driver for lxc afaics
20:27:53 efried powervm with the ssp driver does image caching, but it's not through the ImageCacheManager etc. And it wouldn't respond to any of the conf options. Not sure whether to say that has support or not...
20:28:19 efried oh, a driver note.
20:28:30 mriedem KeithMnemonic: well, with nova list are you using --marker with a server name or id?
20:28:43 mriedem b/c i wouldn't be surprised if nova list doesn't support specifying a server name for the marker like osc does
20:28:58 mriedem efried: i assumed you mean libvirt+xen
20:29:03 mriedem not the xenserver driver
20:29:08 efried uhm
20:29:13 mriedem there are 2 xens
20:29:20 mriedem the one we deprecated in train was xenserver
20:29:20 efried ...oh
20:29:31 mriedem libvirt+xen is also a thing that citrix used to maintain and no longer does
20:29:35 efried o, I see now
20:29:41 mriedem just like libvirt+lxc, libvirt+uml, etc are things
20:29:51 efried but like lxc, libvirt+xen would inherit whatever libvirt has.
20:29:56 mriedem maybe
20:29:59 mriedem usually
20:30:02 mriedem lxc is a weirdo though
20:30:34 mriedem i'd probably mark libvirt+lxc as unknown
20:32:26 efried and libvirt+xen too?
20:32:29 openstackgerrit Matt Riedemann proposed openstack/nova stable/pike: Fix listing deleted servers with a marker https://review.opendev.org/690735
20:32:29 openstack bug 1849409 in OpenStack Compute (nova) train "openstack server list --deleted --limit -1 hangs" [High,In progress] https://launchpad.net/bugs/1849409 - Assigned to Matt Riedemann (mriedem)
20:32:29 openstackgerrit Matt Riedemann proposed openstack/nova stable/pike: Add functional regression test for bug 1849409 https://review.opendev.org/690734
20:37:05 KeithMnemonic mriedem an id, just like with osc which confirms it is just osc that is not doing the marker/deleted correctly
20:41:20 efried boo, support matrix doesn't sphinx
20:43:07 mriedem KeithMnemonic: my point is, if you did: nova list --deleted --marker <server name not id> i think it won't work
20:43:22 mriedem nova cli isn't translating the marker from a name to an id, it just assumes id
20:44:16 sean-k-mooney libvirt+lxc will proably work for precaching but until we add support for resizing the filesystem to mage teh resized image the it will still be kind of borked.
20:44:49 sean-k-mooney normally we just assume cloud-init or the use will grow the filecycle to file the root disk itself
20:44:55 sean-k-mooney but that does not work with lxc
20:45:07 sean-k-mooney or at least if it does i dont know how to do it
20:46:02 KeithMnemonic right, i hear you
20:52:06 openstackgerrit Matt Riedemann proposed openstack/nova stable/ocata: Fix listing deleted servers with a marker https://review.opendev.org/690747
20:52:06 openstack bug 1849409 in OpenStack Compute (nova) train "openstack server list --deleted --limit -1 hangs" [High,In progress] https://launchpad.net/bugs/1849409 - Assigned to Matt Riedemann (mriedem)
20:52:06 openstackgerrit Matt Riedemann proposed openstack/nova stable/ocata: Add functional regression test for bug 1849409 https://review.opendev.org/690746
21:02:28 openstackgerrit Matt Riedemann proposed openstack/nova master: Require Migration object arg to migrate_instance_finish method https://review.opendev.org/690106
21:02:29 openstackgerrit Matt Riedemann proposed openstack/nova master: Stop converting Migration objects to dicts for migrate_instance_start https://review.opendev.org/690115
21:03:24 openstackgerrit Merged openstack/nova master: Add image precaching docs for aggregates https://review.opendev.org/687348
21:05:36 openstackgerrit Eric Fried proposed openstack/nova master: Add image caching to the support matrix https://review.opendev.org/690748
21:05:43 efried mriedem, dansmith: ^
21:15:30 melwitt efried, mriedem: commented on https://review.opendev.org/683988 and while I was looking at the console proxy log I notice the token auth uuid is logged at INFO at different places :( one of which is coming from underlying websockify and not sure how to stop it
21:17:49 efried a websockify bug I guess
21:18:40 melwitt it's not a bug, it's emitting it's 'path' but since noVNC upstream took out the cookie forwarding stuff and had us switch to using the path, the token auth is now in the path
21:21:15 mriedem melwitt: unrelated to this? https://review.opendev.org/#/c/220622/
21:21:44 melwitt mriedem: that fixes one of the two spots
21:22:17 mriedem so are you talking about this? https://zuul.opendev.org/t/openstack/build/8b011c8d92134f27ab59a8d392267b2e/log/controller/logs/screen-n-novnc-cell1.txt.gz#31
21:22:23 mriedem INFO nova.console.websocketproxy [-] 213.32.72.99 - - [23/Oct/2019 19:12:55] 213.32.72.99: Path: '?token=badacaffcc2-6d65-4bc4-a2f5-4081318d0433'
21:22:30 melwitt yes that's the other one
21:22:38 melwitt and that's logged from websockify underneath
21:22:44 mriedem i feel like i remember something recently about a bug with logging and the websockify stuff but can't put my finger on it
21:23:08 melwitt orly? ok. it has always emitted the 'Path:' there but now the token auth is in the path
21:24:56 mriedem wonder if there is a way to plumb a flag down to tell it not to log the path or mask certain keys
21:25:15 mriedem it's not a cve since this is just in the logs
21:25:18 melwitt yeah, I have to go take a gander
21:25:46 melwitt and review the patch you linked. looks pretty simple
21:26:20 mriedem and i have to go sit through maleficent 2, taking one for the team
21:26:23 mriedem speaking of bad stuff
21:26:39 melwitt lol
21:34:02 efried dansmith: what's your stance on testing requirements for the cyborg interaction? tempest > functional > unit, to the point where if there's tempest we can live without functional?
22:03:41 openstackgerrit Dustin Cowles proposed openstack/nova master: Provider Config File: YAML file loading and schema validation https://review.opendev.org/673341
22:03:42 openstackgerrit Dustin Cowles proposed openstack/nova master: Provider Config File: Merge provider configs to provider tree https://review.opendev.org/676522
22:03:42 openstackgerrit Dustin Cowles proposed openstack/nova master: Provider Config File: Function to further validate and retrieve configs https://review.opendev.org/676029
23:58:30 openstackgerrit melanie witt proposed openstack/nova master: Add new policy rule for viewing host status UNKNOWN https://review.opendev.org/679181
#openstack-nova - 2019-10-24
01:02:05 mriedem_afk efried: imo, we have fixtures for all external services we interact with heavily for the most part (cinder/neutron/glance) so functional testing with a cyborg fixture is going to be a lot faster and simpler than waiting for tempest plugin integration tests that are just going to be using a fake driver in cyborg anyway
02:31:56 openstackgerrit Merged openstack/nova stable/rocky: Fix exception translation when creating volume https://review.opendev.org/688089
03:29:12 openstackgerrit Son Pham proposed openstack/nova-specs master: recheck https://review.opendev.org/690789
04:57:59 openstackgerrit melanie witt proposed openstack/nova master: Add new policy rule for viewing host status UNKNOWN https://review.opendev.org/679181
06:33:58 bauzas good morning Nova
06:44:37 openstackgerrit pengyuesheng proposed openstack/os-resource-classes master: Switch to Ussuri jobs https://review.opendev.org/690857
06:47:19 openstackgerrit pengyuesheng proposed openstack/os-vif master: Switch to Ussuri jobs https://review.opendev.org/690860
07:06:07 openstackgerrit Merged openstack/nova master: Fix listing deleted servers with a marker https://review.opendev.org/690429
07:06:13 openstackgerrit Merged openstack/nova stable/queens: Retrun 400 if invalid query parameters are specified https://review.opendev.org/676028
08:30:28 openstackgerrit Lee Yarwood proposed openstack/nova master: libvirt: Ignore volume exceptions during post_live_migration https://review.opendev.org/682622

Earlier   Later