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.

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

Fetch only core set of logs #518

Merged
merged 3 commits into from
Aug 1, 2017
Merged

Conversation

chxchx
Copy link
Contributor

@chxchx chxchx commented Jul 31, 2017

@sebastienvas @yutongz

Reduces cluster log fetching to less than 90 seconds.

@istio-testing
Copy link
Collaborator

@chxchx: The following test failed, say /retest to rerun them all:

Test name Commit Details Rerun command
prow/istio-presubmit.sh 7b98141 link @istio-testing bazel test this

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@sebastienvas
Copy link
Contributor

/approve no-issue

@sebastienvas
Copy link
Contributor

Unfortunately after upgrading the cluster to 1.7.2, we have to push a new version of pilot to istio.

@istio-merge-robot
Copy link

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: chxchx, sebastienvas
We suggest the following additional approver: Douglas-reid

Assign the PR to them by writing /assign @Douglas-reid in a comment when ready.

No associated issue. Update pull-request body to add a reference to an issue, or get approval with /approve no-issue

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 /approve in a comment
You can cancel your approval by writing /approve cancel in a comment

@sebastienvas
Copy link
Contributor

I am force submitting this, since it will help debug 1.7 issues.

@sebastienvas sebastienvas merged commit 785690a into istio:master Aug 1, 2017
mandarjog pushed a commit to mandarjog/istio that referenced this pull request Oct 30, 2017
…missing before (istio#518)

Former-commit-id: 92d94680b4494554da2dc996a5bf772f9c7f2829
rshriram pushed a commit that referenced this pull request Oct 30, 2017
* Fetch only core set of logs

* Turn on cluster log fetching for failing tests


Former-commit-id: 785690a
mandarjog pushed a commit that referenced this pull request Oct 31, 2017
…missing before (#518)

Former-commit-id: 6f8124be3ef18e7ab687b19bb8daeabd7890bad7
vbatts pushed a commit to vbatts/istio that referenced this pull request Oct 31, 2017
* Fetch only core set of logs

* Turn on cluster log fetching for failing tests


Former-commit-id: 785690a
mandarjog pushed a commit that referenced this pull request Nov 2, 2017
* Fetch only core set of logs

* Turn on cluster log fetching for failing tests


Former-commit-id: 785690a
guptasu pushed a commit to guptasu/istio that referenced this pull request Jun 11, 2018
kyessenov pushed a commit to kyessenov/istio that referenced this pull request Aug 13, 2018
howardjohn added a commit to howardjohn/istio that referenced this pull request Jan 12, 2020
danehans pushed a commit to danehans/istio that referenced this pull request Nov 2, 2021
luksa pushed a commit to luksa/istio that referenced this pull request Sep 20, 2022
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.

None yet

5 participants