opendevreview | Merged openstack/tempest master: Add support for ecdsa keys https://review.opendev.org/c/openstack/tempest/+/807465 | 03:30 |
---|---|---|
opendevreview | Ghanshyam proposed openstack/devstack stable/train: Pin Tempest to 26.1.0 tag for stable/train testing https://review.opendev.org/c/openstack/devstack/+/822380 | 03:31 |
opendevreview | Ghanshyam proposed openstack/tempest master: Fix stable/train testing with tempest 26.1.0 https://review.opendev.org/c/openstack/tempest/+/822339 | 03:40 |
opendevreview | Ghanshyam proposed openstack/tempest master: Fix stable/train testing with tempest 26.1.0 https://review.opendev.org/c/openstack/tempest/+/822339 | 03:41 |
opendevreview | Takashi Kajinami proposed openstack/tempest master: Use LOG.warning instead of deprecated LOG.warn https://review.opendev.org/c/openstack/tempest/+/819594 | 04:38 |
*** jpena|off is now known as jpena | 08:38 | |
*** lpiwowar8 is now known as lpiwowar | 08:48 | |
opendevreview | Grzegorz Grasza proposed openstack/devstack master: Global option for enforcing scope (ENFORCE_SCOPE) https://review.opendev.org/c/openstack/devstack/+/814438 | 09:04 |
gmann | vishalmanchanda: ^^ in case you did not read. about horizon moving to django 5.x ^^. chat starting from '<ade_lee> | 13:56 |
gmann | gmann, kopecmartin so - I was wondering how to handle a fips issue with django.' | 13:56 |
opendevreview | Andre Aranha proposed openstack/tempest master: WIP - Refactor ssh.Client to allow other clients https://review.opendev.org/c/openstack/tempest/+/820860 | 13:58 |
vishalmanchanda | gmann: ack. | 14:15 |
ade_lee | vishalmanchanda, hi there - would love to get your feedback on how to handle the django issue | 14:40 |
vishalmanchanda | ade_lee: Let's discuss it in #openstack-horizon channel | 14:41 |
vishalmanchanda | We have horizon weekly meeting at 15 UTC in #openstack-horizon channel | 14:41 |
*** sboyron_ is now known as sboyron | 14:46 | |
gmann | vishalmanchanda: thanks | 14:46 |
ade_lee | gmann, kopecmartin - I see the ecdsa patch merged. gentle reminder about remaining two tempest patches for fips starting with https://review.opendev.org/c/openstack/tempest/+/810808 -- once these merge, many fips gates will be unblocked. | 14:53 |
ade_lee | https://review.opendev.org/c/openstack/tempest/+/810808 and https://review.opendev.org/c/openstack/tempest/+/822560 | 14:54 |
opendevreview | Abhishek Kekane proposed openstack/tempest master: WIP image caching tests https://review.opendev.org/c/openstack/tempest/+/825112 | 14:59 |
opendevreview | Lukas Piwowarski proposed openstack/openstack-health master: WIP: Fix gulp https://review.opendev.org/c/openstack/openstack-health/+/825336 | 15:44 |
opendevreview | James Parker proposed openstack/whitebox-tempest-plugin master: [WIP] Add multiple hugepage size job https://review.opendev.org/c/openstack/whitebox-tempest-plugin/+/825011 | 16:03 |
opendevreview | Martin Kopec proposed openstack/openstack-health master: Test with nodejs16 https://review.opendev.org/c/openstack/openstack-health/+/825342 | 16:11 |
*** jpena is now known as jpena|off | 17:43 | |
jparker | artom: am I missing something with the variables here? They would pass to the role when I had the old job structure inheriting from tempest-multinode-full-py3, but inheriting form whitebox-devstack-multinode does not pass num_hugepages to the role | 19:00 |
opendevreview | Ghanshyam proposed openstack/tempest master: Fix stable/train testing with tempest 26.1.0 https://review.opendev.org/c/openstack/tempest/+/822339 | 19:22 |
artom | jparker, I know there's some weirdness that I don't quite understand with the node groups | 19:24 |
artom | So we had to put the var in both `vars` and under group_vars['compute'] | 19:24 |
jparker | artom: thats what I thought I did, unless I need to move num_hugepages out from tempest? https://review.opendev.org/c/openstack/whitebox-tempest-plugin/+/825011/10/.zuul.yaml#94 | 19:27 |
artom | jparker, ah, wait, so you know how https://review.opendev.org/c/openstack/whitebox-tempest-plugin/+/825011/10/.zuul.yaml#39 is a thing? | 19:29 |
artom | That runs the pre.yaml playbook on the base job | 19:29 |
jparker | artom: right | 19:29 |
artom | ... which a long time ago was written with "everyone gets hugepages" in mind | 19:29 |
artom | Or more precisely, assigns the whitebox-nova-compute role to all hosts | 19:30 |
artom | ... and the whitebox-nova-compute role tries to set up hugepages | 19:30 |
jparker | artom: oh | 19:30 |
jparker | artom: since I removed it form the base job | 19:31 |
jparker | artom: its still calling it | 19:31 |
artom | 'zactly | 19:31 |
jparker | ah gotcha | 19:31 |
artom | We can probably simplify our ansible stuff | 19:31 |
artom | Merge all the existing roles (apart from ensure-pip, which we inherit from higher up) into a 'whitebox-host' role or something | 19:32 |
artom | Or maybe even forego roles altogether, and just go with a bare list of tasks in the playbook | 19:32 |
artom | So one playbook for the base job that sets up the SSH key and stuff | 19:33 |
artom | And another one on top of that that adds hugepages, that only gets called from the hugepages job | 19:33 |
jparker | artom: so would pre-runs in both jobs where the hugepage job inheirts would be additive correct, i.e. the child job would still run whitebox/pre.yaml if I included a new pre-run: in the child job? | 19:35 |
artom | jparker, I believe so yeah | 19:36 |
jparker | I coudl also add a conditional around num_hugepages | 19:36 |
artom | Zuul is smart about inheritance in those cases | 19:36 |
artom | It's additive, not clobberative | 19:36 |
jparker | artom: ok let me carry over the whitebox specific roles over to the pre run playbook and append them after the ensure-pip | 19:38 |
opendevreview | James Parker proposed openstack/whitebox-tempest-plugin master: [WIP] Add multiple hugepage size job https://review.opendev.org/c/openstack/whitebox-tempest-plugin/+/825011 | 20:03 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!