| *** ysandeep|out is now known as ysandeep | 03:04 | |
| *** ysandeep is now known as ysandeep|afk | 04:30 | |
| *** bhagyashris is now known as bhagyashris|ruck | 05:37 | |
| *** ysandeep|afk is now known as ysandeep | 05:39 | |
| *** amoralej|off is now known as amoralej | 06:45 | |
| rdogerrit | Merged gating_scripts master: Allow to select the packages to update for nfvinfo https://review.rdoproject.org/r/c/gating_scripts/+/45654 | 06:50 |
|---|---|---|
| rdogerrit | Merged config master: Simplify buildsys-tags-update for nfvinfo https://review.rdoproject.org/r/c/config/+/45655 | 06:58 |
| *** matbu__ is now known as matbu | 07:03 | |
| ramishra_ | amoralej: Hey! Should we bump openstacksdk to 0.102.0 in @delorean-aoc-temp for testing? | 07:07 |
| ramishra_ | https://github.com/openstack/ansible-collections-openstack/commit/0ade33eb6fc22cd86ccbc289f40a668cf5281f8e | 07:07 |
| amoralej | lemme check something | 07:09 |
| ramishra_ | I don't see zed bumped for it though https://github.com/openstack/releases/blob/master/deliverables/zed/openstacksdk.yaml | 07:09 |
| amoralej | yeah, that's what i'm checking ... | 07:09 |
| ramishra_ | most of the failures in https://review.opendev.org/c/openstack/tripleo-heat-templates/+/859205 are for that | 07:10 |
| amoralej | 0.102 is antelope release | 07:10 |
| amoralej | not zed | 07:10 |
| amoralej | jm1[m], ^ | 07:10 |
| ramishra_ | https://f13b22c11585c38f16e5-3e7bc19bdc58c96c3f64630625f9105c.ssl.cf2.rackcdn.com/859205/4/check/tripleo-ci-centos-9-scenario010-standalone/794fd8b/logs/quickstart_install.log | 07:10 |
| amoralej | for zed, it needs to support 0.101.0 | 07:10 |
| amoralej | if there is some bug qith 101 the fix should be cherrypicked to zed | 07:11 |
| ramishra_ | right, we use master for galaxy install in your test patch https://review.opendev.org/c/openstack/tripleo-quickstart/+/861170 | 07:11 |
| ramishra_ | so it's failing | 07:12 |
| ramishra_ | that all scenario test patch depends-on your patch | 07:12 |
| amoralej | ramishra_, as jm1 last news they still need some patches to land in a-c-o | 07:15 |
| amoralej | let's wait until a-c-o is finished | 07:16 |
| ramishra_ | amoralej: not from tripleo pov, just that they want to get some patches in | 07:16 |
| ramishra_ | which they have been working on | 07:16 |
| amoralej | jm1[m], ^ is this correct? | 07:16 |
| ramishra_ | they're not important for tripleo for zed | 07:17 |
| amoralej | i'm confused now, tbh | 07:17 |
| amoralej | <jm1> amoralej: 8 patches if you need a number ;) | 07:17 |
| amoralej | <jm1> amoralej: might sound worse that it is. we will be able to merge them next week | 07:17 |
| amoralej | that's from friday | 07:18 |
| ramishra_ | I discussed with jm1 and we agreed on that last week.. he would get some patches in before the pre-release but they're not imprtant for tripleo | 07:18 |
| amoralej | then, i'd say we should focus in bump in rdo as soon as posible | 07:21 |
| ramishra_ | amoralej: agreed | 07:21 |
| rdogerrit | Merged rdoinfo master: Promote CBS tags update for yoga-9s-testing https://review.rdoproject.org/r/c/rdoinfo/+/45672 | 07:21 |
| amoralej | and avoid depending on antelope :) | 07:22 |
| rdogerrit | Merged rdoinfo master: Promote CBS tags update for yoga-8s-testing https://review.rdoproject.org/r/c/rdoinfo/+/45673 | 07:22 |
| rdogerrit | Merged rdoinfo master: Promote CBS tags update for xena-8s-testing https://review.rdoproject.org/r/c/rdoinfo/+/45674 | 07:23 |
| rdogerrit | Joel Capitao proposed gating_scripts master: Get MASTER_RELEASE from rdoinfo https://review.rdoproject.org/r/c/gating_scripts/+/45644 | 07:25 |
| *** jpena|off is now known as jpena | 07:30 | |
| jm1 | amoralej, ramishra_: o/ from my pov, on friday we agreed on getting the aoc patches merged until end of this week and then prepare rdo's aoc prerelease based on that. for example, one module update which is not yet in aoc master branch but used by tripleo (not quickstart) is subnets_info. | 07:34 |
| rdogerrit | Joel Capitao proposed gating_scripts master: Get MASTER_RELEASE from rdoinfo https://review.rdoproject.org/r/c/gating_scripts/+/45644 | 07:34 |
| amoralej | jm1, wrt openstacksdk version, see the comments before | 07:35 |
| amoralej | 0.102.0 is antelope | 07:35 |
| ramishra_ | jm1: we merged https://review.opendev.org/c/openstack/tripleo-heat-templates/+/861175 | 07:35 |
| amoralej | we shouldn't depend on it in zed | 07:35 |
| ramishra_ | so that subnet_info dependency us not there | 07:35 |
| ramishra_ | anymore | 07:35 |
| rdogerrit | Joel Capitao proposed rdoinfo master: DNM Test 45645 patch https://review.rdoproject.org/r/c/rdoinfo/+/45647 | 07:35 |
| ramishra_ | so tripleo is kind of ready for the bump, though I leave it to you guys when you want to do it | 07:37 |
| rdogerrit | Joel Capitao proposed gating_scripts master: DNM bypass -testing tag https://review.rdoproject.org/r/c/gating_scripts/+/45649 | 07:37 |
| jm1 | ramishra_: looks like you are trying hard to remove deps to aoc in tripleo :D | 07:38 |
| ramishra_ | and not create blockers for release;) | 07:38 |
| amoralej | i'm fine with both wait or do it now as soon as we have enough time before tripleo release | 07:38 |
| amoralej | and not crete dependencies on antelope releases :) | 07:39 |
| jm1 | ramishra_: do we have a testproject which shows that tq and tripleo are really decoupled as we hoped on friday? | 07:39 |
| jm1 | amoralej: wrt to openstacksdk, we actually want to have the latest openstacksdk in zed because we are constantly fixing issues in sdk's master branch which affect aoc | 07:40 |
| amoralej | latest openstacksdk in zed is 101 | 07:41 |
| ramishra_ | jm1: they're decoupled.. we also merged https://review.opendev.org/c/openstack/python-tripleoclient/+/861354 | 07:41 |
| amoralej | 102 is first in antelope | 07:41 |
| ramishra_ | https://review.opendev.org/c/openstack/tripleo-heat-templates/+/859205 failures are only because you bumped to 0.102.0 in aco | 07:41 |
| amoralej | if there are fixes in antelope required in zed, you can request backport to stable/Zed branch | 07:41 |
| ramishra_ | sdk 0.102.0 | 07:41 |
| jm1 | amoralej: yeah we had to put it into antelope because we were late for zed. but since we are not done with aoc yet we also havent fixed all issues in sdk yet.. | 07:42 |
| amoralej | so, my point is, anything that we plan to use with zed, must support openstacksdk (and others) from zed | 07:42 |
| jm1 | amoralej: we can try with 0.101.0 for now and do this backporting later. 0.102.0 is only marginally better than 0.101.0 anyway | 07:43 |
| *** ysandeep is now known as ysandeep|lunch | 07:43 | |
| ramishra_ | I agree, we should not mix too many things and make zed release more complicated | 07:43 |
| jm1 | amoralej: in theory i agree but with this broken openstacksdk release in zed its not that black and white :/ | 07:44 |
| jm1 | amoralej: anyway, lets stick to 0.101.0 for now | 07:44 |
| amoralej | jm1, do you want me to propose revert of https://review.opendev.org/c/openstack/ansible-collections-openstack/+/861329 ? | 07:45 |
| jm1 | amoralej: nope, use the commit just before that for the prerelease | 07:45 |
| amoralej | jm1, and, if we need a patch to fix something later? | 07:46 |
| amoralej | note stable/zed is going to be a supported release in tripleo | 07:46 |
| amoralej | i'm fine to do that if you want | 07:46 |
| amoralej | but thinking in a zed support, it can be problematic | 07:46 |
| jm1 | amoralej: there is a reason for this bump to 0.102.0. aoc 2.0.0 will definitely need something newer than 0.101.0. again, aoc is still wip and sdk is as well. | 07:47 |
| amoralej | jm1 i understand but then, it should be discussed and fixed in openstacksdk, not in rdo | 07:48 |
| ramishra_ | jm1: and we we use ACO 1.10.0 from galaxy for zed in tripleo-quickstart? | 07:48 |
| jm1 | amoralej: yes, i got your point, we should do a backport of sdk in upstream zed. so for now we can stick to sdk 0.101.0 in rdo zed | 07:49 |
| amoralej | ok, so, iiuc. We'll pin to the commit right before to the 102 bump in a-c-o + openstacksdk-101 | 07:49 |
| amoralej | 0.101 i mean | 07:49 |
| jm1 | amoralej: yes please :) | 07:50 |
| amoralej | ok, wfm | 07:50 |
| jm1 | amoralej: that is what ramishra_ prefers iiuc :) | 07:50 |
| amoralej | then, we need to fix the galaxy part | 07:52 |
| jm1 | amoralej: nope, we dont have to. ramishra_ said that quickstart's aoc is decoupled from tripleo's aoc once the patches he mentioned above have been merged. so we should be fine soon.. | 07:53 |
| jm1 | amoralej: although it would be nice to have a testproject which proofs ramishra_'s point. just to be sure :D | 07:53 |
| amoralej | but we should pin to the same commit in the galaxy installation, shouldn't we? | 07:54 |
| ramishra_ | jm1: it has to be fixed for qucikstart AOC though, you can't use master AOC with sdk 0.101.0 with that patch | 07:54 |
| ramishra_ | amoralej: right | 07:55 |
| jm1 | ramishra_: so tq is still strongly coupled with tripleo through its sdk dependency? cant we drop this dep as well? | 07:56 |
| ramishra_ | jm1: nope | 07:56 |
| jm1 | ramishra_, amoralej: then we need to implement this aoc versioning conditional in tq | 07:58 |
| amoralej | yes, that's my understanding | 07:58 |
| amoralej | jm1, so you want to create one out of my DNM patch that can be merged first? | 07:58 |
| amoralej | and then update in rdo | 07:58 |
| jm1 | amoralej: sure, although i dont know yet how to merge those changes at the same time :/ | 07:59 |
| amoralej | something that we could do is | 07:59 |
| amoralej | in the same commit in tq get the conditional for galaxy + merge the temporary aoc repo with 0.101.0 + your desired commit | 08:00 |
| amoralej | that's 1 | 08:00 |
| amoralej | 2. bump openstacksdk + a-o-c in rdo to the same versions/commit | 08:01 |
| amoralej | 3. remove the temporary repo | 08:01 |
| amoralej | that'd be the way to do everything in one shot | 08:01 |
| amoralej | mmmm | 08:01 |
| amoralej | i think that should work | 08:02 |
| jm1 | amoralej: ok lets try that :) | 08:02 |
| jm1 | amoralej: do we have to wait for promotions between 2 and 3? | 08:03 |
| amoralej | that's what i was thinking about | 08:03 |
| amoralej | yes | 08:03 |
| amoralej | we need to | 08:03 |
| amoralej | btw, we have the desired a-o-c hash already in the temp repo https://trunk.rdoproject.org/centos9-master/aoc-temp/ | 08:04 |
| jm1 | amoralej: great :) | 08:04 |
| amoralej | ed36d82 | 08:04 |
| jm1 | ramishra_: do you know where openstacksdk is installed during tq runs? its not in quickstart's requirements.txt ?!? | 08:04 |
| amoralej | jm1, will you take care of the tq patch? you can take mine as starting point | 08:05 |
| jm1 | amoralej: yes, will update it | 08:05 |
| amoralej | so, i think we have a plan :) | 08:05 |
| amoralej | jm1, ramishra_ ^ i think we all are in the same page now | 08:06 |
| rdogerrit | Joel Capitao proposed gating_scripts master: Get MASTER_RELEASE from rdoinfo https://review.rdoproject.org/r/c/gating_scripts/+/45644 | 08:14 |
| rdogerrit | Joel Capitao proposed gating_scripts master: DNM bypass -testing tag https://review.rdoproject.org/r/c/gating_scripts/+/45649 | 08:15 |
| rdogerrit | Joel Capitao proposed rdoinfo master: DNM Test 45645 patch https://review.rdoproject.org/r/c/rdoinfo/+/45647 | 08:16 |
| rdogerrit | Joel Capitao proposed rdo-jobs master: Get MASTER release from rdoinfo https://review.rdoproject.org/r/c/rdo-jobs/+/45645 | 08:25 |
| rdogerrit | Joel Capitao proposed rdo-jobs master: Get MASTER release from rdoinfo https://review.rdoproject.org/r/c/rdo-jobs/+/45645 | 08:34 |
| rdogerrit | Joel Capitao proposed rdo-jobs master: Get MASTER release from rdoinfo https://review.rdoproject.org/r/c/rdo-jobs/+/45645 | 08:57 |
| *** ysandeep|lunch is now known as ysandeep | 09:54 | |
| slaweq | Hi rdo team, can You review https://review.rdoproject.org/r/c/rdoinfo/+/45211 ? | 09:55 |
| slaweq | thx in advance | 09:55 |
| amoralej | slaweq, there are some comments that need to be addressed | 10:36 |
| amoralej | need to add upstream field | 10:37 |
| slaweq | amoralej ahh, ok, thx | 10:37 |
| slaweq | I probably missed them | 10:37 |
| amoralej | and about maintainer mail addresses | 10:37 |
| *** ysandeep is now known as ysandeep|afk | 10:46 | |
| rdogerrit | Merged rdoinfo master: Bump rdoinfo wallaby tags to latest upper-constraints https://review.rdoproject.org/r/c/rdoinfo/+/45670 | 11:13 |
| *** ysandeep|afk is now known as ysandeep | 11:14 | |
| rdogerrit | Merged rdoinfo master: Bump rdoinfo zed tags to latest upper-constraints https://review.rdoproject.org/r/c/rdoinfo/+/45671 | 11:14 |
| rdogerrit | rdo-trunk created openstack/stevedore-distgit wallaby-rdo: python-stevedore-3.3.2-1 https://review.rdoproject.org/r/c/openstack/stevedore-distgit/+/45676 | 11:19 |
| rdogerrit | rdo-trunk created openstack/stevedore-distgit zed-rdo: python-stevedore-4.0.1-1 https://review.rdoproject.org/r/c/openstack/stevedore-distgit/+/45677 | 11:20 |
| rdogerrit | Merged openstack/stevedore-distgit zed-rdo: python-stevedore-4.0.1-1 https://review.rdoproject.org/r/c/openstack/stevedore-distgit/+/45677 | 11:33 |
| rdogerrit | Merged openstack/stevedore-distgit wallaby-rdo: python-stevedore-3.3.2-1 https://review.rdoproject.org/r/c/openstack/stevedore-distgit/+/45676 | 11:39 |
| rdogerrit | Merged rdo-jobs master: Ignore TripleO jobs for other centos versions in CBS updates https://review.rdoproject.org/r/c/rdo-jobs/+/45656 | 11:57 |
| *** amoralej is now known as amoralej|lunch | 12:05 | |
| rdogerrit | Merged nfvinfo master: Promote CBS tags update for nfvsig-2.16-testing https://review.rdoproject.org/r/c/nfvinfo/+/45174 | 12:11 |
| *** jpena is now known as jpena|off | 12:14 | |
| *** dviroel_ is now known as dviroel | 12:24 | |
| *** jpena|off is now known as jpena | 12:24 | |
| *** amoralej|lunch is now known as amoralej | 13:05 | |
| rdogerrit | Alfredo Moralejo created nfvinfo master: DNM run -release update https://review.rdoproject.org/r/c/nfvinfo/+/45678 | 13:28 |
| *** dasm|off is now known as dasm | 13:50 | |
| rdogerrit | Alfredo Moralejo proposed rdoinfo master: DNM debug weirdo-zed-promote-packstack-scenario003-centos-stream-9 job https://review.rdoproject.org/r/c/rdoinfo/+/44470 | 14:39 |
| rdogerrit | Merged rdo-jobs master: Adds OVB periodic integration jobs for zed https://review.rdoproject.org/r/c/rdo-jobs/+/45537 | 15:02 |
| rdogerrit | evallesp created rdo-infra/ansible-role-dlrn master: Add run-user.sh script. https://review.rdoproject.org/r/c/rdo-infra/ansible-role-dlrn/+/45679 | 15:12 |
| opendevreview | Alfredo Moralejo proposed x/packstack master: Start redis before gnocchi-metricd when coordination uses redis https://review.opendev.org/c/x/packstack/+/861642 | 15:19 |
| amoralej | jcapitao[m], tkajinam ^ that's blocking promotions in zed and master | 15:24 |
| *** dviroel is now known as dviroel|lunch | 15:34 | |
| tkajinam | ugh | 15:38 |
| rdogerrit | Arx Cruz created rdo-infra/ci-config master: Update pull registry https://review.rdoproject.org/r/c/rdo-infra/ci-config/+/45680 | 15:58 |
| rdogerrit | Merged rdo-infra/ci-config master: Update pull registry https://review.rdoproject.org/r/c/rdo-infra/ci-config/+/45680 | 16:23 |
| *** ysandeep is now known as ysandeep|out | 16:26 | |
| *** jpena is now known as jpena|off | 16:27 | |
| *** dviroel|lunch is now known as dviroel | 16:27 | |
| *** marios is now known as marios|out | 16:28 | |
| *** marios|out is now known as marios | 16:29 | |
| *** marios is now known as marios|out | 16:30 | |
| *** amoralej is now known as amoralej|off | 16:50 | |
| *** dviroel is now known as dviroel|out | 21:31 | |
| *** dasm is now known as dasm|off | 22:41 | |
| opendevreview | Merged x/packstack master: Start redis before gnocchi-metricd when coordination uses redis https://review.opendev.org/c/x/packstack/+/861642 | 23:38 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!