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

        Hide
        Andrei Savu added a comment -

        Committed to 0.7 branch.

        Show
        Andrei Savu added a comment - Committed to 0.7 branch.
        Hide
        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!
        Hide
        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
        Andrei Savu added a comment -

        This one is awaiting your review.

        Show
        Andrei Savu added a comment - This one is awaiting your review.
        Hide
        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.
        Hide
        Andrei Savu added a comment -

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

        Show
        Andrei Savu added a comment - ... unless whirr.cluster-user != root

          People

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

            Dates

            • Created:
              Updated:
              Resolved:

              Development