Details

    • Type: Sub-task
    • Status: Resolved
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: 0.8.0
    • Fix Version/s: 1.0.0
    • Component/s: deployment
    • Labels:
      None

      Description

      To let some integration tests to function, user hdfs should be able to login across the nodes with password-less ssh.

        Issue Links

          Activity

          Hide
          cos Konstantin Boudnik added a comment -

          I am thinking of adding ssh keys placement logic into common-hdfs class. Any other thoughts?

          Show
          cos Konstantin Boudnik added a comment - I am thinking of adding ssh keys placement logic into common-hdfs class. Any other thoughts?
          Hide
          jayunit100 jay vyas added a comment -

          One suggestion:, for bigtop, why bot by default allow pwd-less ssh for any user, so that this is never a concern again.... I think that will aide better testing, encourage developers to not write tests that need root to run, and avoid us from having security holes which Are non obvious .

          Show
          jayunit100 jay vyas added a comment - One suggestion:, for bigtop, why bot by default allow pwd-less ssh for any user, so that this is never a concern again.... I think that will aide better testing, encourage developers to not write tests that need root to run, and avoid us from having security holes which Are non obvious .
          Hide
          cos Konstantin Boudnik added a comment - - edited

          This is what I roughly have in mind. Any objections? And it seems to be working well

          Show
          cos Konstantin Boudnik added a comment - - edited This is what I roughly have in mind. Any objections? And it seems to be working well
          Hide
          cos Konstantin Boudnik added a comment -

          One suggestion:, for bigtop, why bot by default allow pwd-less ssh for any user

          I honestly would feel a bit uncomfortable about this. Is it only me?

          Show
          cos Konstantin Boudnik added a comment - One suggestion:, for bigtop, why bot by default allow pwd-less ssh for any user I honestly would feel a bit uncomfortable about this. Is it only me?
          Hide
          rvs Roman Shaposhnik added a comment -

          In general I'm OK with this, but of course only for testing. Speaking of which: can you please make it on/off based on the config.csv with the default being off?

          Show
          rvs Roman Shaposhnik added a comment - In general I'm OK with this, but of course only for testing. Speaking of which: can you please make it on/off based on the config.csv with the default being off?
          Hide
          cos Konstantin Boudnik added a comment -

          an improved version of the patch that makes ssh keys op to be conditional on the setting of testonly_hdfs_sshkeys,yes

          Show
          cos Konstantin Boudnik added a comment - an improved version of the patch that makes ssh keys op to be conditional on the setting of testonly_hdfs_sshkeys,yes
          Hide
          rvs Roman Shaposhnik added a comment -

          +1

          Show
          rvs Roman Shaposhnik added a comment - +1
          Hide
          cos Konstantin Boudnik added a comment -

          committed and pushed to the master

          Show
          cos Konstantin Boudnik added a comment - committed and pushed to the master

            People

            • Assignee:
              cos Konstantin Boudnik
              Reporter:
              cos Konstantin Boudnik
            • Votes:
              0 Vote for this issue
              Watchers:
              3 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Development