Uploaded image for project: 'Bigtop'
  1. Bigtop
  2. BIGTOP-1697

Bootstrap: hook up toolchain into gradle build

    Details

    • Type: Improvement
    • Status: Resolved
    • Priority: Critical
    • Resolution: Fixed
    • Affects Version/s: 0.8.0
    • Fix Version/s: 1.0.0
    • Component/s: build
    • Labels:
      None

      Description

      We can simplify the bootstrap of the development environment by hooking the toolchain right into the gradle build.

      Thanks to the development of gradlew now you don't need to have anything on a system to have a functional build. I'd say a need to run puppet apply manually is going against this great paradigm: users needs to go through README files and try to figure out what to do. I'd rather do
      gradle toolchain that will run sudo puppet apply... under the hood and deliver a functional environment.

        Activity

        Hide
        cos Konstantin Boudnik added a comment - - edited

        Here we go. There might be a better way to do it, but at least this will bridge the gap for now.

        Show
        cos Konstantin Boudnik added a comment - - edited Here we go. There might be a better way to do it, but at least this will bridge the gap for now.
        Hide
        jayunit100 jay vyas added a comment - - edited

        Excellent idea. Im almost on the verge of saying , since the toolchain is really just for building, and we can do that in dockerfiles, someday "apply toolchain" can be replaced w/ "docker pull".

        But thats another conversation.

        +1 ! on the patch

        Show
        jayunit100 jay vyas added a comment - - edited Excellent idea. Im almost on the verge of saying , since the toolchain is really just for building, and we can do that in dockerfiles, someday "apply toolchain" can be replaced w/ "docker pull". But thats another conversation. +1 ! on the patch
        Hide
        cos Konstantin Boudnik added a comment -

        I don't think we should replace it, but perhaps complete one with another?
        Thanks for the review, will commit it shortly.

        Show
        cos Konstantin Boudnik added a comment - I don't think we should replace it, but perhaps complete one with another? Thanks for the review, will commit it shortly.
        Hide
        cos Konstantin Boudnik added a comment -

        Pushed.

        Show
        cos Konstantin Boudnik added a comment - Pushed.

          People

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

            Dates

            • Created:
              Updated:
              Resolved:

              Development