-
Notifications
You must be signed in to change notification settings - Fork 7.7k
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
Update README about flags on cluster logs #547
Conversation
tests/e2e/README.md
Outdated
@@ -19,6 +19,8 @@ Tests could be built and drove manually as a single test or automatically detect | |||
* `--verbose` debug level noise from proxies | |||
* `--istioctl_url` the location of an `istioctl` binary | |||
* `--skip_cleanup` if skip cleanup steps | |||
* `--log_provider` where cluster logs are hosted, only support stackdriver for now |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
how about s/stackdriver/stackdriver
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Done
/assign @sebastienvas |
/approve no-issue |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: chxchx, sebastienvas Associated issue requirement bypassed by: sebastienvas The full list of commands accepted by this bot can be found here.
Needs approval from an approver in each of these OWNERS Files:
You can indicate your approval by writing |
/test all |
/test all [submit-queue is verifying that this PR is safe to merge] |
Automatic merge from submit-queue |
* Add rolling-window for rate limiting. * Address comments. * Update comment. Former-commit-id: 961ec5e317066b8300fe836f105cb1f8d335d9b2
Automatic merge from submit-queue Update README about flags on cluster logs **Release note**: ```release-note NONE ``` Former-commit-id: 93a124b
* Add rolling-window for rate limiting. * Address comments. * Update comment. Former-commit-id: cbdf7f3697d410f9f02e7435ea2d686c36321374
Automatic merge from submit-queue Update README about flags on cluster logs **Release note**: ```release-note NONE ``` Former-commit-id: 93a124b
Automatic merge from submit-queue Update README about flags on cluster logs **Release note**: ```release-note NONE ``` Former-commit-id: 93a124b
* add enable_sds flag to MeshConfig proto definition * address comments - 1. update boolean(sds_enable) to string(uds path) 2. add sds_refresh_delay field * address comment
…stio#263) (istio#547) This adds an option to istiod that allows disabling reading IngressClass resources, which are cluster-scoped. When this is disabled, processing of Ingress resources that use the new (as of Kubernetes v1.18) `ingressClassName` field in their spec is not supported. Only resources that use the deprecated annotation will be processed. Co-authored-by: Marko Lukša <marko.luksa@gmail.com>
Release note: