Uploaded image for project: 'Jenkins Artifactory Plug-in'
  1. Jenkins Artifactory Plug-in
  2. HAP-77

Deploying artifacts from Hudson - not deployed when build is "unstable" (failed tests)

    Details

    • Type: Improvement
    • Status: Resolved
    • Priority: Normal
    • Resolution: Fixed
    • Affects Version/s: 1.3.0
    • Fix Version/s: 1.3.1
    • Component/s: Maven2
    • Labels:
      None

      Description

      Please, see http://goo.gl/LJJ5: "Deploying artifacts from Hudson - not deployed when build is "unstable" (failed tests)"

      We think there can be an option in Hudson to deploy or not deploy artifacts when tests fail

        Attachments

          Activity

          Hide
          emersonf Emerson Farrugia added a comment -

          Why would you want to deploy broken code to Artifactory? Or if the code isn't broken, why are your unit tests failing? Something doesn't add up.

          Show
          emersonf Emerson Farrugia added a comment - Why would you want to deploy broken code to Artifactory? Or if the code isn't broken, why are your unit tests failing? Something doesn't add up.
          Hide
          evgeny.goldin Evgeny Goldin added a comment -

          You're right, but I would like to have a choice of doing or not doing so.
          When deploying with maven - I have an option to ignore failed tests and still deploy everything. Sometimes we need it.

          Show
          evgeny.goldin Evgeny Goldin added a comment - You're right, but I would like to have a choice of doing or not doing so. When deploying with maven - I have an option to ignore failed tests and still deploy everything. Sometimes we need it.
          Hide
          gboissinot Gregory Boissinot added a comment -

          The test step is an optional step in a integration process : we are able to deliver artifacts without tests.
          If there are some failed tests , the Hudson job is set to unstable.
          And there are also lots of others ways to set an Hudson job to unstable (metrics Hudson plugins).

          An option (Deploy even if the build is unstable) is the best way to fix this issue.

          Show
          gboissinot Gregory Boissinot added a comment - The test step is an optional step in a integration process : we are able to deliver artifacts without tests. If there are some failed tests , the Hudson job is set to unstable. And there are also lots of others ways to set an Hudson job to unstable (metrics Hudson plugins). An option (Deploy even if the build is unstable) is the best way to fix this issue.
          Hide
          starwarsfan Yves Schumann added a comment -

          Hi there

          Currently we migrate a huge project with lots of modules from build with Ant on CruiseControl to build with Maven on Hudson. So there are already a lot of modules which have failed tests and which will still be in this state for a while. It's impossible to fix all the failed tests at first and not have complete result set so it would be very much appreciated if this feature could be implemented soon.

          Anyway, thx a lot for your work and the cool plugin.

          Show
          starwarsfan Yves Schumann added a comment - Hi there Currently we migrate a huge project with lots of modules from build with Ant on CruiseControl to build with Maven on Hudson. So there are already a lot of modules which have failed tests and which will still be in this state for a while. It's impossible to fix all the failed tests at first and not have complete result set so it would be very much appreciated if this feature could be implemented soon. Anyway, thx a lot for your work and the cool plugin.
          Hide
          yossis Yossi Shaul added a comment -
          Show
          yossis Yossi Shaul added a comment - Similar issue opened at http://issues.hudson-ci.org/browse/HUDSON-5859
          Hide
          yossis Yossi Shaul added a comment -

          Works for Maven 2 jobs

          Show
          yossis Yossi Shaul added a comment - Works for Maven 2 jobs

            People

            • Assignee:
              yoavl Yoav Landman
              Reporter:
              evgeny.goldin Evgeny Goldin
            • Votes:
              2 Vote for this issue
              Watchers:
              3 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: