Whirr
  1. Whirr
  2. WHIRR-426

Create a convention for naming clusters used for integration tests

    Details

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

      Description

      I'm creating this issue because right now it's not possible for multiple users to run integration tests at the same time under a shared cloud provider account and it's not possible to run the same test against multiple cloud providers on the same machine.

      I suggest that we should name clusters used for integration testing like this:
      whirr.cluster-name=ganglia-itest-$

      {sys:whirr.test.provider}

      -$

      {sys:user.name}

      I know it's not perfect but it should work most of the time.

      1. WHIRR-426.patch
        12 kB
        Andrei Savu

        Issue Links

          Activity

          Hide
          Andrei Savu added a comment -

          Attached patch that should be trivial to review.

          Show
          Andrei Savu added a comment - Attached patch that should be trivial to review.
          Hide
          David Alves added a comment -

          +1 looks good. The patch is simple enough.

          Show
          David Alves added a comment - +1 looks good. The patch is simple enough.
          Hide
          Andrei Savu added a comment -

          Committed. Thanks David for reviewing!

          Show
          Andrei Savu added a comment - Committed. Thanks David for reviewing!
          Hide
          Tom White added a comment -

          Thanks for this Andrei. One question: are hyphens allowed on all cloud providers?

          Show
          Tom White added a comment - Thanks for this Andrei. One question: are hyphens allowed on all cloud providers?
          Hide
          Andrei Savu added a comment -

          Not 100% sure but jclouds is happy to use this as a group name. Works fine on cloudservers & aws-ec2.

          Show
          Andrei Savu added a comment - Not 100% sure but jclouds is happy to use this as a group name. Works fine on cloudservers & aws-ec2.
          Hide
          Tom White added a comment -

          OK, we can revisit if it's ever a problem.

          Show
          Tom White added a comment - OK, we can revisit if it's ever a problem.

            People

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

              Dates

              • Created:
                Updated:
                Resolved:

                Development