juergbi commented on code in PR #2031: URL: https://github.com/apache/buildstream/pull/2031#discussion_r2200664152
########## tests/integration/sandbox.py: ########## @@ -59,3 +60,24 @@ def test_remote_apis_socket(cli, datafiles): result = cli.run(project=project, args=["build", element_name]) assert result.exit_code == 0 + + +# Test configuration with remote action cache for nested REAPI. +@pytest.mark.skipif(not HAVE_SANDBOX, reason="Only available with a functioning sandbox") +@pytest.mark.datafiles(DATA_DIR) +def test_remote_apis_socket_with_action_cache(cli, tmpdir, datafiles): + project = str(datafiles) + element_name = "sandbox/remote-apis-socket.bst" + + with create_artifact_share(os.path.join(str(tmpdir), "remote")) as share: + cli.configure( + { + "remote-execution": { + "storage-service": {"url": share.repo}, + "action-cache-service": {"url": share.repo, "push": True}, Review Comment: I've added these configuration tests. Upload to cache storage-service was missing in case two different CAS remotes are configured. I've added this, reusing code we already had for remote execution. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: commits-unsubscr...@buildstream.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org