Whirr
  1. Whirr
  2. WHIRR-463

Fail fast when running as root

        Details

        • Type: Improvement Improvement
        • Status: Resolved
        • Priority: Major Major
        • Resolution: Fixed
        • Affects Version/s: None
        • Fix Version/s: 0.7.1
        • Component/s: core
        • Labels:
          None

          Description

          Fail fast if the Whirr CLI is running as root and show how whirr.cluster-user can be used to avoid any issues.

            Activity

            Ascending order - Click to sort in descending order
            Andrei Savu created issue -
            Andrei Savu made changes -
            Field Original Value New Value
            Fix Version/s 0.7.1 [ 12319462 ]
            Fix Version/s 0.8.0 [ 12318880 ]
            Hide
            Permalink
            Andrei Savu added a comment -

            ... unless whirr.cluster-user != root

            Show
            Andrei Savu added a comment - ... unless whirr.cluster-user != root
            Andrei Savu made changes -
            Assignee Andrei Savu [ savu.andrei ]
            Andrei Savu made changes -
            Fix Version/s 0.8.0 [ 12318880 ]
            Fix Version/s 0.7.1 [ 12319462 ]
            Hide
            Permalink
            Andrei Savu added a comment -

            With this patch whirr.cluster-user != root is mandatory - enforced by ClusterSpec.

            Show
            Andrei Savu added a comment - With this patch whirr.cluster-user != root is mandatory - enforced by ClusterSpec.
            Andrei Savu made changes -
            Attachment WHIRR-463.patch [ 12513720 ]
            Andrei Savu made changes -
            Status Open [ 1 ] Patch Available [ 10002 ]
            Hide
            Permalink
            Andrei Savu added a comment -

            This one is awaiting your review.

            Show
            Andrei Savu added a comment - This one is awaiting your review.
            Hide
            Permalink
            Tom White added a comment -

            +1 (but please don't change import ordering).

            Show
            Tom White added a comment - +1 (but please don't change import ordering).
            Hide
            Permalink
            Andrei Savu added a comment -

            Committed to trunk (without re-ordering the imports). Thanks Tom!

            Show
            Andrei Savu added a comment - Committed to trunk (without re-ordering the imports). Thanks Tom!
            Andrei Savu made changes -
            Status Patch Available [ 10002 ] Resolved [ 5 ]
            Resolution Fixed [ 1 ]
            Hide
            Permalink
            Andrei Savu added a comment -

            Committed to 0.7 branch.

            Show
            Andrei Savu added a comment - Committed to 0.7 branch.
            Andrei Savu made changes -
            Fix Version/s 0.7.1 [ 12319942 ]
            Component/s core [ 12313574 ]
            Andrei Savu made changes -
            Fix Version/s 0.8.0 [ 12318880 ]

              People

              • Assignee:
                Andrei Savu
                Reporter:
                Andrei Savu
              • Votes:
                0 Vote for this issue
                Watchers:
                0 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved:

                  Development