Details

    • Improvement
    • Resolution: Fixed
    • Trivial
    • Lustre 2.9.0
    • None
    • 9223372036854775807

    Description

      For automation purposes, it would be handy to be able to obtain the NID of the MGS associated with the current node when using command substitutions. This issue will add a %m substitution to the command execution portion of the ldev perl script.

      Attachments

        Activity

          [LU-7060] Add MGS NID substitution to ldev

          Landed to master for 2.9.0

          jgmitter Joseph Gmitter (Inactive) added a comment - Landed to master for 2.9.0

          Oleg Drokin (oleg.drokin@intel.com) merged in patch http://review.whamcloud.com/16129/
          Subject: LU-7060 ldev: Added MGS NID substitution to ldev
          Project: fs/lustre-release
          Branch: master
          Current Patch Set:
          Commit: 0f17fc82a89a40f98f53ad89c8c8982319114ef7

          gerrit Gerrit Updater added a comment - Oleg Drokin (oleg.drokin@intel.com) merged in patch http://review.whamcloud.com/16129/ Subject: LU-7060 ldev: Added MGS NID substitution to ldev Project: fs/lustre-release Branch: master Current Patch Set: Commit: 0f17fc82a89a40f98f53ad89c8c8982319114ef7

          Need one last review on this.

          Thanks!

          dinatale2 Giuseppe Di Natale (Inactive) added a comment - Need one last review on this. Thanks!

          Joe,

          Don't mind at all. Thanks!

          Giuseppe

          dinatale2 Giuseppe Di Natale (Inactive) added a comment - Joe, Don't mind at all. Thanks! Giuseppe

          Giuseppe,
          I see the patch failed sanityn testing some time ago (likely due to LU-7084). Do you mind if I do a quick rebase to kick off testing again in an effort to get a clean test run? I have also added an additional reviewer on our side.
          Thanks.
          Joe

          jgmitter Joseph Gmitter (Inactive) added a comment - Giuseppe, I see the patch failed sanityn testing some time ago (likely due to LU-7084 ). Do you mind if I do a quick rebase to kick off testing again in an effort to get a clean test run? I have also added an additional reviewer on our side. Thanks. Joe

          This is a fairly simple change to ldev. Still needs code reviews. Thanks!

          dinatale2 Giuseppe Di Natale (Inactive) added a comment - This is a fairly simple change to ldev. Still needs code reviews. Thanks!

          Hi Bob,
          Can you take a look at this patch?
          Thanks.
          Joe

          jgmitter Joseph Gmitter (Inactive) added a comment - Hi Bob, Can you take a look at this patch? Thanks. Joe

          Giuseppe Di Natale (dinatale2@llnl.gov) uploaded a new patch: http://review.whamcloud.com/16129
          Subject: LU-7060 ldev: Added MGS NID substitution to ldev
          Project: fs/lustre-release
          Branch: master
          Current Patch Set: 1
          Commit: aa90e3fa4ce5f6807c43fec161be05139b74f181

          gerrit Gerrit Updater added a comment - Giuseppe Di Natale (dinatale2@llnl.gov) uploaded a new patch: http://review.whamcloud.com/16129 Subject: LU-7060 ldev: Added MGS NID substitution to ldev Project: fs/lustre-release Branch: master Current Patch Set: 1 Commit: aa90e3fa4ce5f6807c43fec161be05139b74f181

          People

            bogl Bob Glossman (Inactive)
            dinatale2 Giuseppe Di Natale (Inactive)
            Votes:
            0 Vote for this issue
            Watchers:
            6 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved: