Uploaded image for project: 'Lustre'
  1. Lustre
  2. LU-13153

Function llapi_get_version_string fails

Details

    • Bug
    • Resolution: Fixed
    • Minor
    • Lustre 2.14.0
    • Lustre 2.12.3
    • None
    • 3
    • 9223372036854775807

    Description

      The llapi function llapi_get_version_string always (AFAICT) fails with ENOENT because it passes the argument to get_lustre_param_value in the wrong order.
      I should add that this function is used in jt_get_version that has a fallback to LUSTRE_VERSION_STRING which might have shadowed the error.

      I'm attaching a simple patch.

      Attachments

        Activity

          [LU-13153] Function llapi_get_version_string fails
          pjones Peter Jones added a comment -

          Landed for 2.14

          pjones Peter Jones added a comment - Landed for 2.14

          Oleg Drokin (green@whamcloud.com) merged in patch https://review.whamcloud.com/40680/
          Subject: LU-13153 lib: fix llapi_get_version_string
          Project: fs/lustre-release
          Branch: master
          Current Patch Set:
          Commit: d634e66dc38f97b6697f5ec1d559f0c22aeb4c1d

          gerrit Gerrit Updater added a comment - Oleg Drokin (green@whamcloud.com) merged in patch https://review.whamcloud.com/40680/ Subject: LU-13153 lib: fix llapi_get_version_string Project: fs/lustre-release Branch: master Current Patch Set: Commit: d634e66dc38f97b6697f5ec1d559f0c22aeb4c1d

          Neil Brown (neilb@suse.de) uploaded a new patch: https://review.whamcloud.com/40680
          Subject: LU-13153 lib: fix llapi_get_version_string
          Project: fs/lustre-release
          Branch: master
          Current Patch Set: 1
          Commit: 7a8cd2c24b6e13cf8c289acfedc89a3fb23c952b

          gerrit Gerrit Updater added a comment - Neil Brown (neilb@suse.de) uploaded a new patch: https://review.whamcloud.com/40680 Subject: LU-13153 lib: fix llapi_get_version_string Project: fs/lustre-release Branch: master Current Patch Set: 1 Commit: 7a8cd2c24b6e13cf8c289acfedc89a3fb23c952b

          People

            neilb Neil Brown
            barthelemy Clément Barthelemy (Inactive)
            Votes:
            0 Vote for this issue
            Watchers:
            3 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved: