Description of problem: cinder cron job definition should have randomized sleep command like puppet-heat for decreasing DB spike or lock contention. Version-Release number of selected component (if applicable): Current puppet-cinder. Additional info: Current puppet-cinder configures the cron job like ~~~ # HEADER: This file was autogenerated at 2018-07-13 06:00:08 +0000 by puppet. # HEADER: While it can still be managed manually, it is definitely not recommended. # HEADER: Note particularly that the comments starting with 'Puppet Name' should # HEADER: not be deleted, as doing so could cause duplicate cron jobs. # Puppet Name: cinder-manage db purge PATH=/bin:/usr/bin:/usr/sbin SHELL=/bin/sh 1 0 * * * cinder-manage db purge 30 >>/var/log/cinder/cinder-rowsflush.log 2>&1 ~~~ If many DB cron jobs are executed at the same time, it will trigger lock contention or DB spike. We should have randomized sleep in puppet-heat[0]. [0] https://github.com/openstack/puppet-heat/blob/master/manifests/cron/purge_deleted.pp#L70-L75
Looks like an improvement request, not an actual bug - marking as possible feature request for next releases.
Hello team, A bugzilla for nova[1] is MODIFIED status. Could you consider to implement it until RHOSP13z3? [1] https://bugzilla.redhat.com/1603015 Best Regards, Keigo Noha
Hello Christian, Is it possible to implement this request in RHOSP13z3. A request for nova is already MODIFIED status. This request is fixed at the same time as nova.
Hello Keigo, > Is it possible to implement this request in RHOSP13z3. unfortunately this won't be possible, there is no patch proposed upstream on master yet. As mentioned in comment #1, we see this as possible RFE for a future release, but so far this hasn't been planned for any specific release.
The puppet-cinder patch is currently under review on upstream stable/rocky, after which it will need to be submitted to stable/queens. There is also a THT patch to enable the randomized delay. Assigning to Pablo to track the upstream patches, and manage the downstream backports.
Hi Pablo, The backport to stable/queens was done in upstream. Would you please proceed the backport process into RHOSP13? Thanks, Keigo Noha
This bz requires patches in puppet-cinder, which have merged on stable/queens, and THT. The THT is still under review on upstream master, and will need to be backported.
Hi Alan, Thank you for your review in upstream. https://review.opendev.org/#/c/641563/ doen't have any progress for 2 weeks. Would you be able to ask someone to review the patch and proceed it if it looks good to him/her? Best Regards, Keigo Noha
Hello Keigo. As finally https://review.opendev.org/641563/ was merged, I will continue preparing the equivalents at product level.
Since the problem described in this bug report should be resolved in a recent advisory, it has been closed with a resolution of ERRATA. For information on the advisory, and where to find the updated files, follow the link below. If the solution does not work for you, open a new bug report. https://access.redhat.com/errata/RHBA-2019:1738