Merge pull request #226 from klihub/fixes/mount-injection

[release-0.3] resmgr: inject mount before container state update.
containers · Jan 10, 2024 · 8e8d5ca · 8e8d5ca
2 parents 7a0bf5e + 2071609
commit 8e8d5ca
Showing 1 changed file with 2 additions and 1 deletion.
diff --git a/pkg/resmgr/nri.go b/pkg/resmgr/nri.go
@@ -333,7 +333,6 @@ func (p *nriPlugin) CreateContainer(ctx context.Context, podSandbox *api.PodSand
 		c.UpdateState(cache.ContainerStateStale)
 		return nil, nil, fmt.Errorf("failed to allocate resources: %w", err)
 	}
-	c.UpdateState(cache.ContainerStateCreated)
 
 	c.InsertMount(&cache.Mount{
 		Destination: "/.nri-resource-policy",
@@ -342,6 +341,8 @@ func (p *nriPlugin) CreateContainer(ctx context.Context, podSandbox *api.PodSand
 		Options:     []string{"bind", "ro", "rslave"},
 	})
 
+	c.UpdateState(cache.ContainerStateCreated)
+
 	if err := p.runPostAllocateHooks(event, c); err != nil {
 		m.Error("%s: failed to run post-allocate hooks for %s: %v",
 			event, container.GetName(), err)