Conversation
|
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: woop The full list of commands accepted by this bot can be found here. The pull request process is described here DetailsNeeds approval from an approver in each of these files:
Approvers can indicate their approval by writing |
|
/test test-end-to-end-java-8 |
|
@ches I'm not sure if I have the time to debug the issues on this branch. Are we confident in the changes that have been backported? I think the problem here is likely more the tests/CI than the code. |
|
I thought we had a passing build on #654 and/or #657, I'll take a look at it here.
Currently we consume |
|
It's looking for the |
|
Alright, seems fine now. The non-snapshot jar finding needs to be forward ported as well to master. |
|
/lgtm |
|
@woop: you cannot LGTM your own PR. DetailsIn response to 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. |
|
The changelog generation seems slightly off, mostly the 0.3.7 release version itself: its "Full Changelog" link is compare of v0.4.7...v0.3.7 instead of v0.3.6...v0.3.7, which actually includes the Cassandra store, and adding indexes. There's also a reference to #375 that actually links to 376, not sure if that might have been a mistake in a human edit. |
I noticed the first problem as well when I created the GitHub release. Will try and figure out why that happened. The rest of the changelog was copied over from master. I only generated the 0.3.7 contents. |
What this PR does / why we need it:
Releases v0.3.7. I am not sure how much we need this release, but the point is to test out the release process as a wet run of #476.