mirror of
https://github.com/microsoft/onefuzz.git
synced 2025-06-15 19:38:11 +00:00
increase the polling period in functional tests (#1682)
* increase the polling period * update timeout to 30s * format
This commit is contained in:
@ -34,8 +34,7 @@ from onefuzz.backend import ContainerWrapper, wait
|
|||||||
from onefuzz.cli import execute_api
|
from onefuzz.cli import execute_api
|
||||||
from onefuzztypes.enums import OS, ContainerType, TaskState, VmState
|
from onefuzztypes.enums import OS, ContainerType, TaskState, VmState
|
||||||
from onefuzztypes.models import Job, Pool, Repro, Scaleset, Task
|
from onefuzztypes.models import Job, Pool, Repro, Scaleset, Task
|
||||||
from onefuzztypes.primitives import (Container, Directory, File, PoolName,
|
from onefuzztypes.primitives import Container, Directory, File, PoolName, Region
|
||||||
Region)
|
|
||||||
from pydantic import BaseModel, Field
|
from pydantic import BaseModel, Field
|
||||||
|
|
||||||
LINUX_POOL = "linux-test"
|
LINUX_POOL = "linux-test"
|
||||||
@ -260,13 +259,16 @@ def retry(
|
|||||||
|
|
||||||
|
|
||||||
class TestOnefuzz:
|
class TestOnefuzz:
|
||||||
def __init__(self, onefuzz: Onefuzz, logger: logging.Logger, test_id: UUID) -> None:
|
def __init__(
|
||||||
|
self, onefuzz: Onefuzz, logger: logging.Logger, test_id: UUID, polling_period=30
|
||||||
|
) -> None:
|
||||||
self.of = onefuzz
|
self.of = onefuzz
|
||||||
self.logger = logger
|
self.logger = logger
|
||||||
self.test_id = test_id
|
self.test_id = test_id
|
||||||
self.project = f"test-{self.test_id}"
|
self.project = f"test-{self.test_id}"
|
||||||
self.start_log_marker = f"integration-test-injection-error-start-{self.test_id}"
|
self.start_log_marker = f"integration-test-injection-error-start-{self.test_id}"
|
||||||
self.stop_log_marker = f"integration-test-injection-error-stop-{self.test_id}"
|
self.stop_log_marker = f"integration-test-injection-error-stop-{self.test_id}"
|
||||||
|
self.polling_period = polling_period
|
||||||
|
|
||||||
def setup(
|
def setup(
|
||||||
self,
|
self,
|
||||||
@ -582,7 +584,7 @@ class TestOnefuzz:
|
|||||||
return (not bool(jobs), msg, self.success)
|
return (not bool(jobs), msg, self.success)
|
||||||
|
|
||||||
if poll:
|
if poll:
|
||||||
return wait(check_jobs_impl)
|
return wait(check_jobs_impl, frequency=self.polling_period)
|
||||||
else:
|
else:
|
||||||
_, msg, result = check_jobs_impl()
|
_, msg, result = check_jobs_impl()
|
||||||
self.logger.info(msg)
|
self.logger.info(msg)
|
||||||
@ -724,7 +726,7 @@ class TestOnefuzz:
|
|||||||
msg = "waiting on %d repros" % len(repros)
|
msg = "waiting on %d repros" % len(repros)
|
||||||
return (not bool(repros), msg, self.success)
|
return (not bool(repros), msg, self.success)
|
||||||
|
|
||||||
return wait(check_repro_impl)
|
return wait(check_repro_impl, frequency=self.polling_period)
|
||||||
|
|
||||||
def get_jobs(self) -> List[Job]:
|
def get_jobs(self) -> List[Job]:
|
||||||
jobs = self.of.jobs.list(job_state=None)
|
jobs = self.of.jobs.list(job_state=None)
|
||||||
@ -837,7 +839,7 @@ class TestOnefuzz:
|
|||||||
# order.
|
# order.
|
||||||
|
|
||||||
self.inject_log(self.stop_log_marker)
|
self.inject_log(self.stop_log_marker)
|
||||||
wait(self.check_log_end_marker, frequency=5.0)
|
wait(self.check_log_end_marker, frequency=self.polling_period)
|
||||||
self.logger.info("application insights log flushed")
|
self.logger.info("application insights log flushed")
|
||||||
|
|
||||||
logs = self.of.debug.logs.keyword("error", limit=100000, timespan="PT3H")
|
logs = self.of.debug.logs.keyword("error", limit=100000, timespan="PT3H")
|
||||||
|
Reference in New Issue
Block a user