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

Migrate to Go modules #4

Merged
merged 4 commits into from
Jan 17, 2020
Merged

Migrate to Go modules #4

merged 4 commits into from
Jan 17, 2020

Conversation

sleshchenko
Copy link
Member

@sleshchenko sleshchenko commented Jan 14, 2020

What does this PR do?

This PR replaces Go Dep dependency management with newer approach Go Modules.

It also updates operator-framework to v0.9.0(the minimal buildable version that is compatible with modules).
v0.8.0 fails to compile with the following error message: cannot find module providing package github.com/appscode/jsonpatch

How is this PR tested?

  1. dockerimage is rebuilt
  2. A newer controller is deployed
  3. java-mysql workspace is created
  4. Theia is successfully opened

Signed-off-by: Sergii Leshchenko <sleshche@redhat.com>
Signed-off-by: Sergii Leshchenko <sleshche@redhat.com>
Signed-off-by: Sergii Leshchenko <sleshche@redhat.com>
Copy link
Collaborator

@amisevsk amisevsk left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Generally LGTM, but I get a build failure:

go: github.com/openshift/api@v3.9.1-0.20190424152011-77b8897ec79a+incompatible: invalid pseudo-version: preceding tag (v3.9.0) not found

With version

operator-sdk version: "v0.14.0", commit: "4fb185b9d45682473b4951c9788f845ba2fa5b12", go version: "go1.13.3 linux/amd64"

go.mod Show resolved Hide resolved
@sleshchenko
Copy link
Member Author

go: github.com/openshift/api@v3.9.1-0.20190424152011-77b8897ec79a+incompatible: invalid pseudo-version: preceding tag (v3.9.0) not found

It's strange. It works for me with the specified go and operator-framework versions and even after go clean --modcache. I will investigate it more.

@amisevsk
Copy link
Collaborator

go: github.com/openshift/api@v3.9.1-0.20190424152011-77b8897ec79a+incompatible: invalid pseudo-version: preceding tag (v3.9.0) not found

It's strange. It works for me with the specified go and operator-framework versions and even after go clean --modcache. I will investigate it more.

I figured out this issue, and left a comment on the other PR: #5 (comment)

@sleshchenko sleshchenko merged commit 5de092f into devfile:master Jan 17, 2020
@sleshchenko sleshchenko deleted the goModules branch January 17, 2020 07:43
JPinkney pushed a commit to JPinkney/che-workspace-operator that referenced this pull request May 13, 2021
Add a debug target to the make file
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

Successfully merging this pull request may close these issues.

2 participants