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

tests: lctl list_nids has been deprecated and is still used in test suites

Details

    • Improvement
    • Resolution: Fixed
    • Minor
    • None
    • None
    • None
    • 9223372036854775807

    Description

      The title pretty much sums it up, the idea is to get rid of:

      lctl list_nids: has been deprecated. Please use 'lnetctl net show'

      Attachments

        Issue Links

          is related to

          Bug - A problem which impairs or prevents the functions of the product. LU-10003 lnetctl error "cannot add network: invalid argument"

          • Major - Major loss of function.
          • Resolved

          Activity

            [LU-10790] tests: lctl list_nids has been deprecated and is still used in test suites
            simmonsja James A Simmons added a comment -

            lctl list_nids will never go away. But its being reborn with the Netlink IPv6 work.

            simmonsja James A Simmons added a comment - lctl list_nids will never go away. But its being reborn with the Netlink IPv6 work.
            ashehata Amir Shehata (Inactive) added a comment -

            Why not use this ticket to land the patch. Even though the lctl list_nids hasn't been deprecated yet, it'll eventually be. So if we remove the usage of list_nids from the tests suites, when we do deprecate them, there will be less work to do.

            ashehata Amir Shehata (Inactive) added a comment - Why not use this ticket to land the patch. Even though the lctl list_nids hasn't been deprecated yet, it'll eventually be. So if we remove the usage of list_nids from the tests suites, when we do deprecate them, there will be less work to do.
            bougetq Quentin Bouget (Inactive) added a comment -

            Yes, if this patch does not land and lctl list_nids actually stays deprecated I can always reopen the ticket.

            bougetq Quentin Bouget (Inactive) added a comment - Yes, if this patch does not land and lctl list_nids actually stays deprecated I can always reopen the ticket.
            pjones Peter Jones added a comment -

            So close out this ticket for now?

            pjones Peter Jones added a comment - So close out this ticket for now?
            bougetq Quentin Bouget (Inactive) added a comment -

            I just noticed LU-10003, looks like lctl list_nids might not stay deprecated. I will put this on hold then.

            bougetq Quentin Bouget (Inactive) added a comment - I just noticed LU-10003 , looks like lctl list_nids might not stay deprecated. I will put this on hold then.
            bougetq Quentin Bouget (Inactive) added a comment -

            Quentin Bouget (quentin.bouget@cea.fr) uploaded a new patch: https://review.whamcloud.com/31581
            Subject: LU-10790 tests: replace lctl list_nids with lnetctl net show
            Project: fs/lustre-release
            Branch: master
            Current Patch Set: 2
            Commit: I4ee0a2f0f239aeab37fdb0da156da7f3839bb686

            bougetq Quentin Bouget (Inactive) added a comment - Quentin Bouget (quentin.bouget@cea.fr) uploaded a new patch: https://review.whamcloud.com/31581 Subject: LU-10790 tests: replace lctl list_nids with lnetctl net show Project: fs/lustre-release Branch: master Current Patch Set: 2 Commit: I4ee0a2f0f239aeab37fdb0da156da7f3839bb686

            People

              bougetq Quentin Bouget (Inactive)
              cealustre CEA
              Votes:
              0 Vote for this issue
              Watchers:
              4 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved: