Friday, 2021-08-20

*** pmannidi|brb is now known as pmannidi00:09
opendevreviewMerged openstack/tripleo-heat-templates stable/wallaby: Change tests to run on ansible-core 2.11  https://review.opendev.org/c/openstack/tripleo-heat-templates/+/80499100:28
opendevreviewMerged openstack/python-tripleoclient stable/wallaby: Fetch stack after updating it  https://review.opendev.org/c/openstack/python-tripleoclient/+/80513700:28
*** pmannidi is now known as pmannidi|AFK00:47
*** pmannidi|AFK is now known as pmannidi00:47
opendevreviewBrendan Shephard proposed openstack/tripleo-heat-templates master: Deprecate and replace NetworkDeploymentActions  https://review.opendev.org/c/openstack/tripleo-heat-templates/+/80521701:16
opendevreviewSteve Baker proposed openstack/diskimage-builder master: Move grubenv to EFI dir  https://review.opendev.org/c/openstack/diskimage-builder/+/80400001:19
opendevreviewSteve Baker proposed openstack/diskimage-builder master: Support grubby and the Bootloader Spec  https://review.opendev.org/c/openstack/diskimage-builder/+/80400201:19
opendevreviewSteve Baker proposed openstack/diskimage-builder master: RHEL/Centos 9 does not have package grub2-efi-x64-modules  https://review.opendev.org/c/openstack/diskimage-builder/+/80481601:19
opendevreviewSteve Baker proposed openstack/diskimage-builder master: Add policycoreutils package mappings for RHEL/Centos 9  https://review.opendev.org/c/openstack/diskimage-builder/+/80481701:19
opendevreviewSteve Baker proposed openstack/diskimage-builder master: Add reinstall flag to install-packages, use it in bootloader  https://review.opendev.org/c/openstack/diskimage-builder/+/80481801:19
opendevreviewSteve Baker proposed openstack/diskimage-builder master: Add DIB_YUM_REPO_PACKAGE as an alternative to DIB_YUM_REPO_CONF  https://review.opendev.org/c/openstack/diskimage-builder/+/80481901:19
opendevreviewMerged openstack/python-tripleoclient stable/victoria: Fetch stack after updating it  https://review.opendev.org/c/openstack/python-tripleoclient/+/80513801:26
opendevreviewMerged openstack/python-tripleoclient stable/victoria: Fix network environment validation  https://review.opendev.org/c/openstack/python-tripleoclient/+/80499501:26
opendevreviewMerged openstack/python-tripleoclient stable/wallaby: Remove deprecation warnings for the role command  https://review.opendev.org/c/openstack/python-tripleoclient/+/80471301:26
opendevreviewMerged openstack/tripleo-ansible stable/wallaby: BnR: managed ceph properly to take a backup of the node  https://review.opendev.org/c/openstack/tripleo-ansible/+/80514801:26
opendevreviewMerged openstack/tripleo-ansible stable/wallaby: Adjust molecule jobs for Ansible 2.11  https://review.opendev.org/c/openstack/tripleo-ansible/+/80499001:27
opendevreviewMerged openstack/tripleo-ansible stable/wallaby: Use different name to distinguish between facts and vars in octavia  https://review.opendev.org/c/openstack/tripleo-ansible/+/80429701:27
opendevreviewMerged openstack/tripleo-ansible master: Give user the full path to sosreport  https://review.opendev.org/c/openstack/tripleo-ansible/+/80420001:27
opendevreviewRabi Mishra proposed openstack/puppet-tripleo stable/train: Set authtoken memcached_servers in undercloud services  https://review.opendev.org/c/openstack/puppet-tripleo/+/80532403:00
opendevreviewRabi Mishra proposed openstack/puppet-tripleo stable/train: Set authtoken memcached_servers in undercloud services  https://review.opendev.org/c/openstack/puppet-tripleo/+/80532403:01
opendevreviewTakashi Kajinami proposed openstack/puppet-tripleo stable/train: Set authtoken memcached_servers in undercloud services  https://review.opendev.org/c/openstack/puppet-tripleo/+/80532403:16
opendevreviewTakashi Kajinami proposed openstack/tripleo-heat-templates stable/train: Remove all hieradata overriding for undercloud memcache  https://review.opendev.org/c/openstack/tripleo-heat-templates/+/80532603:21
tkajinamramishra, ^^^ fyi03:23
opendevreviewTakashi Kajinami proposed openstack/puppet-tripleo stable/train: [Train only] Update the authtoken class for undercloud services  https://review.opendev.org/c/openstack/puppet-tripleo/+/80532703:32
opendevreviewMerged openstack/tripleo-common stable/train: Replace upgrades template when providers run again  https://review.opendev.org/c/openstack/tripleo-common/+/80479803:37
opendevreviewTakashi Kajinami proposed openstack/puppet-tripleo master: Remove duplicate spec file  https://review.opendev.org/c/openstack/puppet-tripleo/+/80532803:38
ramishratkajinam:  we don't necessarily need those other backports, but fine if we want03:41
tkajinamramishra, yeah these values are just ignored once the change in puppet-tripleo is merged but having these useless definitions would be confusing03:43
tkajinama problem here was that conflicts were not managed well when we backport tons on work about tls support in memcache03:44
tkajinamthe backports didn't have conflict notes and that resulted in inconsistent implementation in each branches. this is just one of example which was skipped during recent backport to add more parameters for memcache but the backport patch didn't mention it is skipping these files03:45
ramishratkajinam: it was very difficult to debug that BZ to find out the root cause, with 500 from mistral without anything in logs03:46
ramishraI normally don't like backporting tons of stuff if not required03:47
tkajinamthat's true but If we need any backport for some reason then we should do it cleanly without any tricky conflict resolving, leaving some part untouched, ...03:48
ramishratkajinam: ideal world:)03:49
tkajinamramishra, :-D03:49
tkajinamI didn't expect these parts will be updated later so didn't try to backport patches but you know people are interested in Train (for SOME reason) so I should have backported these patches timely03:50
tkajinamthat's what I'd keep in mind in this cycle, for W03:50
tkajinamramishra, by the way, didn't Mistrak or keystone middleware log any errors when memcached is not accessible ?03:51
tkajinams/Mistrak/Mistral/03:51
ramishratkajinam: nope nothing, absolutely.. I just disabled auth while debugging and as it worked fine. I zeroed on middleware. Then noticed the config for memcached03:53
tkajinamramishra, hmm. it might be caused by log level set for some specific modules but I feel like we should fix the behavior somehow03:56
ramishramistral uses pecan, enabling log for pecan did not do anything, but yeah we need some trace for these kind of issues03:58
ramishraI may spend some time on it later03:59
tkajinamit would be good so far is the problem is specific to mistral04:02
tkajinamsome other services like ironic and octavia are using pcan iirc, so if usage of pcan is involved that might need some attention04:02
*** pojadhav- is now known as pojadhav04:09
opendevreviewBrendan Shephard proposed openstack/tripleo-heat-templates master: Deprecate and replace NetworkDeploymentActions  https://review.opendev.org/c/openstack/tripleo-heat-templates/+/80521704:39
opendevreviewBrendan Shephard proposed openstack/tripleo-heat-templates master: Deprecate and replace NetworkDeploymentActions  https://review.opendev.org/c/openstack/tripleo-heat-templates/+/80521704:55
opendevreviewBrendan Shephard proposed openstack/tripleo-heat-templates master: Deprecate and replace NetworkDeploymentActions  https://review.opendev.org/c/openstack/tripleo-heat-templates/+/80521704:57
opendevreviewamolkahat proposed openstack/tripleo-quickstart-extras master: Migrate overcloud-deploy.sh shell script to ansible roles part-1  https://review.opendev.org/c/openstack/tripleo-quickstart-extras/+/80362705:09
opendevreviewamolkahat proposed openstack/tripleo-quickstart-extras master: Migrate overcloud-deploy.sh shell script to ansible roles part-2  https://review.opendev.org/c/openstack/tripleo-quickstart-extras/+/80411305:13
*** ysandeep|away is now known as ysandeep05:29
opendevreviewamolkahat proposed openstack/tripleo-quickstart-extras master: Added stack_name variable to overcloud-delete defaults  https://review.opendev.org/c/openstack/tripleo-quickstart-extras/+/80526005:32
opendevreviewJiri Podivin proposed openstack/tripleo-validations stable/ussuri: Corrected the task 'Get file_descriptors total_limit' to get the 'fs.file-max' value.  https://review.opendev.org/c/openstack/tripleo-validations/+/80224905:58
opendevreviewJiri Podivin proposed openstack/tripleo-validations stable/ussuri: Sort jobs in molecule.yaml file when adding new role  https://review.opendev.org/c/openstack/tripleo-validations/+/79962705:59
opendevreviewJiri Podivin proposed openstack/tripleo-validations stable/ussuri: Corrected the task 'Get file_descriptors total_limit' to get the 'fs.file-max' value.  https://review.opendev.org/c/openstack/tripleo-validations/+/80224905:59
opendevreviewJiri Podivin proposed openstack/tripleo-validations stable/ussuri: Get the undercloud.conf file from where it actually is  https://review.opendev.org/c/openstack/tripleo-validations/+/78564505:59
opendevreviewJiri Podivin proposed openstack/tripleo-validations stable/ussuri: Sort jobs in molecule.yaml file when adding new role  https://review.opendev.org/c/openstack/tripleo-validations/+/79962706:00
opendevreviewJiri Podivin proposed openstack/tripleo-validations stable/ussuri: oslo_config_validator: Setting container network to none  https://review.opendev.org/c/openstack/tripleo-validations/+/80277806:00
opendevreviewMerged openstack/tripleo-heat-templates master: Create the tftpboot ppc64le folder  https://review.opendev.org/c/openstack/tripleo-heat-templates/+/80525306:08
opendevreviewMerged openstack/tripleo-heat-templates master: Enable debug of memcached when Debug is enabled  https://review.opendev.org/c/openstack/tripleo-heat-templates/+/80488006:08
opendevreviewFrancesco Pantano proposed openstack/tripleo-heat-templates stable/wallaby: Move the remaining wallaby jobs to cephadm  https://review.opendev.org/c/openstack/tripleo-heat-templates/+/80503406:21
opendevreviewFrancesco Pantano proposed openstack/tripleo-heat-templates master: Enable CephDashboard for scenario001  https://review.opendev.org/c/openstack/tripleo-heat-templates/+/80489606:22
opendevreviewFrancesco Pantano proposed openstack/tripleo-heat-templates master: Consider GlanceMultistoreConfig when setting glance_pool  https://review.opendev.org/c/openstack/tripleo-heat-templates/+/80502906:22
*** pmannidi is now known as pmannidi|AFK06:43
opendevreviewMerged openstack/tripleo-heat-templates stable/wallaby: Add a role to run independent designate bind backends  https://review.opendev.org/c/openstack/tripleo-heat-templates/+/80406706:55
opendevreviewMerged openstack/tripleo-ansible master: Setting language for the python3.6 tox environment  https://review.opendev.org/c/openstack/tripleo-ansible/+/80511606:56
opendevreviewMerged openstack/python-tripleoclient master: Check for neutron resources with ephemeral Heat  https://review.opendev.org/c/openstack/python-tripleoclient/+/80413906:56
opendevreviewMerged openstack/ansible-role-tripleo-modify-image master: Remove config of container_build_tool  https://review.opendev.org/c/openstack/ansible-role-tripleo-modify-image/+/80406906:56
opendevreviewMerged openstack/tripleo-heat-templates master: Add DeployedCeph boolean parameter  https://review.opendev.org/c/openstack/tripleo-heat-templates/+/79753906:56
opendevreviewchandan kumar proposed openstack/tripleo-quickstart master: Use tripleo.repos.get_hash in repo-setup role  https://review.opendev.org/c/openstack/tripleo-quickstart/+/79148606:56
opendevreviewJuan Badia Payno proposed openstack/tripleo-ansible stable/victoria: BnR: managed ceph properly to take a backup of the node  https://review.opendev.org/c/openstack/tripleo-ansible/+/80527407:22
jpodivinThere seems to be a race condition, of sorts, in the tripleoclient.export.export_stack https://opendev.org/openstack/python-tripleoclient/src/branch/master/tripleoclient/export.py#L78-L100 Depending on how the keys are ordered in the dictionary, we might end up with previously exported data in under a key for data from a JSON file. 07:26
jpodivinThat is, if we hit TypeError or JSONDecodeError while reading the json file. 07:26
jpodivinShould be easy enough to handle, but there is a question of if we want to fail the export immediately after hitting a file decoding issue or not. 07:28
*** rpittau|afk is now known as rpittau07:38
chandankumarfmount: hello, please have a look at https://review.opendev.org/c/openstack/tripleo-heat-templates/+/805034/1#message-7fa06b7e656fd338e942d685a7863010ce0def74 07:38
chandankumarwhen free, thanks :-)07:38
*** jpena|off is now known as jpena07:38
opendevreviewMerged openstack/tripleo-heat-templates master: Add BGP EVPN support to the FRR service  https://review.opendev.org/c/openstack/tripleo-heat-templates/+/79864507:41
fmountchandankumar: o/ thanks for your comment, I think that failure is unrelated to the PS (where I just added an option to remove a warning in the ceph cluster)07:44
fmountnothing special, it's just to see HEALTH_OK instead of HEALTH_WARN when no replica is configured (like the standalone use case)07:45
*** ysandeep is now known as ysandeep|lunch07:45
fmountchandankumar: let's wait for the next run, I'll keep an eye on it07:46
chandankumarfmount: sure, thanks for looking into it .07:46
opendevreviewJiri Podivin proposed openstack/python-tripleoclient master: Tests and documentation for the export_stack function.  https://review.opendev.org/c/openstack/python-tripleoclient/+/80534707:55
opendevreviewJiri Podivin proposed openstack/python-tripleoclient master: New test case and documentation for the export_stack function.  https://review.opendev.org/c/openstack/python-tripleoclient/+/80534707:58
opendevreviewJiri Podivin proposed openstack/python-tripleoclient master: New test case and documentation for the export_stack function.  https://review.opendev.org/c/openstack/python-tripleoclient/+/80534708:01
opendevreviewRabi Mishra proposed openstack/puppet-tripleo stable/train: Set authtoken memcached_servers in undercloud services  https://review.opendev.org/c/openstack/puppet-tripleo/+/80532408:12
opendevreviewJiri Podivin proposed openstack/validations-libs master: Add validation config file mechanism  https://review.opendev.org/c/openstack/validations-libs/+/79246008:12
opendevreviewJiri Podivin proposed openstack/validations-libs master: Default value of the constructor parameter  https://review.opendev.org/c/openstack/validations-libs/+/79648208:12
opendevreviewJiri Podivin proposed openstack/validations-libs master: Validation actions refactor  https://review.opendev.org/c/openstack/validations-libs/+/80290108:12
opendevreviewJiri Podivin proposed openstack/validations-libs master: Expand validations_logs table with reason of failure  https://review.opendev.org/c/openstack/validations-libs/+/80439208:13
opendevreviewJiri Podivin proposed openstack/tripleo-validations master: Fail validation if pacemaker service is not active  https://review.opendev.org/c/openstack/tripleo-validations/+/80518708:13
opendevreviewJiri Podivin proposed openstack/python-tripleoclient stable/wallaby: Moving the 'export_data' dict into constants  https://review.opendev.org/c/openstack/python-tripleoclient/+/80535508:22
opendevreviewJiri Podivin proposed openstack/python-tripleoclient stable/wallaby: Setting upper constraints to wallaby branch of requirements  https://review.opendev.org/c/openstack/python-tripleoclient/+/80535808:33
opendevreviewBrendan Shephard proposed openstack/tripleo-heat-templates master: Deprecate and replace NetworkDeploymentActions  https://review.opendev.org/c/openstack/tripleo-heat-templates/+/80521708:40
*** ysandeep|lunch is now known as ysandeep08:53
opendevreviewJuan Badia Payno proposed openstack/tripleo-ansible stable/victoria: BnR: managed ceph properly to take a backup of the node  https://review.opendev.org/c/openstack/tripleo-ansible/+/80527408:57
opendevreviewJuan Badia Payno proposed openstack/tripleo-ansible stable/ussuri: BnR: managed ceph properly to take a backup of the node  https://review.opendev.org/c/openstack/tripleo-ansible/+/80527608:59
opendevreviewJuan Badia Payno proposed openstack/tripleo-ansible stable/train: BnR: managed ceph properly to take a backup of the node  https://review.opendev.org/c/openstack/tripleo-ansible/+/80527709:00
opendevreviewRabi Mishra proposed openstack/puppet-tripleo stable/train: Set authtoken memcached_servers in undercloud services  https://review.opendev.org/c/openstack/puppet-tripleo/+/80532409:01
opendevreviewamolkahat proposed openstack/tripleo-quickstart-extras master: Added stack_name variable to overcloud-delete defaults  https://review.opendev.org/c/openstack/tripleo-quickstart-extras/+/80526009:06
opendevreviewJuan Badia Payno proposed openstack/tripleo-ansible stable/train: BnR: managed ceph properly to take a backup of the node  https://review.opendev.org/c/openstack/tripleo-ansible/+/80527709:12
opendevreviewamolkahat proposed openstack/tripleo-quickstart-extras master: Migrate overcloud-deploy.sh shell script to ansible roles part-3  https://review.opendev.org/c/openstack/tripleo-quickstart-extras/+/80411709:15
opendevreviewTakashi Kajinami proposed openstack/puppet-tripleo stable/train: [Train only] Update the authtoken class for undercloud services  https://review.opendev.org/c/openstack/puppet-tripleo/+/80532709:59
opendevreviewVijayalakshmi proposed openstack/tripleo-heat-templates master: Update dervice_pci script to handle pci address formats  https://review.opendev.org/c/openstack/tripleo-heat-templates/+/79931810:41
opendevreviewMichele Baldessari proposed openstack/tripleo-ansible master: tripleo_frr: Allow a password to be set in FRR's neighbor section  https://review.opendev.org/c/openstack/tripleo-ansible/+/80354810:45
*** dviroel|ruck|out is now known as dviroel|ruck11:26
*** bhagyashris_ is now known as bhagyashris11:31
*** jpena is now known as jpena|lunch11:36
*** rlandy is now known as rlandy|rover11:45
opendevreviewMerged openstack/tripleo-common master: Fix upload-artifacts and its installation  https://review.opendev.org/c/openstack/tripleo-common/+/80489912:03
opendevreviewJiri Podivin proposed openstack/python-tripleoclient stable/wallaby: Setting upper constraints to wallaby branch of requirements  https://review.opendev.org/c/openstack/python-tripleoclient/+/80535812:08
opendevreviewMerged openstack/tripleo-heat-templates master: Remove deprecated ipv6 corosync parameters  https://review.opendev.org/c/openstack/tripleo-heat-templates/+/73993612:26
opendevreviewJiri Podivin proposed openstack/validations-common master: False positive checks for component jobs  https://review.opendev.org/c/openstack/validations-common/+/80316312:29
opendevreviewJuan Badia Payno proposed openstack/tripleo-ansible stable/train: BnR: managed ceph properly to take a backup of the node  https://review.opendev.org/c/openstack/tripleo-ansible/+/80527712:31
jpodivinhi there. I was wondering if anyone had a bit of time to take a look at https://review.opendev.org/c/openstack/tripleo-validations/+/805235 ? 12:33
jpodivinit's at the bottom of rather large dependency chain ... well tree really. 12:34
jpodivinWe need it in tripleo-validations to unblock the ussuri pipeline12:34
*** jpena|lunch is now known as jpena12:36
opendevreviewwes hayutin proposed openstack/tripleo-quickstart-extras master: Added stack_name variable to overcloud-delete defaults  https://review.opendev.org/c/openstack/tripleo-quickstart-extras/+/80526012:44
opendevreviewMerged openstack/tripleo-heat-templates master: container-puppet.sh idempotency on restart  https://review.opendev.org/c/openstack/tripleo-heat-templates/+/80530112:47
opendevreviewJiri Podivin proposed openstack/validations-libs master: DNM moving callbacks to libs  https://review.opendev.org/c/openstack/validations-libs/+/79918813:20
opendevreviewPooja Jadhav proposed openstack/openstack-tempest-skiplist master: Skip test_attach_volume_negative tempest test  https://review.opendev.org/c/openstack/openstack-tempest-skiplist/+/80538213:20
opendevreviewJohn Fulton proposed openstack/python-tripleoclient master: Introduce "openstack overcloud ceph deploy"  https://review.opendev.org/c/openstack/python-tripleoclient/+/80034113:30
opendevreviewJohn Fulton proposed openstack/python-tripleoclient master: Introduce "openstack overcloud ceph deploy"  https://review.opendev.org/c/openstack/python-tripleoclient/+/80034113:30
opendevreviewJohn Fulton proposed openstack/python-tripleoclient master: Introduce "openstack overcloud ceph deploy"  https://review.opendev.org/c/openstack/python-tripleoclient/+/80034113:39
opendevreviewJuan Badia Payno proposed openstack/tripleo-quickstart master: Install tripleo.operator collections  https://review.opendev.org/c/openstack/tripleo-quickstart/+/80538413:43
opendevreviewMerged openstack/openstack-tempest-skiplist master: Skip test_attach_volume_negative tempest test  https://review.opendev.org/c/openstack/openstack-tempest-skiplist/+/80538213:45
*** rpittau is now known as rpittau|afk13:55
opendevreviewJing Li proposed openstack/diskimage-builder master: Add new element rocky  https://review.opendev.org/c/openstack/diskimage-builder/+/80290214:09
opendevreviewDouglas Viroel proposed openstack/openstack-tempest-skiplist master: Update FloatingIpMultipleRoutersTest in skiplist  https://review.opendev.org/c/openstack/openstack-tempest-skiplist/+/80539314:25
*** ysandeep is now known as ysandeep|away14:31
opendevreviewJames Parker proposed openstack/tripleo-upgrade master: Add different migration actions based on guest  https://review.opendev.org/c/openstack/tripleo-upgrade/+/79837314:44
opendevreviewSorin Sbârnea proposed openstack/tripleo-repos master: Address pyyaml import in ansible module  https://review.opendev.org/c/openstack/tripleo-repos/+/80540014:48
opendevreviewDouglas Viroel proposed openstack/openstack-tempest-skiplist master: Update FloatingIpMultipleRoutersTest in skiplist  https://review.opendev.org/c/openstack/openstack-tempest-skiplist/+/80539315:12
opendevreviewJuan Badia Payno proposed openstack/tripleo-quickstart master: Install tripleo.operator collections  https://review.opendev.org/c/openstack/tripleo-quickstart/+/80538415:17
opendevreviewJiri Podivin proposed openstack/python-tripleoclient master: Eliminating a race condition in the export_stack function  https://review.opendev.org/c/openstack/python-tripleoclient/+/80540915:20
zbrmwhahaha: weshay|ruck : please review/vote/comment on https://review.opendev.org/c/zuul/zuul-jobs/+/803471/ -- no reason to explain why is important for us.15:23
opendevreviewamolkahat proposed openstack/tripleo-quickstart-extras master: Migrate overcloud-deploy.sh shell script to ansible roles part-3  https://review.opendev.org/c/openstack/tripleo-quickstart-extras/+/80411715:34
opendevreviewJiri Podivin proposed openstack/python-tripleoclient master: Enabling cover job in the zuul  https://review.opendev.org/c/openstack/python-tripleoclient/+/80541415:45
weshay|ruckHAPPY TRIPLEO INDEPENDENCE DAY... https://review.opendev.org/c/openstack/releases/+/804871  15:45
weshay|ruckreally was yesterday.. throw a party15:45
fultonjha ha weshay|ruck 15:45
*** jpena is now known as jpena|off15:51
opendevreviewMerged openstack/tripleo-heat-templates master: Enable CephDashboard for scenario001  https://review.opendev.org/c/openstack/tripleo-heat-templates/+/80489615:52
jpodivinfultonj: Hi, I don't want to put it on the review, but I've noticed one more thing, the map var, is actually covering a builtin. A pretty useful one that might be coming to play at some point in the namespace https://docs.python.org/3.8/library/functions.html#map15:53
fultonjjpodivin: good catch, i'll change the name15:54
jpodivinfultonj: thank you! 15:55
*** pojadhav- is now known as pojadhav16:00
opendevreviewFrancesco Pantano proposed openstack/tripleo-heat-templates stable/wallaby: Enable CephDashboard for scenario001  https://review.opendev.org/c/openstack/tripleo-heat-templates/+/80528016:00
opendevreviewMerged openstack/tripleo-heat-templates stable/wallaby: Move the remaining wallaby jobs to cephadm  https://review.opendev.org/c/openstack/tripleo-heat-templates/+/80503416:13
opendevreviewMerged openstack/tripleo-validations stable/ussuri: Trigger molecule jobs when modifying molecule-requirements.txt file  https://review.opendev.org/c/openstack/tripleo-validations/+/80523516:13
opendevreviewMerged openstack/tripleo-validations stable/ussuri: [ussuri-only] Stop requiring ansible 2.8  https://review.opendev.org/c/openstack/tripleo-validations/+/80529016:13
opendevreviewMerged openstack/tripleo-validations stable/victoria: Trigger molecule jobs when modifying molecule-requirements.txt file  https://review.opendev.org/c/openstack/tripleo-validations/+/80523316:13
lbragstado/ hey folks - i'm curious if anyone else would like to weigh in on https://review.opendev.org/c/openstack/tripleo-heat-templates/+/78157116:40
lbragstadspecifically, i'm wondering if we should do one global to turn on secure rbac (EnforceSecureRbac) or if we should keep it at the service level (CinderEnforceSecureRbac)16:41
mwhahahalbragstad: would we ever need to partially enable rbac?16:47
mwhahahaif so, i would not use a global16:47
mwhahahai think it's best to use service specific ones for the foreseeable future16:48
lbragstadthat's a good question - i could see a case where if someone wanted to only put keystone into secure-rbac mode to play with all the different personas16:48
lbragstadthen a service-specific knob seems useful16:48
mwhahahayou could intheory have both16:49
mwhahahaif global is true, use that. else use service value16:50
mwhahahawe used to do something similar for Debug16:50
lbragstadyeah, that's true16:50
lbragstadmy initial thought was to have the global behavior encapsulated in a template (referenced in the patch)16:50
mwhahahawe can also change it later16:51
lbragstadso if an operator really understands what they're doing and they want to opt into the behavior, they include the template16:51
mwhahahai think the service one is the start16:51
lbragstadok16:51
mwhahahaand we come through and add a global when it works everywhere16:51
opendevreviewamolkahat proposed openstack/tripleo-quickstart-extras master: [WIP] Migrate overcloud-deploy.sh shell script to ansible roles part-4  https://review.opendev.org/c/openstack/tripleo-quickstart-extras/+/80033816:51
lbragstadmwhahaha right - ok... another thing too is that the upstream services are trying to get to this new behavior by default (i'm not sure how long that's going to take)16:52
lbragstadbut that will eventually make the global and service specific knobs obsolete16:52
mwhahahayea16:53
* fultonj is on PTO this afternoon17:14
fultonjsee you monday17:14
opendevreviewDamien Ciabrini proposed openstack/tripleo-ansible stable/wallaby: tripleo_push_artifacts: ensure destination exists  https://review.opendev.org/c/openstack/tripleo-ansible/+/80528117:26
opendevreviewBrent Eagles proposed openstack/tripleo-ansible master: DNM: WIP: working on some cert repair playbooks for octavia  https://review.opendev.org/c/openstack/tripleo-ansible/+/80544217:30
opendevreviewDamien Ciabrini proposed openstack/tripleo-ansible stable/wallaby: tripleo_push_artifacts: fix argument parsing  https://review.opendev.org/c/openstack/tripleo-ansible/+/80528217:31
opendevreviewMerged openstack/openstack-tempest-skiplist master: Update FloatingIpMultipleRoutersTest in skiplist  https://review.opendev.org/c/openstack/openstack-tempest-skiplist/+/80539318:48
opendevreviewKevin Carter proposed openstack/python-tripleoclient master: Ensure directories exist before copying  https://review.opendev.org/c/openstack/python-tripleoclient/+/80529118:50
opendevreviewMerged openstack/python-tripleoclient stable/train: [train-only] Honour DeploymentServerBlacklist  https://review.opendev.org/c/openstack/python-tripleoclient/+/80494619:29
opendevreviewDavid Vallee Delisle proposed openstack/tripleo-ansible master: tripleo-kernel: fixing naming of workload protection  https://review.opendev.org/c/openstack/tripleo-ansible/+/80418920:46
opendevreviewMerged openstack/tripleo-docs master: ovs-dpdk: fixing deploy command  https://review.opendev.org/c/openstack/tripleo-docs/+/80297021:05
opendevreviewMerged openstack/tripleo-ha-utils master: setup.cfg: Replace dashes with underscores  https://review.opendev.org/c/openstack/tripleo-ha-utils/+/78820521:47
opendevreviewMerged openstack/tripleo-docs master: setup.cfg: Replace dashes with underscores  https://review.opendev.org/c/openstack/tripleo-docs/+/78820421:55
*** dviroel|ruck is now known as dviroel|out22:04
opendevreviewMerged openstack/tripleo-quickstart-extras master: Fix multinode-ipa job from building-test-packages twice  https://review.opendev.org/c/openstack/tripleo-quickstart-extras/+/80497123:54

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