Details

    • New Feature
    • Status: Closed
    • Major
    • Resolution: Fixed
    • None
    • 3.2.2
    • None

    Description

      Usually we don't want see logs from builds jobs.

      But in case of failure it will be very useful to see what happened in build.

      Especially on CI system where we don't directly have access to build workspace.

       

      Attachments

        Issue Links

          Activity

            We can achieve this by adding postbuild.groovy script with content:

            def buildLog = new File( basedir, 'build.log' )
            assert buildLog.text.contains('[INFO] BUILD SUCCESS')
            

            If assertion will fail log file will be displayed.

            sjaranowski Slawomir Jaranowski added a comment - We can achieve this by adding postbuild.groovy script with content: def buildLog = new File( basedir, 'build.log' ) assert buildLog.text.contains( '[INFO] BUILD SUCCESS' ) If assertion will fail log file will be displayed.

            On failure build - build.log will be printed to mojo log at the and of process be verify goal or by processResults method in run goal.

            I assume that, because printing log during execute tests can cause mixed output of different tests in parallel mode.

            sjaranowski Slawomir Jaranowski added a comment - On failure build - build.log will be printed to mojo log at the and of process be verify goal or by processResults method in run goal. I assume that, because printing log during execute tests can cause mixed output of different tests in parallel mode.

            My proposition in PR - I'm waiting for review.

            TODO - confirm version in @since documentation tag, maybe should be 2.3.0 - according to semver when we add new features we should increment minor version.

            sjaranowski Slawomir Jaranowski added a comment - My proposition in PR - I'm waiting for review. TODO - confirm version in @since documentation tag, maybe should be 2.3.0 - according to semver when we add new features we should increment minor version.

            I can review the code but it would help to get consensus on the feature proposal on the dev mailing list first.

            elharo Elliotte Rusty Harold added a comment - I can review the code but it would help to get consensus on the feature proposal on the dev mailing list first.

            OK, I will write about this on the dev mailing list.

            Thanks for review.

            sjaranowski Slawomir Jaranowski added a comment - OK, I will write about this on the dev mailing list. Thanks for review.
            olamy Olivier Lamy added a comment -

            awesome job!

            Thanks!

            olamy Olivier Lamy added a comment - awesome job! Thanks!

            Also thank you for approval.

            Now I'm waiting for next release. Do you know when next release will place?

            sjaranowski Slawomir Jaranowski added a comment - Also thank you for approval. Now I'm waiting for next release. Do you know when next release will place?
            hudson Hudson added a comment -

            Build succeeded in Jenkins: Maven TLP » maven-invoker-plugin » master #71

            See https://builds.apache.org/job/maven-box/job/maven-invoker-plugin/job/master/71/

            hudson Hudson added a comment - Build succeeded in Jenkins: Maven TLP » maven-invoker-plugin » master #71 See https://builds.apache.org/job/maven-box/job/maven-invoker-plugin/job/master/71/

            People

              olamy Olivier Lamy
              sjaranowski Slawomir Jaranowski
              Votes:
              0 Vote for this issue
              Watchers:
              5 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved:

                Time Tracking

                  Estimated:
                  Original Estimate - Not Specified
                  Not Specified
                  Remaining:
                  Remaining Estimate - 0h
                  0h
                  Logged:
                  Time Spent - 10m
                  10m