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

NVM wrapper hard-coded path

    XMLWordPrintable

Details

    Description

      Once there is nvm installed on the machine, plugin dont inherit and dont respect NVM settings.

      Our case:

      • having NVM at /data/jenkins-misc/nvm/ (same as NVM_DIR system property targets)
      • when using and configure wrapper plugin, all data are generated at jenkins user home- in our case /var/lib/jenkins.nvm (default path)

      Problem is over at following file line (NvmWrapperUtil.java), because there is hardcoded path

      https://github.com/jenkinsci/nvm-wrapper-plugin/blob/97093d1b19467b015565512317d065dc5e752ef7/src/main/java/org/jenkinsci/plugins/nvm/NvmWrapperUtil.java#L21

      Note- because path is hardcoded, it is not possible to override it anyway

      Kindly, please, any possibility to fix this?

      We are started to having serious problem with the free space, because when *.*nvm directory is symlink-ed, than plugin also dont works.

      Thanks in advance

      • Otherwise, if there is any way to solve that, or info who and how to contact, I will appreciate

      Attachments

        Activity

          rsichnyi Roman Sichnyi added a comment -
          • new version of plugin installed 
          • new options on job appeared (as I confirmed above), so it means new version got installed 
          • but in plugin page, I can still see the update available of the nvm-wrapper plugin (I guess you forgot to change the version somewhere or something related to this

           

          same here. we're on jenkins 2.121.3

          rsichnyi Roman Sichnyi added a comment - new version of plugin installed  new options on job appeared (as I confirmed above), so it means new version got installed  but in plugin page, I can still see the update available of the nvm-wrapper plugin (I guess you forgot to change the version somewhere or something related to this   same here. we're on jenkins 2.121.3
          pavenova Pavel Novak added a comment -

          Hi Tome, atoms
          please, can we expect update ? 

          Also, please, review the issue with versioning if possible.

           

          Thanks in advance.

          pavenova Pavel Novak added a comment - Hi Tome, atoms ,  please, can we expect update ?  Also, please, review the issue with versioning if possible.   Thanks in advance.
          atoms Tomas Salazar added a comment - - edited

          Hi, I had worked on a new version, I'm testing it now.

          atoms Tomas Salazar added a comment - - edited Hi, I had worked on a new version, I'm testing it now.
          atoms Tomas Salazar added a comment -

          Resolved on 0.1.7

          atoms Tomas Salazar added a comment - Resolved on 0.1.7
          pavenova Pavel Novak added a comment -

          atoms 

          Hi, I can see you resolved the issue, please, can you confirm also the option for global configuration is in place?

          Thanks 

          pavenova Pavel Novak added a comment - atoms   Hi, I can see you resolved the issue, please, can you confirm also the option for global configuration is in place? Thanks 

          People

            atoms Tomas Salazar
            pavenova Pavel Novak
            Votes:
            1 Vote for this issue
            Watchers:
            4 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved: