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 made changes -
            Resolution New: Fixed [ 1 ]
            Status Original: Reopened [ 4 ] New: Resolved [ 5 ]
            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.
            pjones Peter Jones made changes -
            Reporter Original: Quentin Bouget [ bougetq ] New: CEA [ cealustre ]
            pjones Peter Jones made changes -
            Resolution Original: Duplicate [ 3 ]
            Status Original: Resolved [ 5 ] New: Reopened [ 4 ]
            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 made changes -
            Link New: This issue is related to LU-10003 [ LU-10003 ]
            pjones Peter Jones made changes -
            Resolution New: Duplicate [ 3 ]
            Status Original: Open [ 1 ] New: Resolved [ 5 ]
            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.

            People

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

              Dates

                Created:
                Updated:
                Resolved: