Bigtop
  1. Bigtop
  2. BIGTOP-1256

Spark package deployment should reflect BIGTOP-1183

    Details

    • Type: Bug Bug
    • Status: Resolved
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 0.8.0
    • Fix Version/s: 0.8.0
    • Component/s: debian, rpm
    • Labels:
      None

      Description

      Puppet deployment module for spark were never updated to reflect the package name changes introduced in BIGTOP-1183.

      Class spark needs to insure that

          package { "spark-master":
            ensure => latest,
          }
      

      or

          package { "spark-worker":
            ensure => latest,
          }
      

      for different nodes, instead of just spark

      1. BIGTOP-1258.patch
        2 kB
        Xiaomin Zhang
      2. BIGTOP-1256.patch
        2 kB
        Xiaomin Zhang
      3. BIGTOP-1256.patch
        2 kB
        Xiaomin Zhang

        Issue Links

          Activity

          Konstantin Boudnik created issue -
          Konstantin Boudnik made changes -
          Field Original Value New Value
          Link This issue is broken by BIGTOP-1183 [ BIGTOP-1183 ]
          Konstantin Boudnik made changes -
          Description Puppet deployment module for spark were never updated to reflect the package name changes introduced in BIGTOP-1183.

          Class {{spark}} needs to insure that
          {noformat}
              package { "spark-core":
                ensure => latest,
              }
          {noformat}
          instead of just {{spark}}
          Puppet deployment module for spark were never updated to reflect the package name changes introduced in BIGTOP-1183.

          Class {{spark}} needs to insure that
          {noformat}
              package { "spark-master":
                ensure => latest,
              }
          {noformat}
          or
          {noformat}
              package { "spark-worker":
                ensure => latest,
              }
          {noformat}
          for different nodes, instead of just {{spark}}
          Konstantin Boudnik made changes -
          Link This issue is duplicated by BIGTOP-1260 [ BIGTOP-1260 ]
          Xiaomin Zhang made changes -
          Status Open [ 1 ] Patch Available [ 10002 ]
          Hide
          Xiaomin Zhang added a comment -

          Please find the patch as attached.
          Thanks.

          Show
          Xiaomin Zhang added a comment - Please find the patch as attached. Thanks.
          Xiaomin Zhang made changes -
          Attachment BIGTOP-1258.patch [ 12654998 ]
          Hide
          Xiaomin Zhang added a comment -

          Hi, Konstantin:
          Could you please assign this issue to me and help to review the patch?
          Thanks.

          Show
          Xiaomin Zhang added a comment - Hi, Konstantin: Could you please assign this issue to me and help to review the patch? Thanks.
          Hide
          Konstantin Boudnik added a comment -

          The patch looks good, but I think you put the incorrect name on the patch file. Could you please fix it?

          Show
          Konstantin Boudnik added a comment - The patch looks good, but I think you put the incorrect name on the patch file. Could you please fix it?
          Konstantin Boudnik made changes -
          Assignee Xiaomin Zhang [ zhos ]
          Hide
          Xiaomin Zhang added a comment -

          Oops, my fault. I've uploaded the renamed file.
          Thanks, Cos.

          Show
          Xiaomin Zhang added a comment - Oops, my fault. I've uploaded the renamed file. Thanks, Cos.
          Xiaomin Zhang made changes -
          Attachment BIGTOP-1256.patch [ 12655494 ]
          Hide
          Xiaomin Zhang added a comment -

          The commit message is also updated. Sorry for inconvenience.

          Show
          Xiaomin Zhang added a comment - The commit message is also updated. Sorry for inconvenience.
          Xiaomin Zhang made changes -
          Attachment BIGTOP-1256.patch [ 12655496 ]
          Hide
          Konstantin Boudnik added a comment - - edited

          +1
          I have amended the commit message to use '.' instead of ':' after the JIRA number.

          Show
          Konstantin Boudnik added a comment - - edited +1 I have amended the commit message to use '.' instead of ':' after the JIRA number.
          Hide
          Konstantin Boudnik added a comment -

          Committed to the master. Thank you Xiaomin!

          Show
          Konstantin Boudnik added a comment - Committed to the master. Thank you Xiaomin!
          Konstantin Boudnik made changes -
          Status Patch Available [ 10002 ] Resolved [ 5 ]
          Resolution Fixed [ 1 ]
          Transition Time In Source Status Execution Times Last Executer Last Execution Date
          Open Open Patch Available Patch Available
          114d 16h 46m 1 Xiaomin Zhang 10/Jul/14 16:09
          Patch Available Patch Available Resolved Resolved
          4d 5h 15m 1 Konstantin Boudnik 14/Jul/14 21:24

            People

            • Assignee:
              Xiaomin Zhang
              Reporter:
              Konstantin Boudnik
            • Votes:
              0 Vote for this issue
              Watchers:
              4 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Development