Daniel Grimm

Output usage hint for both unknown subcommand and unknown flag in the same way

4

Now, when you try to run a Command with an unknown sub-command vs. an unknown flag, you'll get: ``` $ testcommand asdsa Error: unknown command "asdsa" for "testcommand" Run 'testcommand...

Rename `chained` to `provider`, as it is more appropriate

3

**Please provide a description of this PR:** The pilot.cni.chained is really about whether to inject a multus-specific annotation rather than whether the CNI plugin is executed in chained or standalone...

Consolidate values API for CNI (values.cni vs values.istio_cni)

4

We currently have two almost-identical looking fields in the values that are concerned with CNI. The first is `values.cni` which if `enabled` is set to `true` will install CNI with...

I tried deploying the workshop myself, using the scripts in the `.workshop/scripts` directory, but ended up with a deployment that was broken in multiple ways: - `%username%` apparently wasn't set;...

Currently, the `run-build-container.sh` script does not work because apt cannot verify the repository signature. This PR fixes that by downloading the key and adjusting the sources.list entry.

We should at least verify that the examples work for every PR

None of this is currently unit tested.

This is taking Ted's work [from his v1.28 branch](https://github.com/tedjpoole/envoy-openssl/tree/release/v1.28), rebased onto upstream v1.30.1. It builds (by running `./openssl/run_envoy_docker.sh './openssl/do_ci.sh debug.server_only'`), it kind of works, but it does log a bunch...

Clean up operator API fields

2

We have a bunch of fields in the Operator API that have long been removed from the helm charts and have no effect anymore. I'm going through the proto file...

**Please provide a description of this PR:** Fixes #51987