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

Update references to renamed Gradle task #1508

Merged
merged 1 commit into from
Aug 5, 2020

Conversation

mark-vieira
Copy link
Contributor

Some of our rest testing tasks were renamed as part of the refactoring done in elastic/elasticsearch#60261. This broke the hadoop build as it still contains references to those old tasks. Right now this is only failing in our release builds due to the new build-tools jar not being published yet but we need to get those builds green to get the updated artifact published.

Copy link
Member

@jbaiera jbaiera left a comment

Choose a reason for hiding this comment

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

LGTM assuming tests are passing with a local build tools snapshot

@mark-vieira
Copy link
Contributor Author

Yes, I was able to run ./gradlew integrationTest locally.

@mark-vieira
Copy link
Contributor Author

@jbaiera We good to merge on this?

@mark-vieira
Copy link
Contributor Author

I'm going to hit the button on this to confirm this fixes the release snapshot builds.

@mark-vieira mark-vieira merged commit fb79a1b into elastic:master Aug 5, 2020
@mark-vieira
Copy link
Contributor Author

This has also been backported to 7.x.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants