From ede49e908c690a6b37d43dbc2de5c989ae6c08a4 Mon Sep 17 00:00:00 2001 From: Balazs Nadasdi Date: Tue, 30 Nov 2021 11:31:57 +0100 Subject: [PATCH 1/2] fix: wait for pidfile --- test/e2e/e2e_test.go | 4 ++++ 1 file changed, 4 insertions(+) diff --git a/test/e2e/e2e_test.go b/test/e2e/e2e_test.go index e2f73d91..37199d43 100644 --- a/test/e2e/e2e_test.go +++ b/test/e2e/e2e_test.go @@ -49,6 +49,8 @@ func TestE2E(t *testing.T) { log.Println("TEST STEP: getting (and verifying) existing MicroVM") Eventually(func(g Gomega) error { + g.Expect(fmt.Sprintf(fcPath, mvmNS, secondMvmID) + "/firecracker.pid").To(BeAnExistingFile()) + // verify that firecracker has started and that a pid has been saved // and that there is actually a running process mvmPid1 = u.ReadPID(fmt.Sprintf(fcPath, mvmNS, mvmID)) @@ -67,6 +69,8 @@ func TestE2E(t *testing.T) { log.Println("TEST STEP: listing all MicroVMs") Eventually(func(g Gomega) error { + g.Expect(fmt.Sprintf(fcPath, mvmNS, secondMvmID) + "/firecracker.pid").To(BeAnExistingFile()) + // verify that firecracker has started and that a pid has been saved // and that there is actually a running process for the new mVM mvmPid2 = u.ReadPID(fmt.Sprintf(fcPath, mvmNS, secondMvmID)) From edb1fec9341481613bc5d1663cae95dea2c416bb Mon Sep 17 00:00:00 2001 From: Balazs Nadasdi Date: Tue, 30 Nov 2021 15:31:44 +0100 Subject: [PATCH 2/2] Update test/e2e/e2e_test.go Co-authored-by: Claudia --- test/e2e/e2e_test.go | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/test/e2e/e2e_test.go b/test/e2e/e2e_test.go index 37199d43..b5bbda4b 100644 --- a/test/e2e/e2e_test.go +++ b/test/e2e/e2e_test.go @@ -49,7 +49,7 @@ func TestE2E(t *testing.T) { log.Println("TEST STEP: getting (and verifying) existing MicroVM") Eventually(func(g Gomega) error { - g.Expect(fmt.Sprintf(fcPath, mvmNS, secondMvmID) + "/firecracker.pid").To(BeAnExistingFile()) + g.Expect(fmt.Sprintf(fcPath, mvmNS, mvmID) + "/firecracker.pid").To(BeAnExistingFile()) // verify that firecracker has started and that a pid has been saved // and that there is actually a running process