Wednesday, 2022-01-12

gmannyuval: cool. 00:36
opendevreviewmelanie witt proposed openstack/nova master: Add wrapper for oslo.concurrency lockutils.ReaderWriterLock()  https://review.opendev.org/c/openstack/nova/+/82428004:16
*** hemna0 is now known as hemna07:38
EugenMayerHow 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:47
opendevreviewyuval proposed openstack/nova-specs master: lightos volume driver spec  https://review.opendev.org/c/openstack/nova-specs/+/82419108:52
yuvaldear core reviewer - I have update the nova spec for lightos driver, can you please review?08:53
gibiyuval: hi! I will take a look08:53
opendevreviewLior Friedman proposed openstack/nova master: support use_multipath for nvme driver  https://review.opendev.org/c/openstack/nova/+/82435308:54
opendevreviewGhanshyam proposed openstack/nova master: Server actions APIs scoped to project scope  https://review.opendev.org/c/openstack/nova/+/82435809:02
opendevreviewLior Friedman proposed openstack/nova master: support use_multipath for nvme driver  https://review.opendev.org/c/openstack/nova/+/82394109:04
*** elodille1 is now known as elodilles09:29
EugenMayerconsidering this older commit to fix nat-reflextion issues https://github.com/openstack/nova/commit/b61e1ea12cd41ea507b1f6496ec1413c93bd679b .. is this even working/used with OVN networking?09:51
sean-k-mooney1EugenMayer: nat reflection is not something done by nova10:02
sean-k-mooney1EugenMayer: the neutron routers i think will do that10:03
EugenMayersean-k-mooney1 i assume it is a neutron topic, but that commit is somehow in the nova namespace10:03
sean-k-mooney1EugenMayer: nova used to have nova networks to use instead of neutron10:03
EugenMayersean-k-mooney1 asking there too - i cannot pin down the right project right now. Thank you for the hint10:03
EugenMayerah - i see. so it was 'nova back then'. Sorry i was not here back then10:03
EugenMayerunderstood. The nova here is the 'old network nova nowdays replaced by neutron'10:04
sean-k-mooney1it was nova when nova did networking10:04
EugenMayerThis also means that particular patch has no affect/sense today. Could be entirely oncovereed10:04
sean-k-mooney1but today its in neutron10:04
EugenMayeri see. As so often, thank you for clearing the fog out for me!10:05
sean-k-mooney1EugenMayer: i think that code has been entirly deleted a release or two ago10:05
sean-k-mooney1https://github.com/openstack/nova/tree/master/nova/network10:05
EugenMayerI see. Do you know of any position right now if people basically are at 'use split DNS or deal with the consequences?'10:06
sean-k-mooney1well its kind of an odd issue10:07
sean-k-mooney1in that you seam to be tryign to use the same ip for two things10:07
sean-k-mooney1using it as the default gateway and as a vm10:07
sean-k-mooney1like you would normally reserve the default gateway ip so it could not be used for floating ips10:09
*** sean-k-mooney1 is now known as sean-k-mooney10:09
opendevreviewMerged openstack/nova master: Enable min pps tempest testing in nova-next  https://review.opendev.org/c/openstack/nova/+/81174810:38
opendevreviewLee Yarwood proposed openstack/nova master: WIP libvirt: Register defaults for undefined hw image properties  https://review.opendev.org/c/openstack/nova/+/80070811:26
opendevreviewLee Yarwood proposed openstack/nova master: WIP manage: Add image_property commands  https://review.opendev.org/c/openstack/nova/+/82439211:26
opendevreviewyuval proposed openstack/nova-specs master: lightos volume driver spec  https://review.opendev.org/c/openstack/nova-specs/+/82419112:24
sean-k-mooneygibi: 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 now12:58
sean-k-mooneymore detail can be included in the documentation and or cidner driver12:58
gibisean-k-mooney: ack, I will check back to that today13:00
sean-k-mooneygibi while i have you regarding teh ttl for healthcheck13:02
gibiyes13:02
sean-k-mooneyso you think i shoudl keep the ttl, remove the stale_multipleer and intoduce a new state unkonwn13:03
sean-k-mooneyand if the ttl is expired for a reading report it as unknown or pass?13:03
sean-k-mooneyor perhaps warn?13:03
gibikeep the ttl, remove the stale_multiplier, do not introduce unknow state13:03
sean-k-mooneyok13:04
gibimove from fail -> pass if ttl expires13:04
gibibasicaly if it is unknown the report pass13:04
gibithat will attract trafic13:04
gibiso if it is still faulty we will detect it 13:04
sean-k-mooneyok that works for nova-api certenly13:04
sean-k-mooneynot sure about other services13:05
sean-k-mooneyi think it shoudl be workable13:06
sean-k-mooneyill incoperate that into the next version13:06
sean-k-mooneyi kind of feel like if all are expired the state shoudl be warn but we can debate that in the implemetion 13:06
sean-k-mooneyunless you feel that shoudl be stated in the spec13:06
sean-k-mooneyfor 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 implemenation13:07
gibiyou view also make sense, we don't know the state so we warn that it might be faulty. 13:07
sean-k-mooneyi.e. only start usign warn after we have the basic pass fail stuff workign13:08
sean-k-mooneygibi ya so the IETF draft recommend treating warn as still "passing" but that somethign might be wrong13:08
gibiI'm OK to define warn, but not use it in the first implementation13:09
gibimaybe we can gather some feedback from the field what operators want to see when the service don't know the state of it dpes13:09
gibi*deps13:09
sean-k-mooneyya that sounds resonabl that we shoudl ask there input13:10
gibiso I won't block on having warn13:10
gibiwith the same logic I would add ttl but not yet add stale_multiplier13:11
gibiwe can add that later if we want to use warn for stale 13:11
opendevreviewJonathan 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/+/82404413:14
chateaulavmorning all13:18
sean-k-mooneyo/13:19
*** dasm|off is now known as dasm14:00
chateaulavkashyap: 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-driver14:01
kashyapchateaulav: It is already linked to in the Whiteboard at the bottom, under "2022 Additions" -- are you looking for something more?14:02
chateaulavok gotcha, i know it has the "Set the URL for this specification" link above blueprint. wasnt sure if that needed to be set or not14:04
chateaulavjust want to make sure you all have everything to hopefully get the spec approved14:04
sean-k-mooneychateaulav: we generally set the url to the html version of the spec when it has merged14:06
sean-k-mooneyyou can set it to the review for now if you want14:06
sean-k-mooney well o kashyap could but ya we normally wait for it to merge before we do that14:07
sean-k-mooneychateaulav: everything looks correct to me form a lanuchpad point of view14:08
sean-k-mooneyso dont worry about that aspect14:08
chateaulavcool, my first spec so wanted to make sure.14:10
sean-k-mooneychateaulav: by the way regarding testing14:12
sean-k-mooneyi see two ways to approch this form a tempest point of view14:12
sean-k-mooneyyou could extend tempest with expicite tests for booting with emulation and add some addtional senario tests gated bhind a feature flag14:13
sean-k-mooneyor 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.sh14:13
sean-k-mooneythe latere woudl inovle adding addtional ci jobs to nova's pipepline14:14
sean-k-mooneyim not sure we really want to add + for every emulated achitrrue to run on every patch14:14
sean-k-mooneybut i would be supproted if haveing one emulated arch tested on every patch and runnign weekly jobs for the rest14:15
chateaulavI think doing the tweak and having one designated arch run would be good14:19
chateaulavas the overall goal is to ensure emulation is functional14:19
sean-k-mooneyack so i would suggest testing aarch64 by default in the check pipline14:19
sean-k-mooneyand then addign all fo them to a the periodic-weekly pipline and then experimental pipeline14:20
sean-k-mooneythat way we can review the status of the other arch jobs in our weekly meeting without much impact to the ci workload14:21
sean-k-mooneyyou could also filter the josb to only run on changes to the libvirt driver and compute module i can help with that14:22
chateaulavsounds good. and greatly appreciated14:22
*** artom__ is now known as artom14:27
opendevreviewyuval proposed openstack/nova-specs master: lightos volume driver spec  https://review.opendev.org/c/openstack/nova-specs/+/82419114:43
opendevreviewyuval proposed openstack/nova-specs master: lightos volume driver spec  https://review.opendev.org/c/openstack/nova-specs/+/82419115:47
yuval^fixed doc auto testing15:47
opendevreviewMerged openstack/nova master: Add wrapper for oslo.concurrency lockutils.ReaderWriterLock()  https://review.opendev.org/c/openstack/nova/+/82428015:56
*** xek_ is now known as xek16:02
opendevreviewJonathan 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/+/82404416:08
opendevreviewTakashi Kajinami proposed openstack/nova master: List auth plugin parameters for [keystone] section  https://review.opendev.org/c/openstack/nova/+/82445816:32
opendevreviewyuval proposed openstack/nova-specs master: lightos volume driver spec  https://review.opendev.org/c/openstack/nova-specs/+/82419117:01
opendevreviewMerged openstack/nova stable/wallaby: Reproduce bug 1953359  https://review.opendev.org/c/openstack/nova/+/82055417:46
opendevreviewMerged openstack/nova stable/wallaby: Extend the reproducer for 1953359 and 1952915  https://review.opendev.org/c/openstack/nova/+/82194317:46
opendevreviewGhanshyam proposed openstack/nova master: Move rule_if_system() method to base test class  https://review.opendev.org/c/openstack/nova/+/82447517:56
opendevreviewGhanshyam proposed openstack/nova master: Server actions APIs scoped to project scope  https://review.opendev.org/c/openstack/nova/+/82435817:57
chateaulavjust 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:23
yuvalchateaulav: I think the pep8 fails if you dont put it under approved/implemented18:56
chateaulavok, just want to make sure im not creating extra work18:58
sean-k-mooneyit shoudl be put under approved19:00
chateaulavok19:00
sean-k-mooneyand the name of the file shoudl match the blueprint name19:00
chateaulavsean-k.mooney: thanks, will fix19:00
sean-k-mooneyyou can run the check locally with "tox -e pep8,docs"19:00
sean-k-mooneywe have a script that will move it form approved to implemented if the feature is complted at the end of the cycle19:01
sean-k-mooneyi or other will run it and it uses the name of the file to check if the blueptirn is marked as complete19:01
opendevreviewJonathan 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/+/82404419:02
chateaulavsean-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:05
sean-k-mooneyit 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 tomorrow19:07
opendevreviewIago Filipe proposed openstack/nova master: Remove deprecated opts from VNC conf  https://review.opendev.org/c/openstack/nova/+/82447819:07
chateaulavgibi had already done so.19:08
chateaulavgibi: sorry for the extra work..19:08
sean-k-mooneyits fine :) dont worry19:08
sean-k-mooneyit need 2 +2 and a +w to merge19:08
chateaulavcool19:08
sean-k-mooneyso we will review it again tomorow/tonight19:09
sean-k-mooneythe cutoff is not untill close of busniess tomrorow anyway but this should be merged by then19:09
opendevreviewIago Filipe proposed openstack/nova master: Remove deprecated opts from VNC conf  https://review.opendev.org/c/openstack/nova/+/82447819:09
sean-k-mooneywith that in mind i should go back to updating my own spec19:11
opendevreviewIago Filipe proposed openstack/nova master: Remove deprecated opts from VNC conf  https://review.opendev.org/c/openstack/nova/+/82447821:19
*** dasm is now known as dasm|off22:45
opendevreviewsean mooney proposed openstack/nova-specs master: add per process healthcheck spec  https://review.opendev.org/c/openstack/nova-specs/+/82127923:38

Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!