kill as root

Author: mishushakov

js/tests/supervisord.test.ts

@@ -19,10 +19,10 @@ sandboxTest('restart after jupyter kill', async ({ sandbox }) => {
   const initialHealth = await waitForHealth(sandbox)
   expect(initialHealth).toBe(true)
 
-  // Kill the jupyter process by pid
-  await sandbox.commands.run(
-    "kill -9 $(pgrep -f 'jupyter server')"
-  )
+  // Kill the jupyter process as root
+  await sandbox.commands.run("kill -9 $(pgrep -f 'jupyter server')", {
+    user: 'root',
+  })
 
   // Wait for supervisord to restart it and health to come back
   const recovered = await waitForHealth(sandbox, 10, 100)
@@ -38,10 +38,10 @@ sandboxTest('restart after code-interpreter kill', async ({ sandbox }) => {
   const initialHealth = await waitForHealth(sandbox)
   expect(initialHealth).toBe(true)
 
-  // Kill the code-interpreter process by pid
-  await sandbox.commands.run(
-    'kill -9 $(cat /var/run/code-interpreter.pid)'
-  )
+  // Kill the code-interpreter process as root
+  await sandbox.commands.run('kill -9 $(cat /var/run/code-interpreter.pid)', {
+    user: 'root',
+  })
 
   // Wait for supervisord to restart it and health to come back
   const recovered = await waitForHealth(sandbox, 10, 100)

python/tests/async/test_async_supervisord.py

@@ -19,7 +19,7 @@ async def test_restart_after_jupyter_kill(async_sandbox: AsyncSandbox):
     assert await wait_for_health(async_sandbox)
 
     # Kill the jupyter process by pid
-    await async_sandbox.commands.run("kill -9 $(pgrep -f 'jupyter server')")
+    await async_sandbox.commands.run("kill -9 $(pgrep -f 'jupyter server')", user="root")
 
     # Wait for supervisord to restart it and health to come back
     assert await wait_for_health(async_sandbox, 10, 100)
@@ -34,7 +34,7 @@ async def test_restart_after_code_interpreter_kill(async_sandbox: AsyncSandbox):
     assert await wait_for_health(async_sandbox)
 
     # Kill the code-interpreter process by pid
-    await async_sandbox.commands.run("kill -9 $(cat /var/run/code-interpreter.pid)")
+    await async_sandbox.commands.run("kill -9 $(cat /var/run/code-interpreter.pid)", user="root")
 
     # Wait for supervisord to restart it and health to come back
     assert await wait_for_health(async_sandbox, 10, 100)

python/tests/sync/test_supervisord.py

@@ -18,8 +18,8 @@ def test_restart_after_jupyter_kill(sandbox: Sandbox):
     # Verify health is up initially
     assert wait_for_health(sandbox)
 
-    # Kill the jupyter process by pid
-    sandbox.commands.run("kill -9 $(pgrep -f 'jupyter server')")
+    # Kill the jupyter process as root
+    sandbox.commands.run("kill -9 $(pgrep -f 'jupyter server')", user="root")
 
     # Wait for supervisord to restart it and health to come back
     assert wait_for_health(sandbox, 10, 100)
@@ -33,8 +33,10 @@ def test_restart_after_code_interpreter_kill(sandbox: Sandbox):
     # Verify health is up initially
     assert wait_for_health(sandbox)
 
-    # Kill the code-interpreter process by pid
-    sandbox.commands.run("kill -9 $(cat /var/run/code-interpreter.pid)")
+    # Kill the code-interpreter process as root
+    sandbox.commands.run(
+        "kill -9 $(cat /var/run/code-interpreter.pid)", user="root"
+    )
 
     # Wait for supervisord to restart it and health to come back
     assert wait_for_health(sandbox, 10, 100)