-
Notifications
You must be signed in to change notification settings - Fork 3.3k
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
HBASE-23236 test yetus 0.11.1 #797
Conversation
Oh the rel/0.11.1 tag has not been pushed yet @busbey ? |
I'd been focused on all the downstream facing stuff and hadn't bothered with the tag in git yet. We're relying on git? |
Yeah we rely on git to get the yetus source. |
okay, let me get a git tag in place. should have my signing keys this evening. |
Have you done @busbey ? Then I will start the jenkins job again to see if it works. |
Just pushed it a few minutes ago. |
💔 -1 overall
This message was automatically generated. |
emmmmm... 'The patch generated 1 ASF License warnings'. The patch does not add new files... Let me take a look. |
'!????? excludes' What's this file? We do not have this file in our repo. @busbey Is it generated during yetus processing? |
that looks like the temp file our personality uses for fetching the untrustworthy tests. not sure why we'd download it into the component directory instead of the patch working directory, but if we do it would explain why it's there. (it wouldn't explain why we're just seeing this now) |
yeah, it's in get_include_exclude_tests_arg. should probably be reading/writing |
Also looks like we don't clean up the excludes file should something go wrong with fetching it that results in wget still writing it out. from the qa run here:
we can see we're exactly on such a path. |
No description provided.