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.

Sign up for GitHub

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

Jump to bottom

ingressroute: add support for traffic mirroring #459

Closed
davecheney opened this issue Jun 25, 2018 · 3 comments · Fixed by #1557
Closed

ingressroute: add support for traffic mirroring #459

davecheney opened this issue Jun 25, 2018 · 3 comments · Fixed by #1557
Labels
blocked/needs-design Categorizes the issue or PR as blocked because it needs a design document. blocked/needs-product Categorizes the issue or PR as blocked because it needs a decision from product management. kind/feature Categorizes issue or PR as related to a new feature. priority/important-longterm Important over the long term, but may not be staffed and/or may need multiple releases to complete.
Milestone
1.0.0-beta.1

Comments

@davecheney
Copy link
Contributor

Feature request from the field, add support for services which get read only mirror traffic. Envoy supports this at the cluster level. Suggest something like this for the spec

spec:
  virtualhost: ...
  routes:
  - match: "/"
     services:
     - name: "red"
        port: 9000
     - name: "green"
        port: 9001
        mirror: true
@davecheney davecheney added kind/feature Categorizes issue or PR as related to a new feature. help wanted Denotes an issue that needs help from a contributor. Must meet "help wanted" guidelines. priority/important-longterm Important over the long term, but may not be staffed and/or may need multiple releases to complete. labels Jun 25, 2018
@davecheney davecheney added this to the 0.7.0 milestone Jun 25, 2018
@davecheney davecheney modified the milestones: 0.7.0, 0.8.0 Oct 23, 2018
@davecheney
Copy link
Contributor Author

Bumping to 0.8, maybe later.

@davecheney davecheney modified the milestones: 0.8.0, 0.9.0 Nov 3, 2018
@davecheney davecheney modified the milestones: 0.9.0, 0.10.0 Jan 9, 2019
@davecheney davecheney modified the milestones: 0.10.0, 0.11.0 Feb 6, 2019
@vaamarnath
Copy link
Contributor

I have already picked an issue (encrypting traffic between envoy and contour) to work during 0.11.0 milestone. Nevertheless, I can get started on this issue if its okay with you @davecheney.

@davecheney davecheney modified the milestones: 0.11.0, 0.12.0 Apr 8, 2019
@timh timh removed this from the 0.12.0 milestone Apr 10, 2019
@davecheney davecheney added blocked/needs-product Categorizes the issue or PR as blocked because it needs a decision from product management. blocked/needs-design Categorizes the issue or PR as blocked because it needs a design document. and removed help wanted Denotes an issue that needs help from a contributor. Must meet "help wanted" guidelines. labels Apr 13, 2019
@davecheney davecheney added this to the 0.15.0 milestone Jun 18, 2019
@davecheney davecheney modified the milestones: 0.15.0, 1.0.0-beta.1 Aug 19, 2019
@davecheney
Copy link
Contributor Author

Delaying this til after 1.0

davecheney added a commit to davecheney/contour that referenced this issue Sep 20, 2019
@davecheney
dag: add support for traffic mirroring
2a056a4 
Fixes projectcontour#459

Add support for per route traffic mirroring.

Signed-off-by: Dave Cheney <[email protected]>
@davecheney davecheney mentioned this issue Sep 20, 2019
Merged
davecheney added a commit to davecheney/contour that referenced this issue Sep 20, 2019
@davecheney
dag: add support for traffic mirroring
54229d7 
Fixes projectcontour#459

Add support for per route traffic mirroring.

Signed-off-by: Dave Cheney <[email protected]>
davecheney added a commit to davecheney/contour that referenced this issue Sep 20, 2019
@davecheney
dag: add support for traffic mirroring
97008ff 
Fixes projectcontour#459

Add support for per route traffic mirroring.

Signed-off-by: Dave Cheney <[email protected]>
davecheney added a commit to davecheney/contour that referenced this issue Sep 20, 2019
@davecheney
dag: add support for traffic mirroring
90fe044 
Fixes projectcontour#459

Add support for per route traffic mirroring.

Signed-off-by: Dave Cheney <[email protected]>
davecheney added a commit to davecheney/contour that referenced this issue Sep 24, 2019
@davecheney
dag: add support for traffic mirroring
d9dc0c2 
Fixes projectcontour#459

Add support for per route traffic mirroring.

Signed-off-by: Dave Cheney <[email protected]>
davecheney added a commit that referenced this issue Sep 24, 2019
@davecheney
dag: add support for traffic mirroring
de19f6b 
Fixes #459

Add support for per route traffic mirroring.

Signed-off-by: Dave Cheney <[email protected]>
davecheney added a commit to davecheney/contour that referenced this issue Sep 24, 2019
@davecheney
docs: document mirror: true
278a65b 
Updates projectcontour#459

Add traffic mirroring documentation.

Signed-off-by: Dave Cheney <[email protected]>
@davecheney davecheney mentioned this issue Sep 24, 2019
Merged
davecheney added a commit to davecheney/contour that referenced this issue Sep 24, 2019
@davecheney
docs: document mirror: true
372b322 
Updates projectcontour#459

Add traffic mirroring documentation.

Signed-off-by: Dave Cheney <[email protected]>
davecheney added a commit that referenced this issue Sep 24, 2019
@davecheney
docs: document mirror: true
9add872 
Updates #459

Add traffic mirroring documentation.

Signed-off-by: Dave Cheney <[email protected]>
davecheney added a commit to davecheney/contour that referenced this issue Sep 25, 2019
@davecheney
internal/dag: add mirror status tests
98c22c8 
Updates projectcontour#459
Updates projectcontour#1569

Clean up builder and add mirror status tests.

Signed-off-by: Dave Cheney <[email protected]>
@davecheney davecheney mentioned this issue Sep 25, 2019
Merged
davecheney added a commit to davecheney/contour that referenced this issue Sep 25, 2019
@davecheney
internal/dag: add mirror status tests
47d7bee 
Updates projectcontour#459
Updates projectcontour#1569

Clean up builder and add mirror status tests.

Signed-off-by: Dave Cheney <[email protected]>
davecheney added a commit that referenced this issue Sep 25, 2019
@davecheney
internal/dag: add mirror status tests
44102ef 
Updates #459
Updates #1569

Clean up builder and add mirror status tests.

Signed-off-by: Dave Cheney <[email protected]>
@jpeach jpeach mentioned this issue Nov 12, 2019
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
blocked/needs-design Categorizes the issue or PR as blocked because it needs a design document. blocked/needs-product Categorizes the issue or PR as blocked because it needs a decision from product management. kind/feature Categorizes issue or PR as related to a new feature. priority/important-longterm Important over the long term, but may not be staffed and/or may need multiple releases to complete.
Projects
None yet
Milestone
1.0.0-beta.1
Development

Successfully merging a pull request may close this issue.

dag: add support for traffic mirroring davecheney/contour
4 participants
@davecheney @timh @stevesloka @vaamarnath