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

Remove meshkit #71

Merged
merged 1 commit into from
Jan 20, 2021
Merged

Remove meshkit #71

merged 1 commit into from
Jan 20, 2021

Conversation

Aisuko
Copy link
Member

@Aisuko Aisuko commented Dec 28, 2020

Signed-off-by: aisuko urakiny@gmail.com

Description

Remove the github.com/kumarabd/gokit v0.2.0 which the package was outdated and import with meshkit each other, you can clone the code and run go mod tidy and you will get the message like below, that may not good idea,

 aisuko@redthirteen  ~/Documents/learn-layer5/smi-conformance   master  go mod tidy
go: finding module for package github.com/layer5io/gokit/logger
go: downloading k8s.io/client-go v0.17.3
go: downloading github.com/layer5io/kuttl v0.4.1-0.20200806180306-b7e46afd657f
go: downloading k8s.io/api v0.17.3
go: downloading sigs.k8s.io/controller-runtime v0.5.1
go: downloading k8s.io/apimachinery v0.17.3
go: downloading github.com/docker/docker v1.4.2-0.20190916154449-92cc603036dd
go: downloading github.com/dustinkirkland/golang-petname v0.0.0-20191129215211-8e5a1ed0cff0
go: downloading sigs.k8s.io/kind v0.7.0
go: downloading golang.org/x/crypto v0.0.0-20191011191535-87dc89f01550
go: downloading github.com/thoas/go-funk v0.6.0
go: downloading github.com/google/shlex v0.0.0-20191202100458-e7afc7fbc510
go: downloading github.com/alessio/shellescape v0.0.0-20190409004728-b115ca0f9053
go: downloading github.com/containerd/containerd v1.2.9
go: downloading github.com/gorilla/mux v1.6.2
go: downloading github.com/gorilla/context v1.1.1
go: downloading github.com/layer5io/gokit v0.1.31
go: found github.com/layer5io/gokit/logger in github.com/layer5io/gokit v0.1.31
go: github.com/layer5io/learn-layer5/smi-conformance imports
        github.com/layer5io/gokit/logger: github.com/layer5io/gokit@v0.1.31: parsing go.mod:
        module declares its path as: github.com/layer5io/meshkit
                but was required as: github.com/layer5io/gokit

So I'd like to fix that, but there may rest of the other issues. But I aim to remove replace github.com/kudobuilder/kuttl => github.com/layer5io/kuttl v0.4.1-0.20200806180306-b7e46afd657f ,because this exists in every adapter, it does not make sense.

This PR fixes #

Notes for Reviewers

Signed commits

  • Yes, I signed my commits.

Signed-off-by: aisuko <urakiny@gmail.com>
@welcome
Copy link

welcome bot commented Dec 28, 2020

Yay, your first pull request! 👍 A contributor will be by to give feedback soon. In the meantime, please review the Layer5 Community Welcome Guide and sure to join the community Slack.
Be sure to double-check that you have signed your commits. Here are instructions for making signing an implicit activity while peforming a commit.

@kumarabd
Copy link
Contributor

@Aisuko You won't be able to remove replace github.com/kudobuilder/kuttl => github.com/layer5io/kuttl v0.4.1-0.20200806180306-b7e46afd657f because this is a fork that we are maintaining in our layer5 project. This has custom changes which are not present in kuttl itself so we would have to have this to run the test harness. I agree with you to remove github.com/kumarabd/gokit and replace with github.com/layer5io/meshkit, same to be done for github.com/layer5io/gokit. After replacing them, all of your errors would get resolved.

@kumarabd kumarabd self-requested a review December 28, 2020 14:38
@Aisuko
Copy link
Member Author

Aisuko commented Dec 29, 2020

@Aisuko You won't be able to remove replace github.com/kudobuilder/kuttl => github.com/layer5io/kuttl v0.4.1-0.20200806180306-b7e46afd657f because this is a fork that we are maintaining in our layer5 project. This has custom changes which are not present in kuttl itself so we would have to have this to run the test harness. I agree with you to remove github.com/kumarabd/gokit and replace with github.com/layer5io/meshkit, same to be done for github.com/layer5io/gokit. After replacing them, all of your errors would get resolved.

Maybe I made you confused @kumarabd , I mean our projects meshkitand meshery-adapter-library all import learn-layer5/smi-conformance, and this repo also import meshkit too. So, we should remove it due to they import each other. And I know we maintain the custom version kuttl and I try to change the way let the replace only keep in this repo. As you know, all the adapter all includes replace kuttl in itself go.mod and I try to solve this. Make any sense to you?

@kumarabd
Copy link
Contributor

kumarabd commented Jan 7, 2021

@Aisuko yes that sounds good. Except we are going to remove learn-layer5 dependency from meshkit. So I think its good to have meshkit in here.

@kumarabd kumarabd merged commit 91dab06 into master Jan 20, 2021
@welcome
Copy link

welcome bot commented Jan 20, 2021

Thanks for your contribution to the Layer5 community! 🎉

Congrats!

      ⭐ Please star the project if you have yet to do so and sure to join the community Slack.

@Aisuko Aisuko deleted the remove-meshkit branch January 21, 2021 01:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants