Monday, 2022-12-12

*** yadnesh|away is now known as yadnesh04:34
gthiemongeI think wallaby, it affects only amphorae with haproxy >=2.406:52
arddennisHi,08:11
arddennisI've tried to rebase 'Experimental multi-az support' https://review.opendev.org/c/openstack/octavia/+/558962 on top of master and test it but faced a strange issue which I am stuck with. 08:11
arddennisComputeCreate.execute requires amphorae and TaskFlow complains that there are no amphorae providers. At the same time this function is supposed to create one. And this info is needed to understand where the other amphora instances are created. Here it is in old patch:https://review.opendev.org/c/openstack/octavia/+/558962/41/octavia/controller/worker/v2/tasks/compute_tasks.py#6108:11
arddennisThe error which I get is:(Epdb) tf.run()08:11
arddennis*** taskflow.exceptions.NotFound: Mapped argument 'amphorae' <= 'amphorae' was not produced by any accessible provider (0 possible providers were scanned)08:11
arddennis(Epdb)08:11
arddennishere it is invoked:https://opendev.org/openstack/octavia/src/branch/master/octavia/controller/worker/v2/controller_worker.py#L36208:11
arddennisI might miss something important and have very basic knowledge of Octavia and a TaskFlow, but I have a strong will to get it working. And I can extend the flavor schema to let the user store a list of desired availability zones for the lb flavor. 08:11
arddennisI tried to debug code across TaskFlow with not much results... and thought that I can ask here as well: 08:11
arddennis* Is there any other way to get a list of the other amphora instances for being created loadbalancer or* at least is there any way to get what kind of amphora type is being created (Master or Backup)08:12
gthiemongearddennis: hi08:13
gthiemongearddennis: so it means that we need an AMPHORAE ('amphorae') object in the store to run CertComputeCreate (https://review.opendev.org/c/openstack/octavia/+/558962/41/octavia/controller/worker/v2/flows/amphora_flows.py )08:15
gthiemongethis object should be provided by the caller of the flow, or by one of the previous tasks08:16
gthiemongearddennis: does it happen during the creation of a LB or during a failover?08:17
arddennisgthiemonge: at the beginning there is no such instances until the first one is created. Then it could be provided. It seems that some check were added to verify providers list instead of just supplying None. Which is expected08:18
arddennisgthiemonge: This is for creation of amphora instances08:18
gthiemongethe amphora object exists in the DB, it is created at https://review.opendev.org/c/openstack/octavia/+/558962/41/octavia/controller/worker/v2/flows/amphora_flows.py#8908:23
gthiemongearddennis: I think I would change this constants.AMPHORAE req to constants.LOADBALANCER_ID, and in the CmputeCreate.execute you should be able to fetch the amphorae from the load balancer object.08:24
arddennisgthiemonge: I will check this now08:24
oschwartHey folks, can anybody review this one https://review.opendev.org/c/openstack/python-octaviaclient/+/857193 ? It is really small and has one +2 already09:37
opendevreviewTom Weininger proposed openstack/octavia master: WIP remove python-neutronclient  https://review.opendev.org/c/openstack/octavia/+/86632710:07
noonedeadpunkHey folks! Any chance to get reviews on https://review.opendev.org/q/topic:octavia%252Fcastellan ?:)13:24
*** yadnesh is now known as yadnesh|away14:52
noonedeadpunkI need a hint on https://review.opendev.org/c/openstack/octavia/+/827456 - when I use oslo-config-generator sections of config are generated, but they're not in docs sections...15:37
noonedeadpunkah, found that15:39
opendevreviewMerged openstack/octavia master: Pass config to castellan  https://review.opendev.org/c/openstack/octavia/+/82745617:58
opendevreviewTom Weininger proposed openstack/octavia master: WIP remove python-neutronclient  https://review.opendev.org/c/openstack/octavia/+/86632718:04
opendevreviewMerged openstack/octavia master: Filter ComputeWaitTimeoutException when jobboard is disabled  https://review.opendev.org/c/openstack/octavia/+/86170418:21
-opendevstatus- NOTICE: Gerrit will be unavailable for a short time as it is upgraded to the 3.6 release19:59

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