Require go.12 in module

[MalloZup]

avoid checksum errors that could be present and require minimal golang
version for fixing it. see (golang/go#29278)

IF we don't have this we will have some issues with the go.sum checksum

info:

On the OBS side, i dunno if this can cause issue I don' think

[jordimassaguerpla]

@MalloZup Could you check which go version is used to build caaspctl in Devel:CaaSP:4.0 and SUSE:SLE-15-SP1:U:P:CASP40? You can use the "osc buildinfo" command with the caaspctl package checkout.

If we use go1.12, I am fine with this change.

[nirmoy]

nirmoy@chapman:~/devel/cilium/home:ndas:branches:Devel:CaaSP:4.0/caaspctl> osc buildinfo|grep go1
  <bdep name="go1.11" version="1.11.9" release="1.12.1" arch="x86_64" project="SUSE:SLE-15:Update" repository="standard" />
nirmoy@chapman:~/devel/cilium/home:ndas:branches:Devel:CaaSP:4.0/caaspctl> 

[MalloZup]

I think we have 2 solution. This would be the cleanest one but I'm not sure how it can impact the pkg (obs, ci and openSUSE leap pkg). 2nd solution could be to remove go sum and put it to gitignore. It might be the less impact solution

[saschagrunert]

Sorry for jumping in here but @MalloZup requested my review. The change LGTM, but I would recommend to check your imports. For example the commit d687e77c8ae9 of the staging repository k8s.io/api seems to come from the master branch and not from the release-1.14 branch.

I think it would be cool to have a module validation as Makefile target, like this:
https://github.com/cri-o/cri-o/blob/b76dad9fd460699f31f9ca7de82d775eb1f1e3ec/Makefile#L210-L214

[jordimassaguerpla]

I just checked. So we have go1.12 because we use that for k8s. However, caaspctl is pulling go1.11, so we could use go1.12 but you need to adapt the spec file to make sure we pull go1.12

[jordimassaguerpla]

I just checked. So we have go1.12 because we use that for k8s. However, caaspctl is pulling go1.11, so we could use go1.12 but you need to adapt the spec file to make sure we pull go1.12

You most probably will have to change this https://github.com/SUSE/caaspctl/blob/master/ci/packaging/suse/caaspctl_spec_template#L30

[MalloZup]

I think this ISSUE require some thinking And discussion. For the moment i will try to don't solve the problem.

Creating an issue we can think later and closing this pr