Removing all tags from a Baremetal or Virtual Guest is done by using …

…an empty string (#144) Closes #139
softlayer · Apr 27, 2017 · fb01e5c · fb01e5c
1 parent 4e8737c
commit fb01e5c
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 10 deletions.
diff --git a/softlayer/resource_softlayer_bare_metal.go b/softlayer/resource_softlayer_bare_metal.go
@@ -526,11 +526,9 @@ func setHardwareTags(id int, d *schema.ResourceData, meta interface{}) error {
 	service := services.GetHardwareService(meta.(ProviderConfig).SoftLayerSession())
 
 	tags := getTags(d)
-	if tags != "" {
-		_, err := service.Id(id).SetTags(sl.String(tags))
-		if err != nil {
-			return fmt.Errorf("Could not set tags on bare metal server %d", id)
-		}
+	_, err := service.Id(id).SetTags(sl.String(tags))
+	if err != nil {
+		return fmt.Errorf("Could not set tags on bare metal server %d", id)
 	}
 
 	return nil

diff --git a/softlayer/resource_softlayer_virtual_guest.go b/softlayer/resource_softlayer_virtual_guest.go
@@ -1114,11 +1114,9 @@ func setGuestTags(id int, d *schema.ResourceData, meta interface{}) error {
 	service := services.GetVirtualGuestService(meta.(ProviderConfig).SoftLayerSession())
 
 	tags := getTags(d)
-	if tags != "" {
-		_, err := service.Id(id).SetTags(sl.String(tags))
-		if err != nil {
-			return fmt.Errorf("Could not set tags on virtual guest %d", id)
-		}
+	_, err := service.Id(id).SetTags(sl.String(tags))
+	if err != nil {
+		return fmt.Errorf("Could not set tags on virtual guest %d", id)
 	}
 
 	return nil