Tuesday, 2022-10-18

rdogerritrdo-trunk created nfvinfo master: Promote CBS tags update for nfvsig-openvswitch2.17-release  https://review.rdoproject.org/r/c/nfvinfo/+/4568100:07
rdogerritrdo-trunk created rdoinfo master: Promote CBS tags update for yoga-9s-release  https://review.rdoproject.org/r/c/rdoinfo/+/4568200:45
rdogerritrdo-trunk created rdoinfo master: Promote CBS tags update for yoga-8s-release  https://review.rdoproject.org/r/c/rdoinfo/+/4568300:46
rdogerritrdo-trunk created rdoinfo master: Promote CBS tags update for xena-8s-release  https://review.rdoproject.org/r/c/rdoinfo/+/4568400:47
opendevreviewTakashi Kajinami proposed x/packstack master: Ceilometer: Use built-in class to run ceilometer-upgrade  https://review.opendev.org/c/x/packstack/+/86168301:37
opendevreviewTakashi Kajinami proposed x/packstack master: Ceilometer: Remove redundant installation of pythonN-redis  https://review.opendev.org/c/x/packstack/+/86168401:39
opendevreviewTakashi Kajinami proposed x/packstack master: Drop firewall rule for ceilometer-api  https://review.opendev.org/c/x/packstack/+/86168501:43
opendevreviewTakashi Kajinami proposed x/packstack master: Drop firewall rule for ceilometer-api  https://review.opendev.org/c/x/packstack/+/86168501:44
opendevreviewTakashi Kajinami proposed x/packstack master: Ceilometer: Use built-in class to run ceilometer-upgrade  https://review.opendev.org/c/x/packstack/+/86168302:48
*** ysandeep|out is now known as ysandeep05:47
*** ramishra_ is now known as ramishra06:38
rdogerritMerged rdoinfo master: Promote CBS tags update for yoga-8s-release  https://review.rdoproject.org/r/c/rdoinfo/+/4568307:09
rdogerritMerged rdoinfo master: Promote CBS tags update for xena-8s-release  https://review.rdoproject.org/r/c/rdoinfo/+/4568407:09
rdogerritJoel Capitao proposed rdo-jobs master: Get MASTER release from rdoinfo  https://review.rdoproject.org/r/c/rdo-jobs/+/4564507:11
rdogerritMerged nfvinfo master: Promote CBS tags update for nfvsig-openvswitch2.17-release  https://review.rdoproject.org/r/c/nfvinfo/+/4568107:12
*** jpena|off is now known as jpena07:17
*** ysandeep is now known as ysandeep|lunch08:16
rdogerritJoel Capitao proposed rdo-jobs master: Get MASTER release from rdoinfo  https://review.rdoproject.org/r/c/rdo-jobs/+/4564508:50
*** marios is now known as marios|call09:00
*** jpodivin__ is now known as jpodivin09:33
*** amoralej|off is now known as amoralej10:21
*** ysandeep|lunch is now known as ysandeep10:37
mharleyHi, team.  I need some guidance on what (if so) to do in this review:  https://review.rdoproject.org/r/c/openstack/barbican-distgit/+/45622.11:07
mharleyThe failing Zuul job is of non-voting.11:07
amoralejmharley, the problemi in that job is related to the minimal version of keystoneclient n the spec11:11
amoralejit's unrelated to your review11:12
amoralejMISMATCH:11:12
amoralejpython-keystonemiddleware >= 9.5.0  (>= 5.1.0 in .spec)11:12
amoralejhttps://github.com/rdo-packages/barbican-distgit/blob/rpm-master/openstack-barbican.spec#L77 11:13
amoralejvs https://github.com/openstack/barbican/blob/master/requirements.txt#L2811:13
rdogerritAlfredo Moralejo proposed openstack/barbican-distgit rpm-master: Fixed typo in openstack-barbican.spec  https://review.rdoproject.org/r/c/openstack/barbican-distgit/+/4562211:13
rdogerritMerged openstack/barbican-distgit wallaby-rdo: Make sure common and python lib is upgraded  https://review.rdoproject.org/r/c/openstack/barbican-distgit/+/4564211:22
mharleyI saw it, @amoralej.  Just wanted to know if I need to change the corresponding requirement needed to be changed.  Thanks.11:23
amoralejmharley, if you can send a separated review for that, it'd be great!!!11:29
rdogerritSławek Kapłoński proposed rdoinfo master: Add whitebox-neutron-tempest-plugin project  https://review.rdoproject.org/r/c/rdoinfo/+/4521111:30
*** marios|call is now known as marios11:33
*** dviroel|out is now known as dviroel11:41
mharleyOk, but leveraging I just started working with RDO issues, what should be done in this case?  To change the distgist's spec file to reflect the upstream's value?11:44
mharley@amoralej11:44
amoralejmharley, yes, that's it. the requirements in the spec should reflect upstream requirements as close as possible11:45
amoralejthere are some very particular exceptions11:45
amoralejbut that's the goal11:45
mharleyAck.  Thank you.12:04
*** ysandeep is now known as ysandeep|away12:20
amoralejtkajinam, my patch https://review.opendev.org/c/x/packstack/+/861642/ didn't help with the order12:28
amoralejredis is still starting at the end12:28
amoralejhttps://logserver.rdoproject.org/openstack-periodic-integration-stable4/review.rdoproject.org/rdoinfo/master/weirdo-zed-promote-packstack-scenario003-centos-stream-9/0631f10/logs/weirdo-project/logs/latest/manifests/38.102.83.49_controller.pp.log.txt.gz12:28
jm1amoralej, ramishra: hey :) we probably have two options to proceed with aoc in zed. option (a) is as discussed, use same aoc version in tq/tqe as in tripleo deployment. option (b) is to completely decouple both versions. 12:34
jm1amoralej, ramishra: (a) https://review.opendev.org/c/openstack/tripleo-quickstart/+/86158612:35
jm1amoralej, ramishra: (b) proof of concept https://review.opendev.org/c/openstack/tripleo-quickstart/+/86158912:36
jm1amoralej, ramishra: i would go for both ;)12:36
jm1rcastillo: ^12:38
*** amoralej is now known as amoralej|lunch12:40
rdogerritMerged openstack/barbican-distgit rpm-master: Fixed typo in openstack-barbican.spec  https://review.rdoproject.org/r/c/openstack/barbican-distgit/+/4562212:47
rdogerritJoel Capitao created openstack/barbican-distgit zed-rdo: Fixed typo in openstack-barbican.spec  https://review.rdoproject.org/r/c/openstack/barbican-distgit/+/4568512:47
rdogerritJoel Capitao created openstack/barbican-distgit yoga-rdo: Fixed typo in openstack-barbican.spec  https://review.rdoproject.org/r/c/openstack/barbican-distgit/+/4568612:51
rdogerritJoel Capitao created openstack/barbican-distgit xena-rdo: Fixed typo in openstack-barbican.spec  https://review.rdoproject.org/r/c/openstack/barbican-distgit/+/4568712:54
rdogerritJoel Capitao created openstack/barbican-distgit wallaby-rdo: Fixed typo in openstack-barbican.spec  https://review.rdoproject.org/r/c/openstack/barbican-distgit/+/4568812:57
*** geguileor is now known as geguileo13:00
*** amoralej|lunch is now known as amoralej13:09
opendevreviewAlfredo Moralejo proposed x/packstack master: Fix order for redis and dependant services  https://review.opendev.org/c/x/packstack/+/86175313:13
opendevreviewTakashi Kajinami proposed x/packstack master: Fix dependency between redis and gnocchi services  https://review.opendev.org/c/x/packstack/+/86175713:20
tkajinamamoralej, I guess this is what we need ^^^13:27
amoralejthanks tkajinam, i'll check the ci logs and compare with https://review.opendev.org/c/x/packstack/+/86175313:28
tkajinam46727:Notice: /Stage[main]/Gnocchi::Metricd/Service[gnocchi-metricd]/ensure: ensure changed 'stopped' to 'running'13:29
tkajinam49102:Notice: /Stage[main]/Redis::Service/Service[redis]/ensure: ensure changed 'stopped' to 'running'13:29
tkajinamyeah seems the order is not at all enforced13:30
amoralejyep13:31
*** dasm|off is now known as dasm13:49
amoralejtkajinam, your patch is working fine14:09
amoraleji'll recheck, another job failed with timeout but it's unrelated14:10
*** ysandeep|away is now known as ysandeep14:31
*** jjoyce|away is now known as jjoyce14:40
*** dviroel is now known as dviroel|dr_appt14:52
opendevreviewAlfredo Moralejo proposed x/packstack master: Establish order between redis and aodh and ceilometer  https://review.opendev.org/c/x/packstack/+/86175315:05
opendevreviewMerged x/packstack master: Ceilometer: Use built-in class to run ceilometer-upgrade  https://review.opendev.org/c/x/packstack/+/86168315:07
*** ysandeep is now known as ysandeep|out15:44
*** marios is now known as marios|out16:02
*** jpena is now known as jpena|off16:31
*** dviroel|dr_appt is now known as dviroel16:49
opendevreviewMerged x/packstack master: Drop firewall rule for ceilometer-api  https://review.opendev.org/c/x/packstack/+/86168516:51
*** amoralej is now known as amoralej|off17:15
opendevreviewMerged x/packstack master: Fix dependency between redis and gnocchi services  https://review.opendev.org/c/x/packstack/+/86175717:21
rdogerritDariusz proposed rdo-infra/ci-config master: Separate configure_users from base role  https://review.rdoproject.org/r/c/rdo-infra/ci-config/+/4566320:13
rdogerritDariusz proposed rdo-infra/ci-config master: Separate configure_journalctl from base role  https://review.rdoproject.org/r/c/rdo-infra/ci-config/+/4566420:13
rdogerritDariusz proposed rdo-infra/ci-config master: Separate configuration of delivery script  https://review.rdoproject.org/r/c/rdo-infra/ci-config/+/4566520:13
rdogerritDariusz proposed rdo-infra/ci-config master: Separate SSH configuration from base role  https://review.rdoproject.org/r/c/rdo-infra/ci-config/+/4566620:13
rdogerritDariusz proposed rdo-infra/ci-config master: Keep teardown playbooks in one place  https://review.rdoproject.org/r/c/rdo-infra/ci-config/+/4566720:13
rdogerritDariusz proposed rdo-infra/ci-config master: Moved cloud configuration to defaults for roles  https://review.rdoproject.org/r/c/rdo-infra/ci-config/+/4566920:13
rdogerritDariusz proposed rdo-infra/ci-config master: Separate configure_users from base role  https://review.rdoproject.org/r/c/rdo-infra/ci-config/+/4566320:32
rdogerritDariusz proposed rdo-infra/ci-config master: Separate configure_journalctl from base role  https://review.rdoproject.org/r/c/rdo-infra/ci-config/+/4566420:32
rdogerritDariusz proposed rdo-infra/ci-config master: Separate configuration of delivery script  https://review.rdoproject.org/r/c/rdo-infra/ci-config/+/4566520:32
rdogerritDariusz proposed rdo-infra/ci-config master: Separate SSH configuration from base role  https://review.rdoproject.org/r/c/rdo-infra/ci-config/+/4566620:32
rdogerritDariusz proposed rdo-infra/ci-config master: Keep teardown playbooks in one place  https://review.rdoproject.org/r/c/rdo-infra/ci-config/+/4566720:32
rdogerritDariusz proposed rdo-infra/ci-config master: Moved cloud configuration to defaults for roles  https://review.rdoproject.org/r/c/rdo-infra/ci-config/+/4566920:32
*** dasm is now known as dasm|off22:15

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