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

ci: Archive sharness trash dirs #5071

Merged
merged 1 commit into from
Jun 17, 2018
Merged

Conversation

magik6k
Copy link
Member

@magik6k magik6k commented Jun 3, 2018

With this CI will archive any leftover sharness trash directories, this should help debugging failing sharness tests.

Example: https://ci.ipfs.team/blue/organizations/jenkins/IPFS%2Fgo-ipfs/detail/feat%2Fci%2Farchive-trash-dirs/46/artifacts

@magik6k magik6k requested a review from Kubuxu as a code owner June 3, 2018 20:56
@ghost ghost assigned magik6k Jun 3, 2018
@ghost ghost added the status/in-progress In progress label Jun 3, 2018
ci/Jenkinsfile Outdated

try {
// archive trash directories if any
def trashdirs = run(returnStdout: true, script: "find test/sharness -name 'trash\\ *' | grep .")
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can this not all be reduced to?

sh "tar -czf sharnessTrashDirs-${osname}.tar.gz trash*/"

(I'm not familiar with Jenkins)

@magik6k magik6k force-pushed the feat/ci/archive-trash-dirs branch 2 times, most recently from 6416075 to f7fa6cf Compare June 4, 2018 08:01
@Stebalien
Copy link
Member

@magik6k this appears to be causing problems on macos (failing in a way that's making the tests fail even though we ignore the results from macos).

@whyrusleeping
Copy link
Member

@magik6k @victorbjelkholm are the mac failures related to this change? or are they independently being super dumb?

@Stebalien
Copy link
Member

Apparently, not related.

@magik6k
Copy link
Member Author

magik6k commented Jun 14, 2018

I've re-run the tests 9x, 2 runs had random (unrelated) fails (1, 2). There appears to be a problem with test time collection on mac, but that's a separate thing (I'll fix in a separate PR)

TL;DR tests failures aren't related

License: MIT
Signed-off-by: Łukasz Magiera <magik6k@gmail.com>
@magik6k magik6k force-pushed the feat/ci/archive-trash-dirs branch from f7fa6cf to 39b76d5 Compare June 15, 2018 13:14
@magik6k
Copy link
Member Author

magik6k commented Jun 15, 2018

rebased

@whyrusleeping whyrusleeping merged commit 2a9de81 into master Jun 17, 2018
@ghost ghost removed the status/in-progress In progress label Jun 17, 2018
@whyrusleeping whyrusleeping deleted the feat/ci/archive-trash-dirs branch June 17, 2018 18:06
@whyrusleeping
Copy link
Member

Seems useful to me.

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.

3 participants