kubernetes-sigs / cluster-api-provider-nested

Cluster API Provider for Nested Clusters
Apache License 2.0
298 stars 65 forks source link

🐛[VC] Failed to do port-forward for a pod in virtual cluster #332

Closed LuBingtan closed 1 year ago

LuBingtan commented 1 year ago

What steps did you take and what happened: I tried to port-forward for a pod in VC:

What did you expect to happen: Port-forward command should be succeeded.

Anything else you would like to add: It seems that the "portForward" here should be "portforward". https://github.com/kubernetes-sigs/cluster-api-provider-nested/blob/cd4b6eb6e248326e6152f7684ac6d25b01fa5fbf/virtualcluster/pkg/vn-agent/server/translate.go#L123-L125

I tried to fix this and have tested on my local environment, and it was working.

Environment:

/kind bug [One or more /area label. See https://github.com/kubernetes-sigs/cluster-api-provider-nested/labels?q=area for the list of labels]

LuBingtan commented 1 year ago

Opened a PR for this (#333 ), plz take a look /cc @wondywang @Fei-Guo @christopherhein