Yashodhan

Hey, I re-tried the two cases on my machine through vagrant several times, and TestRegressionIssue6429 always passed, and TestContainerNoBinaryExists seems to fail. I'm trying to investigate that.

@guni1192 can I ask you to rebase and check again after https://github.com/containers/youki/pull/1347 is merged? It should fix the TestContainerNoBinaryExists failure

@guni1192 can I ask you to rebase and re-run the tests now that #1347 is merged? Thank you :)

@guni1192 @utam0k The CI is passing yay! :tada: :tada: :tada: @guni1192 Thanks a lot for your efforts and this PR :)

Sorry to ping the issue subscribers here, but is this still relevant? Two of the thing mentioned in above have been merged/closed, and the first issue has a related PR...

@utam0k Have you tried `TestWaitStoppedProcess` with the #1252 ? I tried it with changes of that PR, and it seem to pass without any issues.

With merge of #1204 , I think only one or two of the tests are still failing, I will check later and update which ones are failing.

The reason TestContainerKill is failing is because the way it checks for error here https://github.com/containerd/containerd/blob/main/pkg/process/utils.go#L135 is by string comparison of error message (mostly) , and as youki's error message is...

It would appear that with latest main (d99b665283f06b827a8762dc05a4b96e3a98a0df) we are passing all the tests * :tada: :tada: :tada: \* The tests themselves seem to be flaky, I ran the whole...

This is now completed with the merge of #968 !