fix(agent-images): retain SSH client
test / integration-docker (pull_request) Successful in 22s
tracker-policy-pr / check-pr (pull_request) Successful in 22s
test / unit (pull_request) Successful in 46s
lint / lint (push) Successful in 51s
test / integration-firecracker (pull_request) Failing after 6m21s
test / coverage (pull_request) Has been skipped
test / publish-infra (pull_request) Has been skipped
test / integration-docker (pull_request) Successful in 22s
tracker-policy-pr / check-pr (pull_request) Successful in 22s
test / unit (pull_request) Successful in 46s
lint / lint (push) Successful in 51s
test / integration-firecracker (pull_request) Failing after 6m21s
test / coverage (pull_request) Has been skipped
test / publish-infra (pull_request) Has been skipped
This commit is contained in:
@@ -172,7 +172,7 @@ class TestSandboxEscape(unittest.TestCase):
|
||||
# base image without producing five confusing
|
||||
# command-not-found failures down the suite.
|
||||
missing: list[str] = []
|
||||
for tool in ("curl", "git", "dig"):
|
||||
for tool in ("curl", "git", "dig", "ssh"):
|
||||
r = cls._bottle.exec(f"command -v {tool} >/dev/null 2>&1")
|
||||
if r.returncode != 0:
|
||||
missing.append(tool)
|
||||
|
||||
@@ -29,6 +29,14 @@ class TestBuiltinAgentImages(unittest.TestCase):
|
||||
re.compile(r"(?m)^\s*podman(?:\s|\\|$)"),
|
||||
)
|
||||
|
||||
def test_all_install_ssh_client(self):
|
||||
for dockerfile in _AGENT_DOCKERFILES:
|
||||
with self.subTest(provider=dockerfile.parent.name):
|
||||
self.assertRegex(
|
||||
dockerfile.read_text(),
|
||||
re.compile(r"(?m)^\s*openssh-client(?:\s|\\|$)"),
|
||||
)
|
||||
|
||||
|
||||
if __name__ == "__main__":
|
||||
unittest.main()
|
||||
|
||||
Reference in New Issue
Block a user