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

Update deployment README.md to reflect on better deployment experience

    Details

    • Type: Improvement
    • Status: Resolved
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: 1.0.0
    • Fix Version/s: 1.1.0
    • Component/s: documentation
    • Labels:
      None

      Description

      Current puppet deployment readme file directs a user to manually create Hiera's site.yaml file. We already provide a site.yaml under bigtop-deploy/puppet/hieradata. I suggest we change that to copy the template and edit it accordingly instead.

      1. BIGTOP-2065.patch
        2 kB
        Konstantin Boudnik
      2. BIGTOP-2065.patch
        2 kB
        Konstantin Boudnik
      3. BIGTOP-2065.patch
        2 kB
        Konstantin Boudnik

        Activity

        Hide
        cos Konstantin Boudnik added a comment -

        I will hold on committing this, as I want to hear others' opinion about the change. Thoughts?

        Show
        cos Konstantin Boudnik added a comment - I will hold on committing this, as I want to hear others' opinion about the change. Thoughts?
        Hide
        evans_ye Evans Ye added a comment -

        I like the words you added which makes it clearly. But the example with fake values is also useful because sometimes user can't get the idea what to put in.

        Show
        evans_ye Evans Ye added a comment - I like the words you added which makes it clearly. But the example with fake values is also useful because sometimes user can't get the idea what to put in.
        Hide
        cos Konstantin Boudnik added a comment -

        slightly better formatting

        Show
        cos Konstantin Boudnik added a comment - slightly better formatting
        Hide
        cos Konstantin Boudnik added a comment -

        Per Evans suggestion, here's the updated version with the file example.

        Show
        cos Konstantin Boudnik added a comment - Per Evans suggestion, here's the updated version with the file example.
        Hide
        evans_ye Evans Ye added a comment -

        Very nice to me.
        I've committed and pushed to the master.
        Thanks for the update.

        Show
        evans_ye Evans Ye added a comment - Very nice to me. I've committed and pushed to the master. Thanks for the update.
        Hide
        cos Konstantin Boudnik added a comment -

        Thanks Evans!

        Show
        cos Konstantin Boudnik added a comment - Thanks Evans!

          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