Skip to content
This repository has been archived by the owner on Nov 18, 2024. It is now read-only.

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

Open
rjalander opened this issue Feb 4, 2022 · 3 comments

Comments

@rjalander
Copy link
Contributor

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/[email protected]: 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/[email protected]: missing go.sum entry; to add it:
        go mod download github.com/cdfoundation/sig-events/cde/sdk/go
@afrittoli
Copy link
Member

Thanks for the report @rjalander.
One prerequisite for the poc script is to clone the repos and setup the paths accordingly - see

sig-events/poc/poc.sh

Lines 32 to 37 in b58a23c

# - keptn inbound/outbound adapters and tekton cloudevent controller
# https://github.com/salaboy/cdf-events-keptn-adapter - inbound
# https://github.com/salaboy/keptn-cdf-translator - outbound
# https://github.com/tektoncd/experimental - contains tekton cloudevent controller
# should be cloned under $GOROOT/src/github.com/<org>/<repo> or alternatively
# the corresponding PATH environment variables must be set (see the declare section above)
. Could you confirm if you did that?

@rjalander
Copy link
Contributor Author

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
Copy link
Member

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?

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