Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

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 join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants