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

conf-sanity test_32xx fails for separate MGS

Details

    • Bug
    • Resolution: Unresolved
    • Minor
    • None
    • None
    • 3
    • 9223372036854775807

    Description

      It is regression after patch https://review.whamcloud.com/33422 "LU-11556 tests: fix set_persistent_param_and_check breakage". With separate MGS set_persistent_param_and_check run at mgs node, but test stop it, and run local MDS+MGS.

      Attachments

        Issue Links

          Activity

            [LU-11991] conf-sanity test_32xx fails for separate MGS

            Actually, looking at patch https://review.whamcloud.com/33422 "LU-11556 tests: fix set_persistent_param_and_check breakage" it isn't clear that this introduced the problem that you describe. It was just fixing a problem where "$node" was being passed to set_persistent_param_and_check() instead of "$facet", it doesn't do anything related to starting/stopping the MGS device? Is there some other patch that introduced this breakage, or am I missing the connection?

            adilger Andreas Dilger added a comment - Actually, looking at patch https://review.whamcloud.com/33422 " LU-11556 tests: fix set_persistent_param_and_check breakage " it isn't clear that this introduced the problem that you describe. It was just fixing a problem where " $node " was being passed to set_persistent_param_and_check() instead of " $facet ", it doesn't do anything related to starting/stopping the MGS device? Is there some other patch that introduced this breakage, or am I missing the connection?

            Only skipping for 32xx at https://review.whamcloud.com/#/c/33589

            aboyko Alexander Boyko added a comment - Only skipping for 32xx at  https://review.whamcloud.com/#/c/33589
            simmonsja James A Simmons added a comment - - edited

            I see you are resolving this with https://review.whamcloud.com/#/c/33589. Why the separate ticket?

            simmonsja James A Simmons added a comment - - edited I see you are resolving this with  https://review.whamcloud.com/#/c/33589.  Why the separate ticket?

            People

              aboyko Alexander Boyko
              aboyko Alexander Boyko
              Votes:
              0 Vote for this issue
              Watchers:
              3 Start watching this issue

              Dates

                Created:
                Updated: