Wednesday, 2023-01-11

*** ysandeep is now known as ysandeep|ruck05:12
*** marios is now known as marios|rover06:01
marios|rovero/06:01
ysandeep|ruckmarios|rover, good morning o/06:07
marios|rover\o ysandeep|ruck 06:08
jm1Tengu: good morning :) did you have success on the ansible galaxy issue?06:28
marios|roverysandeep|ruck: usual call in 25 ok foryou?06:34
ysandeep|ruckmarios|rover, yes06:35
marios|roverysandeep|ruck: ack sent06:35
*** bhagyashris|brb is now known as bhagyashris06:39
*** bhagyashris is now known as bhagyashris|afk06:39
*** ysandeep|ruck is now known as ysandeep|ruck|brb06:42
*** ysandeep|ruck|brb is now known as ysandeep|ruck07:18
*** ysandeep|ruck is now known as ysandeep|lunch07:22
marios|roverysandeep|lunch:++07:24
ysandeep|lunchzed promoted, wallaby ovn-provider job hitting consistent tempest failure on a test, I will check back after lunch07:25
ysandeep|lunchhttps://logserver.rdoproject.org/openstack-periodic-integration-stable1/opendev.org/openstack/tripleo-ci/master/periodic-tripleo-ci-centos-9-scenario010-ovn-provider-standalone-wallaby/5ca6c98/logs/undercloud/var/log/tempest/failing_tests.log.txt.gz07:25
marios|roverok will check after reviews 07:26
reviewbotDo you want me to add your patch to the Review list? Please type something like add to review list <your_patch> so that I can understand. Thanks.07:26
Tenguhello there07:47
Tengujm1: not yet re-worked on it. And with the coming changes, I'm not sure we'll actually need it... ?07:47
jm1Tengu: too bad, we could really use it in aoc :D07:48
Tengujm1: I'll try to get some time on that again.07:51
*** soniya29 is now known as soniya29|lunch08:01
*** amoralej|off is now known as amoralej08:12
*** jpena|off is now known as jpena08:29
*** ysandeep|lunch is now known as ysandeep|ruck08:29
ysandeep|ruckmarios|rover, ignore my last comment, passed in rerun https://review.rdoproject.org/r/c/testproject/+/39420/24#message-f82f73945ee0bd0b0f950fedd68e65599d95679708:30
marios|roverysandeep|ruck: scen10?08:31
marios|roverysandeep|ruck: it is inconsistent but real thing 08:31
ysandeep|ruckmarios|rover, yes08:31
marios|roverysandeep|ruck: i filed it 08:31
marios|roverysandeep|ruck: https://bugs.launchpad.net/tripleo/+bug/2002478 08:31
marios|roverysandeep|ruck: earliest example from 06 Jan 08:31
marios|roverysandeep|ruck: hits both scen10 jobs ovn one and kvm one 08:32
ysandeep|ruckthanks for the bug report08:32
marios|rovernp ysandeep|ruck 08:34
*** ysandeep|ruck is now known as ysandeep|ruck|afk08:40
akahatreviewbot, add to review list https://review.rdoproject.org/r/c/rdo-jobs/+/4592109:11
reviewbotI have added your review to the Review list09:11
*** soniya29|lunch is now known as soniya2909:18
*** ysandeep|ruck is now known as ysandeep|ruck|afk09:44
chandankumarhello dpawlik 10:40
chandankumardpawlik: please have a look at this https://review.rdoproject.org/r/c/config/+/46570 review when free, thank you :-)10:40
Tengumarios|rover: sorry for your re-loss of vote - I made a really stupid mistake while wanting to split some more tasks...10:57
Tenguapparently the inventory thing is a bit too subtile to run as a dedicated playbook.... bleh.10:57
*** ysandeep__ is now known as ysandeep|ruck11:07
marios|roverTengu: ack will check in few 11:07
marios|rovernp11:07
dpawlikk chandankumar11:09
dpawlikgj chandankumar11:09
*** rlandy|out is now known as rlandy11:15
rlandyamoralej: can you look at https://review.rdoproject.org/r/c/rdo-jobs/+/45921 - are those cbs files used?11:21
amoralejyes, we use it, although we are building the containers on most of our jobs, i'd say, i guess it's fine11:25
marios|roverakahat: o/ can you please check https://review.opendev.org/c/openstack/tripleo-ci/+/865240 linter fail - might need an update11:27
*** bhagyashris|afk is now known as bhagyashris11:32
*** artom_ is now known as artom11:33
Tengujm1: hmmm. I have a really, really bad feeling for the galaxy proxy....11:35
ysandeep|ruckmarios|rover, I am reporting bug for tox-linter failure11:35
rlandysoniya29: hello - spoke with arxcruz - we can cancel today's tempest call11:35
marios|roverthank you ysandeep|ruck 11:35
jm1Tengu: How that?11:35
Tengujm1: the final URI has an access key and a signature..... I have to dig a bit, but iirc, that signature matches the whole URI (minus params).... meaning we won't be able to re-sign the query.11:35
soniya29rlandy, ack11:36
marios|roverysandeep|ruck: (not sure if you were on the call when i said before - might be cos of tox pin to <4 now? let me find the patch from gman11:36
jm1Tengu: argh.... 11:36
soniya29kopecmartin: ^^11:36
rlandymarios|rover: ysandeep|ruck: chandankumar: just fyi - I am starting a list of patches to reduce load on vexx ... 11:37
ysandeep|ruckrlandy: ack, thanks11:37
marios|roverah merged jan6 so not that ysandeep|ruck rlandy fyi https://review.opendev.org/c/openstack/openstack-zuul-jobs/+/867849/4#message-8f5d02b905a35626c23e959d09d3fd6602ee0c4011:37
Tengujm1: hmmm. seems the parameters are different, but it seems to match part of my fear: https://docs.aws.amazon.com/AmazonS3/latest/API/sigv4-query-string-auth.html11:37
rlandymarios|rover: ysandeep|ruck: chandankumar: I am marking them w-1 now - we can review them as a block11:38
rlandyI will talk to them at scrum on thursday11:38
rlandyand we can decide which ones to go with11:38
jm1Tengu: if caching is impossible, we probably have to ping ansible folks :/11:38
marios|roverack rlandy 11:38
ysandeep|ruckmarios|rover, ack, thanks, I will investigate the reason of failure now11:39
Tengujm1: at least, I'd say we can't proxy the s3 query: https://paste.openstack.org/show/bhY99jG9xVCEeKqQNhOr/11:40
Tengustill need to ensure the /artifact/ is properly using the s3 backend11:40
Tengubut it should.11:40
akahatmarios|rover, ack11:41
chandankumarrlandy: ok11:43
Tenguah! interesting.11:43
Tengumaybe it's not THAT bad. there's something wrong in the httpd config.11:43
TenguBAM11:45
Tengujm1: got it.11:45
Tenguwill push a patch shortly.11:45
Tengufirst lunch.11:45
jm1Tengu: :D awesome!11:46
rlandybhagyashris; hi - have a few minutes to chat re: 18?11:53
rlandyans tcib11:53
*** amoralej is now known as amoralej|lunch12:04
Tengujm1: yep, seems to work as expected now. ok. pfff.12:33
Tenguthat was a good chase.12:33
jm1Tengu: do you have patch to vote for? :D12:43
bhagyashrisrlandy, sure12:43
Tengujm1: on its way12:43
Tengujm1: https://review.opendev.org/c/opendev/system-config/+/86981912:43
Tenguthere's a lot of details in the commit message.12:43
TenguI tried to be as precise and verbose as possible so that anyone can check/test on their own.12:43
Tenguif we have ansible-galaxy available in the check job, I'd happily update the tests to use it.12:44
jm1Tengu++ so good!! 12:44
Tenguthe issue was: mixing Location and "the other" ProxyPass syntaxe.12:46
Tenguif I had use Location for both endpoint (as proposed now) or, maybe, moved the ProxyPass /galaxy-s3/ <remote> to the <Location "/">, we'd be good since the merge.12:46
Tengubah. mistakes were made. and I didn't check enough.12:47
rlandybhagyashris: https://meet.google.com/aux-unvt-apq?pli=1&authuser=012:47
*** soniya29 is now known as soniya29|afk13:04
rlandymarios|rover: ysandeep|ruck: going to  miss CIX - need to join the planning call13:11
ysandeep|ruckrlandy, ack13:12
marios|rovernp rlandy i will cover13:12
jm1Tengu: glad it works now. these ansible galaxy connectivity issues were causing random ci failures every day13:12
ysandeep|ruckrlandy, marios|rover fyi.. tox-linters issue was due to latest yamllint release, fixes are up: https://review.opendev.org/q/topic:bug%252F2002526 13:12
marios|roverthanks ysandeep|ruck will check in few 13:13
ysandeep|ruckdetails added here: https://bugs.launchpad.net/tripleo/+bug/2002526/comments/313:13
marios|roverysandeep|ruck++13:13
Tengujm1: should be better once this is merged. 13:13
rlandyysandeep|ruck++ thanks13:14
*** amoralej|lunch is now known as amoralej13:34
dasmo/13:56
*** ysandeep|ruck is now known as ysandeep|ruck|afk14:01
*** ysandeep|ruck|afk is now known as ysandeep|ruck14:30
rlandychandankumar: ready to meet when you are15:03
chandankumarrlandy: yup ready15:05
rlandychandankumar: https://meet.google.com/utc-fonv-bbf?pli=1&authuser=015:06
*** ysandeep|ruck is now known as ysandeep|ruck|afk15:08
chandankumarsee ya people!16:08
rlandychandankumar++16:08
rlandythanks - see you tomorrow16:09
dasmchandankumar: o/ have a good night16:09
marios|rover\o chandankumar 16:13
marios|roverrlandy: please review + workflow if no objection https://review.opendev.org/c/openstack/tripleo-quickstart/+/869820 16:15
marios|roverwhen you next have time 16:15
rlandymarios|rover: ack - wil look later16:17
marios|roverneed sthing rlandy /me off in few 16:24
rlandymarios|rover: no - anything you want me to do - other than above review?16:24
marios|roverarxcruz: o/ tomorrow hand-off call half hour before scrum OK for you?16:25
marios|roversoniya is not here to ask i think?16:25
marios|roverrlandy: ok foryou? ^ 16:25
rlandymarios|rover: ack - works for me16:25
marios|roverrlandy: we should be good 18:24 < rlandy> marios|rover: no - anything you want me to do - other than above review?16:25
rlandymarios|rover: let's go for that time, I have a meeting with soniya earlier that morning16:26
rlandyso she should be around then16:26
marios|roverrlandy: arxcruz: ysandeep|ruck|afk: sent invite for tomorrow half hour before scrum 16:28
ysandeep|ruck|afkmarios|rover, thanks! 16:28
*** ysandeep|ruck|afk is now known as ysandeep|out16:31
* ysandeep|out out, see everyone tomorrow16:32
marios|rover\o 16:33
marios|roverme 216:33
rlandysee you16:33
rlandyfrenzy_friday: http://dashboard-ci.tripleo.org/d/vU75JROVk/cockpit?orgId=1 - numbers here are getting better16:33
frenzy_fridayrlandy, I checked the query behind it - we are also counting the non voting jobs here16:34
frenzy_fridayAnd it shows the pass/fail for both check and gate jobs16:34
frenzy_fridaymaybe that is why it showed 25% fail even with a clean gate16:35
rlandyfrenzy_friday: I think we should ditch non-voting in gate stats16:35
rlandythey don;t impact merges16:35
frenzy_fridayIt includes the non voting check jobs I mean16:35
rlandydasm: ^^ your thoughts16:35
rlandythe pie chart is just gate stats16:35
rlandybut either way non-voting check or gate should not be counted16:35
frenzy_fridayjust gate? I thought I saw check jobs in the query, lemme check16:35
dasmrlandy: makes sense to ditch non-voting ones.16:36
*** marios|rover is now known as marios|out16:36
frenzy_fridayrlandy, I think it shows the check stats as well, do you have time for a quick call? Maybe I am missing something16:38
rlandyfrenzy_friday: sure16:38
frenzy_fridayrlandy, https://meet.google.com/nfz-uikw-kvv16:39
Tenguok Team - see you tomorrow!17:01
rlandyhave a good night17:02
*** jpena is now known as jpena|off17:21
*** amoralej is now known as amoralej|off17:21
*** gthiemon1e is now known as gthiemonge19:43
*** rlandy is now known as rlandy|out22:58
dasmit's time for me23:20
dasmo/23:20
*** dasm is now known as dasm|off23:20

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