code-generator is missing from this repo

[kubernetes]

code-generator is missing from this repo

[k8s-ci-robot]

@Kargakis:

In response to this:

/meow

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

[sttts]

dep is unlike to be used anytime soon.

You might be surprised soon ;)

[0xmichalis]

Another option is to create a dependency link between the generator and the sample repo with an empty import. Could that work?

import _ "k8s.io/code-generator"

[sttts]

Intentional?

Intentional? no. Known? yes. Godeps does not do it itself and the publisher does not fix it up (but could).

[munnerz]

Yep this is a bit of a tricky one right now - does Godeps support explicitly vendoring a package that the project doesn't actually depend upon?

Even if we make the publisher do this, we need to make sure a godeps save ./... in future doesn't wipe out the code-generator dependency (thinking about end-users who might copy and paste this repo to start their own)

[sttts]

(thinking about end-users who might copy and paste this repo to start their own)

An hack/update-godeps.sh could help I think. But I wonder whether we really have to protect the users from the vendoring tools so much. The published repo should of course work. But if users copy the code, they have to know their vendoring tool of choice. And yes, these are a pain in Goland.

[0xmichalis]

Or hack/update-codegen.sh can ensure the generator is cloned in the directory it expects to find it.

[sttts]

Or hack/update-codegen.sh can ensure the generator is cloned in the directory it expects to find it.

Which means it must be aware of its branch and pick the right version. Don't like that either. We need a sane vendoring tool like dep that can reference extra repos.

[0xmichalis]

/meow

[0xmichalis]

Which means it must be aware of its branch and pick the right version. Don't like that either. We need a sane vendoring tool like dep that can reference extra repos.

@sttts today code-generator and sample-controller follow the release branches of kubernetes/kubernetes. Unless that is going to change in the future, I think hack/update-codegen.sh should try to checkout the same branch in code-generatorthat is already checkout out in sample-controller when the script runs. Seems like a viable short-term fix, dep is unlike to be used anytime soon.

[carolynvs]

I am using dep to get started on a fork of this repo. Here's my config for vendoring the code generator (relies on dep v0.4.1+):

required = [
  "k8s.io/code-generator/cmd/client-gen"
]

[[constraint]]
  name = "k8s.io/client-go"
  version = "kubernetes-1.9.2"

[[constraint]]
  name = "k8s.io/api"
  version = "kubernetes-1.9.2"

[[constraint]]
  name = "k8s.io/apimachinery"
  version = "kubernetes-1.9.2"

[[constraint]]
  name = "k8s.io/code-generator"
  version = "kubernetes-1.9.2"

[prune]
  non-go = true
  go-tests = true
  unused-packages = true

  [[prune.project]]
    name = "k8s.io/code-generator"
    unused-packages = false
    non-go = false
    go-tests = false

  [[prune.project]]
    name = "k8s.io/gengo"
    unused-packages = false
    non-go = false
    go-tests = false

I haven't gotten to the point where everything is working so that may need to change some more. 😀

[sttts]

I haven't gotten to the point where everything is working so that may need to change some more.

What is still missing?

Am surprised that we have to disable pruning for required packages, at least the unused-packages = false part.

[carolynvs]

What I posted is a bit of a jenky hack to workaround golang/dep#1545 not being merged yet.

• The root of code-generator package doesn't contain go code, so dep is currently not letting me require it. As a workaround I am requiring a subpackage (cmd/codegen).
• The extra project level prune settings tell dep to not remove the root package contents. Dep thinks the root project is unused because I wasn't able to require the root project.

This is a temporary wart in dep hopefully. Long term, just the required should be sufficient.