starlingx/root
Code Issues Proposed changes

debian: aptly: use environment variable to control timeout

Browse Source 
In case the build server is busy, or the build server is not powerful
enough, aptly operation may timeout thus cause build failure. Add an
environment variable "REPOMGR_REQ_TIMEOUT_FACTOR" to let developer
increate the timeout value in runtime.

`export REPOMGR_REQ_TIMEOUT_FACTOR=10` will increase timeout value
by 10 times.

`unset REPOMGR_REQ_TIMEOUT_FACTOR` or set it to "1" will reset the timeout to
original value.

Story: 2010643
Task: 47770

Test Plan:
PASS: Enlarge timeout value for a long time task.

Signed-off-by: Zhang Xiao <xiao.zhang@windriver.com>
Change-Id: Iefad4d2de144ce6d4ebb5359a7b491e98c855b13
This commit is contained in:
Zhang Xiao 2023-04-03 13:50:52 +08:00
parent 8b1f2f0ea5
commit 8d41a490ce
1 changed files with 7 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
build-tools/stx/aptly_deb_usage.py
View File

@ -284,6 +284,9 @@ class Deb_aptly():
if count not in range(1, 30):
self.logger.error('Requested wait of % minutes is greater than 30 minutes max wait.', count)
return 'EINVAL'
timeout_factor = os.environ.get('REPOMGR_REQ_TIMEOUT_FACTOR')
if timeout_factor and timeout_factor.isdigit() and int(timeout_factor) != 0:
count *= int(timeout_factor)
while count > 0:
count -= 1
try:
@ -291,13 +294,16 @@ class Deb_aptly():
self.aptly.tasks.wait_for_task_by_id(task.id)
except Exception as e:
if count > 0:
self.logger.debug('Aptly task is still running...')
self.logger.debug('Aptly task %d(%s) is still running' % (task.id, task.name))
else:
self.logger.debug('%s' % e)
continue
else:
# return 'SUCCEEDED' or 'FAILED'
return self.aptly.tasks.show(task.id).state
self.logger.warn('Aptly task %d(%s) timeouts.' % (task.id, task.name))
self.logger.info('Environment variable REPOMGR_REQ_TIMEOUT_FACTOR can be used to increase timeout value.')
self.logger.info('For example, set it to "5" can increase the timeout value by 5 times.')
return 'TIMEOUTED'
# Publish a local repository directly, without snapshot or signature