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

[doc error] install crds #3184

Closed
Hokwang opened this issue May 2, 2023 · 12 comments
Closed

[doc error] install crds #3184

Hokwang opened this issue May 2, 2023 · 12 comments
Assignees
Labels
kind/documentation Categorizes issue or PR as related to documentation.

Comments

@Hokwang
Copy link

Hokwang commented May 2, 2023

Describe the bug
In your doc, installation,
image

Steps to reproduce
that command occurs error.

kubectl apply -k "github.com/aws/eks-charts/stable/aws-load-balancer-controller//crds?ref=master"
Alias tip: k apply -k "github.com/aws/eks-charts/stable/aws-load-balancer-controller//crds?ref=master"
error: failed to run '/usr/local/bin/git fetch --depth=1 https://github.com/aws/eks-charts/stable/aws-load-balancer-controller master': remote: Not Found
fatal: repository 'https://github.com/aws/eks-charts/stable/aws-load-balancer-controller/' not found
: exit status 128

Expected outcome
install crds

Environment

  • AWS Load Balancer controller version
  • Kubernetes version 1.24
  • Using EKS (yes/no), if so version? yes

Additional Context:

@RyanRennCN
Copy link

Adding for awareness, this is the same issue noted here:
aws/eks-charts#939

@oliviassss
Copy link
Collaborator

Thanks for the feedback, seems like the Github URL pattern has changed recently, we will review and update the doc.
/kind documentation

@k8s-ci-robot k8s-ci-robot added the kind/documentation Categorizes issue or PR as related to documentation. label May 9, 2023
@oliviassss
Copy link
Collaborator

@Hokwang I could not repro this issue, it could be a github transient issue, can you try again with the command?

@eweidner
Copy link

@oliviassss I can confirm the issue still exists. I've been having this same issue for weeks. Luckily my current clusters have these already installed so I can comment out this step for upgrades. Still having this issue right now...

$ kubectl apply -k "github.com/aws/eks-charts/stable/aws-load-balancer-controller//crds?ref=master"
error: failed to run '/usr/bin/git fetch --depth=1 https://github.com/aws/eks-charts/stable/aws-load-balancer-controller master': remote: Not Found
fatal: repository 'https://github.com/aws/eks-charts/stable/aws-load-balancer-controller/' not found
: exit status 128

@mhenson1
Copy link

I didn't search adequately before creating my ticket and essentially created one for the same problem.
Can't install TargetGroupBinding CRDs with k8s 1.28

I was able to tie it to my client version of kubectl that I was running locally. If using 1.26 it worked, if I upgraded to 1.27 it didn't. As a workaround I had our automation pull down the crds.yaml file and apply it

wget https://raw.githubusercontent.com/aws/eks-charts/master/stable/aws-load-balancer-controller/crds/crds.yaml
kubectl apply -f crds.yaml

@j3mdamas
Copy link

j3mdamas commented Oct 3, 2023

Hi,

I also faced the same issue, but isn't it just a typo?

In the documentation, it's "github.com/aws/eks-charts/stable/aws-load-balancer-controller // crds?ref=master" while running with one forward slash works for me: "github.com/aws/eks-charts/stable/aws-load-balancer-controller / crds?ref=master"

@kasvith
Copy link

kasvith commented Oct 5, 2023

i faced same issue today

@oliviassss
Copy link
Collaborator

@j3mdamas, hi, neither of the links works for me, previously the older link works for me. I think it has something to do with the k8s version as @mhenson1 pointed out, I will update the doc

@oliviassss oliviassss self-assigned this Oct 12, 2023
@oliviassss oliviassss mentioned this issue Oct 12, 2023
6 tasks
@oliviassss
Copy link
Collaborator

Updated the doc to download the crds.yaml and then apply it. Closing this issue as for now, please feel free to reopen if there's any concern

@j3mdamas
Copy link

@j3mdamas, hi, neither of the links works for me, previously the older link works for me. I think it has something to do with the k8s version as @mhenson1 pointed out, I will update the doc

@oliviassss for me your fix seems fine. I just wanted to mention that it worked for me with the slash, and with the following version on kubectl:

Client Version: v1.27.1
Kustomize Version: v5.0.1
Server Version: v1.28.2-eks-f8587cb

@yixuandang
Copy link

aws/eks-charts#939

Same issue here.
╰─ kubectl apply -k "github.com/aws/eks-charts/stable/aws-load-balancer-controller//crds?ref=master"
error: failed to run '/usr/bin/git fetch --depth=1 https://github.com/aws/eks-charts/stable/aws-load-balancer-controller master': remote: Not Found
fatal: repository 'https://github.com/aws/eks-charts/stable/aws-load-balancer-controller/' not found
: exit status 128

@yixuandang
Copy link

https://raw.githubusercontent.com/aws/eks-charts/master/stable/aws-load-balancer-controller/crds/crds.yaml

single "/" worked for me somehow for kubectl 1.29. But I'm pretty sure on kubectl 1.23 I had double "/".

╰─ kubectl apply -k "github.com/aws/eks-charts/stable/aws-load-balancer-controller/crds?ref=master"
customresourcedefinition.apiextensions.k8s.io/ingressclassparams.elbv2.k8s.aws created
customresourcedefinition.apiextensions.k8s.io/targetgroupbindings.elbv2.k8s.aws created

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/documentation Categorizes issue or PR as related to documentation.
Projects
None yet
Development

No branches or pull requests

9 participants