*** ysandeep__ is now known as ysandeep|afk | 03:36 | |
*** pojadhav is now known as pojadhav|ruck | 04:03 | |
*** yadnesh|away is now known as yadnesh | 04:48 | |
* pojadhav|ruck afk for half an hpur | 05:34 | |
* pojadhav|ruck hour* | 05:34 | |
* pojadhav|ruck back | 06:06 | |
*** ysandeep|afk is now known as ysandeep_ | 06:08 | |
ysandeep_ | chandankumar, the jobs you are trying to add here: https://github.com/openshift/release/pull/34299 can be tested in this patch itself, pj-rehearse is a job that trigger those pre-submit jobs in advance. | 06:10 |
---|---|---|
ysandeep_ | chandankumar: Looks like pj-rehearse job behavior is changed now, Earlier it used to trigger by itself, Now we have to add a comment - I have added the comment just now. | 06:10 |
ysandeep_ | we will get the results in few mins, lets see | 06:10 |
chandankumar | ysandeep_: thank you :-) | 06:13 |
ysandeep_ | chandankumar, also I have invited you in https://github.com/tripleo-ci org as an admin, we use this as a source(instead of cloning in our own repos) so that we can update each other commits if we have to. | 06:14 |
chandankumar | ysandeep_: awesome all tests passed | 06:15 |
ysandeep_ | for ex. If I had to push a new commit(with some changes) to your PR, currently I can't but if the PR was send from tripleo-ci repo - I can. | 06:16 |
chandankumar | ysandeep_: Do I also need to include this https://github.com/openstack-k8s-operators/nova-operator/commit/04af1cfb2b07b2f5a12e4dbb0c2b57d4ca8fa66b#diff-76ed074a9305c04054cdebb9e9aad2d818052b07091de1f20cad0bbac34ffb52L235 in my makefile? | 06:16 |
chandankumar | ci tools repo for running tests | 06:17 |
ysandeep_ | chandankumar, looks like those changes are already present: https://github.com/openstack-k8s-operators/ansibleee-operator/blob/main/Makefile#L242 | 06:17 |
chandankumar | ysandeep_: ah yes, I missed that part | 06:18 |
chandankumar | ok cool | 06:19 |
chandankumar | ysandeep_: gh action jobs have started against ansiblee operator from yesterday also | 06:20 |
chandankumar | https://github.com/openstack-k8s-operators/ansibleee-operator/actions/runs/3541594033/jobs/5946046688 | 06:20 |
chandankumar | Does it happened after installing openshift ci github app? | 06:21 |
ysandeep_ | chandankumar, martin added them here: https://github.com/openstack-k8s-operators/ansibleee-operator/commit/389759d337e87df33a069861036a407a78c03e37 | 06:22 |
chandankumar | ysandeep_: ok | 06:23 |
ysandeep_ | added `/pj-rehearse ack` comment as we are satisfied with pre-submit job results, also added /lgtm label, patch will merge soon :D | 06:25 |
ysandeep_ | As there are some changes since you cloned the repo, its rerunning the ci jobs. | 06:27 |
ysandeep_ | chandankumar, fyi..In Prow there is no check/gate concept - if it found base is changed since check jobs are run, it reruns the jobs before merge, | 06:28 |
chandankumar | ysandeep_: so, so in prow, we will be running the same set of tests in both check and gate | 06:36 |
ysandeep_ | chandankumar, correct, basically it retest the same job(incase base changed) if nothing changed afaik.. It even don't retest the jobs. | 06:38 |
chandankumar | and with gh actions, can we customize the set of actions based on pull request and during merge ? | 06:38 |
ysandeep_ | yes gha is very flexible, we can almost anything we want. | 06:40 |
ysandeep_ | can do* | 06:40 |
ysandeep_ | for GHA also there is no check/gate concept, but we can rerun job post merge if we want to. | 06:41 |
ysandeep_ | In GHA natively, there is no way to figure out if the git base changed(what prow does automatically) but we can write our own code to figure that out | 06:42 |
chandankumar | ysandeep_: now I can understand, why you prefer gh action over prow | 07:01 |
ysandeep_ | chandankumar, Github action have its own security constraints specially for pull request: https://securitylab.github.com/research/github-actions-preventing-pwn-requests/, zuul is amazing tool - by design it solved the security constaints with trusted/untrusted repos and its way of handling secrets. | 07:15 |
*** ysandeep_ is now known as ysandeep|brb | 07:17 | |
akahat | @team please review when you are free thanks! https://review.rdoproject.org/r/c/rdo-jobs/+/46183 | 07:27 |
*** amoralej|off is now known as amoralej | 07:43 | |
*** yadnesh is now known as yadnesh|afk | 07:45 | |
*** frenzy_friday is now known as frenzy_friday|rover | 07:46 | |
jpodivin | Hi. I'm seeing tripleo-ci-centos-8-standalone failing on wallaby, it also affects derived jobs. It does seem to be caused by mysql container | 07:58 |
jpodivin | https://storage.gra.cloud.ovh.net/v1/AUTH_dcaab5e32b234d56b626f72581e3644c/zuul_opendev_logs_185/865601/1/check/tripleo-ci-centos-8-standalone/1855d01/logs/undercloud/home/zuul/standalone_deploy.log | 07:58 |
jpodivin | Parsing error :/ do we have a bug for this? | 07:59 |
jpodivin | pojadhav|ruck: ^ | 08:00 |
pojadhav|ruck | jpodivin, checking | 08:01 |
jpodivin | pojadhav|ruck: thanks a lot | 08:03 |
jpodivin | pojadhav|ruck: Here, the container logs https://storage.gra.cloud.ovh.net/v1/AUTH_dcaab5e32b234d56b626f72581e3644c/zuul_opendev_logs_185/865601/1/check/tripleo-ci-centos-8-standalone/1855d01/logs/undercloud/var/log/extra/podman/containers/container-puppet-mysql/stdout.log | 08:03 |
pojadhav|ruck | jpodivin, yes bug is already reported for this https://bugs.launchpad.net/tripleo/+bug/1997939 | 08:08 |
jpodivin | pojadhav|ruck: great, I'm going to update it with my logs | 08:10 |
pojadhav|ruck | jpodivin, sure | 08:10 |
jpodivin | thanks again pojadhav|ruck++ | 08:10 |
pojadhav|ruck | jpodivin, :) | 08:10 |
*** ysandeep|brb is now known as ysandeep_ | 08:14 | |
ysandeep_ | akahat, looking now | 08:14 |
ysandeep_ | folks please add in reviewlist: https://review.opendev.org/c/openstack/tripleo-ci/+/865502 | 08:17 |
reviewbot | I have added your review to the Review list | 08:17 |
ysandeep_ | chandankumar, ^^ to revert back to buildcontainers_venv: true | 08:18 |
ysandeep_ | akahat, Have we already removed the dependency of rdo registry on promoter/copy to quay script? | 08:23 |
akahat | ysandeep, i've patches up for it. | 08:24 |
ysandeep_ | Could you please share those, I couldn't find what was done for promoter/copy to quay in script topic: https://review.rdoproject.org/r/q/topic:rdo_registry_decomission | 08:25 |
ysandeep_ | same* topic | 08:25 |
akahat | ysandeep, yeah.. getting link.. | 08:25 |
akahat | ysandeep, for promoter https://review.rdoproject.org/r/c/rdo-infra/ci-config/+/45898 | 08:25 |
ysandeep_ | sry it was in same topic, my bad | 08:27 |
* pojadhav|ruck lunch | 08:43 | |
* akahat lunch | 08:43 | |
*** yadnesh|afk is now known as yadnesh | 08:45 | |
*** jgilaber_ is now known as jgilaber | 08:50 | |
ysandeep_ | akahat, commented: https://review.rdoproject.org/r/c/rdo-jobs/+/46183 | 09:13 |
ysandeep_ | Maybe we can discuss what would be the correct merge order during review time. | 09:14 |
*** ysandeep_ is now known as ysandeep_lunch | 09:19 | |
jpodivin | pojadhav|ruck: I have a question. We need python3.8 for our linters, unit tests and such, but jammy nodes (where those jobs run) are now using 3.10 | 09:45 |
jpodivin | do you have any suggestions as to how we could get the 3.8 on them, or another approach ? | 09:45 |
chandankumar | jpodivin: is it for master branch jobs? | 09:49 |
jpodivin | chandankumar: not necessarily, but also for them. | 09:49 |
jpodivin | chandankumar: we really need it only for our stable branches. | 09:50 |
jpodivin | code there breaks linters on 3.10 | 09:50 |
chandankumar | jpodivin: can you share the job link or example for stable branches | 09:52 |
chandankumar | for master it is ok to use 3.10 | 09:53 |
jpodivin | https://zuul.opendev.org/t/openstack/build/567689826e5d4ff4b06e20a04df8476a | 09:53 |
jpodivin | https://zuul.opendev.org/t/openstack/build/8f3703d6c66d4c65b7bb3de601803126 | 09:53 |
jpodivin | chandankumar: I ... am not sure. Our software declares compatibility with 3.9 tops | 09:54 |
jpodivin | same with tripleoclient. | 09:54 |
jpodivin | testing it on something more recent is ... well it doesn't break anything now. | 09:54 |
chandankumar | validations-common 1.6 used with train and wallaby ? | 09:54 |
jpodivin | just train | 09:54 |
jpodivin | my point is that we should run tests with the supported version. even if it sort of works. | 09:55 |
chandankumar | jpodivin: for py3.8 can we use nodeset: ubuntu-bionic nodeset | 09:59 |
chandankumar | and rest of it let's use the default nodeset | 09:59 |
chandankumar | https://opendev.org/openstack/openstack-zuul-jobs/src/branch/master/zuul.d/jobs.yaml#L249 | 10:00 |
jpodivin | well, we are running into issues on the linters job, and possibly cover. | 10:00 |
jpodivin | so I'll probably have to subclass those for bionic | 10:01 |
akahat | ysandeep, checking. | 10:07 |
akahat | ysandeep, thanks for review, going with your suggested merge order. | 10:18 |
frenzy_friday|rover | kkula, hey, I couldnt find out yesterday why your dashboard didnt show any data. I'll try again today in some time | 10:39 |
kkula | frenzy_friday|rover: ack, thx! | 10:39 |
*** ysandeep_lunch is now known as ysandeep_ | 10:55 | |
*** frenzy_friday is now known as frenzy_friday|rover | 11:10 | |
ysandeep_ | chandankumar, akahat https://review.rdoproject.org/codesearch/?q=periodic-tripleo-ci-build-containers-centos-8-quay-push-train | 11:24 |
ysandeep_ | chandankumar, https://code.engineering.redhat.com/gerrit/c/openstack/tripleo-ci-internal-jobs/+/432774/4/zuul.d/component-jobs-rhos-17.1-rhel-9.yaml#723 | 11:39 |
ysandeep_ | dviroel, o/ hey around? | 11:51 |
dviroel | ysandeep_: o/ | 12:01 |
ysandeep_ | dviroel, can we meet for podified mtg earlier today, maybe now if you are free. | 12:02 |
ysandeep_ | Ronelle is not around but we can discuss the current status and immedidate next steps. | 12:03 |
dviroel | ysandeep_: can we try to meet in ~1h? | 12:07 |
ysandeep_ | dviroel, nw, lets meet at scheduled time. | 12:10 |
*** ysandeep_ is now known as ysandeep|afk | 12:21 | |
dviroel | ysandeep|afk: hi, when you come back, can you check if we have IBM3 creds on toolbox? thanks | 12:50 |
chandankumar | dviroel: regarding toolbox check pm | 12:54 |
dviroel | chandankumar: tks | 12:58 |
*** amoralej is now known as amoralej|lunch | 13:24 | |
ysandeep|afk | dviroel, looking toolbox | 13:28 |
*** ysandeep|afk is now known as ysandeep_ | 13:28 | |
dviroel | ysandeep_: no need | 13:30 |
dviroel | ysandeep_: we already added creds there | 13:30 |
ysandeep_ | yes looks like you guys added already | 13:30 |
ysandeep_ | yup | 13:30 |
ysandeep_ | chandankumar, podified mtg if you are around | 13:31 |
*** yadnesh is now known as yadnesh|away | 13:53 | |
*** amoralej|lunch is now known as amoralej | 14:10 | |
*** khomesh is now known as khomesh|away | 14:24 | |
pojadhav|ruck | frenzy_friday|rover, will be out in few mins... RR notes updated with posted reruns on upstream component line jobs. please watch them you get free time. | 14:26 |
frenzy_friday|rover | pojadhav|ruck, thanks, have a nice weekend O/ | 14:26 |
pojadhav|ruck | frenzy_friday|rover, same to you :) | 14:27 |
*** pojadhav|ruck is now known as pojadhav|out | 14:29 | |
*** jpena|off is now known as jpena | 14:39 | |
ysandeep_ | chandankumar++ dviroel++ | 14:53 |
* ysandeep_ out, have a great weekend everyone! | 14:53 | |
*** ysandeep_ is now known as ysandeep|out | 14:56 | |
chandankumar | see ya , happy weekend people! | 15:28 |
*** dviroel is now known as dviroel|lunch | 15:58 | |
*** amoralej is now known as amoralej|off | 16:37 | |
*** jpena is now known as jpena|off | 17:01 | |
*** dviroel|lunch is now known as dviroel | 17:06 | |
frenzy_friday|rover | zed promoted | 20:54 |
frenzy_friday|rover | looks like the rest have inconsistent failures. I'll check on monday | 20:55 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!