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

Fix log tailed issue #132

Merged
merged 5 commits into from
May 7, 2019
Merged

Conversation

TommyLike
Copy link
Contributor

@TommyLike TommyLike commented May 6, 2019

When used with label filter option, Kubectl logs would tail logs in 10 lines in default, this patch increases it to 10000

@TommyLike TommyLike force-pushed the bug/fix_log_issue branch 4 times, most recently from 24f1023 to 666cd51 Compare May 6, 2019 10:38
@TommyLike TommyLike changed the title [DNM][WIP] Used for testing Fix log tailed issue May 6, 2019
@TommyLike
Copy link
Contributor Author

@k82cn please take a look

@hzxuzhonghu
Copy link
Collaborator

any reason not the whole logs?

@TommyLike TommyLike force-pushed the bug/fix_log_issue branch from 666cd51 to b5e3428 Compare May 7, 2019 01:23
@TommyLike
Copy link
Contributor Author

@hzxuzhonghu since it's only used in CI environment, it would be great to limit the whole size if the limit can cover most of the cases. But as you said, it's worth discussing.

@hzxuzhonghu
Copy link
Collaborator

It's better if these log files can be downloaded.

@TommyLike
Copy link
Contributor Author

@hzxuzhonghu can be uploaded to external storage, for instance S3.

@hzxuzhonghu
Copy link
Collaborator

/lgtm

@volcano-sh-bot volcano-sh-bot added the lgtm Indicates that a PR is ready to be merged. label May 7, 2019
@k82cn
Copy link
Member

k82cn commented May 7, 2019

/approve

@volcano-sh-bot volcano-sh-bot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label May 7, 2019
@k82cn
Copy link
Member

k82cn commented May 7, 2019

/lgtm
/approve

@k82cn
Copy link
Member

k82cn commented May 7, 2019

/lgtm
/approve

@volcano-sh-bot volcano-sh-bot merged commit d59fc36 into volcano-sh:master May 7, 2019
kevin-wangzefeng pushed a commit to kevin-wangzefeng/volcano that referenced this pull request Jun 28, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
approved Indicates a PR has been approved by an approver from all required OWNERS files. lgtm Indicates that a PR is ready to be merged.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants