Skip to content

Commit ba598bc

Browse files
EXONER4TEDpoiana
authored andcommitted
simplify the check
Signed-off-by: Logan Bond <logantbond@gmail.com>
1 parent c697ed7 commit ba598bc

File tree

3 files changed

+29
-29
lines changed

3 files changed

+29
-29
lines changed

pkg/driverbuilder/builder/build.go

+22-23
Original file line numberDiff line numberDiff line change
@@ -27,29 +27,28 @@ var defaultImageTag = "latest" // This is overwritten when using the Makefile to
2727

2828
// Build contains the info about the on-going build.
2929
type Build struct {
30-
TargetType Type
31-
KernelConfigData string
32-
KernelRelease string
33-
KernelVersion string
34-
DriverVersion string
35-
Architecture string
36-
ModuleFilePath string
37-
ProbeFilePath string
38-
ModuleDriverName string
39-
ModuleDeviceName string
40-
BuilderImage string
41-
BuilderRepos []string
42-
ImagesListers []ImagesLister
43-
KernelUrls []string
44-
GCCVersion string
45-
RepoOrg string
46-
RepoName string
47-
Images ImagesMap
48-
RegistryName string
49-
RegistryUser string
50-
RegistryPassword string
51-
RegistryPlainHTTP bool
52-
BuilderImageNetworkMode string
30+
TargetType Type
31+
KernelConfigData string
32+
KernelRelease string
33+
KernelVersion string
34+
DriverVersion string
35+
Architecture string
36+
ModuleFilePath string
37+
ProbeFilePath string
38+
ModuleDriverName string
39+
ModuleDeviceName string
40+
BuilderImage string
41+
BuilderRepos []string
42+
ImagesListers []ImagesLister
43+
KernelUrls []string
44+
GCCVersion string
45+
RepoOrg string
46+
RepoName string
47+
Images ImagesMap
48+
RegistryName string
49+
RegistryUser string
50+
RegistryPassword string
51+
RegistryPlainHTTP bool
5352
}
5453

5554
func (b *Build) KernelReleaseFromBuildConfig() kernelrelease.KernelRelease {

pkg/driverbuilder/builder/builders.go

-5
Original file line numberDiff line numberDiff line change
@@ -239,11 +239,6 @@ type BuilderImageNetworkMode interface {
239239
BuilderImageNetMode() string
240240
}
241241

242-
// set the default builder image network mode to default
243-
func (b *Build) BuilderImageNetMode() string {
244-
return "default"
245-
}
246-
247242
func (b *Build) GetBuilderImage() string {
248243
if b.hasCustomBuilderImage() {
249244
// BuilderImage MUST have requested GCC installed inside

pkg/driverbuilder/docker.go

+7-1
Original file line numberDiff line numberDiff line change
@@ -205,9 +205,15 @@ func (bp *DockerBuildProcessor) Start(b *builder.Build) error {
205205
Image: builderImage,
206206
}
207207

208+
// check for any overridden builder image network modes by the builder
209+
var builderImageNetMode = container.NetworkMode("default")
210+
if vv, ok := v.(builder.BuilderImageNetworkMode); ok {
211+
builderImageNetMode = container.NetworkMode(vv.BuilderImageNetMode())
212+
}
213+
208214
hostCfg := &container.HostConfig{
209215
AutoRemove: true,
210-
NetworkMode: container.NetworkMode(b.BuilderImageNetworkMode),
216+
NetworkMode: builderImageNetMode,
211217
}
212218

213219
uid := uuid.NewUUID()

0 commit comments

Comments
 (0)