Skip to content

Conversation

@breskeby
Copy link
Contributor

@breskeby breskeby commented Jun 23, 2020

fixes #58351

@breskeby breskeby changed the title Save Shard ID Serializations in Bulk Requests (#56209) Remove minimumRuntime sourceSet from build-tools Jun 23, 2020
@breskeby breskeby requested a review from mark-vieira June 23, 2020 10:16
@@ -66,7 +65,6 @@ public void execute(Task task) {
}
}

@Internal
Copy link
Contributor Author

Choose a reason for hiding this comment

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

that slipped through in the extra sourceSet before as gradles validateProperties task didn't catch it. by moving this to the main sourceSet now we need to fix it here to have :buildSrc:validateProperties pass

@breskeby breskeby self-assigned this Jun 23, 2020
@breskeby breskeby added the :Delivery/Build Build or test infrastructure label Jun 23, 2020
@elasticmachine
Copy link
Collaborator

Pinging @elastic/es-core-infra (:Core/Infra/Build)

Copy link
Contributor

@mark-vieira mark-vieira left a comment

Choose a reason for hiding this comment

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

Yay! 👍

@breskeby breskeby merged commit e7dd178 into elastic:master Jun 24, 2020
breskeby added a commit to breskeby/elasticsearch that referenced this pull request Jun 24, 2020
* Remove minimumRuntime sourceSet from build-tools
* remove license change in loggedexec task
@breskeby breskeby removed the v7.7.2 label Jun 24, 2020
breskeby added a commit to breskeby/elasticsearch that referenced this pull request Jul 1, 2020
* Remove minimumRuntime sourceSet from build-tools
* remove license change in loggedexec task
breskeby added a commit to breskeby/elasticsearch that referenced this pull request Jul 1, 2020
* Remove minimumRuntime sourceSet from build-tools
* remove license change in loggedexec task
@mark-vieira mark-vieira added Team:Delivery Meta label for Delivery team and removed Team:Core/Infra Meta label for core/infra team labels Nov 11, 2020
@jakelandis jakelandis removed the v8.0.0 label Jul 26, 2021
@breskeby breskeby deleted the remove-buildSrc-minimum-runtime branch December 10, 2024 13:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

:Delivery/Build Build or test infrastructure Team:Delivery Meta label for Delivery team v7.8.1 v7.9.0 v8.0.0-alpha1

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Remove 'minimumRuntime' source set from buildSrc

4 participants