Skip to content

Commit 7a9fcdb

Browse files
committed
fix: regression to purge_caches in util/taskcluster.py
1 parent f374adc commit 7a9fcdb

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
@@ -383,14 +383,14 @@ def get_purge_cache_url(provisioner_id, worker_type):
383383

384384
def purge_cache(provisioner_id, worker_type, cache_name):
385385
"""Requests a cache purge from the purge-caches service."""
386+
worker_pool_id = f"{provisioner_id}/{worker_type}"
387+
386388
if testing:
387-
logger.info(f"Would have purged {provisioner_id}/{worker_type}/{cache_name}.")
389+
logger.info(f"Would have purged {worker_pool_id}/{cache_name}.")
388390
else:
389-
logger.info(f"Purging {provisioner_id}/{worker_type}/{cache_name}.")
391+
logger.info(f"Purging {worker_pool_id}/{cache_name}.")
390392
purge_cache_client = get_taskcluster_client("purgeCache")
391-
purge_cache_client.purgeCache(
392-
provisioner_id, worker_type, {"cacheName": cache_name}
393-
)
393+
purge_cache_client.purgeCache(worker_pool_id, {"cacheName": cache_name})
394394

395395

396396
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
@@ -361,7 +361,6 @@ def test_get_current_scopes(responses, root_url):
361361
assert result == ["foo", "bar"]
362362

363363

364-
@pytest.mark.xfail
365364
def test_purge_cache(responses, root_url):
366365
provisioner = "hardware"
367366
worker_type = "mac"

0 commit comments

Comments
 (0)