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

--kube-feature-gates doesn't work in kwokctl --runtime=kind (v0.5.1) #1041

Closed
1 of 5 tasks
Huang-Wei opened this issue Apr 21, 2024 · 4 comments
Closed
1 of 5 tasks
Assignees
Labels
kind/bug Categorizes issue or PR as related to a bug.

Comments

@Huang-Wei
Copy link
Contributor

How to use it?

  • kwok
  • kwokctl --runtime=docker (default runtime)
  • kwokctl --runtime=binary
  • kwokctl --runtime=nerdctl
  • kwokctl --runtime=kind

What happened?

When running the following command:

kwokctl create cluster --runtime kind --kube-feature-gates MatchLabelKeysInPodAffinity=true

It reports:

ERROR: failed to create cluster: unable to decode config: yaml: unmarshal errors:
  line 86: cannot unmarshal !!seq into map[string]bool
ERROR Execute exit err="failed to start cluster \"kwok-kwok\": cmd wait: kind create cluster --config /Users/weih/.kwok/clusters/kwok/kind.yaml --name kwok-kwok --image docker.io/kindest/node:v1.29.0 --wait 1m: exit status 1"

What did you expect to happen?

--kube-feature-gates works as expected.

How can we reproduce it (as minimally and precisely as possible)?

See the steps in "What happened" section

Anything else we need to know?

It seems the bug was introduced back in #440, and continues to v0.5.1. However, recent refactoring PR #1021 removes the kind.yaml.tpl and hence fix this issue.

Kwok version

$ kwok --version
kwok version v0.5.1 go1.21.7 (darwin/arm64)
$ kwokctl --version
kwokctl version v0.5.1 go1.21.7 (darwin/arm64)

OS version

```console # On Linux: $ cat /etc/os-release # paste output here $ uname -a # paste output here

On Darwin:

$ uname -a

paste output here

On Windows:

C:> wmic os get Caption, Version, BuildNumber, OSArchitecture

paste output here

</details>
@Huang-Wei Huang-Wei added the kind/bug Categorizes issue or PR as related to a bug. label Apr 21, 2024
@Huang-Wei
Copy link
Contributor Author

@wzshiming this can be a hot fix in v0.5.2 (if it's planned), the solution can be either revert #440, or cherrypick #1021.

@wzshiming
Copy link
Member

/assign

@wzshiming
Copy link
Member

Fixed by #1047

/close

@k8s-ci-robot
Copy link
Contributor

@wzshiming: Closing this issue.

In response to this:

Fixed by #1047

/close

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.

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

No branches or pull requests

3 participants