Skip to content

Merge release-1.0 into master#7314

Merged
rshriram merged 160 commits intoistio:masterfrom
rshriram:master
Jul 23, 2018
Merged

Merge release-1.0 into master#7314
rshriram merged 160 commits intoistio:masterfrom
rshriram:master

Conversation

@rshriram
Copy link
Copy Markdown
Member

@rshriram rshriram commented Jul 23, 2018

Signed-off-by: Shriram Rajagopalan shriramr@vmware.com

cc @nmittler

quanjielin and others added 30 commits July 3, 2018 12:45
…tio#6818)

* proxy-status diff of what Pilot has not what it has sent to Envoy

* fix copyright year
* fix bug so that destination.service.** attributes are collected (istio#6801)

* remove unnecessary generated attributes finding. (istio#6785)

* modify docker template files for proxyv2 (istio#6790)

* Long-running testing improvements (istio#6800)

* Add values for config map settings, including access log.
More docs.

* Updates and improvements for the stress-testing configs.

* Add values for config map settings, including access log. (istio#6797)

* Add values for config map settings, including access log.
More docs.

* Updates and improvements for the stress-testing configs.

* Address review comments

* Merged wrong files

* Add the setup helm file - this change now depend on the previous PR.

* Sync with remote, remove accidentally added files.

* Another accidental file

* SNI routing from sidecar to gateway with virtual services (istio#6402)

* quick sni matching 1st pass with no refactoring of existing code

* use shriram's api sha

* quick pass at using tls block

* add some validation

* copyright

* fix lint + remove deadcode

* rename protocol tcp_tls -> tls

* update back to istio/api master

* remove accidentally added test file

* add tls block to gateway logic

* add todos

* basic sni wildcard implementation

* add tcp, fix problems with rbac, matching

* better tcp + tls validation

* address code review comments

* remove out of date comment

* update comments

* fix compile error

* use tcp proxy in tcp routing

* add tcp routing e2e test

* add forgotten vs config file + update description of test

* Comments, bug fixes

Signed-off-by: Shriram Rajagopalan <shriramr@vmware.com>

* cleanup gateway tcp test

Signed-off-by: Shriram Rajagopalan <shriramr@vmware.com>

* moving networking test yamls

Signed-off-by: Shriram Rajagopalan <shriramr@vmware.com>

* tcp/tls tests

Signed-off-by: Shriram Rajagopalan <shriramr@vmware.com>

* yaml fixes

Signed-off-by: Shriram Rajagopalan <shriramr@vmware.com>

* fix file switcheroo

* port matches

Signed-off-by: Shriram Rajagopalan <shriramr@vmware.com>

* fix authN plugin overwriting TLS context

Signed-off-by: Shriram Rajagopalan <shriramr@vmware.com>

* more tests - route via egress gateway

Signed-off-by: Shriram Rajagopalan <shriramr@vmware.com>

* yaml fixes

Signed-off-by: Shriram Rajagopalan <shriramr@vmware.com>

* initialize prom variables

Signed-off-by: Shriram Rajagopalan <shriramr@vmware.com>

* split tests

Signed-off-by: Shriram Rajagopalan <shriramr@vmware.com>

* final test fix hopefully

Signed-off-by: Shriram Rajagopalan <shriramr@vmware.com>

* revert gateway tweaks

Signed-off-by: Shriram Rajagopalan <shriramr@vmware.com>
Remove noisy log, makes pilot hard to debug.
* 503s: create DestinationRules before VirtualServices

* Remove redundant route-rule-all-v1-mtls.yaml

* Update the tests

* Fix the bookinfo with auth test
* Updates to accommodate move of api/config/mcp to api/mcp.

* Update dependencies.

* Updated dep and reran due to linter problem. Reverted license file
updates.
Signed-off-by: Kuat Yessenov <kuat@google.com>
With mixer disabled, continuous 503s occur.
* Add proxy version info to envoy metadata

Signed-off-by: Shriram Rajagopalan <shriramr@vmware.com>

* use proxy tag as version

Signed-off-by: Shriram Rajagopalan <shriramr@vmware.com>

* fix tests

Signed-off-by: Shriram Rajagopalan <shriramr@vmware.com>

* update goldens

* detect proxy version in pilot

Signed-off-by: Shriram Rajagopalan <shriramr@vmware.com>

* added istioProxyVersion to helm values.yaml.  updated tests

* remove unused env var logic from istioctl

* undo env var removal

Signed-off-by: Shriram Rajagopalan <shriramr@vmware.com>

* encode version in Docker image

Signed-off-by: Shriram Rajagopalan <shriramr@vmware.com>

* tweaks

Signed-off-by: Shriram Rajagopalan <shriramr@vmware.com>
* add config validation metrics

* s/gauge/counter
* add consistenthash support for httpheader

* wires up source ip for hash policy

* add support for port level traffic policies

* only support port number for traffic policies

port name is deprecated. traffic policies at the port level should only
be inspected for port number when updating a route to use a hash policy.

* bump istio/api

Co-authored-by: Zachary Gershman <zgershman@pivotal.io>

* stop ignoring port level settings on the subset

* subset port level settings override top level

* do not return hash policy when not requested

Co-authored-by: Zachary Gershman <zgershman@pivotal.io>
* Mark gateway ports as H2 in k8s

Signed-off-by: Shriram Rajagopalan <shriramr@vmware.com>

* set alpn protocol match

Signed-off-by: Shriram Rajagopalan <shriramr@vmware.com>

* lint

Signed-off-by: Shriram Rajagopalan <shriramr@vmware.com>

* undo code changes

Signed-off-by: Shriram Rajagopalan <shriramr@vmware.com>
* remove conversion tool

* somewhat questionable pruning of tests without replacing them
* Added support for istioctl kube-inject into pod

* Pod doesn't have an app name label therefore istio-proxy is injected

* And a hashed version value got updated too

* Renamed test and adapt to current branch
* add tests for headless services and stateful sets

Signed-off-by: Shriram Rajagopalan <shriramr@vmware.com>

* fix fqdn

Signed-off-by: Shriram Rajagopalan <shriramr@vmware.com>

* fix pod names

Signed-off-by: Shriram Rajagopalan <shriramr@vmware.com>

* separate headless pod

Signed-off-by: Shriram Rajagopalan <shriramr@vmware.com>

* typo

Signed-off-by: Shriram Rajagopalan <shriramr@vmware.com>

* fix 0 port

* gateway headless test

Signed-off-by: Shriram Rajagopalan <shriramr@vmware.com>

* Revert "gateway headless test"

This reverts commit 98f1e49.
* Add mesh expansion gateway

* Match the old, unused value used by tests

* Fix pilot port number and add ilb gateway

* Move the ports to values.yaml as suggested, add ILB

* The gateway files were missing, fix typo

* Typos, based on review
* Add proxy sha and istio version to docker meta

* Fix the build args, and add a tproxy variant

* tproxy failing in route, may be due to restart

* More small fixes to capture. Tproxy seems to work even if m socket is missing

* Bad cut&paste

* Add missing env based on review
istio#6940)

* Move mixer rules under policy

* Fix lint

* Fix lint

* Rename route-rule to virtual-service

* Use separate rules directories
* Fix upgrade, use Shriram's env

* Format

* Follow up on upgrade fix
…io#6938)

* Include dump_kubernetes.sh in release archive bin dir

* Move dump_kubernetes.sh to tools/

* Update `make dumpsys` (in use by CircleCI)
@googlebot
Copy link
Copy Markdown
Collaborator

So there's good news and bad news.

👍 The good news is that everyone that needs to sign a CLA (the pull request submitter and all commit authors) have done so. Everything is all good there.

😕 The bad news is that it appears that one or more commits were authored or co-authored by someone other than the pull request submitter. We need to confirm that all authors are ok with their commits being contributed to this project. Please have them confirm that here in the pull request.

Note to project maintainer: This is a terminal state, meaning the cla/google commit status will not change from this state. It's up to you to confirm consent of the commit author(s) and merge this pull request when appropriate.

@googlebot googlebot added the cla: no Set by the Google CLA bot to indicate the author of a PR has not signed the Google CLA. label Jul 23, 2018
@rshriram rshriram requested review from hklai and removed request for JimmyCYJ, ZackButcher, ayj, douglas-reid, esnible, geeknoid, linsun, ostromart and ozevren July 23, 2018 01:05
Copy link
Copy Markdown
Contributor

@hklai hklai left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

/lgtm

rshriram and others added 2 commits July 22, 2018 21:43
* Do not allow short names in gateway hosts

Signed-off-by: Shriram Rajagopalan <shriramr@vmware.com>

* allow *

Signed-off-by: Shriram Rajagopalan <shriramr@vmware.com>

* lint

Signed-off-by: Shriram Rajagopalan <shriramr@vmware.com>
@rshriram
Copy link
Copy Markdown
Member Author

/test istio-pilot-e2e-envoyv2-v1alpha3

Copy link
Copy Markdown
Contributor

@nmittler nmittler left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

/lgtm
/approve

@rshriram Thanks for doing this! Just be sure to merge commit so we don't lose the history.

@istio-testing
Copy link
Copy Markdown
Collaborator

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: hklai, nmittler, rshriram

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Details Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@rshriram rshriram merged commit f3a6d52 into istio:master Jul 23, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cla: no Set by the Google CLA bot to indicate the author of a PR has not signed the Google CLA.

Projects

None yet

Development

Successfully merging this pull request may close these issues.