Friday, 2022-03-18

opendevreviewMerged openstack/tripleo-ansible master: bnr - Enable encrypted backups  https://review.opendev.org/c/openstack/tripleo-ansible/+/83278500:05
opendevreviewMerged openstack/tripleo-heat-templates master: Add HostnameFormatDefault for Ceph related roles  https://review.opendev.org/c/openstack/tripleo-heat-templates/+/83395600:05
opendevreviewMerged openstack/tripleo-ansible master: Move creating empty ruleset for deployed server  https://review.opendev.org/c/openstack/tripleo-ansible/+/83411801:21
*** pmannidi|Away is now known as pmannidi01:46
opendevreviewTakashi Kajinami proposed openstack/puppet-tripleo stable/wallaby: Format [keystone_authtoken] memcache_servers when IPv6 is used  https://review.opendev.org/c/openstack/puppet-tripleo/+/83414203:03
opendevreviewTakashi Kajinami proposed openstack/puppet-tripleo stable/wallaby: Format [oslo_cache] memcache_server when IPv6 is used  https://review.opendev.org/c/openstack/puppet-tripleo/+/83414303:03
*** chem is now known as Guest245404:46
opendevreviewMerged openstack/tripleo-ansible stable/wallaby: Creating a default molecule config  https://review.opendev.org/c/openstack/tripleo-ansible/+/83058905:55
opendevreviewChristoph Dwertmann proposed openstack/tripleo-ansible master: Prevent DB backup failure when multiple "galera" containers are present  https://review.opendev.org/c/openstack/tripleo-ansible/+/83349506:49
opendevreviewLuis Tomas Bolivar proposed openstack/tripleo-ansible stable/wallaby: OVN BGP Agent support  https://review.opendev.org/c/openstack/tripleo-ansible/+/83414406:57
opendevreviewMikolaj Ciecierski proposed openstack/tripleo-upgrade stable/ussuri: Run control plane update first, next compute and cephstorage last  https://review.opendev.org/c/openstack/tripleo-upgrade/+/83414507:42
opendevreviewMikolaj Ciecierski proposed openstack/tripleo-upgrade stable/victoria: Run control plane update first, next compute and cephstorage last  https://review.opendev.org/c/openstack/tripleo-upgrade/+/83414607:42
*** arxcruz is now known as arxcruz|rover07:47
*** jpena|off is now known as jpena07:48
opendevreviewDamien Ciabrini proposed openstack/tripleo-common master: Generate database URI for ed25519 passwords  https://review.opendev.org/c/openstack/tripleo-common/+/83361508:12
*** arxcruz|rover is now known as arxcruz|ruck09:00
opendevreviewFrancesco Pantano proposed openstack/tripleo-heat-templates stable/wallaby: Add HostnameFormatDefault for Ceph related roles  https://review.opendev.org/c/openstack/tripleo-heat-templates/+/83414809:37
*** rlandy|PTO is now known as rlandy10:26
opendevreviewBogdan Dobrelya proposed openstack/tripleo-heat-templates master: Add extra debugging from the cache backend  https://review.opendev.org/c/openstack/tripleo-heat-templates/+/83250610:54
*** dviroel|afk is now known as dviroel10:58
opendevreviewMerged openstack/python-tripleoclient master: bnr - Set properly the default inventory file  https://review.opendev.org/c/openstack/python-tripleoclient/+/83032111:00
opendevreviewBogdan Dobrelya proposed openstack/tripleo-heat-templates master: [DNM] testing log_global_request_id  https://review.opendev.org/c/openstack/tripleo-heat-templates/+/83427511:08
opendevreviewJiri Podivin proposed openstack/validations-libs master: Exceptions for Validations Framework  https://review.opendev.org/c/openstack/validations-libs/+/83427711:31
fultonjjbadiapa: 'dict object' has no attribute 'BACKUP_PROG_CRYPT_ENABLED'\n\nThe error appears to be in '/usr/share/ansible/roles/backup_and_restore/tasks/setup_rear.yml 11:39
fultonjhttps://fc1117959d9ea26fc99f-fc178b78e746168ff9a6f79c77a7e799.ssl.cf2.rackcdn.com/832701/2/gate/tripleo-ci-centos-9-scenario001-standalone/2f02c0e/logs/undercloud/home/zuul/ansible.log11:39
fultonjhttps://github.com/openstack/tripleo-ansible/commit/77cb42cec73a669157f35167257214ca57bd190311:39
fultonjhttps://review.opendev.org/c/openstack/tripleo-ansible/+/83278511:40
fultonjhttps://bugs.launchpad.net/tripleo/+bug/196552811:46
fultonjrlandy: ^ fyi11:46
rlandyarxcruz|ruck: ^^11:46
fultonjLewis_: 11:47
fultonjLewis_: sorry typo11:47
arxcruz|ruckrlandy ack added on hackmd 11:48
fultonji'll try a fix, it looks easy enough11:49
arxcruz|ruckfultonj great :) let me know when you finsh the patch, so i can add myself and trackdown 11:49
opendevreviewJohn Fulton proposed openstack/tripleo-ansible master: Do not reference backup/restore config unless defined  https://review.opendev.org/c/openstack/tripleo-ansible/+/83427911:53
fultonjarxcruz|ruck: ^ let's try that11:54
rlandyfultonj++ thanks for quick fix11:55
fultonjno prob11:56
rlandyfultonj: marios is submitting a DNM tets patch to run scenario001 with your change11:56
opendevreviewMarios Andreou proposed openstack/tripleo-heat-templates master: DNM Testing tripleo-ansible/+/834279  https://review.opendev.org/c/openstack/tripleo-heat-templates/+/83428011:57
mariosfultonj: ^^ there 11:57
fultonjYEEEE HAAAAWWW""!!!11:57
fultonj:)11:57
mariosfultonj: ;)11:58
opendevreviewDavid Vallee Delisle proposed openstack/tripleo-ansible stable/wallaby: Adding tripleo_kernel_hugepages  https://review.opendev.org/c/openstack/tripleo-ansible/+/83086911:59
opendevreviewDavid Vallee Delisle proposed openstack/tripleo-ansible stable/wallaby: autodoc shouldn't assume there's a converge playbook  https://review.opendev.org/c/openstack/tripleo-ansible/+/83141512:00
rlandytkajinam: hello ... can you take a look at https://bugs.launchpad.net/tripleo/+bug/1965525 - noticed you were doing some ipv6 work with keystone12:00
rlandypossibly related? https://opendev.org/openstack/puppet-keystone/commit/c2525c6e63c236b470bd935496875545f91e2b2612:00
opendevreviewMarios Andreou proposed openstack/tripleo-heat-templates master: DNM Testing tripleo-ansible/+/834279  https://review.opendev.org/c/openstack/tripleo-heat-templates/+/83428012:02
marios(just restricted to only run standalones fultonj ^^ ) 12:02
fultonjmarios: ah that's smart12:03
fultonjI'm going to borrow that pattern12:03
fultonjbogdando: why Code-Review removed for https://review.opendev.org/c/openstack/tripleo-heat-templates/+/833164 ?12:35
hjensastkajinam: I guess we want to backport the three changes that merged on https://review.opendev.org/q/topic:bug%252F1964824 ?12:50
*** dhill is now known as Guest248112:50
bogdandofultonj: I didn't spot a reply to James... restoring it back12:54
fultonjbogdando: ok, thanks for reviewing it12:54
fultonji'm reviewing it now12:54
bogdandoI still didn't get the "reversed" existing/new logic12:55
bogdandobut as Alan says he tested it, LGTM then12:55
fultonjthe new/existing logic he has makes sense to me12:55
fultonji'm just just reading that shell script more closely to make sure i understand12:56
*** TheJulia is now known as needsleep12:57
*** needsleep is now known as TheJulia12:57
bogdandofultonj, abishop: FTR, in the past there had been another example of similar thing I dealt to https://github.com/kubernetes-sigs/kubespray/commit/0500f27db85405caf9707dce1078ac1055cfcdaf#diff-f2d6c041222aaad478d106c62fdbf8c2dc1237d484143318831447344c1c32d0R513:00
bogdandoand I've tried to recall details out of it...13:00
fultonjok, thanks13:00
bogdandothat's on what my comment was based, including the healthy check and new/existing/inital states13:01
bogdandoin your patch I see the logic kinda reversed13:01
bogdandomay be that's me not getting the case13:01
fultonjdefaulting to existing precludes being able to bootstrap the cluster13:05
* fultonj quoting abishop13:05
fultonjfirst time this code is run is for bootstrap13:05
fultonjthen as long as we're idempotent we're fine13:05
fultonjfollowing that, it's set to new on scale up wich an existing cluster membership13:06
fultonjwhops13:06
fultonjfollowing that, it's set to EXISTING on scale up to add new members13:06
fultonjeither bootstrap idempotently13:06
fultonjor add new members because it already exists13:06
fultonji think the labels have diff context in your patch13:08
fultonjbogdando: 13:08
opendevreviewMerged openstack/tripleo-heat-templates stable/wallaby: Add missing option for rgw/swift compatibility  https://review.opendev.org/c/openstack/tripleo-heat-templates/+/83367913:16
arxcruz|rucktkajinam hey, around:13:21
arxcruz|rucktkajinam wondering if you can take a look at https://bugs.launchpad.net/tripleo/+bug/196552513:21
arxcruz|rucktkajinam it seems related to your patch https://opendev.org/openstack/tripleo-heat-templates/commit/6f8a5bc1ef16f9df5653c36bcf351026a49ea913 13:21
*** dasm|off is now known as dasm13:22
jpodivinarxcruz|ruck: Hi, regarding the https://bugs.launchpad.net/tripleo/+bug/1965426 we have couple of patches in the RDO which should address the issue. 13:23
rlandyarxcruz|ruck: I pinged above with the same13:24
rlandycloudnull: ^^ if tkajinam id not around, can you take a look13:24
arxcruz|ruckjpodivin dobre13:24
arxcruz|ruckrlandy ack, sorry, did not saw it 13:24
rlandynp13:25
jpodivinarxcruz|ruck: Jeste nejsou schvalene :(13:25
opendevreviewLukas Bezdicka proposed openstack/tripleo-heat-templates stable/train: [train-only] Use state file for workaround 1925078  https://review.opendev.org/c/openstack/tripleo-heat-templates/+/83428513:25
arxcruz|ruckjpodivin to je skoda 13:29
bogdandofultonj: right, if ran only for bootstrap, new state should be used13:36
opendevreviewMarios Andreou proposed openstack/tripleo-ci master: Centos8 wallaby jobs teardown - undercloud jobs  https://review.opendev.org/c/openstack/tripleo-ci/+/83428713:37
bogdandoI think the linked patch says the same13:37
*** TheJulia is now known as needssleep13:38
bogdandoand new members added to initial cluster state existing, right13:38
tkajinamarxcruz|ruck, that patch not related at all. the change itself has no effect unless the change in puppet-triple is present.13:43
tkajinamarxcruz|ruck, the error ins keystone indicates an issue with mysql13:44
tkajinamhttps://logserver.rdoproject.org/46/38646/16/check/periodic-tripleo-ci-centos-9-ovb-1ctlr_2comp-featureset020-master/b9ff5d0/logs/overcloud-controller-0/var/log/extra/pcs.txt.gz13:44
tkajinamarxcruz|ruck, ^^^ some resources ar edown because of timeout13:44
arxcruz|rucktkajinam thanks, I will add these info on the bug13:45
arxcruz|ruckdviroel ^13:45
opendevreviewMarios Andreou proposed openstack/python-tripleoclient master: DNM Testing openstack/tripleo-ci/+/834287  https://review.opendev.org/c/openstack/python-tripleoclient/+/83428813:52
opendevreviewMarios Andreou proposed openstack/python-tripleoclient stable/wallaby: DNM Testing openstack/tripleo-ci/+/834287  https://review.opendev.org/c/openstack/python-tripleoclient/+/83431013:54
opendevreviewMarios Andreou proposed openstack/python-tripleoclient stable/train: DNM Testing openstack/tripleo-ci/+/834287  https://review.opendev.org/c/openstack/python-tripleoclient/+/83431113:55
opendevreviewMerged openstack/tripleo-ci master: Remove extra .txt from errors.txt page hyperlink  https://review.opendev.org/c/openstack/tripleo-ci/+/83410413:57
opendevreviewBogdan Dobrelya proposed openstack/tripleo-heat-templates master: Disabling container validation when running redhat-subscription  https://review.opendev.org/c/openstack/tripleo-heat-templates/+/83181313:59
jbadiapafultonj, I was out for lunch14:00
jbadiapathe fix seems ok to me... but I would like to dig a little bit as the role by default is configuring that parameter.14:01
opendevreviewJiri Podivin proposed openstack/tripleo-validations stable/train: DNM override-checkout  https://review.opendev.org/c/openstack/tripleo-validations/+/83431314:04
abishopfultonj, bogdando: hi, just now seeing the discussion you had an hour ago14:07
abishopI generally agree with the points raised, but have a comment on the kubernetes patch that bogdando referenced14:08
bogdandoit is a bit different case, but I think it matches fultonj comments ...14:08
bogdandoso it's good to go I guess14:08
abishopthe "if etcd_cluster_is_healthy.rc != 0" will definitely do the right thing if the etcd cluster hasn't been bootstrapped (the cluster health check will fail because there's no cluster yet!)14:09
bogdandojust that logic build around health checking against scaled up nodes...14:09
abishopbut there may be instances where the cluster has been bootstrapped (so it's "existing") and the health check fails14:10
bogdandoright, iiuc that's covered for tht automagically and is redundant14:10
bogdando?14:10
abishopagree14:10
abishopI think we're all in sync14:10
abishop:D14:10
bogdando:)14:10
bogdandonote that kubespray also implements individual certs for cluster nodes14:11
bogdandothat complicates scenario a bit14:11
abishopyeah, I can imagine that adding complexity14:12
opendevreviewMarios Andreou proposed openstack/tripleo-ci master: Centos8 wallaby jobs teardown - upgrade jobs  https://review.opendev.org/c/openstack/tripleo-ci/+/83432014:24
opendevreviewDouglas Viroel proposed openstack/openstack-tempest-skiplist master: Skip failing test 'test_image_web_download_import'  https://review.opendev.org/c/openstack/openstack-tempest-skiplist/+/83432114:30
opendevreviewMarios Andreou proposed openstack/tripleo-ci master: Centos8 wallaby jobs teardown - upgrade jobs  https://review.opendev.org/c/openstack/tripleo-ci/+/83432014:32
opendevreviewMarios Andreou proposed openstack/tripleo-ci master: Centos8 wallaby jobs teardown - upgrade jobs  https://review.opendev.org/c/openstack/tripleo-ci/+/83432014:34
opendevreviewDouglas Viroel proposed openstack/openstack-tempest-skiplist master: Skip failing test 'test_image_web_download_import'  https://review.opendev.org/c/openstack/openstack-tempest-skiplist/+/83432114:43
opendevreviewDouglas Viroel proposed openstack/openstack-tempest-skiplist master: Skip failing test 'test_image_web_download_import'  https://review.opendev.org/c/openstack/openstack-tempest-skiplist/+/83432114:44
opendevreviewMatthias Runge proposed openstack/tripleo-common master: Remove python-collectd-gnocchi from the image  https://review.opendev.org/c/openstack/tripleo-common/+/83432214:52
rlandyhttps://review.opendev.org/c/openstack/tripleo-heat-templates/+/834280/15:03
rlandymarios: fultonj: ^^ looks to be working15:04
rlandyhttps://review.opendev.org/c/openstack/tripleo-ansible/+/834279/15:04
rlandymarios: fultonj: ^^ can you guys vote and w+ there15:04
fultonji can't vote on my own patch :)15:06
mariosrlandy: checking15:06
mariosrlandy: expediting merge as we need this for gate blocker15:07
rlandyI voted15:07
opendevreviewMerged openstack/openstack-tempest-skiplist master: Skip failing test 'test_image_web_download_import'  https://review.opendev.org/c/openstack/openstack-tempest-skiplist/+/83432115:08
opendevreviewMatthias Runge proposed openstack/tripleo-heat-templates master: Remove support for collectd-gnocchi  https://review.opendev.org/c/openstack/tripleo-heat-templates/+/83432915:15
opendevreviewArx Cruz proposed openstack/openstack-tempest-skiplist master: This test is failing to connect to metadata service  https://review.opendev.org/c/openstack/openstack-tempest-skiplist/+/83433015:16
opendevreviewMerged openstack/tripleo-heat-templates master: Add support for ovn bgp agent  https://review.opendev.org/c/openstack/tripleo-heat-templates/+/78566615:21
*** dviroel is now known as dviroel|lunch15:23
opendevreviewMatthias Runge proposed openstack/tripleo-common master: Remove python-collectd-gnocchi from the image  https://review.opendev.org/c/openstack/tripleo-common/+/83432215:37
opendevreviewMatthias Runge proposed openstack/puppet-tripleo master: Drop support for collectd-gnocchi  https://review.opendev.org/c/openstack/puppet-tripleo/+/83433215:48
*** dviroel|lunch is now known as dviroel16:10
opendevreviewMatthias Runge proposed openstack/tripleo-heat-templates master: Remove support for collectd-gnocchi  https://review.opendev.org/c/openstack/tripleo-heat-templates/+/83432916:30
*** jpena is now known as jpena|off16:48
*** marios is now known as marios|out16:49
opendevreviewMerged openstack/tripleo-validations master: oslo_config_validator: Adding possibility to override namespaces config  https://review.opendev.org/c/openstack/tripleo-validations/+/82756617:18
opendevreviewLuis Tomas Bolivar proposed openstack/tripleo-heat-templates stable/wallaby: Add support for ovn bgp agent  https://review.opendev.org/c/openstack/tripleo-heat-templates/+/83434117:22
opendevreviewMerged openstack/tripleo-validations stable/train: oslo_config_validator: Nova port data changes settings have wrong values  https://review.opendev.org/c/openstack/tripleo-validations/+/83072217:23
*** dviroel_ is now known as dviroel17:46
opendevreviewJohn Fulton proposed openstack/tripleo-quickstart-extras master: Add ceph-install playbook to install ceph before overcloud  https://review.opendev.org/c/openstack/tripleo-quickstart-extras/+/83435219:58
opendevreviewJohn Fulton proposed openstack/tripleo-heat-templates master: WIP: Test deployed Ceph in scenario001  https://review.opendev.org/c/openstack/tripleo-heat-templates/+/83435420:33
opendevreviewJohn Fulton proposed openstack/tripleo-quickstart-extras master: Add ceph-install playbook to install ceph before overcloud  https://review.opendev.org/c/openstack/tripleo-quickstart-extras/+/83435220:34
opendevreviewJohn Fulton proposed openstack/tripleo-quickstart-extras master: Add ceph-install playbook to install ceph before overcloud  https://review.opendev.org/c/openstack/tripleo-quickstart-extras/+/83435220:36
opendevreviewMerged openstack/tripleo-validations master: Removing leftover README docs from roles  https://review.opendev.org/c/openstack/tripleo-validations/+/82749121:09
opendevreviewMerged openstack/tripleo-validations stable/train: Correct the tox option for skipping sdist generation  https://review.opendev.org/c/openstack/tripleo-validations/+/82973621:09
opendevreviewMerged openstack/tripleo-validations stable/victoria: Correct the tox option for skipping sdist generation  https://review.opendev.org/c/openstack/tripleo-validations/+/82973421:09
opendevreviewMerged openstack/tripleo-validations stable/wallaby: Correct the tox option for skipping sdist generation  https://review.opendev.org/c/openstack/tripleo-validations/+/82973321:09
opendevreviewMerged openstack/tripleo-validations stable/victoria: [Victoria-Only] Preventing tox from using local packages  https://review.opendev.org/c/openstack/tripleo-validations/+/83015321:09
opendevreviewMerged openstack/tripleo-heat-templates master: Etcd: Update cluster membership when replacing a node  https://review.opendev.org/c/openstack/tripleo-heat-templates/+/83316421:09
opendevreviewMerged openstack/tripleo-quickstart master: Allow overriding the default base image for convert-image  https://review.opendev.org/c/openstack/tripleo-quickstart/+/83392821:09
opendevreviewMerged openstack/tripleo-ansible master: Do not reference backup/restore config unless defined  https://review.opendev.org/c/openstack/tripleo-ansible/+/83427921:09
opendevreviewJames Slagle proposed openstack/python-tripleoclient master: Handle KeyboardInterrupt for ephemeral Heat cleanup  https://review.opendev.org/c/openstack/python-tripleoclient/+/83097321:16
opendevreviewJames Slagle proposed openstack/python-tripleoclient master: Update overcloud credentials to use ephemeral Heat  https://review.opendev.org/c/openstack/python-tripleoclient/+/83097421:16
opendevreviewJames Slagle proposed openstack/python-tripleoclient master: Remove Heat API dependency for overcloud export  https://review.opendev.org/c/openstack/python-tripleoclient/+/83384321:16
opendevreviewJames Slagle proposed openstack/python-tripleoclient master: Consolidate ephemeral Heat interaction to single try/finally  https://review.opendev.org/c/openstack/python-tripleoclient/+/83097521:16
*** dviroel is now known as dviroel|out21:37
opendevreviewMerged openstack/tripleo-ansible stable/wallaby: Removing playbooks section from default molecule config  https://review.opendev.org/c/openstack/tripleo-ansible/+/83071021:37
opendevreviewMerged openstack/tripleo-ansible stable/wallaby: Run molecule when ceph_spec_bootstrap is changed  https://review.opendev.org/c/openstack/tripleo-ansible/+/83401321:37
opendevreviewMerged openstack/tripleo-validations stable/train: Remove pre-deployment validation  https://review.opendev.org/c/openstack/tripleo-validations/+/82762821:37
opendevreviewMerged openstack/tripleo-validations stable/ussuri: Remove pre-deployment validation  https://review.opendev.org/c/openstack/tripleo-validations/+/82762721:37
opendevreviewMerged openstack/tripleo-quickstart master: Use whole disk images in libvirt deploy  https://review.opendev.org/c/openstack/tripleo-quickstart/+/83392921:37
opendevreviewMerged openstack/tripleo-heat-templates stable/wallaby: Add HostnameFormatDefault for Ceph related roles  https://review.opendev.org/c/openstack/tripleo-heat-templates/+/83414822:19
opendevreviewMartin Magr proposed openstack/tripleo-ansible master: Add module for loading hieradata and facts  https://review.opendev.org/c/openstack/tripleo-ansible/+/83436122:42
opendevreviewMartin Magr proposed openstack/tripleo-ansible master: Add module for loading hieradata and facts  https://review.opendev.org/c/openstack/tripleo-ansible/+/83436122:46
opendevreviewEbbex proposed openstack/tripleo-quickstart master: install-deps.sh: default python_cmd to python3  https://review.opendev.org/c/openstack/tripleo-quickstart/+/83436222:48
opendevreviewChristoph Dwertmann proposed openstack/tripleo-ansible master: Prevent DB backup failure when multiple "galera" containers are present  https://review.opendev.org/c/openstack/tripleo-ansible/+/83349523:27
opendevreviewChristoph Dwertmann proposed openstack/tripleo-ansible master: Prevent DB backup failure when multiple "galera" containers are present  https://review.opendev.org/c/openstack/tripleo-ansible/+/83349523:29

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