| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2022-01-12 | |||
| 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 | |
| 16:32:30 | opendevreview | Takashi Kajinami proposed openstack/nova master: List auth plugin parameters for [keystone] section https://review.opendev.org/c/openstack/nova/+/824458 | |
| 17:01:45 | opendevreview | yuval proposed openstack/nova-specs master: lightos volume driver spec https://review.opendev.org/c/openstack/nova-specs/+/824191 | |
| 17:46:48 | opendevreview | Merged openstack/nova stable/wallaby: Reproduce bug 1953359 https://review.opendev.org/c/openstack/nova/+/820554 | |
| 17:46:54 | opendevreview | Merged openstack/nova stable/wallaby: Extend the reproducer for 1953359 and 1952915 https://review.opendev.org/c/openstack/nova/+/821943 | |
| 17:56:23 | opendevreview | Ghanshyam proposed openstack/nova master: Move rule_if_system() method to base test class https://review.opendev.org/c/openstack/nova/+/824475 | |
| 17:57:19 | opendevreview | Ghanshyam proposed openstack/nova master: Server actions APIs scoped to project scope https://review.opendev.org/c/openstack/nova/+/824358 | |
| 18:23:14 | chateaulav | just a question, i may have miss read a the process docs. for submitting the spec should it be under nova-specs/specs/yoga/ as i currently have it under the implemented directory? | |
| 18:56:54 | yuval | chateaulav: I think the pep8 fails if you dont put it under approved/implemented | |
| 18:58:32 | chateaulav | ok, just want to make sure im not creating extra work | |
| 19:00:24 | sean-k-mooney | it shoudl be put under approved | |
| 19:00:29 | chateaulav | ok | |
| 19:00:35 | sean-k-mooney | and the name of the file shoudl match the blueprint name | |
| 19:00:44 | chateaulav | sean-k.mooney: thanks, will fix | |
| 19:00:53 | sean-k-mooney | you can run the check locally with "tox -e pep8,docs" | |
| 19:01:41 | sean-k-mooney | we have a script that will move it form approved to implemented if the feature is complted at the end of the cycle | |
| 19:01:59 | sean-k-mooney | i or other will run it and it uses the name of the file to check if the blueptirn is marked as complete | |
| 19:02:54 | 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 | |
| 19:05:28 | chateaulav | sean-k-mooney: ack, placed it in the right area now. my bad... lol. dont know if the +2 needs readded with the new patchset or if that auto adds itself. | |
| 19:07:07 | sean-k-mooney | it will be lost but ill try and review it tomorrow if i agree its ready i can fast approve but if not we will get to it tomorrow | |
| 19:07:40 | opendevreview | Iago Filipe proposed openstack/nova master: Remove deprecated opts from VNC conf https://review.opendev.org/c/openstack/nova/+/824478 | |