add various fixes
diff --git a/acceptance/rackspace/db/v1/config_group_test.go b/acceptance/rackspace/db/v1/config_group_test.go
index 85c3756..81bd40a 100644
--- a/acceptance/rackspace/db/v1/config_group_test.go
+++ b/acceptance/rackspace/db/v1/config_group_test.go
@@ -4,13 +4,14 @@
import (
"github.com/rackspace/gophercloud/acceptance/tools"
+ os "github.com/rackspace/gophercloud/openstack/db/v1/configurations"
"github.com/rackspace/gophercloud/pagination"
config "github.com/rackspace/gophercloud/rackspace/db/v1/configurations"
"github.com/rackspace/gophercloud/rackspace/db/v1/instances"
)
func (c *context) createConfigGrp() {
- opts := config.CreateOpts{
+ opts := os.CreateOpts{
Name: tools.RandomString("config_", 5),
Values: map[string]interface{}{
"connect_timeout": 300,
@@ -33,7 +34,7 @@
}
func (c *context) updateConfigGrp() {
- opts := config.UpdateOpts{
+ opts := os.UpdateOpts{
Name: tools.RandomString("new_name_", 5),
Values: map[string]interface{}{
"connect_timeout": 250,
@@ -45,9 +46,9 @@
}
func (c *context) replaceConfigGrp() {
- opts := config.UpdateOpts{
+ opts := os.UpdateOpts{
Values: map[string]interface{}{
- "expire_logs_days": 7,
+ "big_tables": 1,
},
}
diff --git a/acceptance/rackspace/db/v1/instance_test.go b/acceptance/rackspace/db/v1/instance_test.go
index f6e1f03..5441763 100644
--- a/acceptance/rackspace/db/v1/instance_test.go
+++ b/acceptance/rackspace/db/v1/instance_test.go
@@ -5,7 +5,6 @@
import (
"testing"
- "github.com/rackspace/gophercloud"
"github.com/rackspace/gophercloud/acceptance/tools"
"github.com/rackspace/gophercloud/pagination"
"github.com/rackspace/gophercloud/rackspace/db/v1/instances"
@@ -69,8 +68,12 @@
// TEARDOWN
c.deleteUsers()
c.deleteDBs()
- c.detachAndDeleteReplica()
- c.deleteInstance()
+
+ c.restartInstance()
+ c.WaitUntilActive(c.instanceID)
+
+ c.deleteInstance(c.replicaID)
+ c.deleteInstance(c.instanceID)
}
func (c *context) createInstance() {
@@ -114,10 +117,10 @@
c.Logf("Getting instance: %#v", instance)
}
-func (c *context) deleteInstance() {
- err := instances.Delete(c.client, c.instanceID).ExtractErr()
+func (c *context) deleteInstance(id string) {
+ err := instances.Delete(c.client, id).ExtractErr()
c.AssertNoErr(err)
- c.Logf("Deleted instance %s", c.instanceID)
+ c.Logf("Deleted instance %s", id)
}
func (c *context) enableRootUser() {
@@ -164,25 +167,3 @@
c.Logf("Default config group for instance %s: %#v", c.instanceID, config)
c.AssertNoErr(err)
}
-
-func (c *context) detachAndDeleteReplica() {
- err := instances.DetachReplica(c.client, c.replicaID).ExtractErr()
- c.AssertNoErr(err)
- c.Logf("Detached replica %s from instance %s", c.replicaID, c.instanceID)
-
- err = instances.Delete(c.client, c.replicaID).ExtractErr()
- c.AssertNoErr(err)
- c.Logf("Deleted replica %s", c.replicaID)
-
- // Check that it's deleted
- err = gophercloud.WaitFor(60, func() (bool, error) {
- _, err := instances.Get(c.client, c.replicaID).Extract()
- if casted, ok := err.(*gophercloud.UnexpectedResponseCodeError); ok && casted.Actual == 404 {
- return true, nil
- }
- if err != nil {
- return false, err
- }
- return false, nil
- })
-}