| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2022-01-11 | |||
| 19:16:17 | opendevreview | Jonathan Race proposed openstack/os-traits master: Adds Pick guest CPU architecture based on host arch in libvirt driver support https://review.opendev.org/c/openstack/os-traits/+/824050 | |
| 21:02:39 | opendevreview | Merged openstack/placement master: Spec: support any trait in allocation candidates https://review.opendev.org/c/openstack/placement/+/649992 | |
| 21:05:37 | opendevreview | Merged openstack/placement master: Spec: support mixing required traits with any traits https://review.opendev.org/c/openstack/placement/+/649368 | |
| 22:47:15 | opendevreview | yuval proposed openstack/nova master: NO MERGE - activate lightbits third party ci on nova repo https://review.opendev.org/c/openstack/nova/+/824255 | |
| 23:02:26 | yuval | if someone is still awake - I have enabled the third party ci | |
| #openstack-nova - 2022-01-12 | |||
| 00:36:11 | gmann | yuval: cool. | |
| 04:16:05 | opendevreview | melanie witt proposed openstack/nova master: Add wrapper for oslo.concurrency lockutils.ReaderWriterLock() https://review.opendev.org/c/openstack/nova/+/824280 | |
| 08:47:43 | EugenMayer | How do you all handle nat reflection issues. so a vm/server connecting to an other service via the public IP, which happens to be hosted in the same openstack and is also the IP of the gateway for the commucation. So usualy nat reflection - any hints? | |
| 08:52:22 | opendevreview | yuval proposed openstack/nova-specs master: lightos volume driver spec https://review.opendev.org/c/openstack/nova-specs/+/824191 | |
| 08:53:13 | yuval | dear core reviewer - I have update the nova spec for lightos driver, can you please review? | |
| 08:53:30 | gibi | yuval: hi! I will take a look | |
| 08:54:38 | opendevreview | Lior Friedman proposed openstack/nova master: support use_multipath for nvme driver https://review.opendev.org/c/openstack/nova/+/824353 | |
| 09:02:56 | opendevreview | Ghanshyam proposed openstack/nova master: Server actions APIs scoped to project scope https://review.opendev.org/c/openstack/nova/+/824358 | |
| 09:04:30 | opendevreview | Lior Friedman proposed openstack/nova master: support use_multipath for nvme driver https://review.opendev.org/c/openstack/nova/+/823941 | |
| 09:51:45 | EugenMayer | considering this older commit to fix nat-reflextion issues https://github.com/openstack/nova/commit/b61e1ea12cd41ea507b1f6496ec1413c93bd679b .. is this even working/used with OVN networking? | |
| 10:02:42 | sean-k-mooney1 | EugenMayer: nat reflection is not something done by nova | |
| 10:03:09 | sean-k-mooney1 | EugenMayer: the neutron routers i think will do that | |
| 10:03:11 | EugenMayer | sean-k-mooney1 i assume it is a neutron topic, but that commit is somehow in the nova namespace | |
| 10:03:34 | sean-k-mooney1 | EugenMayer: nova used to have nova networks to use instead of neutron | |
| 10:03:37 | EugenMayer | sean-k-mooney1 asking there too - i cannot pin down the right project right now. Thank you for the hint | |
| 10:03:57 | EugenMayer | ah - i see. so it was 'nova back then'. Sorry i was not here back then | |
| 10:04:13 | EugenMayer | understood. The nova here is the 'old network nova nowdays replaced by neutron' | |
| 10:04:34 | sean-k-mooney1 | it was nova when nova did networking | |
| 10:04:39 | EugenMayer | This also means that particular patch has no affect/sense today. Could be entirely oncovereed | |
| 10:04:45 | sean-k-mooney1 | but today its in neutron | |
| 10:05:02 | EugenMayer | i see. As so often, thank you for clearing the fog out for me! | |
| 10:05:17 | sean-k-mooney1 | EugenMayer: i think that code has been entirly deleted a release or two ago | |
| 10:05:54 | sean-k-mooney1 | https://github.com/openstack/nova/tree/master/nova/network | |
| 10:06:10 | EugenMayer | I see. Do you know of any position right now if people basically are at 'use split DNS or deal with the consequences?' | |
| 10:07:12 | sean-k-mooney1 | well its kind of an odd issue | |
| 10:07:26 | sean-k-mooney1 | in that you seam to be tryign to use the same ip for two things | |
| 10:07:39 | sean-k-mooney1 | using it as the default gateway and as a vm | |
| 10:09:01 | sean-k-mooney1 | like you would normally reserve the default gateway ip so it could not be used for floating ips | |
| 10:38:46 | opendevreview | Merged openstack/nova master: Enable min pps tempest testing in nova-next https://review.opendev.org/c/openstack/nova/+/811748 | |
| 11:26:42 | opendevreview | Lee Yarwood proposed openstack/nova master: WIP libvirt: Register defaults for undefined hw image properties https://review.opendev.org/c/openstack/nova/+/800708 | |
| 11:26:42 | opendevreview | Lee Yarwood proposed openstack/nova master: WIP manage: Add image_property commands https://review.opendev.org/c/openstack/nova/+/824392 | |
| 12:24:58 | opendevreview | yuval proposed openstack/nova-specs master: lightos volume driver spec https://review.opendev.org/c/openstack/nova-specs/+/824191 | |
| 12:58:33 | sean-k-mooney | gibi: bauzas im ok with the latest version of ^ by the way. given the scope fo the nova change is small i think this is more then enough detail for now | |
| 12:58:57 | sean-k-mooney | more detail can be included in the documentation and or cidner driver | |
| 13:00:39 | gibi | sean-k-mooney: ack, I will check back to that today | |
| 13:02:37 | sean-k-mooney | gibi while i have you regarding teh ttl for healthcheck | |
| 13:02:50 | gibi | yes | |
| 13:03:10 | sean-k-mooney | so you think i shoudl keep the ttl, remove the stale_multipleer and intoduce a new state unkonwn | |
| 13:03:28 | sean-k-mooney | and if the ttl is expired for a reading report it as unknown or pass? | |
| 13:03:52 | sean-k-mooney | or perhaps warn? | |
| 13:03:53 | gibi | keep the ttl, remove the stale_multiplier, do not introduce unknow state | |
| 13:04:00 | sean-k-mooney | ok | |
| 13:04:02 | gibi | move from fail -> pass if ttl expires | |
| 13:04:16 | gibi | basicaly if it is unknown the report pass | |
| 13:04:22 | gibi | that will attract trafic | |
| 13:04:38 | gibi | so if it is still faulty we will detect it | |
| 13:04:56 | sean-k-mooney | ok that works for nova-api certenly | |
| 13:05:20 | sean-k-mooney | not sure about other services | |
| 13:06:00 | sean-k-mooney | i think it shoudl be workable | |
| 13:06:14 | sean-k-mooney | ill incoperate that into the next version | |
| 13:06:40 | sean-k-mooney | i kind of feel like if all are expired the state shoudl be warn but we can debate that in the implemetion | |
| 13:06:49 | sean-k-mooney | unless you feel that shoudl be stated in the spec | |
| 13:07:24 | sean-k-mooney | for now i think im going to follow dansmith's advise of sepcify that warn is a viald value but in initally do not use it in the implemenation | |
| 13:07:46 | gibi | you view also make sense, we don't know the state so we warn that it might be faulty. | |
| 13:08:03 | sean-k-mooney | i.e. only start usign warn after we have the basic pass fail stuff workign | |
| 13:08:42 | sean-k-mooney | gibi ya so the IETF draft recommend treating warn as still "passing" but that somethign might be wrong | |
| 13:09:10 | gibi | I'm OK to define warn, but not use it in the first implementation | |
| 13:09:40 | gibi | maybe we can gather some feedback from the field what operators want to see when the service don't know the state of it dpes | |
| 13:09:43 | gibi | *deps | |
| 13:10:15 | sean-k-mooney | ya that sounds resonabl that we shoudl ask there input | |
| 13:10:43 | gibi | so I won't block on having warn | |
| 13:11:01 | gibi | with the same logic I would add ttl but not yet add stale_multiplier | |
| 13:11:11 | gibi | we can add that later if we want to use warn for stale | |
| 13:14:53 | opendevreview | Jonathan Race proposed openstack/nova-specs master: Adds Pick guest CPU architecture based on host arch in libvirt driver support https://review.opendev.org/c/openstack/nova-specs/+/824044 | |
| 13:18:09 | chateaulav | morning all | |
| 13:19:10 | sean-k-mooney | o/ | |
| 14:01:08 | chateaulav | kashyap: can you link the spec: https://review.opendev.org/c/openstack/nova-specs/+/824044 to the blueprint: https://blueprints.launchpad.net/nova/+spec/pick-guest-arch-based-on-host-arch-in-libvirt-driver | |
| 14:02:30 | kashyap | chateaulav: It is already linked to in the Whiteboard at the bottom, under "2022 Additions" -- are you looking for something more? | |
| 14:04:00 | chateaulav | ok gotcha, i know it has the "Set the URL for this specification" link above blueprint. wasnt sure if that needed to be set or not | |
| 14:04:25 | chateaulav | just want to make sure you all have everything to hopefully get the spec approved | |
| 14:06:47 | sean-k-mooney | chateaulav: we generally set the url to the html version of the spec when it has merged | |
| 14:06:58 | sean-k-mooney | you can set it to the review for now if you want | |
| 14:07:46 | sean-k-mooney | well o kashyap could but ya we normally wait for it to merge before we do that | |
| 14:08:37 | sean-k-mooney | chateaulav: everything looks correct to me form a lanuchpad point of view | |
| 14:08:48 | sean-k-mooney | so dont worry about that aspect | |
| 14:10:18 | chateaulav | cool, my first spec so wanted to make sure. | |
| 14:12:14 | sean-k-mooney | chateaulav: by the way regarding testing | |
| 14:12:32 | sean-k-mooney | i see two ways to approch this form a tempest point of view | |
| 14:13:09 | sean-k-mooney | you could extend tempest with expicite tests for booting with emulation and add some addtional senario tests gated bhind a feature flag | |
| 14:13:58 | sean-k-mooney | or we coudl just use tempest as is but tweak the job to use a non native arch image and set teh image proerty in via devstack local.sh | |
| 14:14:38 | sean-k-mooney | the latere woudl inovle adding addtional ci jobs to nova's pipepline | |
| 14:14:57 | sean-k-mooney | im not sure we really want to add + for every emulated achitrrue to run on every patch | |
| 14:15:25 | sean-k-mooney | but i would be supproted if haveing one emulated arch tested on every patch and runnign weekly jobs for the rest | |
| 14:19:12 | chateaulav | I think doing the tweak and having one designated arch run would be good | |
| 14:19:45 | chateaulav | as the overall goal is to ensure emulation is functional | |
| 14:19:46 | sean-k-mooney | ack so i would suggest testing aarch64 by default in the check pipline | |
| 14:20:10 | sean-k-mooney | and then addign all fo them to a the periodic-weekly pipline and then experimental pipeline | |
| 14:21:20 | sean-k-mooney | that way we can review the status of the other arch jobs in our weekly meeting without much impact to the ci workload | |
| 14:22:00 | sean-k-mooney | you could also filter the josb to only run on changes to the libvirt driver and compute module i can help with that | |
| 14:22:46 | chateaulav | sounds good. and greatly appreciated | |
| 14:43:45 | opendevreview | yuval proposed openstack/nova-specs master: lightos volume driver spec https://review.opendev.org/c/openstack/nova-specs/+/824191 | |
| 15:47:21 | opendevreview | yuval proposed openstack/nova-specs master: lightos volume driver spec https://review.opendev.org/c/openstack/nova-specs/+/824191 | |
| 15:47:43 | yuval | ^fixed doc auto testing | |
| 15:56:19 | opendevreview | Merged openstack/nova master: Add wrapper for oslo.concurrency lockutils.ReaderWriterLock() https://review.opendev.org/c/openstack/nova/+/824280 | |
| 16:08:39 | opendevreview | Jonathan Race proposed openstack/nova-specs master: Adds Pick guest CPU architecture based on host arch in libvirt driver support https://review.opendev.org/c/openstack/nova-specs/+/824044 | |