Skip to content

Commit 13866bf

Browse files
committed
fix: regression to purge_caches in util/taskcluster.py
1 parent 82d2e97 commit 13866bf

File tree

2 files changed

+5
-6
lines changed

2 files changed

+5
-6
lines changed

src/taskgraph/util/taskcluster.py

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -384,14 +384,14 @@ def get_purge_cache_url(provisioner_id, worker_type):
384384

385385
def purge_cache(provisioner_id, worker_type, cache_name):
386386
"""Requests a cache purge from the purge-caches service."""
387+
worker_pool_id = f"{provisioner_id}/{worker_type}"
388+
387389
if testing:
388-
logger.info(f"Would have purged {provisioner_id}/{worker_type}/{cache_name}.")
390+
logger.info(f"Would have purged {worker_pool_id}/{cache_name}.")
389391
else:
390-
logger.info(f"Purging {provisioner_id}/{worker_type}/{cache_name}.")
392+
logger.info(f"Purging {worker_pool_id}/{cache_name}.")
391393
purge_cache_client = get_taskcluster_client("purgeCache")
392-
purge_cache_client.purgeCache(
393-
provisioner_id, worker_type, {"cacheName": cache_name}
394-
)
394+
purge_cache_client.purgeCache(worker_pool_id, {"cacheName": cache_name})
395395

396396

397397
def send_email(address, subject, content, link):

test/test_util_taskcluster.py

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -359,7 +359,6 @@ def test_get_current_scopes(responses, root_url):
359359
assert result == ["foo", "bar"]
360360

361361

362-
@pytest.mark.xfail
363362
def test_purge_cache(responses, root_url):
364363
provisioner = "hardware"
365364
worker_type = "mac"

0 commit comments

Comments
 (0)