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

releasenotes-builder: there is problem on Travis to fush output #254

Closed
romani opened this issue Jul 15, 2017 · 2 comments
Closed

releasenotes-builder: there is problem on Travis to fush output #254

romani opened this issue Jul 15, 2017 · 2 comments

Comments

@romani
Copy link
Member

romani commented Jul 15, 2017

it is not rare case when build is failing https://travis-ci.org/checkstyle/checkstyle/jobs/253921585 but there is no details on failure, it is started recently(about a month)

tail of log file is:

[WARN] Issue #4708 "idea: second part of violations" is not closed!
[WARN] Issue #4711 "idea: code style violations" is not closed!
[WARN] Issue #4394 "pitest: increase mutation coverage for pitest-checkstyle-api profile to 100%" is not closed!
[WARN] Issue #4707 "idea: abstraction violations" is not closed!
[WARN] Issue #4592 "Split BaseCheckTestSupport into AbstractPathTestSupport, AbstractModuleTestSupport, and AbstractTreeTestSupport" is not closed!
[WARN] Issue #4681 "review all suppressed IntellijIdea inspections with comment 'till ...'" is not closed!
[WARN] Issue #4675 "pitest: increase mutation coverage for pitest-checks-misc profile to 100%" is not closed!
[WARN] Issue #4398 "pitest: increase mutation coverage for pitest-checkstyle-tree-walker profile to 100%" is not closed!
[WARN] Issue #4399 "pitest: increase mutation coverage for pitest-checkstyle-main profile to 100%" is not closed!
[WARN] I

There is some ERROR in validation that result in error exit code that stop whole shell processing, but looks like jvm output is not flushed and partly lost.

Workaround:
run it on local, or try to find from recent commits, what issue is missing labels.

This particular problem was in missed label of checkstyle/checkstyle#3301.
Fixed a label.
I will not relaunch build on Travis to keep it in history.

@rnveach
Copy link
Member

rnveach commented Jun 30, 2018

@romani is this still an issue?

@romani
Copy link
Member Author

romani commented Jun 30, 2018

usually we resolve this by "sleep 5" to let travis flush all buffers before it stops execution.

I do not see such hack at https://github.com/checkstyle/checkstyle/blob/master/.ci/travis/xtr_releasenotes-gen.sh#L45

lets close issue for now, ones it come back we can reopen

@romani romani closed this as completed Jun 30, 2018
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

No branches or pull requests

2 participants