*** macz_ has quit IRC | 00:00 | |
*** k_mouza has joined #openstack-nova | 00:10 | |
*** k_mouza has quit IRC | 00:14 | |
*** brinzhang has joined #openstack-nova | 00:21 | |
*** tosky has quit IRC | 00:23 | |
*** zzzeek has quit IRC | 00:29 | |
*** zzzeek has joined #openstack-nova | 00:30 | |
*** derekh has quit IRC | 00:44 | |
*** k_mouza has joined #openstack-nova | 00:57 | |
*** k_mouza has quit IRC | 01:02 | |
*** martinkennelly has quit IRC | 01:28 | |
*** zzzeek has quit IRC | 01:30 | |
*** zzzeek has joined #openstack-nova | 01:33 | |
*** k_mouza has joined #openstack-nova | 02:43 | |
*** k_mouza has quit IRC | 02:47 | |
*** LinPeiWen has quit IRC | 03:08 | |
*** mkrai has joined #openstack-nova | 03:19 | |
*** psachin has joined #openstack-nova | 03:33 | |
*** k_mouza has joined #openstack-nova | 03:54 | |
*** k_mouza has quit IRC | 03:59 | |
*** sapd1 has joined #openstack-nova | 04:12 | |
*** mkrai has quit IRC | 04:14 | |
gmann | lyarwood: seems like tempest test cleanup change did not fix the nova-ceph-multistore job - https://storage.gra.cloud.ovh.net/v1/AUTH_dcaab5e32b234d56b626f72581e3644c/zuul_opendev_logs_548/763487/3/check/nova-ceph-multistore/5480be7/testr_results.html | 04:35 |
---|---|---|
gmann | lyarwood: got the issue, its class level vs test level cleanup. fixing in https://review.opendev.org/c/openstack/tempest/+/764407 | 04:53 |
openstackgerrit | Ghanshyam Mann proposed openstack/nova master: DNM: testing fix for nova-ceph-multistore https://review.opendev.org/c/openstack/nova/+/764408 | 04:55 |
gmann | testing here ^^ | 04:55 |
*** LinPeiWen has joined #openstack-nova | 04:56 | |
*** brinzhang has quit IRC | 05:22 | |
*** brinzhang has joined #openstack-nova | 05:22 | |
*** brinzhang has quit IRC | 05:24 | |
*** brinzhang has joined #openstack-nova | 05:24 | |
*** rcernin has quit IRC | 05:27 | |
*** evrardjp has quit IRC | 05:33 | |
*** evrardjp has joined #openstack-nova | 05:33 | |
*** vishalmanchanda has joined #openstack-nova | 05:41 | |
*** brinzhang has quit IRC | 05:47 | |
*** rcernin has joined #openstack-nova | 05:49 | |
*** zzzeek has quit IRC | 05:53 | |
*** brinzhang has joined #openstack-nova | 05:54 | |
*** ratailor has joined #openstack-nova | 05:54 | |
*** zzzeek has joined #openstack-nova | 05:56 | |
*** rcernin has quit IRC | 06:01 | |
*** mkrai has joined #openstack-nova | 06:12 | |
*** rcernin has joined #openstack-nova | 06:12 | |
*** hemanth_n has joined #openstack-nova | 06:16 | |
*** sapd1_x has joined #openstack-nova | 06:21 | |
*** sapd1_x has quit IRC | 06:26 | |
*** rcernin has quit IRC | 06:33 | |
*** rcernin has joined #openstack-nova | 06:37 | |
*** belmoreira has joined #openstack-nova | 06:53 | |
*** rcernin has quit IRC | 07:01 | |
*** lpetrut has joined #openstack-nova | 07:04 | |
gmann | lyarwood: its late here, going to bed. I pinged other tempest core to merger the tempest change once it pass the job in nova testing patch | 07:06 |
*** zzzeek has quit IRC | 07:10 | |
*** zzzeek has joined #openstack-nova | 07:12 | |
*** slaweq has joined #openstack-nova | 07:27 | |
*** mkrai has quit IRC | 07:36 | |
*** zzzeek has quit IRC | 07:38 | |
*** zzzeek has joined #openstack-nova | 07:38 | |
*** damien_r has joined #openstack-nova | 07:49 | |
*** damien_r has quit IRC | 07:53 | |
*** rcernin has joined #openstack-nova | 07:54 | |
openstackgerrit | Wenping Song proposed openstack/nova master: Nova supports password encrypted VNC https://review.opendev.org/c/openstack/nova/+/622336 | 07:56 |
*** damien_r has joined #openstack-nova | 08:00 | |
*** andrewbonney has joined #openstack-nova | 08:08 | |
bauzas | good morning Nova | 08:09 |
*** rcernin has quit IRC | 08:11 | |
*** martinkennelly has joined #openstack-nova | 08:13 | |
*** tesseract has joined #openstack-nova | 08:15 | |
*** rpittau|afk is now known as rpittau | 08:17 | |
*** rcernin has joined #openstack-nova | 08:19 | |
*** rcernin has quit IRC | 08:20 | |
openstackgerrit | Wenping Song proposed openstack/nova master: Cyborg shelve/unshelve support https://review.opendev.org/c/openstack/nova/+/729563 | 08:36 |
*** ociuhandu has joined #openstack-nova | 08:38 | |
*** ociuhandu has quit IRC | 08:43 | |
*** k_mouza has joined #openstack-nova | 08:45 | |
*** takamatsu has quit IRC | 08:46 | |
*** k_mouza has quit IRC | 08:49 | |
*** zzzeek has quit IRC | 08:49 | |
*** zzzeek has joined #openstack-nova | 08:50 | |
*** mgoddard has joined #openstack-nova | 08:50 | |
lyarwood | gmann: ack thanks, I'm not sure if that's going to work tbh, it looks like my patch was working the DELETEs were just racing | 08:55 |
lyarwood | gmann: on my bad, the DELETE I was looking at was for the volume attachment | 08:59 |
*** happyhemant has joined #openstack-nova | 08:59 | |
*** k_mouza has joined #openstack-nova | 09:00 | |
*** Qianbiao has joined #openstack-nova | 09:00 | |
*** k_mouza has quit IRC | 09:05 | |
*** zzzeek has quit IRC | 09:08 | |
openstackgerrit | Wenping Song proposed openstack/nova master: Cyborg shelve/unshelve support https://review.opendev.org/c/openstack/nova/+/729563 | 09:10 |
*** mkrai has joined #openstack-nova | 09:11 | |
lyarwood | gmann: okay that's also failing still | 09:12 |
*** derekh has joined #openstack-nova | 09:12 | |
*** zzzeek has joined #openstack-nova | 09:12 | |
openstackgerrit | Lee Yarwood proposed openstack/nova master: zuul: Skip test_attach_scsi_disk_with_config_drive in nova-ceph-multistore https://review.opendev.org/c/openstack/nova/+/764424 | 09:13 |
lyarwood | gibi / stephenfin / bauzas ; ^ lets skip this test in the ceph job for now until we get is passing again | 09:13 |
*** Yumeng has joined #openstack-nova | 09:17 | |
*** sapd1_x has joined #openstack-nova | 09:18 | |
* bauzas clicks | 09:18 | |
*** xek_ has joined #openstack-nova | 09:19 | |
lyarwood | that regex was written prior to coffe on a Friday morning after a long week and several night shifts with my child so lets wait to see if it skips the test before +W'ing ;) | 09:20 |
* lyarwood likes to get his excuses in early | 09:21 | |
*** k_mouza has joined #openstack-nova | 09:23 | |
bauzas | lyarwood: I don't see a problem with the regex itself, and the test name is long so I'm pretty sure we don't remove another test too | 09:23 |
bauzas | https://github.com/openstack/tempest/search?q=test_attach_scsi_disk_with_config_drive proves me I was right | 09:24 |
lyarwood | cool thansk | 09:25 |
lyarwood | thanks* | 09:25 |
*** sapd1_x has quit IRC | 09:27 | |
*** zzzeek has quit IRC | 09:27 | |
*** zzzeek has joined #openstack-nova | 09:29 | |
openstackgerrit | Lee Yarwood proposed openstack/nova stable/victoria: zuul: Introduce nova-evacuate https://review.opendev.org/c/openstack/nova/+/764430 | 09:34 |
openstackgerrit | Lee Yarwood proposed openstack/nova stable/victoria: zuul: Merge nova-evacuate into nova-multinode-live-migration https://review.opendev.org/c/openstack/nova/+/764431 | 09:34 |
openstackgerrit | Lee Yarwood proposed openstack/nova stable/victoria: zuul: Replace nova-live-migration with zuulv3 jobs https://review.opendev.org/c/openstack/nova/+/764432 | 09:34 |
lyarwood | elod: ^ okay here's my attempt to unfsck the stable/victoria evacuation stuff, one of the squashed changes hasn't merged on master yet but I just wanted to start testing | 09:34 |
*** songwenping_ has quit IRC | 09:36 | |
elod | lyarwood: wow, sounds good! thanks for the info (and for working on the issue :)), I'll look at them | 09:37 |
elod | \o/ | 09:38 |
lyarwood | elod: np and apologies it took so long | 09:38 |
Yumeng | hi bauzas gibi and sean-k-mooney: about the cyborg-owned vgpu support spec, I've discussed all your concerns with the team and updated the spec https://review.opendev.org/c/openstack/nova-specs/+/750116/ please see my update at your convenience. | 09:39 |
*** macz_ has joined #openstack-nova | 09:44 | |
*** macz_ has quit IRC | 09:49 | |
lyarwood | stephenfin: random one, any idea why we trigger a full gate run with docs changes? | 09:59 |
lyarwood | https://review.opendev.org/c/openstack/nova/+/764329 - I guess it isn't full but still that's odd. | 10:00 |
*** mgoddard has quit IRC | 10:02 | |
lyarwood | oh fail | 10:02 |
lyarwood | you changed nova/conf/pci.py | 10:02 |
lyarwood | that triggers things, ignore me | 10:02 |
*** dtantsur|afk is now known as dtantsur | 10:04 | |
*** mgoddard has joined #openstack-nova | 10:05 | |
stephenfin | yeah | 10:07 |
stephenfin | it's the conf change that triggers things | 10:08 |
stephenfin | I considered changing the regex to skip changes to that but that's silly | 10:08 |
lyarwood | yeah I'm not sure it's worth it | 10:10 |
*** zzzeek has quit IRC | 10:10 | |
*** zzzeek has joined #openstack-nova | 10:10 | |
stephenfin | lyarwood: I've held off on +W of that, as requested. Feel free to stick it on if it works as expected | 10:14 |
*** sapd1_x has joined #openstack-nova | 10:15 | |
*** sapd1_x has quit IRC | 10:21 | |
openstackgerrit | Alexandre arents proposed openstack/nova master: libvirt: Abort live-migration job when monitoring fails https://review.opendev.org/c/openstack/nova/+/764435 | 10:24 |
lyarwood | stephenfin: ack will do thanks | 10:24 |
*** diconico07 has quit IRC | 10:27 | |
*** nautik has quit IRC | 10:27 | |
stephenfin | holy s***, I always forget just how rough nova.pci is /o\ | 10:33 |
stephenfin | some serious TLC needed there | 10:33 |
*** brinzhang has quit IRC | 10:41 | |
*** mkrai has quit IRC | 10:42 | |
*** adrianc has joined #openstack-nova | 10:51 | |
*** hemanth_n has quit IRC | 10:59 | |
*** mkrai has joined #openstack-nova | 11:07 | |
*** ratailor has quit IRC | 11:14 | |
openstackgerrit | Stephen Finucane proposed openstack/nova master: pci: Improve docs for '_filter_*' functions https://review.opendev.org/c/openstack/nova/+/764376 | 11:35 |
openstackgerrit | Stephen Finucane proposed openstack/nova master: pci: Add logging for filtering https://review.opendev.org/c/openstack/nova/+/764377 | 11:35 |
openstackgerrit | Stephen Finucane proposed openstack/nova master: pci: Improve testing of 'nova.pci.request' module https://review.opendev.org/c/openstack/nova/+/764446 | 11:35 |
*** macz_ has joined #openstack-nova | 11:38 | |
sean-k-mooney | Yumeng: thanks im onloy around for a few hours today but ill try to review it before i leave | 11:40 |
sean-k-mooney | stephenfin: it is rough but it works. we very very rarely have any bugs or issue related to nova.pci we the bugs related to sriov/pci are almost always in the compute manager | 11:41 |
Yumeng | sean-k-mooney: That's great! Thanks sean. Just take your time. ^^ | 11:41 |
*** ociuhandu has joined #openstack-nova | 11:42 | |
*** macz_ has quit IRC | 11:42 | |
*** dklyle has quit IRC | 11:55 | |
*** ociuhandu has quit IRC | 11:56 | |
*** tosky has joined #openstack-nova | 11:57 | |
*** ociuhandu has joined #openstack-nova | 12:04 | |
*** adrianc has quit IRC | 12:07 | |
*** sapd1_x has joined #openstack-nova | 12:07 | |
openstackgerrit | Lee Yarwood proposed openstack/nova stable/victoria: zuul: Replace nova-live-migration with zuulv3 jobs https://review.opendev.org/c/openstack/nova/+/764432 | 12:08 |
*** ociuhandu has quit IRC | 12:08 | |
*** adrianc has joined #openstack-nova | 12:11 | |
*** sapd1_x has quit IRC | 12:12 | |
*** raildo has joined #openstack-nova | 12:13 | |
*** ociuhandu has joined #openstack-nova | 12:15 | |
openstackgerrit | Stephen Finucane proposed openstack/nova stable/train: Validate id as integer for os-aggregates https://review.opendev.org/c/openstack/nova/+/764455 | 12:23 |
openstackgerrit | Stephen Finucane proposed openstack/nova stable/stein: Validate id as integer for os-aggregates https://review.opendev.org/c/openstack/nova/+/764308 | 12:26 |
stephenfin | hmm, cherry-picking via the web UI is curiously slow now :-\ | 12:28 |
openstackgerrit | Stephen Finucane proposed openstack/nova stable/rocky: Validate id as integer for os-aggregates https://review.opendev.org/c/openstack/nova/+/764309 | 12:28 |
openstackgerrit | Stephen Finucane proposed openstack/nova stable/queens: Validate id as integer for os-aggregates https://review.opendev.org/c/openstack/nova/+/764310 | 12:29 |
lyarwood | FWIW git-review is also slow for me so I assume it's something on the backend | 12:46 |
lyarwood | not the ui | 12:46 |
gibi | yeah, the whole UI feels slower than the old one | 12:51 |
*** xek__ has joined #openstack-nova | 12:58 | |
*** ociuhandu has quit IRC | 12:59 | |
*** xek_ has quit IRC | 13:00 | |
*** macz_ has joined #openstack-nova | 13:02 | |
*** macz_ has quit IRC | 13:07 | |
openstackgerrit | Merged openstack/nova master: zuul: Skip test_attach_scsi_disk_with_config_drive in nova-ceph-multistore https://review.opendev.org/c/openstack/nova/+/764424 | 13:27 |
*** ociuhandu has joined #openstack-nova | 13:30 | |
openstackgerrit | Lee Yarwood proposed openstack/nova master: compute: Don't detach volumes when RescheduledException raised without retry https://review.opendev.org/c/openstack/nova/+/757894 | 13:32 |
openstackgerrit | Lee Yarwood proposed openstack/nova master: zuul: Add devstack-plugin-ceph-compute-local-ephemeral to experimental https://review.opendev.org/c/openstack/nova/+/743220 | 13:34 |
openstackgerrit | Lee Yarwood proposed openstack/nova master: libvirt: Skip encryption metadata lookups if secret already exists on host https://review.opendev.org/c/openstack/nova/+/764246 | 13:35 |
openstackgerrit | Lee Yarwood proposed openstack/nova master: nova-live-migration: Disable *all* virt services during negative tests https://review.opendev.org/c/openstack/nova/+/762623 | 13:35 |
*** ociuhandu has quit IRC | 13:38 | |
openstackgerrit | Lee Yarwood proposed openstack/nova stable/victoria: zuul: Skip test_attach_scsi_disk_with_config_drive in nova-ceph-multistore https://review.opendev.org/c/openstack/nova/+/764464 | 13:42 |
lyarwood | ^ elod / bauzas ; looks like we also need this on stable/victoria | 13:42 |
bauzas | roger. | 13:42 |
elod | ACK | 13:46 |
*** tobberydberg_ has quit IRC | 13:53 | |
*** tobberydberg has joined #openstack-nova | 13:56 | |
*** mgoddard has quit IRC | 13:56 | |
*** ociuhandu has joined #openstack-nova | 14:12 | |
*** ociuhandu has quit IRC | 14:12 | |
*** ociuhandu has joined #openstack-nova | 14:14 | |
*** ociuhandu has quit IRC | 14:20 | |
*** sapd1_x has joined #openstack-nova | 14:20 | |
*** ociuhandu has joined #openstack-nova | 14:33 | |
*** derekh has quit IRC | 14:36 | |
*** derekh has joined #openstack-nova | 14:36 | |
*** ociuhandu has quit IRC | 14:41 | |
*** zzzeek has quit IRC | 14:44 | |
*** zzzeek has joined #openstack-nova | 14:48 | |
*** rpittau is now known as rpittau|afk | 14:51 | |
*** mgoddard has joined #openstack-nova | 14:51 | |
*** hack-char has quit IRC | 14:51 | |
*** hack-char has joined #openstack-nova | 14:54 | |
stephenfin | lyarwood, elod: In theory, we could now +2 +W all stable backports and they won't be able to merge because of dansmith's cherry-pick test, right? | 14:55 |
elod | stephenfin: why? O.o | 14:56 |
elod | how do you mean? | 14:56 |
stephenfin | take this change (thanks for the reviews, btw :)) https://review.opendev.org/c/openstack/nova/+/764455 | 14:56 |
stephenfin | that's for stable/train but I've got (identical) cherry-picks all the way back to stable/queens | 14:57 |
stephenfin | so in theory, if you were reviewing one, you could in theory review the others at the same time, and they'd have to merge in series because the pep8 job would fail otherwise | 14:58 |
stephenfin | *they'd have to merge in order | 14:58 |
elod | oh, so you mean we could +2+W in advance back till queens in this case | 14:58 |
stephenfin | Yup, exactly. That's much clearer :) | 14:58 |
elod | yeah, I guess that should work :] | 14:59 |
elod | I'm thinking now on cases where "somehowe-something-changes" but we don't realize and just recheck and patch merges... | 15:00 |
*** lpetrut has quit IRC | 15:02 | |
elod | hmmm. in case of a change in newer branch, the hash changes, so in that case it won't be a problem, on older branches the patches simply does not merge due to the same cherry-pick check | 15:02 |
elod | so that should work too | 15:02 |
stephenfin | yup, that what I'm thinking anyway | 15:03 |
*** ociuhandu has joined #openstack-nova | 15:03 | |
elod | we just have to be careful that the hash of patch on branch N+1 is really there in patch on branch N | 15:04 |
stephenfin | well I assume the test does that | 15:04 |
stephenfin | what we need to validate is that the hash corresponds to what we think it does (as opposed to an arbitrary patch) | 15:04 |
elod | no, it just checks whether the listed hashes are merged somewhere | 15:05 |
gmann | lyarwood: humm that seems like image deleted with server seems, debugging | 15:05 |
stephenfin | oh, so you could in theory cherry-pick staight from master to e.g. stable/queens and it would pass | 15:05 |
stephenfin | okay, yeah, that's one to watch for | 15:06 |
stephenfin | but once we do that...! :D | 15:06 |
elod | :) well, we could try that... ;) | 15:09 |
openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Revert "zuul: Skip test_attach_scsi_disk_with_config_drive in nova-ceph-multistore" https://review.opendev.org/c/openstack/nova/+/764311 | 15:11 |
*** vishalmanchanda has quit IRC | 15:22 | |
*** tesseract has quit IRC | 15:24 | |
*** diconico07 has joined #openstack-nova | 15:26 | |
*** mkrai has quit IRC | 15:29 | |
*** nautik has joined #openstack-nova | 15:31 | |
*** k_mouza has quit IRC | 15:32 | |
*** Qianbiao has quit IRC | 15:44 | |
lyarwood | /o\ sorry ssh timed out and I didn't notice | 15:48 |
lyarwood | gmann: yeah that's odd | 15:48 |
gmann | lyarwood: or we are not passing the image_id for delete. testing changes in above revert, it should work now | 15:49 |
gmann | now i am passing image_id and image should be found and deleted correctly | 15:50 |
*** k_mouza has joined #openstack-nova | 15:50 | |
lyarwood | gmann: kk | 15:51 |
lyarwood | stephenfin: https://review.opendev.org/c/openstack/nova/+/764246 - would you mind taking a swing at that before you call it a day? | 15:52 |
stephenfin | sure | 15:52 |
lyarwood | https://review.opendev.org/c/openstack/nova/+/743220 is also trival | 15:52 |
gmann | gibi: as placement is under Nova now, this enable to notify the placement changes on nova channel too - https://review.opendev.org/c/openstack/project-config/+/764469 | 15:53 |
gmann | as next step we can probably merge both channel (delete placement channel basically ) | 15:54 |
gmann | while pushing few policy patches in placement i realized there are not bot to nova channel | 15:54 |
*** mgoddard has quit IRC | 15:56 | |
openstackgerrit | Lee Yarwood proposed openstack/nova stable/victoria: zuul: Replace nova-live-migration with zuulv3 jobs https://review.opendev.org/c/openstack/nova/+/764432 | 15:56 |
*** tesseract has joined #openstack-nova | 15:57 | |
lyarwood | elod / bauzas ; okay the ceph change is now squashed into that change so we hopefully have a passing gate again on stable/victoria | 15:57 |
bauzas | all good | 15:57 |
lyarwood | actually pep8 will fail until https://review.opendev.org/c/openstack/nova/+/762623 lands FWIW | 15:57 |
lyarwood | so bauzas / gibi / stephenfin ; ^ reviews on that would be appreciated | 15:58 |
bauzas | ack | 15:58 |
gibi | gmann: ohh, so the governance patch has been merged. cool. yes I agree that we shoudl redirect the placement channel traffic to nova | 15:58 |
*** happyhemant has quit IRC | 15:59 | |
gmann | gibi: ok, I will send email to request infra admin to do that. it can be done via some redirect way I think. | 16:01 |
gibi | gmann: thanks! | 16:01 |
gmann | gibi: sent, may be you can confirm there and I will follow up with infra team - http://lists.openstack.org/pipermail/openstack-discuss/2020-November/019092.html | 16:07 |
gibi | gmann: ack | 16:07 |
*** macz_ has joined #openstack-nova | 16:09 | |
*** macz_ has quit IRC | 16:10 | |
*** ociuhandu_ has joined #openstack-nova | 16:21 | |
*** ociuhandu has quit IRC | 16:24 | |
*** ociuhandu_ has quit IRC | 16:26 | |
*** k_mouza has quit IRC | 16:28 | |
stephenfin | lyarwood: very soft -1 but I think it's worth thinking about before we proceed https://review.opendev.org/c/openstack/nova/+/764246 | 16:28 |
stephenfin | I'm pretty sure we've discussed this exact question before, in fact | 16:28 |
lyarwood | stephenfin: ack looking | 16:30 |
lyarwood | stephenfin: yeah we have, I might turn that into a hacking check of some kind | 16:31 |
stephenfin | I'm thinking the same thing. A modification of H904 perhaps | 16:32 |
*** k_mouza has joined #openstack-nova | 16:32 | |
gibi | stephenfin: good catch on non the delayed LOG interpolation. | 16:33 |
openstackgerrit | Lee Yarwood proposed openstack/nova master: libvirt: Skip encryption metadata lookups if secret already exists on host https://review.opendev.org/c/openstack/nova/+/764246 | 16:34 |
lyarwood | stephenfin: ack yeah we should add f-strings to that check | 16:35 |
*** ociuhandu has joined #openstack-nova | 16:40 | |
*** ociuhandu has quit IRC | 16:44 | |
lyarwood | lol so just found a wonderful gerrit bug | 16:49 |
lyarwood | if you're writing a comment don't leave your cursor anywhere outside of the textbox | 16:49 |
lyarwood | otherwise while text does end up in the box the ui will also react to any shortcuts you happen to be hitting at the sametime | 16:49 |
lyarwood | on google-chrome at least | 16:50 |
*** tesseract has quit IRC | 16:50 | |
sean-k-mooney | ya that sometimes happened before | 16:52 |
sean-k-mooney | i never really figured out why | 16:52 |
sean-k-mooney | its not new to the new version | 16:52 |
sean-k-mooney | you coudl work around it by disabling fast rendering | 16:52 |
lyarwood | ah kk, first time I've seen it | 16:52 |
sean-k-mooney | but that is not an option now | 16:52 |
*** tosky has quit IRC | 17:04 | |
*** belmoreira has quit IRC | 17:08 | |
*** hack-char has quit IRC | 17:13 | |
openstackgerrit | Elod Illes proposed openstack/nova stable/pike: Validate id as integer for os-aggregates https://review.opendev.org/c/openstack/nova/+/764479 | 17:15 |
*** hack-char has joined #openstack-nova | 17:16 | |
*** psachin has quit IRC | 17:32 | |
*** derekh has quit IRC | 17:41 | |
openstackgerrit | Stephen Finucane proposed openstack/nova master: utils: Sanitize purely numeric TLD in hostnames https://review.opendev.org/c/openstack/nova/+/764482 | 17:42 |
stephenfin | artom: sean-k-mooney: Have at it ^ | 17:43 |
stephenfin | I think that's the most sensible solution of the poor selection available to us | 17:43 |
* stephenfin would like 'openstack server create --hostname foo.bar ...' but doesn't want to do the work | 17:43 | |
stephenfin | :) | 17:44 |
artom | stephenfin, yeah, a new spec/API is too much work for something like this | 17:44 |
lyarwood | incoming | 17:44 |
* lyarwood needs to rebuild a test setup | 17:44 | |
openstackgerrit | Lee Yarwood proposed openstack/nova master: Add encryption fields to BlockDeviceMapping object and table https://review.opendev.org/c/openstack/nova/+/760453 | 17:46 |
openstackgerrit | Lee Yarwood proposed openstack/nova master: image_meta: Add ephemeral encryption properties https://review.opendev.org/c/openstack/nova/+/760454 | 17:46 |
openstackgerrit | Lee Yarwood proposed openstack/nova master: WIP virt: Add ephemeral encryption flag https://review.opendev.org/c/openstack/nova/+/760455 | 17:46 |
openstackgerrit | Lee Yarwood proposed openstack/nova master: WIP scheduler: Add an ephemeral encryption pre filter https://review.opendev.org/c/openstack/nova/+/760456 | 17:46 |
openstackgerrit | Lee Yarwood proposed openstack/nova master: func: Allow compute_driver to be set and used by _IntegratedTestBase https://review.opendev.org/c/openstack/nova/+/764484 | 17:46 |
openstackgerrit | Lee Yarwood proposed openstack/nova master: BlockDeviceMapping: Add is_local property https://review.opendev.org/c/openstack/nova/+/764485 | 17:46 |
openstackgerrit | Lee Yarwood proposed openstack/nova master: WIP compute: Update bdms with ephemeral encryption details when requested https://review.opendev.org/c/openstack/nova/+/764486 | 17:46 |
*** READ10 has joined #openstack-nova | 17:56 | |
openstackgerrit | Stephen Finucane proposed openstack/nova master: pci: Improve docs for '_filter_*' functions https://review.opendev.org/c/openstack/nova/+/764376 | 17:56 |
openstackgerrit | Stephen Finucane proposed openstack/nova master: pci: Add logging for filtering https://review.opendev.org/c/openstack/nova/+/764377 | 17:56 |
openstackgerrit | Stephen Finucane proposed openstack/nova master: pci: Improve testing of 'nova.pci.request' module https://review.opendev.org/c/openstack/nova/+/764446 | 17:56 |
*** READ10 has quit IRC | 17:59 | |
*** jangutter has quit IRC | 17:59 | |
*** jangutter has joined #openstack-nova | 17:59 | |
sean-k-mooney | stephenfin: we should jsut block anything with a . in it | 18:01 |
sean-k-mooney | stephenfin: if you want to allow FQDNS you shoudl write a spec and add a new microverion. this has implecatoin for cloud init and anything that assumes the hostname is a host name soe im not ok with adding support for hostnames as a bug fix | 18:04 |
sean-k-mooney | *support for FQDNs | 18:05 |
*** dtantsur is now known as dtantsur|afk | 18:05 | |
artom | Yeah, if it were up to me, I'd add a new microversion that did 2 things: 1. remove all the sanitization that we do, and just fail with a 400 if we get a bad hostname and 2. decouples VM names from dns_name, adding a separate argument for the latter | 18:13 |
artom | Piling on more obscure magic doesn't seem like the way forward | 18:13 |
artom | Wait, is instance.hostname exposed anywhere? | 18:16 |
artom | IOW, what do we actually use it for? | 18:16 |
sean-k-mooney | we use it for cloud init | 18:17 |
sean-k-mooney | its in the metadata | 18:17 |
sean-k-mooney | how your vm end up being called the same as the server name | 18:17 |
sean-k-mooney | but the vm also has a display name which can be modifed after and that is what we show in the cli/ui | 18:17 |
sean-k-mooney | the hostname cannot be changed | 18:18 |
sean-k-mooney | at lease i dont think it can | 18:18 |
artom | Ah, so the hostname isn't API-visible | 18:19 |
artom | OK, this mellows me a bit | 18:20 |
*** jangutter has quit IRC | 18:21 | |
*** jangutter has joined #openstack-nova | 18:21 | |
sean-k-mooney | artom: it is | 18:24 |
sean-k-mooney | its in the server details reponce | 18:24 |
sean-k-mooney | i think its just readonly once set | 18:24 |
artom | Ah, yeah, in the extended attributes | 18:25 |
artom | Bleah | 18:25 |
sean-k-mooney | OS-EXT-SRV-ATTR:hostname": "new-server-test", | 18:25 |
sean-k-mooney | its admin only by defaul but its in the api | 18:26 |
sean-k-mooney | you can change the server name vai a put but you cant change the hostname | 18:27 |
sean-k-mooney | i mean you can ssh in and manually change ti but its not changeable via the api | 18:27 |
sean-k-mooney | when you update the cerver name it updates the display name | 18:28 |
*** jangutter has quit IRC | 18:38 | |
*** jangutter has joined #openstack-nova | 18:39 | |
*** jangutter has quit IRC | 18:39 | |
*** jangutter has joined #openstack-nova | 18:40 | |
*** CeeMac has quit IRC | 18:55 | |
*** tosky has joined #openstack-nova | 19:03 | |
gmann | lyarwood: gibi bauzas skipped test in ceph job is passing now with tempest fix as depends-on - https://review.opendev.org/c/openstack/nova/+/764311 | 19:16 |
*** ralonsoh has quit IRC | 19:23 | |
*** k_mouza has quit IRC | 19:23 | |
*** whoami-rajat__ has quit IRC | 19:31 | |
*** zzzeek has quit IRC | 19:42 | |
*** zzzeek has joined #openstack-nova | 19:43 | |
*** ociuhandu has joined #openstack-nova | 19:45 | |
*** ociuhandu has quit IRC | 19:50 | |
*** mugsie has quit IRC | 19:59 | |
*** mugsie has joined #openstack-nova | 20:01 | |
*** zzzeek has quit IRC | 20:06 | |
*** zzzeek has joined #openstack-nova | 20:07 | |
*** slaweq has quit IRC | 20:26 | |
*** andrewbonney has quit IRC | 20:38 | |
*** sapd1_x has quit IRC | 21:04 | |
*** sapd1_x has joined #openstack-nova | 21:17 | |
*** cgoncalves has quit IRC | 21:21 | |
*** osmanlicilegi has quit IRC | 21:22 | |
*** k_mouza has joined #openstack-nova | 21:24 | |
*** Yumeng has quit IRC | 21:26 | |
*** xek_ has joined #openstack-nova | 21:26 | |
*** redrobot has quit IRC | 21:27 | |
*** xek__ has quit IRC | 21:29 | |
*** k_mouza has quit IRC | 21:29 | |
openstackgerrit | Merged openstack/nova stable/train: docs: Clarify configuration steps for PF devices https://review.opendev.org/c/openstack/nova/+/764329 | 21:46 |
*** efried has joined #openstack-nova | 21:54 | |
*** hack-char has quit IRC | 22:02 | |
*** hack-char has joined #openstack-nova | 22:04 | |
*** martinkennelly has quit IRC | 23:04 | |
*** xek_ has quit IRC | 23:08 | |
*** tosky has quit IRC | 23:42 | |
*** k_mouza has joined #openstack-nova | 23:42 | |
*** k_mouza has quit IRC | 23:47 |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!