Details

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

      Description

      There's a use case where stack components could be installed from more than one repo. Say, a stable version and hotfix updates just for some of the components. However, how deployment code only allows to specify one repo at the time (see bigtop-deploy/puppet/manifests/bigtop_repo.pp)

      Would be great to be able to specify a list of repo URLs and have a number of repos files generated.

        Activity

        Hide
        evans_ye Evans Ye added a comment -

        +1.
        Committed and pushed.
        Thanks for the beautiful code.

        Show
        evans_ye Evans Ye added a comment - +1. Committed and pushed. Thanks for the beautiful code.
        Hide
        cos Konstantin Boudnik added a comment -

        Actually, it won't break the provisioner, because you still can use the single repo URL. And it should work as before.

        Show
        cos Konstantin Boudnik added a comment - Actually, it won't break the provisioner, because you still can use the single repo URL. And it should work as before.
        Hide
        evans_ye Evans Ye added a comment -

        Oh. It seems the beautiful code nicely passed my test on centos-7. Let me try more.

        Show
        evans_ye Evans Ye added a comment - Oh. It seems the beautiful code nicely passed my test on centos-7. Let me try more.
        Hide
        evans_ye Evans Ye added a comment -

        LGTM. but this will break Provisioner and Sandbox right after merged.
        Let me provide a patch first so that we can get these properly tested.

        Show
        evans_ye Evans Ye added a comment - LGTM. but this will break Provisioner and Sandbox right after merged. Let me provide a patch first so that we can get these properly tested.
        Hide
        rvs Roman Shaposhnik added a comment -

        LGTM!

        Show
        rvs Roman Shaposhnik added a comment - LGTM!
        Hide
        cos Konstantin Boudnik added a comment -

        That should do it.
        To avoid any confusion, later I want to reformat the file to be compliant with out styling guidelines.

        I'd appreciate the second opinion on this, esp from Olaf Flebbe and Evans Ye.
        Thank you!

        Show
        cos Konstantin Boudnik added a comment - That should do it. To avoid any confusion, later I want to reformat the file to be compliant with out styling guidelines. I'd appreciate the second opinion on this, esp from Olaf Flebbe and Evans Ye . Thank you!
        Hide
        cos Konstantin Boudnik added a comment -

        The default repo needs to be bumped to 1.2.0 (or whatever the latest would be)

        Show
        cos Konstantin Boudnik added a comment - The default repo needs to be bumped to 1.2.0 (or whatever the latest would be)

          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