opencurve / curve-operator

Curve Storage Orchestration for Kubernetes
23 stars 21 forks source link

Feat: upgrade go version 1.18 #53

Closed fengshunli closed 1 year ago

fengshunli commented 1 year ago

compile success?

yes

caoxianfei1 commented 1 year ago

compile success?

yes

compile success?

yes

you can make run for test.

fengshunli commented 1 year ago

compile success?

yes

compile success?

yes

you can make run for test.

make test ok

fengshunli commented 1 year ago
image

make run ok

fengshunli commented 1 year ago

use local kustomize

caoxianfei1 commented 1 year ago

use local kustomize

it is necessary to ensure make kustomize work.

fengshunli commented 1 year ago

make kustomize

image

everything is normal @caoxianfei1

caoxianfei1 commented 1 year ago

Older version should work under newer versions of the Go compiler without changes. However, new programs may not compile under older versions of the Go compiler. Is an upgrade necessary?

fengshunli commented 1 year ago

Yes, you need to upgrade go version 1.18 or above

caoxianfei1 commented 1 year ago

make kustomize

image everything is normal @caoxianfei1

make run cannot trigger download kustomize. you can execute make kustomize to verify

caoxianfei1 commented 1 year ago

make kustomize

image everything is normal @caoxianfei1

make run cannot trigger download kustomize. you can delete local kustomize and execute make kustomize to verify.

fengshunli commented 1 year ago

make kustomize

image everything is normal @caoxianfei1

make run cannot trigger download kustomize. you can delete local kustomize and execute make kustomize to verify.

make kustomize

image everything is normal @caoxianfei1

make run cannot trigger download kustomize. you can delete local kustomize and execute make kustomize to verify.

image

You can't go get here, you need go install to install it before it can be executed @caoxianfei1

fengshunli commented 1 year ago

@caoxianfei1 merge

caoxianfei1 commented 1 year ago

/cicheck

fengshunli commented 1 year ago

the Dockerfile need to update golang version to 1.18 together not to a new pr.

done @caoxianfei1

caoxianfei1 commented 1 year ago

/cicheck