| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2019-06-18 | |||
| 13:36:51 | sean-k-mooney | hard reboot we kill teh runnign vm. unplug all the ports and disk redeine the vm and plug all the port/disk/devices again from scratch as if it was a first boot | |
| 13:37:32 | jangutter | hamdyk: You could split out the _set_config_VIFHostDevice section from https://review.opendev.org/#/c/658785 | |
| 13:38:31 | jangutter | hamdyk: That should be pretty non-controversial. | |
| 13:55:22 | openstackgerrit | Artom Lifshitz proposed openstack/nova master: Revert resize: wait for events according to hybrid plug https://review.opendev.org/644881 | |
| 13:55:23 | openstackgerrit | Artom Lifshitz proposed openstack/nova master: [DNM] testing bug/1813789 revert resize events https://review.opendev.org/664442 | |
| 13:55:23 | openstackgerrit | Artom Lifshitz proposed openstack/nova master: WIP: Include direct-physical in compute manager events check https://review.opendev.org/664431 | |
| 13:55:59 | openstackgerrit | Josephine Seifert proposed openstack/nova-specs master: Spec for the Nova part of Image Encryption https://review.opendev.org/608696 | |
| 14:07:31 | hamdyk | jangutter: will do | |
| 14:35:40 | mdbooth | lyarwood: LGTM, but missing unit test I think. | |
| 14:38:19 | ohwhyosa | Hello peoples! | |
| 14:38:37 | ohwhyosa | What is the best way to change the keymap in spice / novnc per instance? | |
| 14:38:56 | ohwhyosa | I saw the keymap param in nova.conf and I saw also that it's deprecated | |
| 14:39:05 | ohwhyosa | But that is more global | |
| 14:39:33 | ohwhyosa | Say I have a french person, a german person and an american person accesing their instances, surely they can change the keympa? | |
| 14:40:06 | sean-k-mooney | ohwhyosa: the best way is for them to do it in the os when they log in | |
| 14:40:34 | lyarwood | mdbooth: ack thanks | |
| 14:40:40 | lyarwood | is gerrit dying for anyone else? | |
| 14:40:50 | sean-k-mooney | ohwhyosa: that or they can try and set it in there spice client | |
| 14:40:58 | ohwhyosa | sean-k-mooney, But then, for example, the cirros pass with a :) at the end becomes complicated, and if the only way they had of accessing the machine was via console they'd be screweeeed | |
| 14:41:04 | ohwhyosa | sean-k-mooney, Thanks anyway! | |
| 14:41:24 | ohwhyosa | Hmmm sean-k-mooney the spice client, the web one? | |
| 14:41:34 | sean-k-mooney | ohwhyosa: no a desktop one | |
| 14:42:01 | sean-k-mooney | you can install a spice client locally and use that to conect to the vm directly | |
| 14:42:21 | sean-k-mooney | that client should be able to set the keymap i think | |
| 14:42:48 | ohwhyosa | sean-k-mooney, great! thanks! | |
| 14:46:48 | sean-k-mooney | stephenfin: you worked on ^ a bit do we have any documenation that could help | |
| 14:48:57 | artom | sean-k-mooney, errr, don't think support anything besides the webclient for SPICE | |
| 14:49:03 | artom | Because of the websocket proxy thing | |
| 14:49:18 | artom | The native client would need to understand websockets | |
| 14:50:23 | sean-k-mooney | im not sure about that i think you can get the direct socket url via the api | |
| 14:50:51 | artom | IIRC you get the websocket HTTP URL | |
| 14:50:55 | sean-k-mooney | but i think you do need to configure you hosts to support it | |
| 14:51:15 | stephenfin | sean-k-mooney: afaik it should be configurable via the VNC/Spice client, if said client supports such a thing | |
| 14:51:20 | sean-k-mooney | maybe im not sure i have used it with novnc in the past | |
| 14:51:25 | stephenfin | noVNC does since version 1.0, I think | |
| 14:52:35 | sean-k-mooney | artom: i would not be surprised if there are spice clients tha tsupprot websocket streams too by they way | |
| 14:53:30 | artom | sean-k-mooney, maybe, but back when I tried https://review.opendev.org/#/c/442040/ there weren't | |
| 14:54:58 | sean-k-mooney | oh ok. is that just a limitation for spice? im pretty sure i have used remote viewer and other clinets to connect to no vnc in the past | |
| 14:55:15 | artom | All consoles, I think | |
| 14:55:37 | artom | Because you can't get to the compute host directly, you need to go through a proxy on the publicly accessible controller | |
| 14:55:54 | artom | And all of our proxies use websockets | |
| 14:55:55 | sean-k-mooney | i could have been using virt-manager to connect via the host... | |
| 14:56:06 | artom | To be able to authenticate the connection first | |
| 14:56:29 | sean-k-mooney | well that the thing i thought we supported directly connecting to the host without the proxy and that was required for non web clients | |
| 14:56:44 | sean-k-mooney | anyway got to grab coffee before a meeting | |
| 15:03:09 | ohwhyosa | thanks sean-k-mooney artom stephenfin !! | |
| 15:12:26 | Nick_A | We are trying to inject passwords and keys with libguestfs-tools. keys work fine but not passwords. tried with config drive on and off. any ideas? | |
| 15:38:07 | openstackgerrit | Lee Yarwood proposed openstack/nova master: libvirt: flatten rbd images when unshelving an instance https://review.opendev.org/457886 | |
| 15:46:52 | Nick_A | virt-customize works, so it seems libguestfs isn't the issue | |
| 15:52:34 | openstackgerrit | Stephen Finucane proposed openstack/nova-specs master: Additional upgrade clarifications for cpu-resources https://review.opendev.org/666032 | |
| 16:06:25 | dansmith | mriedem: can you circle back to this soonishly? https://review.opendev.org/#/c/644881/ | |
| 16:08:56 | mriedem | https://www.youtube.com/watch?v=yKNQzFxV-qg | |
| 16:09:02 | mriedem | mm | |
| 16:09:03 | mriedem | buh | |
| 16:09:04 | mriedem | snuf | |
| 16:09:57 | dansmith | https://www.youtube.com/watch?v=9MOAhnQbC6s | |
| 16:10:17 | mriedem | NSFW | |
| 16:21:23 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Fix GET /servers/detail host_status performance regression https://review.opendev.org/666042 | |
| 16:24:38 | Nick_A | Is this the place to ask about nova-lxd? | |
| 16:24:44 | dansmith | no. | |
| 16:27:51 | sean-k-mooney | Nick_A: nova-lxd is an out of tree dirver that is effectivly only supported by cannonical | |
| 16:28:23 | Nick_A | gotcha | |
| 16:29:09 | sean-k-mooney | Nick_A: https://github.com/openstack/nova-lxd#support-and-discussions looks like they use #lxcontainers | |
| 16:29:19 | Nick_A | having trouble getting initial root passwords injected with it for lxc and with libguestfs for kvm | |
| 16:30:33 | sean-k-mooney | Nick_A: its not listed here but it does not work for libvirt lxc https://docs.openstack.org/nova/latest/user/support-matrix.html#operation_set_admin_password | |
| 16:30:54 | sean-k-mooney | Nick_A: so i would not be surprised if it was not supported for lxd also | |
| 16:31:56 | sean-k-mooney | Nick_A: https://github.com/openstack/nova-lxd/blob/bbdd85d5ce70e9b959c6b70a8ebc704ae41a1a79/doc/source/support_matrix/support-matrix.ini#L245-L260 its marked as unknon in there own documuentation so i dont think its suppored | |
| 16:32:50 | sean-k-mooney | Nick_A: if you are using config drive it might work https://github.com/openstack/nova-lxd/blob/d0faf787d92e20a55faf35ea6d95dd4d7ae7e598/nova/virt/lxd/driver.py#L584-L590 | |
| 16:33:02 | Nick_A | we've been trying with and without config drive | |
| 16:33:23 | Nick_A | reviewing your links thank you | |
| 16:33:46 | sean-k-mooney | ya i would suggest using user-data and cloud init instead | |
| 16:34:00 | sean-k-mooney | file injection is deprecated for some time now | |
| 17:35:07 | artom | dansmith, any chance I could get you to revisit https://review.opendev.org/#/c/644881/ hopefully one last time? I've added logging, and commented pointing out where they're happening in gate jobs. Should show that the patch works as intended and alleviate your reluctance to merge something you can't test | |
| 17:35:23 | dansmith | artom: I poked mriedem above to look at it | |
| 17:35:27 | dansmith | artom: I will after he does | |
| 17:35:33 | artom | dansmith, ack, appreciated | |
| 17:36:42 | artom | Am I the pony in this situation? | |
| 17:36:44 | artom | I've been worse | |
| 17:41:03 | dansmith | honestly, I have no idea | |
| 18:20:01 | mriedem | i'm trying to get some timings on this host_status perf regression before i kill this devstack env and then i'll get on that resize revert change | |
| 18:56:27 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Fix GET /servers/detail host_status performance regression https://review.opendev.org/666042 | |
| 19:24:52 | openstackgerrit | Merged openstack/nova master: Add Migration.cross_cell_move and get_by_uuid https://review.opendev.org/614012 | |
| 19:24:59 | openstackgerrit | Merged openstack/nova master: Fix enabled_filters default value in admin config docs https://review.opendev.org/665751 | |
| 19:36:37 | openstackgerrit | François Palin proposed openstack/nova stable/queens: Include all network devices in nova diagnostics https://review.opendev.org/666152 | |
| 19:38:09 | mriedem | man https://review.opendev.org/#/c/644881/ is still queued after nearly 6 hours | |
| 19:38:12 | openstackgerrit | François Palin proposed openstack/nova stable/queens: Include all network devices in nova diagnostics https://review.opendev.org/666152 | |
| 20:19:10 | openstackgerrit | Merged openstack/nova master: Deprecate RetryFilter https://review.opendev.org/663953 | |
| 20:24:46 | mriedem | artom: dansmith: well the last time i commented was PS23 and now it's at PS32 so pretty hard to do a diff (unless I eyeball the patch sets in separate tabs), but i think i've found enough to -1 for now anyway | |
| 20:25:25 | artom | mriedem, cheers :) | |
| 20:25:40 | artom | (Whoever implements "diff of diffs" for Gerrit will have songs written about them) | |
| 20:26:37 | mriedem | mostly easy stuff | |
| 20:27:17 | mriedem | but you've missed some of the 35 conditionals from the existing libvirt driver _create_domain_and_network logic to determine if we should wait for events | |
| 20:28:11 | artom | Oh only 35? | |
| 21:17:47 | sean-k-mooney | dansmith: im pretty sure i know the answer to this but the order in which https://github.com/openstack/nova/blob/master/nova/scheduler/request_filter.py#L157-L161 is defined is not ment to change the behavior of the prefilters right? | |
| 21:18:59 | sean-k-mooney | because if i reorder that alphabetically then it breaks unit test by changing behvior | |
| 21:19:02 | sean-k-mooney | ALL_REQUEST_FILTERS = [ | |
| 21:19:03 | sean-k-mooney | require_tenant_aggregate, | |
| 21:19:05 | sean-k-mooney | map_az_to_placement_aggregate, | |
| 21:19:07 | sean-k-mooney | require_image_type_support, | |
| 21:19:09 | sean-k-mooney | ] | |