| rdogerrit | rdo-trunk rdo-trunk proposed nfvinfo master: Promote CBS tags update for nfvsig-openvswitch2.17-testing https://review.rdoproject.org/r/c/nfvinfo/+/53367 | 00:04 |
|---|---|---|
| rdogerrit | rdo-trunk rdo-trunk proposed nfvinfo master: Promote CBS tags update for nfvsig-openvswitch3.3-testing https://review.rdoproject.org/r/c/nfvinfo/+/53369 | 00:04 |
| rdogerrit | Yatin Karel proposed nfvinfo master: Update ovn24.03 to latest FDP rebuild https://review.rdoproject.org/r/c/nfvinfo/+/53414 | 06:09 |
| rdogerrit | Ananya Banerjee proposed rdo-jobs master: Adds task to call test role https://review.rdoproject.org/r/c/rdo-jobs/+/53077 | 06:32 |
| rdogerrit | Ananya Banerjee proposed rdo-jobs master: Adds task to call test role https://review.rdoproject.org/r/c/rdo-jobs/+/53077 | 06:33 |
| rdogerrit | Merged rdoinfo master: Bump rdoinfo dalmatian-uc tags to latest upper-constraints or latest Puppetfile https://review.rdoproject.org/r/c/rdoinfo/+/53413 | 06:43 |
| *** ykarel_ is now known as ykarel | 07:01 | |
| ykarel | Hi, is the issue with jobs in https://review.rdoproject.org/r/q/project:nfvinfo known? | 07:10 |
| ykarel | fails due to 8-stream EOL | 07:10 |
| rdogerrit | Joan Francesc Gilabert proposed rdo-jobs master: Make logs collection for adoption less verbose https://review.rdoproject.org/r/c/rdo-jobs/+/53415 | 07:15 |
| karolinku[m] | ykarel: let me check | 07:37 |
| karolinku[m] | jobs are based on centos8, I'll propose a patch | 07:52 |
| ykarel | karolinku[m], thx | 07:55 |
| rdogerrit | Marios Andreou proposed rdo-jobs master: Add final two compute nodes for data plane adoption multinode job https://review.rdoproject.org/r/c/rdo-jobs/+/53365 | 08:12 |
| rdogerrit | Karolina Kula proposed rdo-jobs master: Run tox-linters on centos9 nodeset https://review.rdoproject.org/r/c/rdo-jobs/+/53416 | 08:30 |
| rdogerrit | Yatin Karel proposed nfvinfo master: Update ovn24.03 to latest FDP rebuild https://review.rdoproject.org/r/c/nfvinfo/+/53414 | 08:38 |
| rdogerrit | Karolina Kula proposed nfvinfo master: Switch to py36 tox jobs https://review.rdoproject.org/r/c/nfvinfo/+/53417 | 08:48 |
| rdogerrit | Karolina Kula proposed nfvinfo master: Switch to py36 tox jobs https://review.rdoproject.org/r/c/nfvinfo/+/53417 | 08:53 |
| ykarel | s/py36/py39 | 08:58 |
| ykarel | karolinku[m], ^ | 08:58 |
| ykarel | also can you update nodeset for tox-linters in ^ | 08:58 |
| karolinku[m] | yes, but tox-linters is wider issue, not only for nfvinfo | 08:59 |
| ykarel | karolinku[m], yes so you plan to fix base-minimal job in config/zuul.d/jobs.yaml then? | 09:02 |
| ykarel | https://github.com/rdo-infra/review.rdoproject.org-config/blob/82191e4d4efc2312f40432971396d9e847dcfd73/zuul.d/jobs.yaml#L48 | 09:03 |
| karolinku[m] | yes, but let me confirm it with jcapitao | 09:04 |
| ykarel | ack | 09:04 |
| ykarel | 8-stream is already doomed so seems should be fine to switch base itself | 09:04 |
| rdogerrit | Ananya Banerjee proposed rdo-jobs master: Adds task to call test role https://review.rdoproject.org/r/c/rdo-jobs/+/53077 | 09:11 |
| rdogerrit | Karolina Kula proposed config master: Switch base-minimal job to use centos 9 nodeset https://review.rdoproject.org/r/c/config/+/53418 | 09:11 |
| jcapitao[m] | I proposed to amoralej to switch base to CS9 couple of weeks ago | 09:13 |
| jcapitao[m] | but we decided to not switch because it would impact all jobs even the ones we are not stakeholders | 09:13 |
| karolinku[m] | so, we should define new jobs? | 09:14 |
| jcapitao[m] | so we prefered to switch for the RDO jobs only | 09:14 |
| jcapitao[m] | each stakeholders should override nodeset of their job with CS9 | 09:15 |
| jcapitao[m] | I'm checking the latest builds in RDO Zuul Gerrit | 09:21 |
| jcapitao[m] | to check if all jobs have migrated to CS9 | 09:21 |
| jcapitao[m] | there are still jobs that haven't migrated to CS9 | 09:28 |
| jcapitao[m] | karolinku: so I guess we have to override tox-linters and tox-py39 by defining them in https://github.com/rdo-infra/review.rdoproject.org-config/blob/master/zuul.d/jobs.yaml | 09:34 |
| jcapitao[m] | by just defining the nodeset CS9 attribute | 09:34 |
| karolinku[m] | yes, that's more safe solution | 10:06 |
| ykarel | jcapitao[m], what all other stackholders still running 8-stream jobs? | 10:09 |
| jcapitao[m] | https://review.rdoproject.org/zuul/builds?result=RETRY_LIMIT&skip=0 | 10:11 |
| jcapitao[m] | mostly ex-tripleo/openstack-k8s-operators | 10:11 |
| ykarel | and tripleo already dead | 10:12 |
| ykarel | arxcruz, ^ why tripleo jobs still run? | 10:13 |
| jcapitao[m] | IIRC some of them are still used (jobs not rebranded) | 10:14 |
| ykarel | me not sure if many of those jobs still relevant except podified/edpm ones | 10:18 |
| jcapitao[m] | yeah I'm not sure either | 10:20 |
| jcapitao[m] | well since it's broken since a week.. | 10:20 |
| jcapitao[m] | maybe we can move to CS9 already | 10:20 |
| ykarel | you mean move base jobs, right? | 10:27 |
| jcapitao[m] | yeah | 10:28 |
| ykarel | +1 to that | 10:30 |
| karolinku[m] | you mean switch base job? like https://review.rdoproject.org/r/c/config/+/53418 ? | 10:32 |
| jcapitao[m] | yes | 10:32 |
| karolinku[m] | according to procedure I should first change only base-minial-test | 10:33 |
| karolinku[m] | but tox-linters will fail also in this case | 10:33 |
| jcapitao[m] | yeah the job relies also on tox-linters, so we need to add the job tox-linters running on CS9 alongside your change | 10:37 |
| jcapitao[m] | that way it will run on CS9 | 10:37 |
| jcapitao[m] | if all goes well, we'll merge it, and then remove the added job afterward | 10:38 |
| karolinku[m] | do we want to first merge base-minial-test? | 10:38 |
| jcapitao[m] | no need | 10:40 |
| jcapitao[m] | we can do both together | 10:40 |
| jcapitao[m] | hmm | 10:40 |
| jcapitao[m] | actually | 10:41 |
| jcapitao[m] | yes we should first merge base-minial-test to be able to test right | 10:41 |
| rdogerrit | Karolina Kula proposed config master: Switch base-minimal job to use centos 9 nodeset https://review.rdoproject.org/r/c/config/+/53418 | 10:43 |
| ykarel | still fails | 10:53 |
| ykarel | we can first change https://github.com/rdo-infra/rdo-jobs/blob/0558edd436d12569f34a05b932cc1fa9a00eb39f/zuul.d/jobs.yaml#L7 | 10:53 |
| ykarel | add nodeset there, that should allow to change in config repo | 10:54 |
| rdogerrit | Karolina Kula proposed config master: Switch base-minimal job to use centos 9 nodeset https://review.rdoproject.org/r/c/config/+/53418 | 10:55 |
| karolinku[m] | hmm, I though it inherit after current change in base-test | 10:56 |
| jcapitao[m] | it loads the zuul configuration currently defined | 11:01 |
| jcapitao[m] | we have to force merge this one | 11:01 |
| jcapitao[m] | it's step 3 of the process | 11:04 |
| ykarel | we can change rdo-jobs above, and then can change in config both minimal and minimal-test together | 11:04 |
| ykarel | and then can revert rdo-jobs one | 11:05 |
| rdogerrit | Merged rdo-jobs master: Make logs collection for adoption less verbose https://review.rdoproject.org/r/c/rdo-jobs/+/53415 | 11:06 |
| jcapitao[m] | ykarel: yeah that's another way to do it | 11:07 |
| rdogerrit | Karolina Kula proposed config master: Switch base-minimal-test job to use centos 9 nodeset https://review.rdoproject.org/r/c/config/+/53418 | 11:18 |
| rdogerrit | Karolina Kula proposed config master: Switch base-minimal-test job to use centos 9 nodeset https://review.rdoproject.org/r/c/config/+/53418 | 11:19 |
| karolinku[m] | I will follow describer procedure | 11:19 |
| karolinku[m] | now it's pure change in base-minimal-test | 11:19 |
| ykarel | but tox-linter will fail there, that's why i suggested the another way | 11:20 |
| karolinku[m] | force merge will be needed here | 11:22 |
| ykarel | for this particular case the described procedure doesn't matter much(as we just change nodeset and nothing trusted-project specific), but ok if we want to go that way and have force merge permissions | 11:22 |
| karolinku[m] | jcapitao: are you able to force-merge? | 11:22 |
| jcapitao[m] | arf, I don't have force-merge permission .. | 11:25 |
| jcapitao[m] | so let's do it in rdo-jobs as proposed by ykarel | 11:25 |
| karolinku[m] | ack | 11:40 |
| rdogerrit | Takashi Kajinami proposed openstack/swift-distgit rpm-master: Fix inconsistent config file ownership https://review.rdoproject.org/r/c/openstack/swift-distgit/+/53419 | 12:45 |
| rdogerrit | Bogdan Dobrelya proposed rdo-jobs master: Extend hostnamemap for multi-cell deployments https://review.rdoproject.org/r/c/rdo-jobs/+/53192 | 13:00 |
| rdogerrit | Karolina Kula proposed rdo-jobs master: Run tox-linters on centos9 nodeset https://review.rdoproject.org/r/c/rdo-jobs/+/53416 | 13:10 |
| rdogerrit | Karolina Kula proposed rdo-jobs master: Run tox-linters on centos9 nodeset https://review.rdoproject.org/r/c/rdo-jobs/+/53416 | 13:10 |
| rdogerrit | pkomarov komarov proposed rdo-jobs master: WIP add edpm_bootstrap_command to vars before adoption and tests https://review.rdoproject.org/r/c/rdo-jobs/+/53119 | 13:39 |
| rdogerrit | pkomarov komarov proposed rdo-jobs master: Add edpm_bootstrap_command to vars before adoption and tests https://review.rdoproject.org/r/c/rdo-jobs/+/53119 | 13:40 |
| rdogerrit | Karolina Kula proposed rdo-jobs master: Run tox-linters on centos9 nodeset https://review.rdoproject.org/r/c/rdo-jobs/+/53416 | 13:42 |
| rdogerrit | Karolina Kula proposed config master: Switch base-minimal-test job to use centos 9 nodeset https://review.rdoproject.org/r/c/config/+/53418 | 14:52 |
| karolinku[m] | ykarel: we can't shadow tox-linters in https://review.rdoproject.org/r/c/rdo-jobs/+/53416/4 | 14:53 |
| ykarel | karolinku[m], had you tried changing base: job in rdo-jobs already? | 14:54 |
| ykarel | i would have expected that to work | 14:54 |
| ykarel | i don't see that in any of the patches | 14:55 |
| ykarel | let me update on what i mean | 14:55 |
| karolinku[m] | I think we will ask for force merge https://review.rdoproject.org/r/c/config/+/53418 | 14:56 |
| ykarel | that should also work | 14:57 |
| rdogerrit | Yatin Karel proposed rdo-jobs master: Switch base job to run on 9-stream nodes https://review.rdoproject.org/r/c/rdo-jobs/+/53420 | 15:01 |
| ykarel | updated ^ in case we need to go this route | 15:01 |
| ykarel | it's green if we want to merge it | 15:08 |
| rdogerrit | Merged config master: Switch base-minimal-test job to use centos 9 nodeset https://review.rdoproject.org/r/c/config/+/53418 | 15:09 |
| rdogerrit | Joel Capitao proposed config master: DNM test config jobs https://review.rdoproject.org/r/c/config/+/53421 | 15:14 |
| rdogerrit | Yatin Karel proposed nfvinfo master: Switch to py39 tox jobs https://review.rdoproject.org/r/c/nfvinfo/+/53417 | 15:15 |
| jcapitao[m] | ykarel: I left a comment in https://review.rdoproject.org/r/c/nfvinfo/+/53414 | 15:16 |
| rdogerrit | Yatin Karel proposed nfvinfo master: Update ovn24.03 to latest FDP rebuild https://review.rdoproject.org/r/c/nfvinfo/+/53414 | 15:17 |
| ykarel | jcapitao[m], yeap was updating that only :), also rebased on above patch | 15:17 |
| jcapitao[m] | nice | 15:18 |
| rdogerrit | Merged nfvinfo master: Switch to py39 tox jobs https://review.rdoproject.org/r/c/nfvinfo/+/53417 | 15:22 |
| ykarel | jcapitao[m], can you also review https://review.rdoproject.org/r/c/nfvinfo/+/53414 | 16:11 |
| jcapitao[m] | ykarel: merging it | 16:13 |
| rdogerrit | Merged nfvinfo master: Update ovn24.03 to latest FDP rebuild https://review.rdoproject.org/r/c/nfvinfo/+/53414 | 16:18 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!