Tighten up the timing around Dockerfile commands. (#514)
Right now this timing also includes the snapshot time.
This commit is contained in:
parent
ee2baf8025
commit
2255837142
|
|
@ -227,6 +227,7 @@ func (s *stageBuilder) build() error {
|
||||||
return err
|
return err
|
||||||
}
|
}
|
||||||
files = command.FilesToSnapshot()
|
files = command.FilesToSnapshot()
|
||||||
|
timing.DefaultRun.Stop(t)
|
||||||
|
|
||||||
if !s.shouldTakeSnapshot(index, files) {
|
if !s.shouldTakeSnapshot(index, files) {
|
||||||
continue
|
continue
|
||||||
|
|
@ -250,7 +251,6 @@ func (s *stageBuilder) build() error {
|
||||||
if err := s.saveSnapshotToImage(command.String(), tarPath); err != nil {
|
if err := s.saveSnapshotToImage(command.String(), tarPath); err != nil {
|
||||||
return err
|
return err
|
||||||
}
|
}
|
||||||
timing.DefaultRun.Stop(t)
|
|
||||||
}
|
}
|
||||||
if err := cacheGroup.Wait(); err != nil {
|
if err := cacheGroup.Wait(); err != nil {
|
||||||
logrus.Warnf("error uploading layer to cache: %s", err)
|
logrus.Warnf("error uploading layer to cache: %s", err)
|
||||||
|
|
|
||||||
Loading…
Reference in New Issue