| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2019-10-23 | |||
| 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 | |
| 13:34:41 | mriedem | melwitt: thanks for fixing http://grafana.openstack.org/d/-iKINcImz/ceph-failure-rate?orgId=1 - nova master looks good now | |
| 13:53:26 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Make nova-next multinode and drop tempest-slow-py3 https://review.opendev.org/683988 | |
| 14:19:51 | openstackgerrit | Merged openstack/os-resource-classes master: Switch to Ussuri jobs https://review.opendev.org/690857 | |
| 14:56:49 | openstackgerrit | Eric Fried proposed openstack/os-resource-classes master: Stop testing with py2 https://review.opendev.org/690996 | |
| 14:59:21 | openstackgerrit | Eric Fried proposed openstack/os-traits master: Stop testing with py2 https://review.opendev.org/690997 | |
| 15:01:42 | melwitt | mriedem: np. re: train and the other stable branches, when I browsed on https://graphite.opendev.org I couldn't seen to find any jobs for our stable/train branch and no data for stein and rocky. is graphite no longer the thing to look at or? I just didn't know what was going on, so I stuck to only updating master for now | |
| 15:09:01 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Use admin neutron client to query ports for binding https://review.opendev.org/690999 | |
| 15:13:02 | mriedem | melwitt: yeah i noticed the same when i was poking through graphite this morning looking at those stable branches | |
| 15:13:15 | mriedem | i'm also not sure why glance isn't showing anything on master since the jobs for glance are correct | |
| 15:13:24 | mriedem | there is just no data in graphite for some reason | |
| 15:13:32 | melwitt | yeah, so I was concerned that if I added train, it wouldn't work. I need to ask -infra | |
| 15:13:38 | melwitt | oh hm | |
| 15:24:20 | openstackgerrit | Merged openstack/os-vif master: Switch to Ussuri jobs https://review.opendev.org/690860 | |
| 15:29:48 | mriedem | melwitt: just a few things in your host_status policy patch https://review.opendev.org/#/c/679181/ | |
| 15:30:41 | melwitt | thanks | |
| 15:33:03 | melwitt | how to do that | |
| 15:33:03 | melwitt | I'll try to adjust the sleep thing like you suggested. I had based it on this https://github.com/openstack/nova/blob/master/nova/tests/functional/test_server_group.py#L420 after having some trouble setting the down time lower. but I didn't try the sleep 0.5 thing. but, as I'm typing this, I think we have a way of advancing time in tests with oslo.timeutils. so that would be an ideal way to make this better. I'm gonna see if I can find | |
| 15:33:57 | melwitt | and if that works, I can propose a change to improve test_server_group as well since it's a 10 second slow thing too | |
| 15:34:19 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Use admin neutron client to gather port resource requests https://review.opendev.org/691005 | |
| 15:42:50 | mriedem | ah yeah i forgot about the oslo timeutils stuff | |
| 15:48:25 | mriedem | novaclient 16.0.0 extension list removal also breaks osc https://github.com/openstack/python-openstackclient/blob/45af14ca72b44c03f14189ac61a29253faf03bd7/openstackclient/common/extension.py#L94 | |
| 15:48:26 | mriedem | weee | |
| 16:46:50 | KeithMnemonic | mriedem , melwitt can i ask you some more about the osc thing from yesterday. the simple fix mriedem is not resolving it. i seems to be as you said with how find_resource does the search. but trying to capture/analyze this is ran into something that is probably obvious here but not to me. I can curl compute/servers/detail?deleted=True and get a list of deleted, if I add in the marker to the same curl i get | |
| 16:46:51 | KeithMnemonic | a 401 Unauthroized. any thoughts on why that happens? | |
| 16:47:14 | KeithMnemonic | same token,.... | |
| 16:47:40 | KeithMnemonic | only diff id &marker= . for non deleted it also works with the same token | |
| 16:47:53 | KeithMnemonic | and a marker | |
| 17:05:52 | KeithMnemonic | i figured that part out | |
| 17:15:02 | mriedem | idk off the top of my head why you'd get a 401 in that case | |
| 17:35:28 | openstackgerrit | Artom Lifshitz proposed openstack/nova master: Helper to start computes with different HostInfos https://review.opendev.org/686832 | |