builder: remove dependency on image
Signed-off-by: Tibor Vass <tibor@docker.com> Upstream-commit: 9be1ec60d4d4fe63d5ef6e1ec36c585b548a00d0 Component: engine
This commit is contained in:
@ -11,7 +11,6 @@ import (
|
||||
|
||||
"github.com/docker/docker/api/types"
|
||||
"github.com/docker/docker/daemon"
|
||||
"github.com/docker/docker/image"
|
||||
"github.com/docker/docker/runconfig"
|
||||
)
|
||||
|
||||
@ -112,9 +111,9 @@ type Backend interface {
|
||||
// TODO: use digest reference instead of name
|
||||
|
||||
// GetImage looks up a Docker image referenced by `name`.
|
||||
GetImage(name string) (*image.Image, error)
|
||||
GetImage(name string) (Image, error)
|
||||
// Pull tells Docker to pull image referenced by `name`.
|
||||
Pull(name string) (*image.Image, error)
|
||||
Pull(name string) (Image, error)
|
||||
// ContainerWsAttachWithLogs attaches to container.
|
||||
ContainerWsAttachWithLogs(name string, cfg *daemon.ContainerWsAttachWithLogsConfig) error
|
||||
// ContainerCreate creates a new Docker container and returns potential warnings
|
||||
|
||||
@ -18,8 +18,8 @@ import (
|
||||
"strings"
|
||||
|
||||
"github.com/Sirupsen/logrus"
|
||||
"github.com/docker/docker/builder"
|
||||
derr "github.com/docker/docker/errors"
|
||||
"github.com/docker/docker/image"
|
||||
flag "github.com/docker/docker/pkg/mflag"
|
||||
"github.com/docker/docker/pkg/nat"
|
||||
"github.com/docker/docker/pkg/signal"
|
||||
@ -210,7 +210,7 @@ func from(b *Builder, args []string, attributes map[string]bool, original string
|
||||
}
|
||||
|
||||
var (
|
||||
image *image.Image
|
||||
image builder.Image
|
||||
err error
|
||||
)
|
||||
// TODO: don't use `name`, instead resolve it to a digest
|
||||
|
||||
@ -24,7 +24,6 @@ import (
|
||||
"github.com/docker/docker/builder"
|
||||
"github.com/docker/docker/builder/dockerfile/parser"
|
||||
"github.com/docker/docker/daemon"
|
||||
"github.com/docker/docker/image"
|
||||
"github.com/docker/docker/pkg/archive"
|
||||
"github.com/docker/docker/pkg/httputils"
|
||||
"github.com/docker/docker/pkg/ioutils"
|
||||
@ -403,11 +402,11 @@ func containsWildcards(name string) bool {
|
||||
return false
|
||||
}
|
||||
|
||||
func (b *Builder) processImageFrom(img *image.Image) error {
|
||||
b.image = img.ID().String()
|
||||
func (b *Builder) processImageFrom(img builder.Image) error {
|
||||
b.image = img.ID()
|
||||
|
||||
if img.Config != nil {
|
||||
b.runConfig = img.Config
|
||||
b.runConfig = img.Config()
|
||||
}
|
||||
|
||||
// The default path will be blank on Windows (set by HCS)
|
||||
|
||||
9
components/engine/builder/image.go
Normal file
9
components/engine/builder/image.go
Normal file
@ -0,0 +1,9 @@
|
||||
package builder
|
||||
|
||||
import "github.com/docker/docker/runconfig"
|
||||
|
||||
// Image represents a Docker image used by the builder.
|
||||
type Image interface {
|
||||
ID() string
|
||||
Config() *runconfig.Config
|
||||
}
|
||||
Reference in New Issue
Block a user