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

[Backport release-1.30] Move generated CRDs into their own directory #4985

Merged

Conversation

twz123
Copy link
Member

@twz123 twz123 commented Sep 16, 2024

Backport to release-1.30:

This backport makes the CI for Windows unit tests much less flaky.

Introduce the static/_crds/* directories, which replace the
static/manifests/*/CustomResourceDefinition directories. The underscore
in the name prevents go tools from descending into the folder when
looking for go files. This makes it safe to run go commands at the same
time as generating the CRDs in this folder.

This prevents errors like the following:

  go: finding module for package github.com/k0sproject/k0s/static/manifests/v1beta1/CustomResourceDefinition
  go: finding module for package github.com/k0sproject/k0s/static/manifests/helm/CustomResourceDefinition
  go: finding module for package github.com/k0sproject/k0s/static/manifests/autopilot/CustomResourceDefinition
  go: finding module for package github.com/k0sproject/k0s/static/manifests/etcd/CustomResourceDefinition
  go: github.com/k0sproject/k0s/static/manifests/autopilot/CustomResourceDefinition: no matching versions for query "latest"
  go: github.com/k0sproject/k0s/static/manifests/etcd/CustomResourceDefinition: no matching versions for query "latest"
  go: github.com/k0sproject/k0s/static/manifests/helm/CustomResourceDefinition: no matching versions for query "latest"
  go: github.com/k0sproject/k0s/static/manifests/v1beta1/CustomResourceDefinition: no matching versions for query "latest"
  make: *** [Makefile:113: go.sum] Error 1

Signed-off-by: Tom Wieczorek <[email protected]>
(cherry picked from commit 583cf5d)
@twz123 twz123 added the chore label Sep 16, 2024
@twz123 twz123 marked this pull request as ready for review September 16, 2024 20:03
@twz123 twz123 requested a review from a team as a code owner September 16, 2024 20:03
@twz123 twz123 merged commit 70cdb88 into k0sproject:release-1.30 Sep 17, 2024
80 checks passed
@twz123 twz123 deleted the backport-4894-to-release-1.30 branch September 17, 2024 10:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants