Friday, 2022-12-02

*** rlandy is now known as rlandy|out00:18
*** yadnesh|away is now known as yadnesh05:01
ysandeepgood morning folks o/05:34
*** marios is now known as marios|ruck06:14
marios|ruck\o/06:14
ysandeepmarios|ruck, happy friday and a calm week ahead in ruck/rover land o/06:21
marios|rucko/06:23
*** ysandeep is now known as ysandeep|brb07:12
*** pojadhav is now known as pojadhav|lunch08:21
*** frenzy_friday|rover is now known as frenzy_friday08:33
*** jpena|off is now known as jpena08:38
*** pojadhav|lunch is now known as pojadhav|08:43
*** pojadhav| is now known as pojadhav08:43
*** yadnesh is now known as yadnesh|afk08:49
*** ysandeep|brb is now known as ysandeep08:51
*** amoralej|off is now known as amoralej08:54
frenzy_fridayrlandy|out, thanks. I'll put patches to add fs64 internal to pipeline and criteria08:56
frenzy_fridayfolks, pls add to your review lists: https://review.rdoproject.org/r/c/rdo-jobs/+/46330 (fs64 trigger), https://review.rdoproject.org/r/c/rdo-infra/ci-config/+/46329 (fs64 criteria) 08:57
reviewbotI have added your review to the Review list08:57
reviewbotI have added your review to the Review list08:57
*** yadnesh|afk is now known as yadnesh10:00
*** ysandeep is now known as ysandeep|afk10:01
marios|ruckchandankumar: ysandeep|afk: dviroel_: bhagyashris: please vote when you get a minute https://review.opendev.org/c/openstack/tripleo-ci/+/866441 10:54
chandankumarfrenzy_friday: commented10:55
chandankumarhttps://review.rdoproject.org/r/c/rdo-infra/ci-config/+/46329/2#message-1dd13cd33ea56212e830ffdc935a6b913bf5094d - misisng10:56
chandankumar*missing10:56
frenzy_fridaypojadhav, bhagyashris hey, I tried to address some of the comments in the 9-1 nodeset patch (https://code.engineering.redhat.com/gerrit/c/openstack/sf-config/+/435898) It is still in progress. Pls add to your review lists when you get some time10:56
reviewbotI have added your review to the Review list10:56
frenzy_fridaychandankumar, thanks, updated10:58
marios|ruckfrenzy_friday: ok? https://bugs.launchpad.net/tripleo/+bug/1998348/comments/4 11:01
frenzy_fridaymarios|ruck, thanks. Forgot to close it yesterday *facepalm*11:02
marios|ruckfrenzy_friday: np just sanity checking in case i missed something else 11:02
*** dviroel_ is now known as dviroel11:05
*** rlandy|out is now known as rlandy|rover11:10
rlandy|rovermarios|ruck: hi there - want to touch base after review time?11:11
marios|ruckrlandy|rover: sure will be a bit late to reviews getting some food 11:13
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.11:13
marios|ruckbrb11:13
rlandy|roverfrenzy_friday: want to join review time to talk about patches?11:17
*** ysandeep|afk is now known as ysandeep11:17
rlandy|roverfrenzy_friday: https://code.engineering.redhat.com/gerrit/c/openstack/sf-config/+/435898 - looking at that11:18
pojadhavfrenzy_friday, ack11:49
ysandeepdviroel: hey o/ you were investigating running ovb on same compute, did it work?11:49
ysandeepdviroel, before you joined review call we were discussing ovb random failure, i think the random issues we have is i.e because of network/resource constraints - May be we have better results with running on same compute/with bigger flavors - maybe running overcloud on same compute will help us.11:52
dviroelysandeep: hey o/, yes, the affinitty was working11:54
ysandeepdviroel: nice, good to know that, thanks!11:54
dviroelysandeep: but my initial tests were still failing with related network issues11:54
dviroelysandeep: maybe we should give a try again, and do more testing11:55
ysandeepdviroel, did you already tried with bigger flavors for overcloud nodes?11:55
dviroelysandeep: yes, this one definitely didn't change anything in my tests. But i didn't tried affinity + big flavors.11:56
dviroelysandeep: i have more fait on affinity approach11:57
dviroelbut needs more investigation, I think11:57
ysandeepdviroel, okay so big flavor alone didn't help and affinity alone also didn't help(but we did limited tests)11:57
dviroelcorrect11:57
dviroelthats why I didn't close affinity card11:57
dviroelysandeep: https://issues.redhat.com/browse/TRIPLEOCI-116711:58
ysandeepdviroel, and what does it take to enable affinity, do we need patches in open-virtual-baremetal repo?11:58
ysandeepdviroel, thanks reading jira11:58
dviroelysandeep: yes, i abandoned, but we can restore. OVB partches + testproject is enough11:59
dviroelysandeep: yeah, there is some comments showing that affinity was on, but still, need more testing to see if is effective.12:01
dviroelysandeep: FYI, ovb patch is a starting patch, extra-node is not included12:01
ysandeepdviroel, ack o/ thanks for info12:02
dviroelrlandy|rover: sorry, I didn't find a time to help you with https://code.engineering.redhat.com/gerrit/c/tripleo-environments/+/435580 - but testing in my machine, looks like it works. Added my vote+comments there.12:04
chandankumarysandeep: dviroel https://github.com/tripleo-ci/ansibleee-operator/pull/2 github depends with github pr works fine12:23
ysandeepchandankumar, wohooo great! 12:26
dviroelchandankumar: this is great :)12:29
ysandeepchandankumar, where is pcp_edpm_ansiblee_operator_github_check definition?12:29
chandankumarso it is not deploying any crc12:44
chandankumardviroel: ysandeep https://github.com/rdo-infra/review.rdoproject.org-config/commit/00dfcb59654e484b2eee7703404f94531f2bbb7c12:45
chandankumarI first wanted to check whether we can clone depends on or not12:45
chandankumarit works fine12:45
chandankumarNow I can focus on fixing crc job12:45
*** frenzy_friday is now known as frenzy_friday|food12:54
ysandeepchandankumar, ack o/ zuul++12:55
rlandy|roverdviroel: it's ok -this seems to work13:01
*** amoralej is now known as amoralej|off13:06
rlandy|roverfrenzy_friday|food: chatting with pojadhav 13:06
rlandy|roverwe're talking about the 9.1 work - where hers is blocked by the nodeset13:07
rlandy|roverfrenzy_friday|food: pls ping when you are back13:07
*** frenzy_friday|food is now known as frenzy_friday13:13
frenzy_fridayrlandy|rover, I am back13:13
pojadhavrlandy|rover, https://hackmd.io/HhVO_RuySBGpbhYG2Ll0hg13:13
pojadhavfrenzy_friday, please join us https://meet.google.com/yoq-muit-gvw?authuser=013:14
rlandy|rovermarios|ruck: rekicking ipa jobs for 17 promo13:28
marios|ruckrlandy|rover: thanks13:30
rlandy|roverchandankumar: next gen meeting13:31
marios|ruck rlandy|rover: chandankumar: ysandeep: please vote when you can https://review.rdoproject.org/r/c/rdo-infra/ci-config/+/46217 13:32
marios|ruckrlandy|rover: (revert zed criteria^^)13:32
jbadiapamarios|ruck, I've just seen your last comment about standalone_ffu job and periodic jobs13:50
jbadiapaThe idea is to make it voiting at the same time that we add it to the periodic pipeline? and remove it as non-voting ? but this only will happen when the job is stable13:52
marios|ruckjbadiapa: o/ we need it gating but in order to make it gating it also needs to run in the integration promotion line (since the result of that is what we run in the gate)13:56
marios|ruckjbadiapa: so ideally that work should start asap once the check/gate issue is resolved13:56
jbadiapaI think the issue is solved... it was all the time there but I havent checked all the required patches to get it mork13:57
jbadiapamarios|ruck: my question was on how to do it, becasue I'm not sure how to get a stable job if we dont have it as non-voting for a while13:58
marios|ruckjbadiapa: not sure what you mean but we have to try fix the current bug, then see what else is broken, keep fixing till it is green 13:59
marios|ruckwas it ever green jbadiapa ?13:59
marios|ruckjbadiapa: but anyway once we get it green we need to make it vote asap to prevent this happening again13:59
jbadiapamarios|ruck, yes , it was green on the tht test patch, because the missing patch was as depends-on14:00
jbadiapamarios|ruck: so I can say as non-voting job, it has never been green. I expect to be green on the following executions14:01
jbadiapamarios|ruck: https://review.opendev.org/c/openstack/tripleo-heat-templates/+/861154 << this is the missing patch I was refering before14:05
marios|ruckjbadiapa: ok then now we should sort out the integration line ... here is a reference to help you start (undercloud-ffu https://github.com/rdo-infra/rdo-jobs/blob/e29b6b9bfc23934f1bf3ecabf3f88a0215bae282/zuul.d/integration-jobs-wallaby.yaml#L63 )14:06
marios|ruckrlandy|rover: ysandeep: per envC discussion we need to remove that https://code.engineering.redhat.com/gerrit/c/openstack/tripleo-ci-internal-config/+/436919 14:07
jbadiapamarios|ruck: thx14:09
*** dasm|off is now known as dasm14:14
dasmo/14:14
jm1dasm: o/14:48
ysandeepdviroel++ chandankumar++ rlandy++14:48
ysandeepmarios|ruck, looking14:49
rlandy|roverysandeep++ also14:50
rlandy|rovermaking significant progress 14:50
dasmjm1: 'o14:51
rlandy|rovermarios|ruck: wrt envC ... 14:51
rlandy|rovercreate-baremetal-trigger-for-downstreamopenstack/tripleo-cimasteropenstack-periodic-integration-main2 mins 0 secs2022-12-02 01:07:49SUCCESS14:51
rlandy|rover^^ that's the job that needs to be removed14:52
rlandy|roverhttps://code.engineering.redhat.com/gerrit/c/openstack/tripleo-ci-internal-config/+/436919  removes starting the job is a trigger happens14:52
rlandy|roveriiuc, we should stop the trigger14:52
ysandeepboth will work, but better to kill the trigger: https://github.com/rdo-infra/rdo-jobs/blob/master/zuul.d/integration-pipeline-main.yaml#L36-L3714:53
ysandeepbecause ^^ will stop posting gerrit patches14:53
marios|ruckysandeep: rlandy|rover: ack https://review.rdoproject.org/r/c/rdo-jobs/+/4633414:56
ysandeepmarios|ruck, thanks voted14:57
dviroelysandeep: fyi, the automantion that updates the imageset on hive: https://github.com/openshift/release/commit/78457793fdaa6c5d953ba8bd64c884c87bdf71ba14:58
rlandy|rovermarios|ruck: thanks +2'ed - as did ysandeep so you can merge when ready14:58
ysandeepdviroel: thanks for info, good to know that we test latest minor release of openshift 15:00
dviroelyep15:00
rlandy|rovermarios|ruck: looks like master is also falling over in deplot for fs03915:01
rlandy|roverdeploy15:02
marios|ruckrlandy|rover: ack but wrt 'something merged in 39 and not 64 conversation earlier - don't think so/didn't find and also see inheritance https://github.com/rdo-infra/rdo-jobs/blob/e29b6b9bfc23934f1bf3ecabf3f88a0215bae282/zuul.d/integration-jobs-cs9-master.yaml#L86-L8715:03
rlandy|rovernice - all green on train run15:05
rlandy|rovermarios|ruck: wrt c8 wallaby ..15:07
rlandy|roverperiodic-tripleo-ci-centos-8-undercloud-upgrade-ffu-wallaby is still in criteria15:08
rlandy|roverdidn't run in the line15:08
marios|ruckrlandy|rover: it runs in 9 line and reports to 8 dlrn15:14
rlandy|rovermarios|ruck: ok- so we just wait for the 9 line to run?15:14
marios|ruckrlandy|rover: that is next on my plate to sanity check that is all good 15:15
marios|ruckrlandy|rover: i mean after latest merges for the mixed rhel rework15:15
marios|ruckrlandy|rover: will check in bit15:15
rlandy|roverk - wil leave that with you15:15
marios|ruckrlandy|rover: see https://review.rdoproject.org/zuul/buildset/da77a70bc7cb4644a2538fa36490799c Pipeline openstack-periodic-integration-stable1  periodic-tripleo-ci-centos-8-undercloud-upgrade-ffu-wallaby 15:16
*** ysandeep is now known as ysandeep|out15:31
* ysandeep|out out, happy weekend everyone o/15:31
*** dviroel is now known as dviroel|lunhc15:43
* dviroel|lunhc lunch15:43
jbadiapamarios|ruck: Added you to the https://review.rdoproject.org/r/c/rdo-jobs/+/46336 "Add periodic-tripleo-ci-centos-8-standalone-upgrade-ffu-wallaby" tested at https://review.rdoproject.org/r/c/testproject/+/46335 16:08
marios|ruckjbadiapa: thanks adding to check but probably monday 16:10
marios|ruckjbadiapa: also you can add to the ci review list 16:11
marios|ruckreviewbot: please add to review list https://review.rdoproject.org/r/c/rdo-jobs/+/4633616:11
reviewbotI have added your review to the Review list16:11
jbadiapamarios|ruck: great! thx16:14
*** dviroel|lunhc is now known as dviroel16:19
*** yadnesh is now known as yadnesh|away16:26
rlandy|rover17 promoted16:28
rlandy|rovermarios|ruck: hey - out of meetings for a hour ...16:29
rlandy|roverguessing you are EoD soon16:29
rlandy|rovermarios|ruck: anything to follow except master promo?16:29
marios|ruckrlandy|rover: https://review.rdoproject.org/r/c/rdo-infra/ci-config/+/46337 16:32
marios|ruckrlandy|rover: the mixed rhel jobs are reporting correctly commented @ https://issues.redhat.com/browse/TRIPLEOCI-1300?focusedCommentId=21357300&page=com.atlassian.jira.plugin.system.issuetabpanels%3Acomment-tabpanel#comment-21357300 16:32
rlandy|rovermarios|ruck: ok on  https://review.rdoproject.org/r/c/rdo-infra/ci-config/+/46337  - just removing standalone16:33
marios|ruckrlandy|rover: i think we really need to remove the 8 line (next week) but otherwise jobs seem to be reporting OK. we have some disconnect between when the 8 line runs vs 9 line (value of ci-testing for 8 different) that will be addressed when we stop 816:33
rlandy|roveryep16:33
marios|ruckrlandy|rover: for now we should align the requirements with the 8 in 9 line jobs16:33
marios|ruckrlandy|rover: so i posted that 4637716:33
rlandy|rovergets confusing to track16:33
marios|ruckrlandy|rover: will revisit next week and start clearup16:33
rlandy|roversure16:33
marios|ruckof the 8 line i mean 16:33
marios|ruckrlandy|rover: o/ have a good friday & w/e16:47
*** marios|ruck is now known as marios|out16:50
rlandy|rovermarios|out; you too16:52
rlandy|roverlunch - brb16:52
rlandy|roverdviroel: hey - need your help pls ... 17:25
rlandy|roverhttps://github.com/openshift/release/pull/3443517:25
rlandy|roverfirst there was one failure17:25
rlandy|roverso I reran - now there are two17:25
rlandy|rover   [PANICKED] Test Panicked17:25
rlandy|rover  In [AfterSuite] at: /usr/local/go/src/runtime/panic.go:220 @ 12/02/22 16:45:33.876 17:25
*** jpena is now known as jpena|off17:31
dviroelrlandy|rover: seems to be an issue with prow cluster17:32
dviroellets rerun17:32
rlandy|roverdviroel: ok - nvm the test - I panicked 17:32
dviroelhahah17:33
dviroelhttps://prow.ci.openshift.org/job-history/gs/origin-ci-test/pr-logs/directory/rehearse-34435-pull-ci-openstack-k8s-operators-nova-operator-master-unit17:33
dviroelthis one run just once17:33
dviroelneed /pj-rehearse17:33
dviroelrlandy|rover: we need to ping nova team I think, the error is:   runtime error: invalid memory address or nil pointer dereference17:43
dviroel /go/src/github.com/openstack-k8s-operators/nova-operator/test/functional/suite_test.go:169 +0x4517:44
rlandy|roverdviroel: sorry - meeting - will look after17:46
* dviroel going afk for some hours17:58
* dviroel brazilian break17:58
*** dviroel is now known as dviroel|afk17:58
jm1dviroel|afk: what makes a brazilian break different from a regular break?18:00
rlandy|roverjm1: brazil gets time off to watch the games19:08

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