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

Attempt to create existing net returns EINVAL

Details

    • 3
    • 9223372036854775807

    Description

      When "lnetctl net add" is called for an interface/net pair that already exists, the error code returned is EINVAL. It should be EEXIST, so the user knows that the net is already configured.

      [olaf@diane lustre]$ sudo lnet/utils/lnetctl net add --net tcp1 --if enp0s3
      [olaf@diane lustre]$ sudo lnet/utils/lnetctl net add --net tcp1 --if enp0s3
      add:
          - net:
                errno: -22
                descr: "cannot add network: Invalid argument"
      

      Attachments

        Activity

          [LU-12626] Attempt to create existing net returns EINVAL

          Oleg Drokin (green@whamcloud.com) merged in patch https://review.whamcloud.com/35953/
          Subject: LU-12626 lnet: create existing net returns EEXIST
          Project: fs/lustre-release
          Branch: b2_12
          Current Patch Set:
          Commit: 2a59f48ee2bed7f80ae193edaabc67abc3445d0a

          gerrit Gerrit Updater added a comment - Oleg Drokin (green@whamcloud.com) merged in patch https://review.whamcloud.com/35953/ Subject: LU-12626 lnet: create existing net returns EEXIST Project: fs/lustre-release Branch: b2_12 Current Patch Set: Commit: 2a59f48ee2bed7f80ae193edaabc67abc3445d0a

          Minh Diep (mdiep@whamcloud.com) uploaded a new patch: https://review.whamcloud.com/35953
          Subject: LU-12626 lnet: create existing net returns EEXIST
          Project: fs/lustre-release
          Branch: b2_12
          Current Patch Set: 1
          Commit: 3a1ef49a40bf6dfac13695dd0ad75abda4bf48d1

          gerrit Gerrit Updater added a comment - Minh Diep (mdiep@whamcloud.com) uploaded a new patch: https://review.whamcloud.com/35953 Subject: LU-12626 lnet: create existing net returns EEXIST Project: fs/lustre-release Branch: b2_12 Current Patch Set: 1 Commit: 3a1ef49a40bf6dfac13695dd0ad75abda4bf48d1
          pjones Peter Jones added a comment -

          Landed for 2.13

          pjones Peter Jones added a comment - Landed for 2.13

          Oleg Drokin (green@whamcloud.com) merged in patch https://review.whamcloud.com/35681/
          Subject: LU-12626 lnet: create existing net returns EEXIST
          Project: fs/lustre-release
          Branch: master
          Current Patch Set:
          Commit: 4aa71267cc0317e126843509f1c5b237f469414b

          gerrit Gerrit Updater added a comment - Oleg Drokin (green@whamcloud.com) merged in patch https://review.whamcloud.com/35681/ Subject: LU-12626 lnet: create existing net returns EEXIST Project: fs/lustre-release Branch: master Current Patch Set: Commit: 4aa71267cc0317e126843509f1c5b237f469414b

          Olaf Faaland-LLNL (faaland1@llnl.gov) uploaded a new patch: https://review.whamcloud.com/35681
          Subject: LU-12626 lnet: create existing net returns EEXIST
          Project: fs/lustre-release
          Branch: master
          Current Patch Set: 1
          Commit: 20b8dc05a948505148bd07d51b0b7f55918603ef

          gerrit Gerrit Updater added a comment - Olaf Faaland-LLNL (faaland1@llnl.gov) uploaded a new patch: https://review.whamcloud.com/35681 Subject: LU-12626 lnet: create existing net returns EEXIST Project: fs/lustre-release Branch: master Current Patch Set: 1 Commit: 20b8dc05a948505148bd07d51b0b7f55918603ef
          pjones Peter Jones added a comment -

          Olaf

          You've marked this as llnlfixready - does that mean that you'll be pushing the fix?

          Peter

          pjones Peter Jones added a comment - Olaf You've marked this as llnlfixready - does that mean that you'll be pushing the fix? Peter

          People

            ofaaland Olaf Faaland
            ofaaland Olaf Faaland
            Votes:
            0 Vote for this issue
            Watchers:
            3 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved: