Details

    • Type: Bug Bug
    • Status: Closed Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 2.0.1
    • Fix Version/s: 2.1
    • Labels:
      None
    • Number of attachments :
      0

      Description

      Add a text input to specify task command.

        Activity

        Hide
        Fabrice Bellingard added a comment -

        Works well Julien.

        There's just an improvement to do on the Sonar URL (again!) when a task is specified: the URL should not be extracted from "ANALYSIS SUCCESSFUL, you can browse http://XXX" but from "INFO: Server: http://XXXX". Indeed, if we take the example of Views computation, there will be several "ANALYSIS SUCCESSFUL, you can browse http://XXX" occurences in the log (as many as the total number of views) so the SONAR URL will point to the last analysed view.

        Show
        Fabrice Bellingard added a comment - Works well Julien. There's just an improvement to do on the Sonar URL (again!) when a task is specified: the URL should not be extracted from "ANALYSIS SUCCESSFUL, you can browse http://XXX " but from "INFO: Server: http://XXXX ". Indeed, if we take the example of Views computation, there will be several "ANALYSIS SUCCESSFUL, you can browse http://XXX " occurences in the log (as many as the total number of views) so the SONAR URL will point to the last analysed view.
        Hide
        Julien HENRY added a comment -

        There is no point to get the INFO:... URL as it will not be the public URL so it may not be reachable by users.
        In current state most of tasks will not have an URL in Sonar "ProjectAction" but anyway tasks will mostly be visible by Jenkins administrators so this is not a big deal.

        Show
        Julien HENRY added a comment - There is no point to get the INFO:... URL as it will not be the public URL so it may not be reachable by users. In current state most of tasks will not have an URL in Sonar "ProjectAction" but anyway tasks will mostly be visible by Jenkins administrators so this is not a big deal.
        Hide
        Fabrice Bellingard added a comment -

        Indeed Julien!
        Tested.

        Show
        Fabrice Bellingard added a comment - Indeed Julien! Tested.

          People

          • Assignee:
            Julien HENRY
            Reporter:
            Julien HENRY
          • Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: