Uploaded image for project: 'Data Management'
  1. Data Management
  2. DM-12526

Jenkins should artifact *.failed files

    Details

    • Type: Story
    • Status: Done
    • Resolution: Done
    • Fix Version/s: None
    • Component/s: Continuous Integration
    • Labels:
      None

      Description

      Per discussion on #dm-square, there are still some C++ tests that are not being driven by pytest. Failures in these tests will not be recorded by pytest in a junit xml file. A stop gap solution until all tests are executed under pytest is to artifact the *.failed files.

        Attachments

          Issue Links

            Activity

            Hide
            jhoblitt Joshua Hoblitt added a comment -

            I have merged artifacting */.failed.

            Fritz Mueller Could you retry your problematic build and verify that the .failed files are being recorded?

            Show
            jhoblitt Joshua Hoblitt added a comment - I have merged artifacting * / .failed . Fritz Mueller Could you retry your problematic build and verify that the .failed files are being recorded?
            Hide
            salnikov Andy Salnikov added a comment -

            Joshua Hoblitt, I have "fixed" qserv unit tests to fail and ran that through jenkins and it indeed shows "*.failed" files from those unit tests in artifacts now (https://ci.lsst.codes/blue/organizations/jenkins/stack-os-matrix/detail/stack-os-matrix/27012/artifacts). Thanks for adding that! I know Fritz is busy today so I'll mark ticket as reviewed.

            Show
            salnikov Andy Salnikov added a comment - Joshua Hoblitt , I have "fixed" qserv unit tests to fail and ran that through jenkins and it indeed shows "*.failed" files from those unit tests in artifacts now ( https://ci.lsst.codes/blue/organizations/jenkins/stack-os-matrix/detail/stack-os-matrix/27012/artifacts ). Thanks for adding that! I know Fritz is busy today so I'll mark ticket as reviewed.
            Hide
            jhoblitt Joshua Hoblitt added a comment -

            Good news. Thanks!

            Show
            jhoblitt Joshua Hoblitt added a comment - Good news. Thanks!

              People

              • Assignee:
                jhoblitt Joshua Hoblitt
                Reporter:
                jhoblitt Joshua Hoblitt
                Reviewers:
                Fritz Mueller
                Watchers:
                Andy Salnikov, Fritz Mueller, Joshua Hoblitt, Tim Jenness
              • Votes:
                0 Vote for this issue
                Watchers:
                4 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved:

                  Summary Panel