openstack
/
openstack
Code Issues Proposed changes

Update git submodules 

* Update devstack from branch 'master'
  to 0c9dbbbe1150abce7e5cf9929f1686a762d29bbc
  - Merge "Allow others to override NOVA_SERVICE_REPORT_INTERVAL"
  - Allow others to override NOVA_SERVICE_REPORT_INTERVAL
    
    While the patch where this was first introduced and set to 120 [1] is
    sensible for the vast majority of jobs, it's conceivable that some
    jobs might want a different value.
    
    Specifically, the whitebox-tempest-plugin changes configurations and
    restarts Nova services, and to do so it waits for the service status
    to update in the API before continuing with the tests. With the report
    interval set to 120 and the down time threshold set to 720, the
    service would continue showing 'up' in the API long after it was
    actually down, causing the wait to time out.
    
    Whitebox is a low-traffic project with only a couple of devstack jobs
    that run tempest tests sequentially (concurrency=1). Its CI is also
    pretty stable. It seems legitimate for it to keep the old default
    values of report_interval and service_down_time.
    
    This patch keeps the 120 default for NOVA_SERVICE_REPORT_INTERVAL, but
    makes it configurable by individual jobs. Since the original patch
    also introduced CINDER_SERVICE_REPORT_INTERVAL as a constant, make
    that configurable as well.
    
    [1] https://review.opendev.org/c/openstack/devstack/+/890439
    
    Needed-by: https://review.opendev.org/c/openstack/whitebox-tempest-plugin/+/891612
    Change-Id: I64fa2059537ea072a38fb4900d3c7d2d8f0ce429
This commit is contained in:
Zuul 2023-09-12 17:07:55 +00:00 committed by Gerrit Code Review
parent 7ca7b6a044
commit 34bc283fad
1 changed files with 1 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
devstack

@ -1 +1 @@
Subproject commit 428cb8a0a942164b2b13d1c08e3b746e306feb00
Subproject commit 0c9dbbbe1150abce7e5cf9929f1686a762d29bbc