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

docs: use ConfigMap for Cilium job based installation example #9039

Conversation

mattwillsher
Copy link

@mattwillsher mattwillsher commented Jul 20, 2024

Pull Request

What? (description)

Passing the command args to cilium CLI results in an error where the list elements are interpreted as a single argument, producing an error. Attempts to split them out lead to issues parsing the capabilities list.

Using a ConfigMap avoids these issues.

#9034 changes the same example.

Why? (reasoning)

I don't understand why the example in the docs didn't work for me, but this new approach worked both when applied directly and when run via the talos Terraform provider.

Acceptance

Please use the following checklist:

  • you linked an issue (if applicable)
  • you included tests (if applicable)
  • you ran conformance (make conformance)
  • you formatted your code (make fmt)
  • you linted your code (make lint)
  • you generated documentation (make docs)
  • you ran unit-tests (make unit-tests)

See make help for a description of the available targets.

Passing the command args to cilium CLI results in are error where list elements are
interpreted as a single argument, producing an error. Attempts to split them out
lead to issue parsing the capabilities list.

Use a ConfigMap avoids these issues.

Signed-off-by: Matt Willsher <[email protected]>
Copy link

github-actions bot commented Sep 4, 2024

This PR is stale because it has been open 45 days with no activity.

@github-actions github-actions bot added the Stale label Sep 4, 2024
@mattwillsher mattwillsher deleted the docs-cilium-method-5-config-map branch September 11, 2024 19:29
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.

1 participant