search

cdfoundation / sig-events

CDF Events Special Interest Group
Apache License 2.0
48 stars 34 forks source link

POC : Installing keptn Inbound fails - ko reference is not a valid import path #113

Open rjalander opened 2 years ago

rjalander commented 2 years ago

Applying config file for keptn-cdf-translator is failed with below error when executing poc.sh on a local server,

Installed all the prerequisite softwares and cloned the repositories,

<Server>:~/go/src/github.com/salaboy/keptn-cdf-translator$ ko apply -f config/
role.rbac.authorization.k8s.io/keptn-token-reader unchanged
serviceaccount/keptn-cdevents unchanged
rolebinding.rbac.authorization.k8s.io/keptn-cdevents-keptn-token-reader unchanged
Error: error processing import paths in "config/keptn-cdevents.yaml": error resolving image references: found strict reference but ko://github.com/salaboy/keptn-cdf-translator is not a valid import path: go/build: go list github.com/salaboy/keptn-cdf-translator: exit status 1
go: github.com/cdfoundation/sig-events/cde/sdk/go@v0.0.0-20210705175621-39a606459f2b: missing go.sum entry; to add it:
        go mod download github.com/cdfoundation/sig-events/cde/sdk/go
go: github.com/cdfoundation/sig-events/cde/sdk/go@v0.0.0-20210705175621-39a606459f2b: missing go.sum entry; to add it:
        go mod download github.com/cdfoundation/sig-events/cde/sdk/go
afrittoli commented 2 years ago

Thanks for the report @rjalander. One prerequisite for the poc script is to clone the repos and setup the paths accordingly - see https://github.com/cdfoundation/sig-events/blob/b58a23cd7f1553cd435818e48d1843f6c74570ba/poc/poc.sh#L32-L37. Could you confirm if you did that?

rjalander commented 2 years ago

Yes @afrittoli, I have cloned the repos and set the GOROOT and GOPATH before running the script.

While debugging further found that, there is a permission issue with the generated file ../salaboy/keptn-cdf-translator/go.sum file, updating the permission for the file (chmod 775 go.sum) solved this issue.

afrittoli commented 2 years ago

Thanks @rjalander. go.sum is not in https://github.com/salaboy/keptn-cdf-translator, perhaps we should add it to avoid this kind of issus - @salaboy wdyt?