TestSnapshotPattern uses default SG for servers

[Andrea Frittoli]

TestSnapshotPattern uses default SG for servers, and it assumes that the "loginable" rules do not exist there.

The test should create its own SG (like other test do - e.g. TestServerBasicOps), and create the "loginable" rules in it.

[Andrea Frittoli (andrea-frittoli)]

The issue still exists.

[Sean Dague (sdague)]

This should be simple enough to add a function to this tests which creates a new security group for the servers in question.

[OpenStack Infra (hudson-openstack)]

Fix proposed to branch: master
Review: https://review.openstack.org/86049

[OpenStack Infra (hudson-openstack)]

Reviewed: https://review.openstack.org/86049
Committed: https://git.openstack.org/cgit/openstack/tempest/commit/?id=0f1e11c1ef4054a0a3645fde2b824153dc849fdc
Submitter: Jenkins
Branch: master

commit 0f1e11c1ef4054a0a3645fde2b824153dc849fdc
Author: Grishkin <email address hidden>
Date: Sun May 4 20:44:52 2014 +0400

    Reworked scenario tests to use their own non-default security groups

    Moved _create_security_group_nova to base OfficialClientTest class, so
    that now it can be used by all scenario tests. All scenario tests now
    create security group with "loginable" rules explicitly and attach it to
    servers instead of using "default" security group.

    Change-Id: If82c681b4a7f62808f46b03c57d113f8f5c97f6f
    Closes-Bug: #1247060