Maybe we should also check that pid matches the pid shown in runc list? Or is that not really worth the effort?

It is not necessary to check that pid matches the pid shown in runc list. but we can check the content of the file as following:

run cat pid.txt
[ "status" -eq 0 ]
[[ ${lines[0]} =~ [0-9]+ ]]

I'm not sure that it's "not necessary". I don't think we have a test that makes sure that pid-file contains the right information? That's why I was suggesting it. But yeah, that change is also fine.

I thought for a moment. the pid is sent by C program to Go program and write to file by Go. It is necessary to check it matches. I update the patch with

run cat pid.txt
[ "status" -eq 0 ]
[[ ${lines[0]} =~ [0-9]+ ]]

I will open another pr to achieve this, because other integration tests also need to check, eg: start_hello.bats,start_detached.bats.

thanks

Alright, sure.