• Icon: Bug Bug
    • Resolution: Fixed
    • Icon: Minor Minor
    • influxdb-plugin
    • None
    • Jenkins 2.89.3
      InfluxDB plugin 1.14

      When entering a custom prefix for InfluxDB that includes a dash the plugin is incorrectly substituting the dash for an underscore. This isn't necessary and is confusing if you're using the prefix to query the data.  The AbstractPointGenerator is incorrectly passing the custom prefix to the measurementName() method. This isn't necessary, it's not being used in the measurement name but rather in the Prefix tag.  Tag values can have underscore.

       

          [JENKINS-50575] Dash changed to underscore in custom prefix

          Aleksi Simell added a comment -

          "Tag values can have underscore."

          You mean to say tag values can have dashes? Otherwise this is already the functionality.

          Aleksi Simell added a comment - "Tag values can have underscore." You mean to say tag values can have dashes? Otherwise this is already the functionality.

          Yes, that's what I meant

          Mandy Prescott-Courville added a comment - Yes, that's what I meant

          karan magdani added a comment - - edited

          Why are we replacing the '-' to '_' ?

          I want to use custom prefix and keep the '-' 

          "Tag values can have underscore."

          I think what Mandy meant was that if we want '_' we can use underscore. Why does the code need to convert '-' to '_'

           

          karan magdani added a comment - - edited Why are we replacing the '-' to '_' ? I want to use custom prefix and keep the '-'  "Tag values can have underscore." I think what Mandy meant was that if we want '_ ' we can use underscore. Why does the code need to convert '-' to ' _'  

          Aleksi Simell added a comment - - edited

          Removed replacing dashes with underscores in tag values. Will be available in next release.

          Aleksi Simell added a comment - - edited Removed replacing dashes with underscores in tag values. Will be available in next release.

            aleksisimell Aleksi Simell
            mandyjprescott Mandy Prescott-Courville
            Votes:
            1 Vote for this issue
            Watchers:
            3 Start watching this issue

              Created:
              Updated:
              Resolved: