Skip to content

Update#4

Merged
ydh926 merged 238 commits intoydh926:masterfrom
istio:master
Feb 8, 2020
Merged

Update#4
ydh926 merged 238 commits intoydh926:masterfrom
istio:master

Conversation

@ydh926
Copy link
Copy Markdown
Owner

@ydh926 ydh926 commented Feb 8, 2020

Please provide a description for what this PR is for.

And to help us figure out who should review this PR, please
put an X in all the areas that this PR affects.

[ ] Configuration Infrastructure
[ ] Docs
[ ] Installation
[ ] Networking
[ ] Performance and Scalability
[ ] Policies and Telemetry
[ ] Security
[ ] Test and Release
[ ] User Experience
[ ] Developer Infrastructure

bianpengyuan and others added 30 commits January 20, 2020 19:13
* add stats and mx wasm plugin into proxy image

* simplify wasm plugins downloading

* format & clean up
* Update root.go

* Update root.go

* Add Example cobra command for manifest apply

* Ran make gen

* Update operator/cmd/mesh/root.go

Formatting

Co-Authored-By: John Howard <howardjohn@google.com>

* Update root.go

Co-authored-by: John Howard <howardjohn@google.com>
* Fix: sse duplicate xds push

* fix ut
* Unify the readiness path for galley and istiod validation webook servers.
* Use the correct port for checking galley's readiness.
…20173)

* add e2e test for STS

* revise

* revise

* add test

* update comment

* sync master

* use fake JWT
* Add e2e tests with Jwt audience check. Also, add test case to show OR still work with the same issuer

* Fix make gen

* Add alpha policy for a to assert that it will be ignored

* Rename test case
* Set Host field appropriately when passing through probes

If someone sets a Host header, we shouldn't overwrite it back to
localhost. The health check might be affected by the value of that
header. We should ensure the host field is set to the provided host
header, if set on the probe

* cover :authority too
* Add ICP to IOP translation to manifest migrate command

* Comments

* Fix gen, tests
* remove the ability for pilot to register CRDs itself

Installing and modifying CRDs requires elevated cluster-wide
permissions. This should be done by a separate install step
(istio-init) or by an operator.

* fix controller_test.go

* make gen
… when using mTLS (#20126)

Update based on config

Fix tests

Trying to fix  tests

Just do for TCP

Enable Metadata Exchange for TCP when mTLS is on

cleanup

cleanup

Fixing lint  errors and running make gen

Fix make gen

Running make gen after rebase

Fix license file

Add option to enable metadata exchange
Fix licensing file

Add e2e test

Fix e2e test

Fix e2e test

Fix tests

Fix cleanup

Fix based on feedback

Add meta exc config to filter_types.go

Run make gen

Using INSERT_BEFORE to adding metadata exchange filter in the top of the list

Fix spacing nit

Fixed based on feedback

Fixed based on feedback

Fixed based on feedback

Fixed dup in comments

Co-authored-by: Nupur Garg <37600866+gargnupur@users.noreply.github.com>
* Improve nodeagent tests and error handling.

* Small fix.
* Remove certmanager CRD from e2e test

* format
This is now obsolete
* add e2e test for STS

* revise

* revise

* add test

* update comment

* sync master

* add test

* format

* add test

* add more tests

* revise

* add timeout test

* revise

* update comments

* format

* remove duplicated file
* Update deps to latest patch versions

* Update licenses/
Morven Cao and others added 29 commits February 4, 2020 20:13
* update api version.

* fix test.
* Rename operator/cmd/manager to operator/cmd/operator

* Update operator binary name to 'operator'

* Add back operator/cmd/mesh.go

* Update generated files

* Revert deletion of operator/Makefile.core.mk
* Re-enable CNI test

* Fixes
* Respect targetPort in gateway listener generation

* Change listening ports to unprivileged ports (>1023)

* Update operator's golden files

* Fix linting issues
* Remove python proto gen from operator

We don't use this, just clean up some extra junk

* fix
* Add analyzer for v1alpha1/Policy using JWT

It is possible that JWT authentication when used with the
v1alpha1/Policy API is misconfigured leading users to believe that their
cluster is secure when it is not. Warn the users of a misconfiguration;
the associated K8s service's port name should be prefixed with
http|http2|https.

* lint_istio job was failing due to v1alpha1/Policy use
* Remove all traces of nodeagent

Followup to #20777

* remove binary build
* Make istiod.enabled always true

* Gen
* Setup ExternalName instances only for specified ports

For #13479

Not sure if this is the right way to go

* fix typo
We need #18135 done to prevent
regressions in these tests
Message DeploymentRequiresServiceAssociated was receiving an extra parameter to its Sprintf formatter, which resulted in error messages like:

Warn [IST0117] (Deployment loadgenerator.hipster) No service associated with
this deployment. Service mesh deployments must be associated with a
service.%!(EXTRA string=loadgenerator)

This change removes the extra parameter (the deployment under question is already listed as the origin).
* Remove dependency on SSE in analyzer

- This change is required for istiod work and it allows
to use K8SCoreV1Services directly instead of SSE in analyzer's
destination host

* Address code review comments
* Apply beta peer authentication policy down to workload level

* Clean up

* Lint

* Check beta policy for auto mtls. This can be removed when EP metadata take into account the policy

* Use explicit peerauthentication policy for permissive, as we haven't remove old mesh policy during installation

* pilot/pkg/security/authn/v1beta1/policy_applier.go

* Move all test for beta mTLS api to the end

* Change to namespace policy

* Revert cluster.go

* Change peer authn consolidation algorithm for UNSET (inheritant mode)

* Reimplement getMostSpecificConfig (now composePeerAuthentication) which also consolidate port-level policies.

* Fix inheritance: do not inherit if it is weaker than the current mode

* Remove debug logs

* Change test policy to namespace level to make sure they are clean up properly with the existing test setup.

* Address comment

* Lint

* Simplify logic to pick the oldest

* fix typo

* Update function comment
* Merge

* fix merge

* Don't cache secrets loaded from file, no need for it.

It would allow rotation without restart or watching - envoy should ask
for new cert if it expires.

* Merge issues

* revert comment

* Move initialization of root cert file.

If pilot is killed (or crashes), the code path will find existing cert
and not generate a new one, resulting in wrong key getting patched.

* Fix the handling of 'user-supplied' certs

* Move method to initialize pub key to istio_ca

* format/fix

* Added few more comments and details to the flag

* Don't initialize DNS listener if the CA is missing

* Remove duplicated policy

* Fix bugs related to duplicated env variables.

* Format

* Merge errors

* Sync files

* Add extra config

* Cosmetic changes

* Add comments

* Make gen

* make gen again
* Derive istio namespace from CR, related refactor and cleanups

* Update goldens

* Lint

* Fix initialism

* Update goldens

* Restore previous injection goldens

* Review comments

* Fix injector input format to GenManifests
* Removal bad merge artifact

This is not used, was meant to be deleted

* gen
* fix sts port node metadata

* fix tes

* fix lint

* revert port to integer

* clean up

* more clean up
* initial commit

* refactor and add test output

* refactor the translate

* gen proto

* update make gen

* further clean up of naming files

* resolve conflict

* resolve comments

* address comment and resolve conflict

* update error message

* resolve conflict

* resolve conflict again
…n components (#20809)

* Add initial leader election

* Cleanup

* Add unit tests

* user leader election for ns controller, validation controller

* Log about ingress

* Add graceful shutdown

* Fix tests, add injector controller

* Fix lint

* Add renewal when lease is droppped, failing test

* wip

* Get everything working

* Fix build

* lint
* feat(grpc status codes): report grpc status codes via mixer

* response.grpc_status is a string

* force string instead of auto-detect duration

* update manifest files as well

* update to match istio/proxy PR

Signed-off-by: Douglas Reid <douglas-reid@users.noreply.github.com>

* update golden files

Signed-off-by: Douglas Reid <douglas-reid@users.noreply.github.com>

* add generated file

Signed-off-by: Douglas Reid <douglas-reid@users.noreply.github.com>

* running make gen a second time produces new results

Signed-off-by: Douglas Reid <douglas-reid@users.noreply.github.com>
*  update default profile

* fix gen
…20943)

* feat(injector): add initial support for canonical service revision labeling

Signed-off-by: Douglas Reid <douglas-reid@users.noreply.github.com>

* update bootstrap to match ICS proposal for v2 support
* Templating the cni validation container name

* Adding changed proto with new field

* gencheck

* Updating golden yaml

* More gencheck
* Apply peer authentication policy at port level

* Lint

* Fix e2e TestReachability/beta-per-port-mtls

* Fix mix api test

* Add log to test to debug resource apply/delete

* Add require env kube for new tests
@ydh926 ydh926 merged commit cdcda39 into ydh926:master Feb 8, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.