Skip to content

Conversation

@shaneutt
Copy link
Member

@shaneutt shaneutt commented Dec 2, 2021

What type of PR is this?

/kind bug
/kind regression

What this PR does / why we need it:

Previously I was able to deploy the Gateway CRDs with the following one-liner in CI:

$ kubectl kustomize https://github.com/kubernetes-sigs/gateway-api/config/crd | kubectl apply -f -

With the changes in 0d40986 this now fails:

$ kubectl kustomize https://github.com/kubernetes-sigs/gateway-api/config/crd | kubectl apply -f -
Error: accumulating resources: accumulation err='accumulating resources from 'v1alpha2/gateway.networking.k8s.io_gatewayclasses.yaml': evalsymlink failure on '/tmp/kustomize-2449990972/config/crd/v1alpha2/gateway.networking.k8s.io_gatewayclasses.yaml' : lstat /tmp/kustomize-2449990972/config/crd/v1alpha2: no such file or directory': evalsymlink failure on '/tmp/kustomize-2449990972/config/crd/v1alpha2/gateway.networking.k8s.io_gatewayclasses.yaml' : lstat /tmp/kustomize-2449990972/config/crd/v1alpha2: no such file or directory

This patch updates the kustomize resource list to match the new location of the stable CRDs.

A recent commit changed the directory name for the stable CRDs from
"v1alpha2" to "stable". The result is that CI I had running that was
deploying the CRDs via kustomize is now failing. This patch changes
the resource list of the kustomize configuration to point to the new
location of the CRDs.
@k8s-ci-robot k8s-ci-robot added kind/bug Categorizes issue or PR as related to a bug. kind/regression Categorizes issue or PR as related to a regression from a prior release. cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. size/S Denotes a PR that changes 10-29 lines, ignoring generated files. labels Dec 2, 2021
@hbagdi
Copy link
Contributor

hbagdi commented Dec 2, 2021

/lgtm

@k8s-ci-robot k8s-ci-robot added the lgtm "Looks good to me", indicates that a PR is ready to be merged. label Dec 2, 2021
@shaneutt shaneutt changed the title fix: kustomize pathing to stable CRDs fix: kustomize resource paths Dec 2, 2021
@robscott
Copy link
Member

robscott commented Dec 3, 2021

Thanks!

/approve

@k8s-ci-robot
Copy link
Contributor

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: robscott, shaneutt

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@k8s-ci-robot k8s-ci-robot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Dec 3, 2021
@k8s-ci-robot k8s-ci-robot merged commit 5f6560f into kubernetes-sigs:master Dec 3, 2021
@shaneutt shaneutt deleted the patch-1 branch December 3, 2021 18:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

approved Indicates a PR has been approved by an approver from all required OWNERS files. cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. kind/bug Categorizes issue or PR as related to a bug. kind/regression Categorizes issue or PR as related to a regression from a prior release. lgtm "Looks good to me", indicates that a PR is ready to be merged. size/S Denotes a PR that changes 10-29 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants