*** pmannidi|brb is now known as pmannidi | 00:09 | |
opendevreview | Merged openstack/tripleo-heat-templates stable/wallaby: Change tests to run on ansible-core 2.11 https://review.opendev.org/c/openstack/tripleo-heat-templates/+/804991 | 00:28 |
---|---|---|
opendevreview | Merged openstack/python-tripleoclient stable/wallaby: Fetch stack after updating it https://review.opendev.org/c/openstack/python-tripleoclient/+/805137 | 00:28 |
*** pmannidi is now known as pmannidi|AFK | 00:47 | |
*** pmannidi|AFK is now known as pmannidi | 00:47 | |
opendevreview | Brendan Shephard proposed openstack/tripleo-heat-templates master: Deprecate and replace NetworkDeploymentActions https://review.opendev.org/c/openstack/tripleo-heat-templates/+/805217 | 01:16 |
opendevreview | Steve Baker proposed openstack/diskimage-builder master: Move grubenv to EFI dir https://review.opendev.org/c/openstack/diskimage-builder/+/804000 | 01:19 |
opendevreview | Steve Baker proposed openstack/diskimage-builder master: Support grubby and the Bootloader Spec https://review.opendev.org/c/openstack/diskimage-builder/+/804002 | 01:19 |
opendevreview | Steve 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/+/804816 | 01:19 |
opendevreview | Steve Baker proposed openstack/diskimage-builder master: Add policycoreutils package mappings for RHEL/Centos 9 https://review.opendev.org/c/openstack/diskimage-builder/+/804817 | 01:19 |
opendevreview | Steve Baker proposed openstack/diskimage-builder master: Add reinstall flag to install-packages, use it in bootloader https://review.opendev.org/c/openstack/diskimage-builder/+/804818 | 01:19 |
opendevreview | Steve 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/+/804819 | 01:19 |
opendevreview | Merged openstack/python-tripleoclient stable/victoria: Fetch stack after updating it https://review.opendev.org/c/openstack/python-tripleoclient/+/805138 | 01:26 |
opendevreview | Merged openstack/python-tripleoclient stable/victoria: Fix network environment validation https://review.opendev.org/c/openstack/python-tripleoclient/+/804995 | 01:26 |
opendevreview | Merged openstack/python-tripleoclient stable/wallaby: Remove deprecation warnings for the role command https://review.opendev.org/c/openstack/python-tripleoclient/+/804713 | 01:26 |
opendevreview | Merged openstack/tripleo-ansible stable/wallaby: BnR: managed ceph properly to take a backup of the node https://review.opendev.org/c/openstack/tripleo-ansible/+/805148 | 01:26 |
opendevreview | Merged openstack/tripleo-ansible stable/wallaby: Adjust molecule jobs for Ansible 2.11 https://review.opendev.org/c/openstack/tripleo-ansible/+/804990 | 01:27 |
opendevreview | Merged openstack/tripleo-ansible stable/wallaby: Use different name to distinguish between facts and vars in octavia https://review.opendev.org/c/openstack/tripleo-ansible/+/804297 | 01:27 |
opendevreview | Merged openstack/tripleo-ansible master: Give user the full path to sosreport https://review.opendev.org/c/openstack/tripleo-ansible/+/804200 | 01:27 |
opendevreview | Rabi Mishra proposed openstack/puppet-tripleo stable/train: Set authtoken memcached_servers in undercloud services https://review.opendev.org/c/openstack/puppet-tripleo/+/805324 | 03:00 |
opendevreview | Rabi Mishra proposed openstack/puppet-tripleo stable/train: Set authtoken memcached_servers in undercloud services https://review.opendev.org/c/openstack/puppet-tripleo/+/805324 | 03:01 |
opendevreview | Takashi Kajinami proposed openstack/puppet-tripleo stable/train: Set authtoken memcached_servers in undercloud services https://review.opendev.org/c/openstack/puppet-tripleo/+/805324 | 03:16 |
opendevreview | Takashi Kajinami proposed openstack/tripleo-heat-templates stable/train: Remove all hieradata overriding for undercloud memcache https://review.opendev.org/c/openstack/tripleo-heat-templates/+/805326 | 03:21 |
tkajinam | ramishra, ^^^ fyi | 03:23 |
opendevreview | Takashi Kajinami proposed openstack/puppet-tripleo stable/train: [Train only] Update the authtoken class for undercloud services https://review.opendev.org/c/openstack/puppet-tripleo/+/805327 | 03:32 |
opendevreview | Merged openstack/tripleo-common stable/train: Replace upgrades template when providers run again https://review.opendev.org/c/openstack/tripleo-common/+/804798 | 03:37 |
opendevreview | Takashi Kajinami proposed openstack/puppet-tripleo master: Remove duplicate spec file https://review.opendev.org/c/openstack/puppet-tripleo/+/805328 | 03:38 |
ramishra | tkajinam: we don't necessarily need those other backports, but fine if we want | 03:41 |
tkajinam | ramishra, yeah these values are just ignored once the change in puppet-tripleo is merged but having these useless definitions would be confusing | 03:43 |
tkajinam | a problem here was that conflicts were not managed well when we backport tons on work about tls support in memcache | 03:44 |
tkajinam | the 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 files | 03:45 |
ramishra | tkajinam: it was very difficult to debug that BZ to find out the root cause, with 500 from mistral without anything in logs | 03:46 |
ramishra | I normally don't like backporting tons of stuff if not required | 03:47 |
tkajinam | that'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 |
ramishra | tkajinam: ideal world:) | 03:49 |
tkajinam | ramishra, :-D | 03:49 |
tkajinam | I 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 timely | 03:50 |
tkajinam | that's what I'd keep in mind in this cycle, for W | 03:50 |
tkajinam | ramishra, by the way, didn't Mistrak or keystone middleware log any errors when memcached is not accessible ? | 03:51 |
tkajinam | s/Mistrak/Mistral/ | 03:51 |
ramishra | tkajinam: nope nothing, absolutely.. I just disabled auth while debugging and as it worked fine. I zeroed on middleware. Then noticed the config for memcached | 03:53 |
tkajinam | ramishra, hmm. it might be caused by log level set for some specific modules but I feel like we should fix the behavior somehow | 03:56 |
ramishra | mistral uses pecan, enabling log for pecan did not do anything, but yeah we need some trace for these kind of issues | 03:58 |
ramishra | I may spend some time on it later | 03:59 |
tkajinam | it would be good so far is the problem is specific to mistral | 04:02 |
tkajinam | some other services like ironic and octavia are using pcan iirc, so if usage of pcan is involved that might need some attention | 04:02 |
*** pojadhav- is now known as pojadhav | 04:09 | |
opendevreview | Brendan Shephard proposed openstack/tripleo-heat-templates master: Deprecate and replace NetworkDeploymentActions https://review.opendev.org/c/openstack/tripleo-heat-templates/+/805217 | 04:39 |
opendevreview | Brendan Shephard proposed openstack/tripleo-heat-templates master: Deprecate and replace NetworkDeploymentActions https://review.opendev.org/c/openstack/tripleo-heat-templates/+/805217 | 04:55 |
opendevreview | Brendan Shephard proposed openstack/tripleo-heat-templates master: Deprecate and replace NetworkDeploymentActions https://review.opendev.org/c/openstack/tripleo-heat-templates/+/805217 | 04:57 |
opendevreview | amolkahat 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/+/803627 | 05:09 |
opendevreview | amolkahat 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/+/804113 | 05:13 |
*** ysandeep|away is now known as ysandeep | 05:29 | |
opendevreview | amolkahat proposed openstack/tripleo-quickstart-extras master: Added stack_name variable to overcloud-delete defaults https://review.opendev.org/c/openstack/tripleo-quickstart-extras/+/805260 | 05:32 |
opendevreview | Jiri 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/+/802249 | 05:58 |
opendevreview | Jiri 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/+/799627 | 05:59 |
opendevreview | Jiri 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/+/802249 | 05:59 |
opendevreview | Jiri 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/+/785645 | 05:59 |
opendevreview | Jiri 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/+/799627 | 06:00 |
opendevreview | Jiri Podivin proposed openstack/tripleo-validations stable/ussuri: oslo_config_validator: Setting container network to none https://review.opendev.org/c/openstack/tripleo-validations/+/802778 | 06:00 |
opendevreview | Merged openstack/tripleo-heat-templates master: Create the tftpboot ppc64le folder https://review.opendev.org/c/openstack/tripleo-heat-templates/+/805253 | 06:08 |
opendevreview | Merged openstack/tripleo-heat-templates master: Enable debug of memcached when Debug is enabled https://review.opendev.org/c/openstack/tripleo-heat-templates/+/804880 | 06:08 |
opendevreview | Francesco Pantano proposed openstack/tripleo-heat-templates stable/wallaby: Move the remaining wallaby jobs to cephadm https://review.opendev.org/c/openstack/tripleo-heat-templates/+/805034 | 06:21 |
opendevreview | Francesco Pantano proposed openstack/tripleo-heat-templates master: Enable CephDashboard for scenario001 https://review.opendev.org/c/openstack/tripleo-heat-templates/+/804896 | 06:22 |
opendevreview | Francesco Pantano proposed openstack/tripleo-heat-templates master: Consider GlanceMultistoreConfig when setting glance_pool https://review.opendev.org/c/openstack/tripleo-heat-templates/+/805029 | 06:22 |
*** pmannidi is now known as pmannidi|AFK | 06:43 | |
opendevreview | Merged openstack/tripleo-heat-templates stable/wallaby: Add a role to run independent designate bind backends https://review.opendev.org/c/openstack/tripleo-heat-templates/+/804067 | 06:55 |
opendevreview | Merged openstack/tripleo-ansible master: Setting language for the python3.6 tox environment https://review.opendev.org/c/openstack/tripleo-ansible/+/805116 | 06:56 |
opendevreview | Merged openstack/python-tripleoclient master: Check for neutron resources with ephemeral Heat https://review.opendev.org/c/openstack/python-tripleoclient/+/804139 | 06:56 |
opendevreview | Merged openstack/ansible-role-tripleo-modify-image master: Remove config of container_build_tool https://review.opendev.org/c/openstack/ansible-role-tripleo-modify-image/+/804069 | 06:56 |
opendevreview | Merged openstack/tripleo-heat-templates master: Add DeployedCeph boolean parameter https://review.opendev.org/c/openstack/tripleo-heat-templates/+/797539 | 06:56 |
opendevreview | chandan kumar proposed openstack/tripleo-quickstart master: Use tripleo.repos.get_hash in repo-setup role https://review.opendev.org/c/openstack/tripleo-quickstart/+/791486 | 06:56 |
opendevreview | Juan 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/+/805274 | 07:22 |
jpodivin | There 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 |
jpodivin | That is, if we hit TypeError or JSONDecodeError while reading the json file. | 07:26 |
jpodivin | Should 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 rpittau | 07:38 | |
chandankumar | fmount: hello, please have a look at https://review.opendev.org/c/openstack/tripleo-heat-templates/+/805034/1#message-7fa06b7e656fd338e942d685a7863010ce0def74 | 07:38 |
chandankumar | when free, thanks :-) | 07:38 |
*** jpena|off is now known as jpena | 07:38 | |
opendevreview | Merged openstack/tripleo-heat-templates master: Add BGP EVPN support to the FRR service https://review.opendev.org/c/openstack/tripleo-heat-templates/+/798645 | 07:41 |
fmount | chandankumar: 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 |
fmount | nothing 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|lunch | 07:45 | |
fmount | chandankumar: let's wait for the next run, I'll keep an eye on it | 07:46 |
chandankumar | fmount: sure, thanks for looking into it . | 07:46 |
opendevreview | Jiri Podivin proposed openstack/python-tripleoclient master: Tests and documentation for the export_stack function. https://review.opendev.org/c/openstack/python-tripleoclient/+/805347 | 07:55 |
opendevreview | Jiri Podivin proposed openstack/python-tripleoclient master: New test case and documentation for the export_stack function. https://review.opendev.org/c/openstack/python-tripleoclient/+/805347 | 07:58 |
opendevreview | Jiri Podivin proposed openstack/python-tripleoclient master: New test case and documentation for the export_stack function. https://review.opendev.org/c/openstack/python-tripleoclient/+/805347 | 08:01 |
opendevreview | Rabi Mishra proposed openstack/puppet-tripleo stable/train: Set authtoken memcached_servers in undercloud services https://review.opendev.org/c/openstack/puppet-tripleo/+/805324 | 08:12 |
opendevreview | Jiri Podivin proposed openstack/validations-libs master: Add validation config file mechanism https://review.opendev.org/c/openstack/validations-libs/+/792460 | 08:12 |
opendevreview | Jiri Podivin proposed openstack/validations-libs master: Default value of the constructor parameter https://review.opendev.org/c/openstack/validations-libs/+/796482 | 08:12 |
opendevreview | Jiri Podivin proposed openstack/validations-libs master: Validation actions refactor https://review.opendev.org/c/openstack/validations-libs/+/802901 | 08:12 |
opendevreview | Jiri Podivin proposed openstack/validations-libs master: Expand validations_logs table with reason of failure https://review.opendev.org/c/openstack/validations-libs/+/804392 | 08:13 |
opendevreview | Jiri Podivin proposed openstack/tripleo-validations master: Fail validation if pacemaker service is not active https://review.opendev.org/c/openstack/tripleo-validations/+/805187 | 08:13 |
opendevreview | Jiri Podivin proposed openstack/python-tripleoclient stable/wallaby: Moving the 'export_data' dict into constants https://review.opendev.org/c/openstack/python-tripleoclient/+/805355 | 08:22 |
opendevreview | Jiri Podivin proposed openstack/python-tripleoclient stable/wallaby: Setting upper constraints to wallaby branch of requirements https://review.opendev.org/c/openstack/python-tripleoclient/+/805358 | 08:33 |
opendevreview | Brendan Shephard proposed openstack/tripleo-heat-templates master: Deprecate and replace NetworkDeploymentActions https://review.opendev.org/c/openstack/tripleo-heat-templates/+/805217 | 08:40 |
*** ysandeep|lunch is now known as ysandeep | 08:53 | |
opendevreview | Juan 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/+/805274 | 08:57 |
opendevreview | Juan 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/+/805276 | 08:59 |
opendevreview | Juan 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/+/805277 | 09:00 |
opendevreview | Rabi Mishra proposed openstack/puppet-tripleo stable/train: Set authtoken memcached_servers in undercloud services https://review.opendev.org/c/openstack/puppet-tripleo/+/805324 | 09:01 |
opendevreview | amolkahat proposed openstack/tripleo-quickstart-extras master: Added stack_name variable to overcloud-delete defaults https://review.opendev.org/c/openstack/tripleo-quickstart-extras/+/805260 | 09:06 |
opendevreview | Juan 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/+/805277 | 09:12 |
opendevreview | amolkahat 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/+/804117 | 09:15 |
opendevreview | Takashi Kajinami proposed openstack/puppet-tripleo stable/train: [Train only] Update the authtoken class for undercloud services https://review.opendev.org/c/openstack/puppet-tripleo/+/805327 | 09:59 |
opendevreview | Vijayalakshmi proposed openstack/tripleo-heat-templates master: Update dervice_pci script to handle pci address formats https://review.opendev.org/c/openstack/tripleo-heat-templates/+/799318 | 10:41 |
opendevreview | Michele 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/+/803548 | 10:45 |
*** dviroel|ruck|out is now known as dviroel|ruck | 11:26 | |
*** bhagyashris_ is now known as bhagyashris | 11:31 | |
*** jpena is now known as jpena|lunch | 11:36 | |
*** rlandy is now known as rlandy|rover | 11:45 | |
opendevreview | Merged openstack/tripleo-common master: Fix upload-artifacts and its installation https://review.opendev.org/c/openstack/tripleo-common/+/804899 | 12:03 |
opendevreview | Jiri Podivin proposed openstack/python-tripleoclient stable/wallaby: Setting upper constraints to wallaby branch of requirements https://review.opendev.org/c/openstack/python-tripleoclient/+/805358 | 12:08 |
opendevreview | Merged openstack/tripleo-heat-templates master: Remove deprecated ipv6 corosync parameters https://review.opendev.org/c/openstack/tripleo-heat-templates/+/739936 | 12:26 |
opendevreview | Jiri Podivin proposed openstack/validations-common master: False positive checks for component jobs https://review.opendev.org/c/openstack/validations-common/+/803163 | 12:29 |
opendevreview | Juan 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/+/805277 | 12:31 |
jpodivin | hi 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 |
jpodivin | it's at the bottom of rather large dependency chain ... well tree really. | 12:34 |
jpodivin | We need it in tripleo-validations to unblock the ussuri pipeline | 12:34 |
*** jpena|lunch is now known as jpena | 12:36 | |
opendevreview | wes hayutin proposed openstack/tripleo-quickstart-extras master: Added stack_name variable to overcloud-delete defaults https://review.opendev.org/c/openstack/tripleo-quickstart-extras/+/805260 | 12:44 |
opendevreview | Merged openstack/tripleo-heat-templates master: container-puppet.sh idempotency on restart https://review.opendev.org/c/openstack/tripleo-heat-templates/+/805301 | 12:47 |
opendevreview | Jiri Podivin proposed openstack/validations-libs master: DNM moving callbacks to libs https://review.opendev.org/c/openstack/validations-libs/+/799188 | 13:20 |
opendevreview | Pooja Jadhav proposed openstack/openstack-tempest-skiplist master: Skip test_attach_volume_negative tempest test https://review.opendev.org/c/openstack/openstack-tempest-skiplist/+/805382 | 13:20 |
opendevreview | John Fulton proposed openstack/python-tripleoclient master: Introduce "openstack overcloud ceph deploy" https://review.opendev.org/c/openstack/python-tripleoclient/+/800341 | 13:30 |
opendevreview | John Fulton proposed openstack/python-tripleoclient master: Introduce "openstack overcloud ceph deploy" https://review.opendev.org/c/openstack/python-tripleoclient/+/800341 | 13:30 |
opendevreview | John Fulton proposed openstack/python-tripleoclient master: Introduce "openstack overcloud ceph deploy" https://review.opendev.org/c/openstack/python-tripleoclient/+/800341 | 13:39 |
opendevreview | Juan Badia Payno proposed openstack/tripleo-quickstart master: Install tripleo.operator collections https://review.opendev.org/c/openstack/tripleo-quickstart/+/805384 | 13:43 |
opendevreview | Merged openstack/openstack-tempest-skiplist master: Skip test_attach_volume_negative tempest test https://review.opendev.org/c/openstack/openstack-tempest-skiplist/+/805382 | 13:45 |
*** rpittau is now known as rpittau|afk | 13:55 | |
opendevreview | Jing Li proposed openstack/diskimage-builder master: Add new element rocky https://review.opendev.org/c/openstack/diskimage-builder/+/802902 | 14:09 |
opendevreview | Douglas Viroel proposed openstack/openstack-tempest-skiplist master: Update FloatingIpMultipleRoutersTest in skiplist https://review.opendev.org/c/openstack/openstack-tempest-skiplist/+/805393 | 14:25 |
*** ysandeep is now known as ysandeep|away | 14:31 | |
opendevreview | James Parker proposed openstack/tripleo-upgrade master: Add different migration actions based on guest https://review.opendev.org/c/openstack/tripleo-upgrade/+/798373 | 14:44 |
opendevreview | Sorin Sbârnea proposed openstack/tripleo-repos master: Address pyyaml import in ansible module https://review.opendev.org/c/openstack/tripleo-repos/+/805400 | 14:48 |
opendevreview | Douglas Viroel proposed openstack/openstack-tempest-skiplist master: Update FloatingIpMultipleRoutersTest in skiplist https://review.opendev.org/c/openstack/openstack-tempest-skiplist/+/805393 | 15:12 |
opendevreview | Juan Badia Payno proposed openstack/tripleo-quickstart master: Install tripleo.operator collections https://review.opendev.org/c/openstack/tripleo-quickstart/+/805384 | 15:17 |
opendevreview | Jiri Podivin proposed openstack/python-tripleoclient master: Eliminating a race condition in the export_stack function https://review.opendev.org/c/openstack/python-tripleoclient/+/805409 | 15:20 |
zbr | mwhahaha: 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 |
opendevreview | amolkahat 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/+/804117 | 15:34 |
opendevreview | Jiri Podivin proposed openstack/python-tripleoclient master: Enabling cover job in the zuul https://review.opendev.org/c/openstack/python-tripleoclient/+/805414 | 15:45 |
weshay|ruck | HAPPY TRIPLEO INDEPENDENCE DAY... https://review.opendev.org/c/openstack/releases/+/804871 | 15:45 |
weshay|ruck | really was yesterday.. throw a party | 15:45 |
fultonj | ha ha weshay|ruck | 15:45 |
*** jpena is now known as jpena|off | 15:51 | |
opendevreview | Merged openstack/tripleo-heat-templates master: Enable CephDashboard for scenario001 https://review.opendev.org/c/openstack/tripleo-heat-templates/+/804896 | 15:52 |
jpodivin | fultonj: 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#map | 15:53 |
fultonj | jpodivin: good catch, i'll change the name | 15:54 |
jpodivin | fultonj: thank you! | 15:55 |
*** pojadhav- is now known as pojadhav | 16:00 | |
opendevreview | Francesco Pantano proposed openstack/tripleo-heat-templates stable/wallaby: Enable CephDashboard for scenario001 https://review.opendev.org/c/openstack/tripleo-heat-templates/+/805280 | 16:00 |
opendevreview | Merged openstack/tripleo-heat-templates stable/wallaby: Move the remaining wallaby jobs to cephadm https://review.opendev.org/c/openstack/tripleo-heat-templates/+/805034 | 16:13 |
opendevreview | Merged openstack/tripleo-validations stable/ussuri: Trigger molecule jobs when modifying molecule-requirements.txt file https://review.opendev.org/c/openstack/tripleo-validations/+/805235 | 16:13 |
opendevreview | Merged openstack/tripleo-validations stable/ussuri: [ussuri-only] Stop requiring ansible 2.8 https://review.opendev.org/c/openstack/tripleo-validations/+/805290 | 16:13 |
opendevreview | Merged openstack/tripleo-validations stable/victoria: Trigger molecule jobs when modifying molecule-requirements.txt file https://review.opendev.org/c/openstack/tripleo-validations/+/805233 | 16:13 |
lbragstad | o/ hey folks - i'm curious if anyone else would like to weigh in on https://review.opendev.org/c/openstack/tripleo-heat-templates/+/781571 | 16:40 |
lbragstad | specifically, 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 |
mwhahaha | lbragstad: would we ever need to partially enable rbac? | 16:47 |
mwhahaha | if so, i would not use a global | 16:47 |
mwhahaha | i think it's best to use service specific ones for the foreseeable future | 16:48 |
lbragstad | that'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 personas | 16:48 |
lbragstad | then a service-specific knob seems useful | 16:48 |
mwhahaha | you could intheory have both | 16:49 |
mwhahaha | if global is true, use that. else use service value | 16:50 |
mwhahaha | we used to do something similar for Debug | 16:50 |
lbragstad | yeah, that's true | 16:50 |
lbragstad | my initial thought was to have the global behavior encapsulated in a template (referenced in the patch) | 16:50 |
mwhahaha | we can also change it later | 16:51 |
lbragstad | so if an operator really understands what they're doing and they want to opt into the behavior, they include the template | 16:51 |
mwhahaha | i think the service one is the start | 16:51 |
lbragstad | ok | 16:51 |
mwhahaha | and we come through and add a global when it works everywhere | 16:51 |
opendevreview | amolkahat 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/+/800338 | 16:51 |
lbragstad | mwhahaha 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 |
lbragstad | but that will eventually make the global and service specific knobs obsolete | 16:52 |
mwhahaha | yea | 16:53 |
* fultonj is on PTO this afternoon | 17:14 | |
fultonj | see you monday | 17:14 |
opendevreview | Damien Ciabrini proposed openstack/tripleo-ansible stable/wallaby: tripleo_push_artifacts: ensure destination exists https://review.opendev.org/c/openstack/tripleo-ansible/+/805281 | 17:26 |
opendevreview | Brent Eagles proposed openstack/tripleo-ansible master: DNM: WIP: working on some cert repair playbooks for octavia https://review.opendev.org/c/openstack/tripleo-ansible/+/805442 | 17:30 |
opendevreview | Damien Ciabrini proposed openstack/tripleo-ansible stable/wallaby: tripleo_push_artifacts: fix argument parsing https://review.opendev.org/c/openstack/tripleo-ansible/+/805282 | 17:31 |
opendevreview | Merged openstack/openstack-tempest-skiplist master: Update FloatingIpMultipleRoutersTest in skiplist https://review.opendev.org/c/openstack/openstack-tempest-skiplist/+/805393 | 18:48 |
opendevreview | Kevin Carter proposed openstack/python-tripleoclient master: Ensure directories exist before copying https://review.opendev.org/c/openstack/python-tripleoclient/+/805291 | 18:50 |
opendevreview | Merged openstack/python-tripleoclient stable/train: [train-only] Honour DeploymentServerBlacklist https://review.opendev.org/c/openstack/python-tripleoclient/+/804946 | 19:29 |
opendevreview | David Vallee Delisle proposed openstack/tripleo-ansible master: tripleo-kernel: fixing naming of workload protection https://review.opendev.org/c/openstack/tripleo-ansible/+/804189 | 20:46 |
opendevreview | Merged openstack/tripleo-docs master: ovs-dpdk: fixing deploy command https://review.opendev.org/c/openstack/tripleo-docs/+/802970 | 21:05 |
opendevreview | Merged openstack/tripleo-ha-utils master: setup.cfg: Replace dashes with underscores https://review.opendev.org/c/openstack/tripleo-ha-utils/+/788205 | 21:47 |
opendevreview | Merged openstack/tripleo-docs master: setup.cfg: Replace dashes with underscores https://review.opendev.org/c/openstack/tripleo-docs/+/788204 | 21:55 |
*** dviroel|ruck is now known as dviroel|out | 22:04 | |
opendevreview | Merged openstack/tripleo-quickstart-extras master: Fix multinode-ipa job from building-test-packages twice https://review.opendev.org/c/openstack/tripleo-quickstart-extras/+/804971 | 23:54 |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!