Uploaded image for project: 'Jenkins'
  1. Jenkins
  2. JENKINS-37387

Add @Symbol("ansible") to Ansible's ToolDescriptor

    XMLWordPrintable

    Details

    • Similar Issues:
    • Epic Link:

      Attachments

        Activity

        Ascending order - Click to sort in descending order
        Hide
        Permalink
        escoem Emilio Escobar added a comment -

        Hi Andrew Bayer, IIUC, all you want is to add @Symbol("ansible") at https://github.com/jenkinsci/ansible-plugin/blob/master/src/main/java/org/jenkinsci/plugins/ansible/AnsibleInstallation.java#L116

        isn't it?

        Show
        escoem Emilio Escobar added a comment - Hi Andrew Bayer , IIUC, all you want is to add @Symbol("ansible") at https://github.com/jenkinsci/ansible-plugin/blob/master/src/main/java/org/jenkinsci/plugins/ansible/AnsibleInstallation.java#L116 isn't it?
        Hide
        Permalink
        michelzanini Michel Zanini added a comment -

        I think this would solve an issue I am having where I can't declare Ansible as tool on a declarative pipeline.

        I wish to be able to do:

        tools {
    maven '3.5.4'
    ansible '2.3.2.0'
  }

        And have ansible installation work just like maven, jdk, etc...
        But right now this is failing because the class has not being named with @Symbol("ansible").

        Do you know if there are any workaround?

        I tried this:

        tools {
    org.jenkinsci.plugins.ansible. AnsibleInstallation '2.3.2.0'
  }

        But it does not work...

        Show
        michelzanini Michel Zanini added a comment - I think this would solve an issue I am having where I can't declare Ansible as tool on a declarative pipeline. I wish to be able to do: tools { maven '3.5.4' ansible '2.3.2.0' } And have ansible installation work just like maven, jdk, etc... But right now this is failing because the class has not being named with @Symbol("ansible"). Do you know if there are any workaround? I tried this: tools { org.jenkinsci.plugins.ansible. AnsibleInstallation '2.3.2.0' } But it does not work...
        Hide
        Permalink
        michelzanini Michel Zanini added a comment -

        The fix for this is merged on master but still not released.

        Show
        michelzanini Michel Zanini added a comment - The fix for this is merged on master but still not released.

          People

          Assignee:
          escoem Emilio Escobar
          Reporter:
          abayer Andrew Bayer
          Votes:
          0 Vote for this issue
          Watchers:
          3 Start watching this issue

            Dates

            Created:
            Updated: