Merge pull request #3355 from thomasmckay/1316-exportactionlogsworker
1316 - exportactionlogsworker period to 60sec
This commit is contained in:
commit
c8bb6e0ec5
2 changed files with 4 additions and 2 deletions
|
@ -22,7 +22,7 @@ from util.useremails import send_logs_exported_email
|
||||||
logger = logging.getLogger(__name__)
|
logger = logging.getLogger(__name__)
|
||||||
|
|
||||||
|
|
||||||
POLL_PERIOD_SECONDS = 1
|
POLL_PERIOD_SECONDS = app.config.get('EXPORT_ACTION_LOGS_WORKER_POLL_PERIOD', 60)
|
||||||
|
|
||||||
EXPORT_LOGS_STORAGE_PATH = app.config.get('EXPORT_ACTION_LOGS_STORAGE_PATH', 'exportedactionlogs')
|
EXPORT_LOGS_STORAGE_PATH = app.config.get('EXPORT_ACTION_LOGS_STORAGE_PATH', 'exportedactionlogs')
|
||||||
MAXIMUM_WORK_PERIOD_SECONDS = app.config.get('EXPORT_ACTION_LOGS_MAXIMUM_SECONDS', 60 * 60) # 1 hour
|
MAXIMUM_WORK_PERIOD_SECONDS = app.config.get('EXPORT_ACTION_LOGS_MAXIMUM_SECONDS', 60 * 60) # 1 hour
|
||||||
|
|
|
@ -7,7 +7,7 @@ from httmock import urlmatch, HTTMock
|
||||||
|
|
||||||
from data import model
|
from data import model
|
||||||
from data.logs_model import logs_model
|
from data.logs_model import logs_model
|
||||||
from workers.exportactionlogsworker import ExportActionLogsWorker
|
from workers.exportactionlogsworker import ExportActionLogsWorker, POLL_PERIOD_SECONDS
|
||||||
|
|
||||||
from test.fixtures import *
|
from test.fixtures import *
|
||||||
|
|
||||||
|
@ -64,3 +64,5 @@ def test_process_queue_item(namespace, repo_name, expects_logs, app):
|
||||||
|
|
||||||
assert created_json['export_id'] == 'someid'
|
assert created_json['export_id'] == 'someid'
|
||||||
assert len(created_json['logs']) == (expected_count + 1)
|
assert len(created_json['logs']) == (expected_count + 1)
|
||||||
|
|
||||||
|
assert POLL_PERIOD_SECONDS == 60
|
||||||
|
|
Reference in a new issue