*** ysandeep is now known as ysandeep|ruck | 05:12 | |
*** marios is now known as marios|rover | 06:01 | |
marios|rover | o/ | 06:01 |
---|---|---|
ysandeep|ruck | marios|rover, good morning o/ | 06:07 |
marios|rover | \o ysandeep|ruck | 06:08 |
jm1 | Tengu: good morning :) did you have success on the ansible galaxy issue? | 06:28 |
marios|rover | ysandeep|ruck: usual call in 25 ok foryou? | 06:34 |
ysandeep|ruck | marios|rover, yes | 06:35 |
marios|rover | ysandeep|ruck: ack sent | 06:35 |
*** bhagyashris|brb is now known as bhagyashris | 06:39 | |
*** bhagyashris is now known as bhagyashris|afk | 06:39 | |
*** ysandeep|ruck is now known as ysandeep|ruck|brb | 06:42 | |
*** ysandeep|ruck|brb is now known as ysandeep|ruck | 07:18 | |
*** ysandeep|ruck is now known as ysandeep|lunch | 07:22 | |
marios|rover | ysandeep|lunch:++ | 07:24 |
ysandeep|lunch | zed promoted, wallaby ovn-provider job hitting consistent tempest failure on a test, I will check back after lunch | 07:25 |
ysandeep|lunch | https://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.gz | 07:25 |
marios|rover | ok will check after reviews | 07:26 |
reviewbot | Do 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 |
Tengu | hello there | 07:47 |
Tengu | jm1: not yet re-worked on it. And with the coming changes, I'm not sure we'll actually need it... ? | 07:47 |
jm1 | Tengu: too bad, we could really use it in aoc :D | 07:48 |
Tengu | jm1: I'll try to get some time on that again. | 07:51 |
*** soniya29 is now known as soniya29|lunch | 08:01 | |
*** amoralej|off is now known as amoralej | 08:12 | |
*** jpena|off is now known as jpena | 08:29 | |
*** ysandeep|lunch is now known as ysandeep|ruck | 08:29 | |
ysandeep|ruck | marios|rover, ignore my last comment, passed in rerun https://review.rdoproject.org/r/c/testproject/+/39420/24#message-f82f73945ee0bd0b0f950fedd68e65599d956797 | 08:30 |
marios|rover | ysandeep|ruck: scen10? | 08:31 |
marios|rover | ysandeep|ruck: it is inconsistent but real thing | 08:31 |
ysandeep|ruck | marios|rover, yes | 08:31 |
marios|rover | ysandeep|ruck: i filed it | 08:31 |
marios|rover | ysandeep|ruck: https://bugs.launchpad.net/tripleo/+bug/2002478 | 08:31 |
marios|rover | ysandeep|ruck: earliest example from 06 Jan | 08:31 |
marios|rover | ysandeep|ruck: hits both scen10 jobs ovn one and kvm one | 08:32 |
ysandeep|ruck | thanks for the bug report | 08:32 |
marios|rover | np ysandeep|ruck | 08:34 |
*** ysandeep|ruck is now known as ysandeep|ruck|afk | 08:40 | |
akahat | reviewbot, add to review list https://review.rdoproject.org/r/c/rdo-jobs/+/45921 | 09:11 |
reviewbot | I have added your review to the Review list | 09:11 |
*** soniya29|lunch is now known as soniya29 | 09:18 | |
*** ysandeep|ruck is now known as ysandeep|ruck|afk | 09:44 | |
chandankumar | hello dpawlik | 10:40 |
chandankumar | dpawlik: please have a look at this https://review.rdoproject.org/r/c/config/+/46570 review when free, thank you :-) | 10:40 |
Tengu | marios|rover: sorry for your re-loss of vote - I made a really stupid mistake while wanting to split some more tasks... | 10:57 |
Tengu | apparently the inventory thing is a bit too subtile to run as a dedicated playbook.... bleh. | 10:57 |
*** ysandeep__ is now known as ysandeep|ruck | 11:07 | |
marios|rover | Tengu: ack will check in few | 11:07 |
marios|rover | np | 11:07 |
dpawlik | k chandankumar | 11:09 |
dpawlik | gj chandankumar | 11:09 |
*** rlandy|out is now known as rlandy | 11:15 | |
rlandy | amoralej: can you look at https://review.rdoproject.org/r/c/rdo-jobs/+/45921 - are those cbs files used? | 11:21 |
amoralej | yes, we use it, although we are building the containers on most of our jobs, i'd say, i guess it's fine | 11:25 |
marios|rover | akahat: o/ can you please check https://review.opendev.org/c/openstack/tripleo-ci/+/865240 linter fail - might need an update | 11:27 |
*** bhagyashris|afk is now known as bhagyashris | 11:32 | |
*** artom_ is now known as artom | 11:33 | |
Tengu | jm1: hmmm. I have a really, really bad feeling for the galaxy proxy.... | 11:35 |
ysandeep|ruck | marios|rover, I am reporting bug for tox-linter failure | 11:35 |
rlandy | soniya29: hello - spoke with arxcruz - we can cancel today's tempest call | 11:35 |
marios|rover | thank you ysandeep|ruck | 11:35 |
jm1 | Tengu: How that? | 11:35 |
Tengu | jm1: 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 |
soniya29 | rlandy, ack | 11:36 |
marios|rover | ysandeep|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 gman | 11:36 |
jm1 | Tengu: argh.... | 11:36 |
soniya29 | kopecmartin: ^^ | 11:36 |
rlandy | marios|rover: ysandeep|ruck: chandankumar: just fyi - I am starting a list of patches to reduce load on vexx ... | 11:37 |
ysandeep|ruck | rlandy: ack, thanks | 11:37 |
marios|rover | ah merged jan6 so not that ysandeep|ruck rlandy fyi https://review.opendev.org/c/openstack/openstack-zuul-jobs/+/867849/4#message-8f5d02b905a35626c23e959d09d3fd6602ee0c40 | 11:37 |
Tengu | jm1: 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.html | 11:37 |
rlandy | marios|rover: ysandeep|ruck: chandankumar: I am marking them w-1 now - we can review them as a block | 11:38 |
rlandy | I will talk to them at scrum on thursday | 11:38 |
rlandy | and we can decide which ones to go with | 11:38 |
jm1 | Tengu: if caching is impossible, we probably have to ping ansible folks :/ | 11:38 |
marios|rover | ack rlandy | 11:38 |
ysandeep|ruck | marios|rover, ack, thanks, I will investigate the reason of failure now | 11:39 |
Tengu | jm1: at least, I'd say we can't proxy the s3 query: https://paste.openstack.org/show/bhY99jG9xVCEeKqQNhOr/ | 11:40 |
Tengu | still need to ensure the /artifact/ is properly using the s3 backend | 11:40 |
Tengu | but it should. | 11:40 |
akahat | marios|rover, ack | 11:41 |
chandankumar | rlandy: ok | 11:43 |
Tengu | ah! interesting. | 11:43 |
Tengu | maybe it's not THAT bad. there's something wrong in the httpd config. | 11:43 |
Tengu | BAM | 11:45 |
Tengu | jm1: got it. | 11:45 |
Tengu | will push a patch shortly. | 11:45 |
Tengu | first lunch. | 11:45 |
jm1 | Tengu: :D awesome! | 11:46 |
rlandy | bhagyashris; hi - have a few minutes to chat re: 18? | 11:53 |
rlandy | ans tcib | 11:53 |
*** amoralej is now known as amoralej|lunch | 12:04 | |
Tengu | jm1: yep, seems to work as expected now. ok. pfff. | 12:33 |
Tengu | that was a good chase. | 12:33 |
jm1 | Tengu: do you have patch to vote for? :D | 12:43 |
bhagyashris | rlandy, sure | 12:43 |
Tengu | jm1: on its way | 12:43 |
Tengu | jm1: https://review.opendev.org/c/opendev/system-config/+/869819 | 12:43 |
Tengu | there's a lot of details in the commit message. | 12:43 |
Tengu | I tried to be as precise and verbose as possible so that anyone can check/test on their own. | 12:43 |
Tengu | if we have ansible-galaxy available in the check job, I'd happily update the tests to use it. | 12:44 |
jm1 | Tengu++ so good!! | 12:44 |
Tengu | the issue was: mixing Location and "the other" ProxyPass syntaxe. | 12:46 |
Tengu | if 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 |
Tengu | bah. mistakes were made. and I didn't check enough. | 12:47 |
rlandy | bhagyashris: https://meet.google.com/aux-unvt-apq?pli=1&authuser=0 | 12:47 |
*** soniya29 is now known as soniya29|afk | 13:04 | |
rlandy | marios|rover: ysandeep|ruck: going to miss CIX - need to join the planning call | 13:11 |
ysandeep|ruck | rlandy, ack | 13:12 |
marios|rover | np rlandy i will cover | 13:12 |
jm1 | Tengu: glad it works now. these ansible galaxy connectivity issues were causing random ci failures every day | 13:12 |
ysandeep|ruck | rlandy, 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|rover | thanks ysandeep|ruck will check in few | 13:13 |
ysandeep|ruck | details added here: https://bugs.launchpad.net/tripleo/+bug/2002526/comments/3 | 13:13 |
marios|rover | ysandeep|ruck++ | 13:13 |
Tengu | jm1: should be better once this is merged. | 13:13 |
rlandy | ysandeep|ruck++ thanks | 13:14 |
*** amoralej|lunch is now known as amoralej | 13:34 | |
dasm | o/ | 13:56 |
*** ysandeep|ruck is now known as ysandeep|ruck|afk | 14:01 | |
*** ysandeep|ruck|afk is now known as ysandeep|ruck | 14:30 | |
rlandy | chandankumar: ready to meet when you are | 15:03 |
chandankumar | rlandy: yup ready | 15:05 |
rlandy | chandankumar: https://meet.google.com/utc-fonv-bbf?pli=1&authuser=0 | 15:06 |
*** ysandeep|ruck is now known as ysandeep|ruck|afk | 15:08 | |
chandankumar | see ya people! | 16:08 |
rlandy | chandankumar++ | 16:08 |
rlandy | thanks - see you tomorrow | 16:09 |
dasm | chandankumar: o/ have a good night | 16:09 |
marios|rover | \o chandankumar | 16:13 |
marios|rover | rlandy: please review + workflow if no objection https://review.opendev.org/c/openstack/tripleo-quickstart/+/869820 | 16:15 |
marios|rover | when you next have time | 16:15 |
rlandy | marios|rover: ack - wil look later | 16:17 |
marios|rover | need sthing rlandy /me off in few | 16:24 |
rlandy | marios|rover: no - anything you want me to do - other than above review? | 16:24 |
marios|rover | arxcruz: o/ tomorrow hand-off call half hour before scrum OK for you? | 16:25 |
marios|rover | soniya is not here to ask i think? | 16:25 |
marios|rover | rlandy: ok foryou? ^ | 16:25 |
rlandy | marios|rover: ack - works for me | 16:25 |
marios|rover | rlandy: we should be good 18:24 < rlandy> marios|rover: no - anything you want me to do - other than above review? | 16:25 |
rlandy | marios|rover: let's go for that time, I have a meeting with soniya earlier that morning | 16:26 |
rlandy | so she should be around then | 16:26 |
marios|rover | rlandy: arxcruz: ysandeep|ruck|afk: sent invite for tomorrow half hour before scrum | 16:28 |
ysandeep|ruck|afk | marios|rover, thanks! | 16:28 |
*** ysandeep|ruck|afk is now known as ysandeep|out | 16:31 | |
* ysandeep|out out, see everyone tomorrow | 16:32 | |
marios|rover | \o | 16:33 |
marios|rover | me 2 | 16:33 |
rlandy | see you | 16:33 |
rlandy | frenzy_friday: http://dashboard-ci.tripleo.org/d/vU75JROVk/cockpit?orgId=1 - numbers here are getting better | 16:33 |
frenzy_friday | rlandy, I checked the query behind it - we are also counting the non voting jobs here | 16:34 |
frenzy_friday | And it shows the pass/fail for both check and gate jobs | 16:34 |
frenzy_friday | maybe that is why it showed 25% fail even with a clean gate | 16:35 |
rlandy | frenzy_friday: I think we should ditch non-voting in gate stats | 16:35 |
rlandy | they don;t impact merges | 16:35 |
frenzy_friday | It includes the non voting check jobs I mean | 16:35 |
rlandy | dasm: ^^ your thoughts | 16:35 |
rlandy | the pie chart is just gate stats | 16:35 |
rlandy | but either way non-voting check or gate should not be counted | 16:35 |
frenzy_friday | just gate? I thought I saw check jobs in the query, lemme check | 16:35 |
dasm | rlandy: makes sense to ditch non-voting ones. | 16:36 |
*** marios|rover is now known as marios|out | 16:36 | |
frenzy_friday | rlandy, I think it shows the check stats as well, do you have time for a quick call? Maybe I am missing something | 16:38 |
rlandy | frenzy_friday: sure | 16:38 |
frenzy_friday | rlandy, https://meet.google.com/nfz-uikw-kvv | 16:39 |
Tengu | ok Team - see you tomorrow! | 17:01 |
rlandy | have a good night | 17:02 |
*** jpena is now known as jpena|off | 17:21 | |
*** amoralej is now known as amoralej|off | 17:21 | |
*** gthiemon1e is now known as gthiemonge | 19:43 | |
*** rlandy is now known as rlandy|out | 22:58 | |
dasm | it's time for me | 23:20 |
dasm | o/ | 23:20 |
*** dasm is now known as dasm|off | 23:20 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!