Skip to content

Commit

Permalink
fix interference manager build bug in makefile (#9)
Browse files Browse the repository at this point in the history
Signed-off-by: songtao98 <songtao2603060@gmail.com>
  • Loading branch information
songtao98 authored Mar 2, 2023
1 parent 9af16a8 commit 9071d03
Show file tree
Hide file tree
Showing 2 changed files with 6 additions and 3 deletions.
4 changes: 3 additions & 1 deletion Makefile
Original file line number Diff line number Diff line change
Expand Up @@ -17,6 +17,8 @@ ENVTEST_K8S_VERSION = 1.22
# Set license header files.
LICENSE_HEADER_GO ?= hack/boilerplate/boilerplate.go.txt

PACKAGES ?= $(shell go list ./...)

# Get the currently used golang install path (in GOPATH/bin, unless GOBIN is set)
ifeq (,$(shell go env GOBIN))
GOBIN=$(shell go env GOPATH)/bin
Expand Down Expand Up @@ -93,7 +95,7 @@ build: build-interference-manager

.PHONY: build-interference-manager
build-interference-manager: ## Build interference-manager binary.
go build -o cmd/interference-manager main.go
go build -o bin/interference-manager cmd/interference-manager/main.go

# If you wish built the manager image targeting other platforms you can use the --platform flag.
# (i.e. docker build --platform linux/arm64 ). However, you must enable docker buildKit for it.
Expand Down
5 changes: 3 additions & 2 deletions cmd/interference-manager/main.go
Original file line number Diff line number Diff line change
Expand Up @@ -18,8 +18,6 @@ package main

import (
"flag"
interferencev1alpha1 "github.com/koordinator-sh/koordetector/apis/interference/v1alpha1"
"github.com/koordinator-sh/koordetector/pkg/interferencemanager/controllers"
"os"

// Import all Kubernetes client auth plugins (e.g. Azure, GCP, OIDC, etc.)
Expand All @@ -33,6 +31,9 @@ import (
"sigs.k8s.io/controller-runtime/pkg/healthz"
"sigs.k8s.io/controller-runtime/pkg/log/zap"
//+kubebuilder:scaffold:imports

interferencev1alpha1 "github.com/koordinator-sh/koordetector/apis/interference/v1alpha1"
"github.com/koordinator-sh/koordetector/pkg/interferencemanager/controllers"
)

var (
Expand Down

0 comments on commit 9071d03

Please sign in to comment.