InTheForest/moby
1
0
Fork 0
mirror of https://github.com/moby/moby.git synced 2026-01-11 18:51:37 +00:00
Code Issues Packages Projects Releases Wiki Activity

Merge pull request #47003 from LarsSven/fix-container-start-time

Browse Source 
Move StartedAt time to before starting the container
This commit is contained in:
Paweł Gronowski
2024-05-07 14:58:27 +02:00
committed by GitHub
parent 8e14f278c4 d4f61f92fd
commit 440836a8cf
6 changed files with 25 additions and 13 deletions
Download Patch File Download Diff File Expand all files Collapse all files

3
daemon/start.go
View File

@@ -179,6 +179,7 @@ func (daemon *Daemon) containerStart(ctx context.Context, daemonCfg *configStore
}
}()
startupTime := time.Now()
// TODO(mlaventure): we need to specify checkpoint options here
tsk, err := ctr.NewTask(context.TODO(), // Passing ctx caused integration tests to be stuck in the cleanup phase
checkpointDir, container.StreamConfig.Stdin() != nil || container.Config.Tty,
@@ -204,7 +205,7 @@ func (daemon *Daemon) containerStart(ctx context.Context, daemonCfg *configStore
}
container.HasBeenManuallyRestarted = false
container.SetRunning(ctr, tsk, true)
container.SetRunning(ctr, tsk, startupTime)
container.HasBeenStartedBefore = true
daemon.setStateCounter(container)