Skip to content
Snippets Groups Projects

Deprecate `worker_version` usage and support `worker_run` where it was missing

Merged Eva Bardou requested to merge deprecate-worker-version into master
All threads resolved!
13 files
+ 532
115
Compare changes
  • Side-by-side
  • Inline
Files
13
@@ -72,7 +72,7 @@ class BaseWorker:
self.parser.add_argument(
"-c",
"--config",
help="Alternative configuration file when running without a Worker Version ID",
help="Alternative configuration file when running without a Worker Run ID",
type=open,
)
self.parser.add_argument(
@@ -94,7 +94,7 @@ class BaseWorker:
"--dev",
help=(
"Run worker in developer mode. "
"Worker will be in read-only state even if a worker_version is supplied. "
"Worker will be in read-only state even if a worker run is supplied. "
),
action="store_true",
default=False,
@@ -176,6 +176,14 @@ class BaseWorker:
"""
return self.args.dev or self.worker_run_id is None
@property
def worker_version_id(self):
"""Deprecated property previously used to retrieve the current WorkerVersion ID.
:raises DeprecationWarning: Whenever `worker_version_id` is used.
"""
raise DeprecationWarning("`worker_version_id` usage is deprecated")
def setup_api_client(self):
"""
Create an ArkindexClient to make API requests towards Arkindex instances.
@@ -243,10 +251,6 @@ class BaseWorker:
# Load worker version information
worker_version = worker_run["worker_version"]
# Store worker version id
self.worker_version_id = worker_version["id"]
self.worker_details = worker_version["worker"]
logger.info(f"Loaded {worker_run['summary']} from API")
Loading