Skip to content

Conversation

@jasontedor
Copy link
Member

Today we are by-hand maintaining a list of CCR QA sub-projects that the check task depends on. This commit simplifies this by finding these sub-projects automatically and adding their check task as dependencies of the CCR check task.

Today we are by-hand maintaining a list of CCR QA sub-projects that the
check task depends on. This commit simplifies this by finding these
sub-projects automatically and adding their check task as dependencies
of the CCR check task.
@jasontedor jasontedor added review :Distributed Indexing/CCR Issues around the Cross Cluster State Replication features labels Aug 21, 2018
@jasontedor jasontedor requested a review from martijnvg August 21, 2018 13:46
@elasticmachine
Copy link
Collaborator

Pinging @elastic/es-distributed

Copy link
Member

@martijnvg martijnvg left a comment

Choose a reason for hiding this comment

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

👍 nice one!

@jasontedor
Copy link
Member Author

@elasticmachine retest this please

@jasontedor jasontedor merged commit 9623cf6 into elastic:ccr Aug 21, 2018
jasontedor added a commit that referenced this pull request Aug 21, 2018
Today we are by-hand maintaining a list of CCR QA sub-projects that the
check task depends on. This commit simplifies this by finding these
sub-projects automatically and adding their check task as dependencies
of the CCR check task.
@jasontedor jasontedor deleted the find-ccr-qa-automatically branch August 21, 2018 16:53
jasontedor added a commit to jasontedor/elasticsearch that referenced this pull request Aug 24, 2018
Today we are by-hand maintaining a list of CCR QA sub-projects that the
check task depends on. This commit simplifies this by finding these
sub-projects automatically and adding their check task as dependencies
of the CCR check task.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

:Distributed Indexing/CCR Issues around the Cross Cluster State Replication features

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants