Skip to content

GitLab

Closed
Created by James Willenbring@jmwilleMaintainer

Four MueLu tests broken in CI build in push to MueLu on 2/19/2018

Created by: bartlettroscoe

CC: @trilinos/muelu, @trilinos/framework

Next Action Status

The original failure was caused due to a violation of the additive test assumption of branches due to a 10 day time lag between when the branch was tested and when it was merged. See https://github.com/trilinos/Trilinos/pull/2171#issuecomment-367530716 and #2312.

Description

The push of the merge commit:

commit 22e7f91ed633a22d6d44baf46d8f1e4fca7bfb3e
Merge: 52f3559 d851185
Author: Luc Berger <lberge@sandia.gov>
Date:   Mon Feb 19 15:38:53 2018 -0700

    Merge pull request #2171 from lucbv/develop
    
    MueLu_structured_aggregation

broke the standard CI build as shown at:

which involved breaking the four tests:

  • MueLu_UnitTestsEpetra_MPI_1
  • MueLu_UnitTestsEpetra_MPI_4
  • MueLu_UnitTestsTpetra_MPI_1
  • MueLu_UnitTestsTpetra_MPI_4

This was the only merge commit pulled in the CI iteration as shown at:

Therefore, I will back out this merge commit and run the checkin-test-sems.sh script to fix this ASAP (but I will only run the MueLu test to speed this up). Since the CI build passed before this merge commit was pushed, that should be enough testing. Otherwise, everyone's automated PR testing for changes to MueLu or upsstream from MueLu will fail because of this.