Wednesday, 2022-04-20

*** pojadhav|out is now known as pojadhav|ruck02:08
*** pojadhav is now known as pojadhav|ruck04:36
opendevreviewMerged openstack/devstack stable/ussuri: Mark our source trees as safe for git to use as other users  https://review.opendev.org/c/openstack/devstack/+/83774905:42
*** pojadhav|ruck is now known as pojadhav|brb06:07
*** pojadhav|brb is now known as pojadhav|ruck06:25
*** pojadhav|ruck is now known as pojadhav|dr_appt06:30
*** jpena|off is now known as jpena07:01
*** pojadhav|dr_appt is now known as pojadhav|ruck07:02
opendevreviewyatin proposed openstack/devstack master: Fix doc and user create script to set homedir permissions  https://review.opendev.org/c/openstack/devstack/+/83864507:10
*** pojadhav|ruck is now known as pojadhav|lunch08:37
*** pojadhav|lunch is now known as pojadhav|ruck09:49
*** ralonsoh_ is now known as ralonsoh10:15
opendevreviewElod Illes proposed openstack/devstack stable/stein: Mark our source trees as safe for git to use as other users  https://review.opendev.org/c/openstack/devstack/+/83867910:16
opendevreviewElod Illes proposed openstack/devstack stable/rocky: Mark our source trees as safe for git to use as other users  https://review.opendev.org/c/openstack/devstack/+/83868110:18
opendevreviewElod Illes proposed openstack/devstack stable/queens: Mark our source trees as safe for git to use as other users  https://review.opendev.org/c/openstack/devstack/+/83868210:18
opendevreviewElod Illes proposed openstack/devstack stable/pike: Mark our source trees as safe for git to use as other users  https://review.opendev.org/c/openstack/devstack/+/83868310:18
sean-k-mooneyclarkb: i think i found why the job failed if you move the wsgi python home directive i think it will fix at least some of the job failures10:20
sean-k-mooneyhttps://review.opendev.org/c/openstack/devstack/+/558930/26#message-64dc9429605bbdd1dc0b8e974655c0576d01c02c=10:20
sean-k-mooneythe local.confs for the jobs that passed dont have horizon enabled https://zuul.opendev.org/t/openstack/build/577af265295441d4adac64fb290ddbf7/log/controller/logs/local_conf.txt10:21
opendevreviewAndre Aranha proposed openstack/tempest master: Test fips job on centos-9  https://review.opendev.org/c/openstack/tempest/+/83160712:09
opendevreviewAndre Aranha proposed openstack/tempest master: Test fips job on centos-9  https://review.opendev.org/c/openstack/tempest/+/83160712:19
opendevreviewAndre Aranha proposed openstack/tempest master: Add multinode FIPS job to the experimental queue  https://review.opendev.org/c/openstack/tempest/+/82658012:25
opendevreviewBrian Rosmaita proposed openstack/devstack stable/train: Make neutron-grenade non-voting on stable/train  https://review.opendev.org/c/openstack/devstack/+/83872213:44
opendevreviewsean mooney proposed openstack/devstack master: Add option to install everything in global venvs  https://review.opendev.org/c/openstack/devstack/+/55893014:05
sean-k-mooneyclarkb: tested that locally and it seams to fix horizon14:06
opendevreviewAlex Yefimov proposed openstack/tempest master: Fixes Unittest failure in test_users  https://review.opendev.org/c/openstack/tempest/+/83872614:17
opendevreviewAlex Yefimov proposed openstack/tempest master: fixes:   skips test_user::test_user_update_own_password when keystone minimum_password_age > 0  https://review.opendev.org/c/openstack/tempest/+/83872614:26
opendevreviewAlex Yefimov proposed openstack/tempest master: fixes:   skips test_user::test_user_update_own_password when keystone minimum_password_age > 0  https://review.opendev.org/c/openstack/tempest/+/83872614:31
opendevreviewuggla proposed openstack/tempest master: [WIP] Test tempest for Nova and Manila  https://review.opendev.org/c/openstack/tempest/+/83874314:56
dansmithgmann: sorry to bother again, but the logsearch patch to consume this is already +2, so would be good to start generating this data if we can: https://review.opendev.org/c/openstack/devstack/+/83713915:08
gmanndansmith: sure, I will check today for sure, need to go out for my son appointment. I will review after coming back.15:11
dansmithgmann: I think this is more important than your son's appointment, but.. okay. (extreme sarcasm)15:13
dansmithgmann: thanks :)15:13
gmann:)15:13
opendevreviewuggla proposed openstack/tempest master: [WIP] Test tempest for Nova and Manila  https://review.opendev.org/c/openstack/tempest/+/83874315:18
clarkbsean-k-mooney: thanks I think some of the other failures are due to similar reasons to horizon. There are a bunch of apache templates that all need updating potentially15:19
*** artom__ is now known as artom15:19
opendevreviewuggla proposed openstack/tempest master: [WIP] Test tempest for Nova and Manila  https://review.opendev.org/c/openstack/tempest/+/83874315:25
sean-k-mooneyack ya i saw some tls related issues which i have not seen locally since im disabling the tls proxy currently i need to test https://review.opendev.org/c/openstack/devstack/+/838043 in a multi node case to confirm it15:25
sean-k-mooneybut that shoudl fix tls for centos 9 stream in multi node15:25
sean-k-mooneywe use a sideefct fo enableing tls-proxy on the compute node to update the ca on the compute15:26
opendevreviewuggla proposed openstack/tempest master: [WIP] Test tempest for Nova and Manila  https://review.opendev.org/c/openstack/tempest/+/83874315:34
opendevreviewuggla proposed openstack/tempest master: [WIP] Test tempest for Nova and Manila  https://review.opendev.org/c/openstack/tempest/+/83874315:43
opendevreviewJames Parker proposed openstack/whitebox-tempest-plugin master: Test resize with mem_page_size in flavor  https://review.opendev.org/c/openstack/whitebox-tempest-plugin/+/82477215:46
opendevreviewMartin Kopec proposed openstack/tempest master: Switch to ecdsa ssh key type by default  https://review.opendev.org/c/openstack/tempest/+/83875315:59
opendevreviewribaudr proposed openstack/tempest master: [WIP] Test tempest for Nova and Manila  https://review.opendev.org/c/openstack/tempest/+/83874316:07
*** jpena is now known as jpena|off17:13
opendevreviewJames proposed openstack/tempest master: Test fails due to wrong assertRaises catch label  https://review.opendev.org/c/openstack/tempest/+/83876717:34
opendevreviewJames Parker proposed openstack/whitebox-tempest-plugin master: Test resize with mem_page_size in flavor  https://review.opendev.org/c/openstack/whitebox-tempest-plugin/+/82477218:13
jparkerartom: I'm in no rush to get https://review.opendev.org/c/openstack/whitebox-tempest-plugin/+/824772 merged, but I wonder since there is a solution to test this downstream if I should remove the depends upon and disable it running upstream until we can get a solution for getting multi hugepages configured upstream for the computes?18:14
artomjparker, oh, I left a comment on https://review.opendev.org/c/openstack/whitebox-tempest-plugin/+/82501118:48
jparkerartom: yea I saw, I'm just not sure when I'll have the bandwidth to get the hugepage configuration working upstream. But since the deployment is working downstream it might make sense to commit just the testcases by themselves and have them run downstream18:50
jparkerartom: I'm also ok with taking your approach with the comment, just not sure when I'll be able to take a look at it again18:51
artomjparker, I can help push it over the line, if you want18:53
gmanndansmith: +1 on perf data one, 2 comments but nothing blocking https://review.opendev.org/c/openstack/devstack/+/83713918:53
artomI'd rather to do it "properly" upstream if at all possible18:53
jparkerartom: I don't mean to say it wont be added upstream for coverage, just that that tests can be added upstream and will be running downstream now. And when there is time a follow up commit can be done to get the computes upstream configured properly with multiple hugepages18:56
dansmithgmann: thanks, will flip that default and push18:57
jparkerartom: there isn't a rush so I dont mind getting everyone upstream done first, but until thats done there won't be any coverage upstream or down18:58
gmanndansmith: +1, thanks 18:58
opendevreviewDan Smith proposed openstack/devstack master: Gather performance data after tempest  https://review.opendev.org/c/openstack/devstack/+/83713918:59
mnaseris there a way using devstack to unset an ini value (opposite of iniset?)19:05
mnaserok, found it `inidelete [-sudo] config-file section option`19:06
dansmithdang beat me19:07
*** artom__ is now known as artom19:07
jm1dansmith, gmann: May I ask you to merge this patch series into devstack to get its train/stein/rocky/queens/pike branches working again? :) This currently breaks our train job in the ansible openstack collection. https://review.opendev.org/q/topic:bug%252F196879819:11
dansmithjm1: I do whatever gmann tells me to do19:12
opendevreviewMohammed Naser proposed openstack/devstack master: nova: unset cpu_model on aarch64  https://review.opendev.org/c/openstack/devstack/+/83877519:13
jm1dansmith: hehe ^^ did not know whom to ask and so i asked the original patch authors with +w permissions ;)19:15
gmannjm1: dansmith as fix are there I am +2, we can merge them 19:15
dansmithjm1: I'm not sure why t is dependent on s, according to rosmaita's comments on that one19:15
dansmithgmann: I followed with +Ws19:15
dansmithare we still running grenade on t?19:16
gmanndansmith: we have grenade job voting in stable/train which we do not need to19:16
dansmithah19:16
gmannyeah19:16
dansmithgmann: so I should avoid +W on that until s merges, which is just now heading to gate right?19:16
gmanndansmith: we can wait in case they are not enqueue correctly. 19:16
dansmithack, threw my +2 on there so either can +W when the time comes19:17
dansmithsounds like jm1 wants pike (egads!) anyway19:17
jm1dansmith: as long as train is fixed i am totally happy ^^19:17
jm1dansmith: just asked for all because someone backported your patch to all those branches19:18
dansmithjm1: okay, train is the odd one out19:19
dansmithbut I guess I can +W and then anyone can recheck it when the stein patch merges19:19
jm1dansmith: awesome, will (re)check in my morning :)19:20
jm1dansmith++ gmann++ thank you very much for helping out ☺️👍19:21
dansmithnp19:21
gmannyw!19:32
opendevreviewMohammed Naser proposed openstack/devstack master: ovn: use bundled ovs  https://review.opendev.org/c/openstack/devstack/+/83878219:43
opendevreviewMohammed Naser proposed openstack/devstack master: ovn: use bundled ovs  https://review.opendev.org/c/openstack/devstack/+/83878219:50
opendevreviewGhanshyam proposed openstack/devstack stable/ussuri: Test Tempest 30.0.0 on ussuri  https://review.opendev.org/c/openstack/devstack/+/83805120:26
opendevreviewMerged openstack/devstack stable/pike: Mark our source trees as safe for git to use as other users  https://review.opendev.org/c/openstack/devstack/+/83868320:59
opendevreviewMerged openstack/devstack stable/queens: Mark our source trees as safe for git to use as other users  https://review.opendev.org/c/openstack/devstack/+/83868220:59
opendevreviewMerged openstack/devstack stable/stein: Mark our source trees as safe for git to use as other users  https://review.opendev.org/c/openstack/devstack/+/83867922:26

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