Skip to content
This repository has been archived by the owner on Sep 12, 2023. It is now read-only.

Master fails to build #160

Closed
Jeffwan opened this issue Sep 19, 2021 · 4 comments
Closed

Master fails to build #160

Jeffwan opened this issue Sep 19, 2021 · 4 comments

Comments

@Jeffwan
Copy link
Member

Jeffwan commented Sep 19, 2021

➜  common git:(master) ✗ go build ./...
go: github.com/kubeflow/common/pkg/reconciler.v1/common: package github.com/go-logr/logr imported from implicitly required module; to add missing requirements, run:
        go get github.com/go-logr/[email protected]
go: github.com/kubeflow/common/test_job/reconciler.v1/test_job: package github.com/go-logr/logr imported from implicitly required module; to add missing requirements, run:
        go get github.com/go-logr/[email protected]

dependency is missing in go.mod and it's imported implicitly.

/cc @zw0610

@Jeffwan
Copy link
Member Author

Jeffwan commented Sep 19, 2021

After go get github.com/go-logr/[email protected], I get other errors.

➜  common git:(master) ✗ go build ./...
pkg/reconciler.v1/common/interface.go:27:2: no required module provides package sigs.k8s.io/controller-runtime; to add it:
        go get sigs.k8s.io/controller-runtime
pkg/reconciler.v1/common/gang.go:18:2: no required module provides package sigs.k8s.io/controller-runtime/pkg/client; to add it:
        go get sigs.k8s.io/controller-runtime/pkg/client
pkg/reconciler.v1/common/gang_volcano.go:32:2: no required module provides package sigs.k8s.io/controller-runtime/pkg/controller/controllerutil; to add it:
        go get sigs.k8s.io/controller-runtime/pkg/controller/controllerutil
pkg/reconciler.v1/common/reconciler.go:24:2: no required module provides package sigs.k8s.io/controller-runtime/pkg/log; to add it:
        go get sigs.k8s.io/controller-runtime/pkg/log
pkg/reconciler.v1/common/reconciler.go:25:2: no required module provides package sigs.k8s.io/controller-runtime/pkg/manager; to add it:
        go get sigs.k8s.io/controller-runtime/pkg/manager

@Jeffwan
Copy link
Member Author

Jeffwan commented Sep 19, 2021

After adding sigs.k8s.io/controller-runtime v0.7.2, it shows some reconciler errors

# github.com/kubeflow/common/pkg/reconciler.v1/common
pkg/reconciler.v1/common/reconciler.go:60:24: cannot use jobInter (type *KubeflowJobReconciler) as type JobInterface in assignment:
        *KubeflowJobReconciler does not implement JobInterface (missing ExtractJobStatus method)
pkg/reconciler.v1/common/reconciler.go:66:24: cannot use jobInter (type *KubeflowJobReconciler) as type JobInterface in assignment:
        *KubeflowJobReconciler does not implement JobInterface (missing ExtractJobStatus method)
pkg/reconciler.v1/common/reconciler.go:73:34: cannot use jobInter (type *KubeflowJobReconciler) as type JobInterface in assignment:
        *KubeflowJobReconciler does not implement JobInterface (missing ExtractJobStatus method)

@gaocegege
Copy link
Member

Do we have CI to test if it can be built successfully?

@Jeffwan
Copy link
Member Author

Jeffwan commented Sep 22, 2021

@gaocegege As you know, CI was removed somehow. #161 Address this issue and we can close it. Github Action has been added

@Jeffwan Jeffwan closed this as completed Sep 22, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants