feat: delete container on failed hooks

README.md

@@ -16,7 +16,6 @@ This is a personal project for me to explore and better understand the OCI Runti
 - [ ] Implement [Cgroups v2](https://github.com/opencontainers/runtime-spec/blob/main/config-linux.md#control-groups)
 - [ ] Implement optional [Seccomp](https://github.com/opencontainers/runtime-spec/blob/main/config-linux.md#seccomp)
 - [ ] Implement optional [AppArmor](https://github.com/opencontainers/runtime-spec/blob/main/config.md#linux-process)
-- [ ] Rollback (step 12)
 - [ ] Build, version and package
 - [ ] Refactor and tidy-up
 

container/container_init.go

@@ -19,10 +19,18 @@ import (
 
 func (c *Container) Init(reexecCmd string, reexecArgs []string) error {
 	if err := c.ExecHooks("createRuntime"); err != nil {
+		if err := c.Delete(true); err != nil {
+			return fmt.Errorf("delete container: %w", err)
+		}
+
 		return fmt.Errorf("execute createRuntime hooks: %w", err)
 	}
 
 	if err := c.ExecHooks("createContainer"); err != nil {
+		if err := c.Delete(true); err != nil {
+			return fmt.Errorf("delete container: %w", err)
+		}
+
 		return fmt.Errorf("execute createContainer hooks: %w", err)
 	}
 

container/container_reexec.go

@@ -252,6 +252,10 @@ func (c *Container) Reexec() error {
 	}
 
 	if err := c.ExecHooks("startContainer"); err != nil {
+		if err := c.Delete(true); err != nil {
+			return fmt.Errorf("delete container: %w", err)
+		}
+
 		return fmt.Errorf("execute startContainer hooks: %w", err)
 	}
 

container/container_start.go

@@ -23,8 +23,8 @@ func (c *Container) Start() error {
 	}
 
 	if err := c.ExecHooks("prestart"); err != nil {
-		if err := c.ExecHooks("poststop"); err != nil {
-			fmt.Println("Warning: failed to execute poststop hooks: ", err)
+		if err := c.Delete(true); err != nil {
+			return fmt.Errorf("delete container: %w", err)
 		}
 
 		return fmt.Errorf("execute prestart hooks: %w", err)