| rdogerrit | rdo-trunk created rdoinfo master: Promote CBS tags update for ussuri-8-release https://review.rdoproject.org/r/c/rdoinfo/+/34561 | 00:12 |
|---|---|---|
| rdogerrit | rdo-trunk created rdoinfo master: Promote CBS tags update for wallaby-8s-release https://review.rdoproject.org/r/c/rdoinfo/+/34562 | 00:12 |
| rdogerrit | rdo-trunk created rdoinfo master: Bump rdoinfo xena-uc tags to latest upper-constraints https://review.rdoproject.org/r/c/rdoinfo/+/34563 | 00:13 |
| rdogerrit | rdo-trunk created rdoinfo master: Promote CBS tags update for victoria-8-release https://review.rdoproject.org/r/c/rdoinfo/+/34564 | 00:13 |
| rdogerrit | rdo-trunk created rdoinfo master: Promote CBS tags update for victoria-8s-release https://review.rdoproject.org/r/c/rdoinfo/+/34565 | 00:13 |
| rdogerrit | rdo-trunk created rdoinfo master: Promote CBS tags update for ussuri-8s-release https://review.rdoproject.org/r/c/rdoinfo/+/34566 | 00:14 |
| *** rlandy|bbl is now known as rlandy | 01:49 | |
| *** ykarel|away is now known as ykarel | 04:31 | |
| rdogerrit | Merged rdoinfo master: Promote CBS tags update for victoria-8-release https://review.rdoproject.org/r/c/rdoinfo/+/34564 | 04:41 |
| rdogerrit | Merged rdoinfo master: Promote CBS tags update for ussuri-8s-release https://review.rdoproject.org/r/c/rdoinfo/+/34566 | 04:48 |
| rdogerrit | Merged rdoinfo master: Promote CBS tags update for victoria-8s-release https://review.rdoproject.org/r/c/rdoinfo/+/34565 | 04:48 |
| *** iurygregory_ is now known as iurygregory | 06:38 | |
| rdogerrit | Merged rdoinfo master: Bump rdoinfo xena-uc tags to latest upper-constraints https://review.rdoproject.org/r/c/rdoinfo/+/34563 | 06:46 |
| *** amoralej|away is now known as amoralej | 07:02 | |
| rdogerrit | Merged rdoinfo master: Promote CBS tags update for ussuri-8-release https://review.rdoproject.org/r/c/rdoinfo/+/34561 | 07:30 |
| *** rpittau|afk is now known as rpittau | 07:31 | |
| amoralej | jcapitao, ykarel so https://review.rdoproject.org/r/c/rdo-infra/ci-config/+/34419 is ready to merge? | 07:59 |
| jcapitao | amoralej: yes | 08:00 |
| ykarel | jcapitao, i posted some comments on train one can u check | 08:02 |
| ykarel | wrt ^^ yes master one is ready, but was thinking if there are some restrictions on reusing jobs in train c7, then may restructure 34419 | 08:05 |
| jcapitao | ykarel: hmm yesteraday I saw a restriction while testing, but now it appears there is a solution if I change something in 34419 | 08:13 |
| ykarel | jcapitao, okk can comment and update, so we can check | 08:13 |
| ykarel | saying because in future we have c9 jobs comming as well | 08:14 |
| ykarel | and so focusing more on reusing stuffs where possible | 08:14 |
| *** bhagyashris_ is now known as bhagyashris|ruck | 08:24 | |
| jcapitao | ykarel: yeah, agree | 08:26 |
| jcapitao | ykarel: I replied to your comment | 08:26 |
| ykarel | ack checking | 08:27 |
| jcapitao | ykarel: maybe we can add {os_release} = centos7/8/9 in job template definition wdyt ? | 08:36 |
| jcapitao | from weirdo-{openstack_release}-centos7-promote-{playbook} to weirdo-{openstack_release}-{os_release}-promote-{playbook} | 08:36 |
| jcapitao | and add it in 34419 patch | 08:37 |
| jcapitao | that way we can template it for whatever release | 08:37 |
| ykarel | jcapitao, actually i thought over that and with that there will be some unnecessary jobs added | 08:40 |
| ykarel | so for each distro and release and playbook, total jobs:- no of distro * no of release * no of playbook | 08:43 |
| jcapitao | ykarel: ah right, it will create centos7 jobs for each openstack release and packstack/puppet-openstack | 08:43 |
| jcapitao | right | 08:43 |
| ykarel | there are some pros too if we go that way, as job name itself will say about what release it's running for | 08:44 |
| jcapitao | maybe we can hide them with view configuration | 08:45 |
| ykarel | but in that case we need to seperate releases , like train have both c7,c8 jobs, ussuri,victoria c8, wallaby c8,c9 and xena c9 | 08:45 |
| ykarel | so i think view will just make required jobs visible, but they will still be there | 08:49 |
| jcapitao | right, by creating a dedicated release project in weirdo-main.yml | 08:49 |
| jcapitao | with distro_release and playbook as variable | 08:50 |
| ykarel | yes with distro in name, somethink like weirdo-promote-jobs-{c7/c8/c9} | 08:50 |
| ykarel | ohk u mean the reverse | 08:51 |
| jcapitao | ykarel: yes, but yours is better | 08:52 |
| ykarel | jcapitao, wrt now it appears there is a solution if I change something in 34419, u mean ^ itself or something else? | 08:56 |
| jcapitao | ykarel: I thought to put train-centos7 as value for openstack_release variable | 08:57 |
| ykarel | iirc that's also used by weirdo | 08:57 |
| jcapitao | but this breaks the code later on execution, and we can't manipulate variable unfortunately | 08:59 |
| ykarel | yes version="{openstack_release}" weirdo-promote-ansible-playbook-builder and that's passed to weirdo | 08:59 |
| jcapitao | right | 08:59 |
| ykarel | ok let's see if we can get through the limitation in some better way | 09:03 |
| ykarel | jcapitao, wouldn't it work if we don't pass centos7 at all in copyartifact considering we are running all these jobs as multijob | 09:03 |
| ykarel | and c7 and c8 jobs will be different | 09:04 |
| ykarel | iirc that's the limitation u referring too | 09:04 |
| ykarel | hmm but we are binding to scripts based on that name, so tricky, can ignore this | 09:06 |
| ykarel | me leaves for lunch, can check later | 09:06 |
| jcapitao | exact that's the issue | 09:06 |
| *** ykarel is now known as ykarel|lunch | 09:06 | |
| *** ykarel|lunch is now known as ykarel | 11:11 | |
| *** jcapitao is now known as jcapitao_lunch | 11:12 | |
| jcapitao_lunch | ykarel: I'm currently squashing my 2 patches (34419 and 34560) and introducing {distro_release} | 11:17 |
| ykarel | jcapitao_lunch, ack let's discuss post ur lunch | 11:18 |
| *** dviroel|out is now known as dviroel | 11:19 | |
| rdogerrit | User marios created rdo-jobs master: Adds missing standalone-upgrade-train for train integration line https://review.rdoproject.org/r/c/rdo-jobs/+/34568 | 11:42 |
| *** jcapitao_lunch is now known as jcapitao | 11:54 | |
| jcapitao | ykarel: so, should we go with {distro_release} ? | 11:56 |
| jcapitao | I'd say yes | 11:56 |
| *** amoralej is now known as amoralej|lunch | 12:05 | |
| ykarel | jcapitao, i thought about reusing, and only way i could get was to restructure existing jobs(rdo-promote-get-hash) and make names without distro name in it and handling release specific logic in script itself scripts/centos-master-current-tripleo.sh | 12:08 |
| ykarel | but doesn't seem it much worth to go this way | 12:08 |
| ykarel | jcapitao, also i came across https://docs.openstack.org/infra/jenkins-job-builder/builders.html#builders.conditional-step | 12:09 |
| ykarel | which seems should be able to work by using conditionally invoking builders | 12:10 |
| ykarel | i searched in jenkins available plugin search but didn't find it in library | 12:11 |
| jcapitao | that's interesting the conditional-step | 12:11 |
| ykarel | from home page, it says This plugin is up for adoption! We are looking for new maintainers. Visit our Adopt a Plugin initiative for more information. | 12:12 |
| ykarel | https://plugins.jenkins.io/conditional-buildstep/ | 12:12 |
| ykarel | so if that get's unsupported we may face issues later, so i think we can avoid using it | 12:15 |
| jcapitao | right | 12:16 |
| ykarel | and go with distro_release specific itself | 12:16 |
| ykarel | and also take opinion from amoralej|lunch | 12:16 |
| rdogerrit | User marios created rdo-jobs master: Remove train undercloud-upgrade job it is not valid https://review.rdoproject.org/r/c/rdo-jobs/+/34569 | 12:24 |
| rdogerrit | User bshewale created rdo-infra/ci-config master: [DNM] Fix ppc containers manifest push https://review.rdoproject.org/r/c/rdo-infra/ci-config/+/34570 | 12:34 |
| *** marios_ is now known as marios | 12:40 | |
| *** marios is now known as marios|call | 13:10 | |
| apevec[m] | amoralej: I saw on centos-devel "the latest ansible release requires python 3.8" - do you know where is that coming from? We do have Ansible 2.9 working on py3.9 in CS9 right? | 13:13 |
| *** amoralej|lunch is now known as amoralej | 13:13 | |
| amoralej | apevec[m], we've run weirdo with ansible in cs9 | 13:14 |
| amoralej | but i'm not sure if there is any specific known issue | 13:14 |
| amoralej | i'm not aware of | 13:14 |
| apevec[m] | yeah, as many things, this seems to be hear-say... | 13:15 |
| apevec[m] | I guess it's the fact that Ansible is testing with py3.8 only | 13:15 |
| apevec[m] | their exec env containers are CS8 + py3.8 module | 13:16 |
| amoralej | very likely | 13:19 |
| rlandy | apevec[m]: amoralej: hi - we have a hacked up a testproject job to run container builds for c9: https://review.rdoproject.org/r/c/testproject/+/18953 | 13:27 |
| amoralej | oh, those are good news | 13:27 |
| rlandy | the tripleo-common depends-on creates all the changes to the dockerfiles listed in the etherpad | 13:28 |
| *** marios|call is now known as marios | 13:34 | |
| rdogerrit | User pojadhav created rdo-infra/ci-config master: Refactor centos-7 FS037 minor update job to match with upstream https://review.rdoproject.org/r/c/rdo-infra/ci-config/+/34572 | 13:54 |
| *** rpittau is now known as rpittau|afk | 14:08 | |
| rdogerrit | Joel Capitao proposed rdo-infra/ci-config master: Add promote-master-current-tripleo job in new jenkins https://review.rdoproject.org/r/c/rdo-infra/ci-config/+/34419 | 14:42 |
| *** ykarel is now known as ykarel|away | 14:46 | |
| jcapitao | ykarel: ^ | 14:48 |
| rdogerrit | User pojadhav created rdo-jobs master: Refactor CentOS-7 FS030 job to match with upstream https://review.rdoproject.org/r/c/rdo-jobs/+/34573 | 14:53 |
| rdogerrit | User pojadhav created rdo-infra/ci-config master: Refactor CentOS-7 FS030 job to match with upstream https://review.rdoproject.org/r/c/rdo-infra/ci-config/+/34574 | 14:58 |
| rdogerrit | Joel Capitao proposed rdo-infra/ci-config master: Add rdo_trunk-promote-train-centos7-current-tripleo job https://review.rdoproject.org/r/c/rdo-infra/ci-config/+/34560 | 14:59 |
| rdogerrit | User pojadhav created config master: Refactor CentOS-7 FS030 job to match with upstream https://review.rdoproject.org/r/c/config/+/34575 | 15:05 |
| rdogerrit | User pojadhav proposed rdo-jobs master: Refactor CentOS-7 FS030 job to match with upstream https://review.rdoproject.org/r/c/rdo-jobs/+/34573 | 15:21 |
| rdogerrit | Joel Capitao proposed rdo-infra/ci-config master: Add promote-master-current-tripleo job in new jenkins https://review.rdoproject.org/r/c/rdo-infra/ci-config/+/34419 | 16:07 |
| rdogerrit | Joel Capitao proposed rdo-infra/ci-config master: Add remaining tripleo promotion jobs https://review.rdoproject.org/r/c/rdo-infra/ci-config/+/34560 | 16:08 |
| *** marios is now known as marios|out | 16:35 | |
| *** amoralej is now known as amoralej|off | 16:45 | |
| rdogerrit | wes hayutin created rdo-infra/ci-config master: WIP: clean up crazy vars https://review.rdoproject.org/r/c/rdo-infra/ci-config/+/34576 | 22:44 |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!