Move site to provider level
This commit is contained in:
@@ -35,6 +35,5 @@ resource "unifi_wlan" "test" {
|
||||
|
||||
## TODO
|
||||
|
||||
* [ ] Move site to provider level? (or use 2 value IDs?)
|
||||
* [ ] WLAN Groups (data source for default?)
|
||||
* [ ] User Groups (data source for default?)
|
||||
|
||||
@@ -27,11 +27,16 @@ func Provider() terraform.ResourceProvider {
|
||||
Required: true,
|
||||
DefaultFunc: schema.EnvDefaultFunc("UNIFI_API", ""),
|
||||
},
|
||||
"allow_insecure": {
|
||||
Type: schema.TypeBool,
|
||||
Optional: true,
|
||||
Default: false,
|
||||
"site": {
|
||||
Type: schema.TypeString,
|
||||
Required: true,
|
||||
DefaultFunc: schema.EnvDefaultFunc("UNIFI_SITE", "default"),
|
||||
},
|
||||
// "allow_insecure": {
|
||||
// Type: schema.TypeBool,
|
||||
// Optional: true,
|
||||
// Default: false,
|
||||
// },
|
||||
},
|
||||
DataSourcesMap: map[string]*schema.Resource{
|
||||
// "scaffolding_data_source": dataSourceScaffolding(),
|
||||
@@ -50,10 +55,12 @@ func configure(p *schema.Provider) schema.ConfigureFunc {
|
||||
user := d.Get("username").(string)
|
||||
pass := d.Get("password").(string)
|
||||
baseURL := d.Get("api_url").(string)
|
||||
site := d.Get("site").(string)
|
||||
//insecure := d.Get("allow_insecure").(bool)
|
||||
|
||||
c := &client{
|
||||
c: &unifi.Client{},
|
||||
c: &unifi.Client{},
|
||||
site: site,
|
||||
}
|
||||
|
||||
c.c.SetBaseURL(baseURL)
|
||||
@@ -69,5 +76,6 @@ func configure(p *schema.Provider) schema.ConfigureFunc {
|
||||
}
|
||||
|
||||
type client struct {
|
||||
c *unifi.Client
|
||||
c *unifi.Client
|
||||
site string
|
||||
}
|
||||
|
||||
@@ -1,8 +1,8 @@
|
||||
package provider
|
||||
|
||||
import (
|
||||
"testing"
|
||||
"os"
|
||||
"testing"
|
||||
|
||||
"github.com/hashicorp/terraform-plugin-sdk/helper/resource"
|
||||
"github.com/hashicorp/terraform-plugin-sdk/helper/schema"
|
||||
@@ -38,3 +38,18 @@ func importStep(name string, ignore ...string) resource.TestStep {
|
||||
|
||||
return step
|
||||
}
|
||||
|
||||
func preCheck(t *testing.T) {
|
||||
variables := []string{
|
||||
"UNIFI_USERNAME",
|
||||
"UNIFI_PASSWORD",
|
||||
"UNIFI_API",
|
||||
}
|
||||
|
||||
for _, variable := range variables {
|
||||
value := os.Getenv(variable)
|
||||
if value == "" {
|
||||
t.Fatalf("`%s` must be set for acceptance tests!", variable)
|
||||
}
|
||||
}
|
||||
}
|
||||
|
||||
@@ -14,19 +14,11 @@ func resourceNetwork() *schema.Resource {
|
||||
Read: resourceNetworkRead,
|
||||
Update: resourceNetworkUpdate,
|
||||
Delete: resourceNetworkDelete,
|
||||
|
||||
// TODO: handle site + ID (or name)
|
||||
// Importer: &schema.ResourceImporter{
|
||||
// State: schema.ImportStatePassthrough,
|
||||
// },
|
||||
Importer: &schema.ResourceImporter{
|
||||
State: schema.ImportStatePassthrough,
|
||||
},
|
||||
|
||||
Schema: map[string]*schema.Schema{
|
||||
"site": {
|
||||
Type: schema.TypeString,
|
||||
Optional: true,
|
||||
Default: "default",
|
||||
ForceNew: true,
|
||||
},
|
||||
"name": {
|
||||
Type: schema.TypeString,
|
||||
Required: true,
|
||||
@@ -75,8 +67,6 @@ func resourceNetwork() *schema.Resource {
|
||||
func resourceNetworkCreate(d *schema.ResourceData, meta interface{}) error {
|
||||
c := meta.(*client)
|
||||
|
||||
site := d.Get("site").(string)
|
||||
|
||||
req := &unifi.Network{
|
||||
Name: d.Get("name").(string),
|
||||
Purpose: d.Get("purpose").(string),
|
||||
@@ -96,7 +86,7 @@ func resourceNetworkCreate(d *schema.ResourceData, meta interface{}) error {
|
||||
// IPV6PDStop string `json:"ipv6_pd_stop"` // "::7d1"
|
||||
}
|
||||
|
||||
resp, err := c.c.CreateNetwork(site, req)
|
||||
resp, err := c.c.CreateNetwork(c.site, req)
|
||||
if err != nil {
|
||||
return err
|
||||
}
|
||||
@@ -109,10 +99,9 @@ func resourceNetworkCreate(d *schema.ResourceData, meta interface{}) error {
|
||||
func resourceNetworkRead(d *schema.ResourceData, meta interface{}) error {
|
||||
c := meta.(*client)
|
||||
|
||||
site := d.Get("site").(string)
|
||||
id := d.Id()
|
||||
|
||||
_, err := c.c.GetNetwork(site, id)
|
||||
_, err := c.c.GetNetwork(c.site, id)
|
||||
if _, ok := err.(*unifi.NotFoundError); ok {
|
||||
d.SetId("")
|
||||
return nil
|
||||
@@ -131,11 +120,10 @@ func resourceNetworkUpdate(d *schema.ResourceData, meta interface{}) error {
|
||||
func resourceNetworkDelete(d *schema.ResourceData, meta interface{}) error {
|
||||
c := meta.(*client)
|
||||
|
||||
site := d.Get("site").(string)
|
||||
name := d.Get("name").(string)
|
||||
id := d.Id()
|
||||
|
||||
err := c.c.DeleteNetwork(site, id, name)
|
||||
err := c.c.DeleteNetwork(c.site, id, name)
|
||||
if _, ok := err.(*unifi.NotFoundError); ok {
|
||||
return nil
|
||||
}
|
||||
|
||||
@@ -8,6 +8,7 @@ import (
|
||||
|
||||
func TestAccNetwork_basic(t *testing.T) {
|
||||
resource.ParallelTest(t, resource.TestCase{
|
||||
PreCheck: func() { preCheck(t) },
|
||||
Providers: providers,
|
||||
// TODO: CheckDestroy: ,
|
||||
Steps: []resource.TestStep{
|
||||
@@ -17,7 +18,7 @@ func TestAccNetwork_basic(t *testing.T) {
|
||||
// testCheckNetworkExists(t, "name"),
|
||||
),
|
||||
},
|
||||
// importStep("unifi_network.test"),
|
||||
importStep("unifi_network.test"),
|
||||
},
|
||||
})
|
||||
}
|
||||
|
||||
@@ -14,19 +14,11 @@ func resourceWLAN() *schema.Resource {
|
||||
Read: resourceWLANRead,
|
||||
Update: resourceWLANUpdate,
|
||||
Delete: resourceWLANDelete,
|
||||
|
||||
// TODO: handle site + ID (or name)
|
||||
// Importer: &schema.ResourceImporter{
|
||||
// State: schema.ImportStatePassthrough,
|
||||
// },
|
||||
Importer: &schema.ResourceImporter{
|
||||
State: schema.ImportStatePassthrough,
|
||||
},
|
||||
|
||||
Schema: map[string]*schema.Schema{
|
||||
"site": {
|
||||
Type: schema.TypeString,
|
||||
Optional: true,
|
||||
Default: "default",
|
||||
ForceNew: true,
|
||||
},
|
||||
"name": {
|
||||
Type: schema.TypeString,
|
||||
Required: true,
|
||||
@@ -48,10 +40,8 @@ func resourceWLAN() *schema.Resource {
|
||||
func resourceWLANCreate(d *schema.ResourceData, meta interface{}) error {
|
||||
c := meta.(*client)
|
||||
|
||||
site := d.Get("site").(string)
|
||||
|
||||
// TODO: allow passing these defaults
|
||||
wlanGroups, err := c.c.ListWLANGroup(site)
|
||||
wlanGroups, err := c.c.ListWLANGroup(c.site)
|
||||
if err != nil {
|
||||
return err
|
||||
}
|
||||
@@ -66,7 +56,7 @@ func resourceWLANCreate(d *schema.ResourceData, meta interface{}) error {
|
||||
return fmt.Errorf("unable to find default WLAN group")
|
||||
}
|
||||
|
||||
userGroups, err := c.c.ListUserGroup(site)
|
||||
userGroups, err := c.c.ListUserGroup(c.site)
|
||||
if err != nil {
|
||||
return err
|
||||
}
|
||||
@@ -107,7 +97,7 @@ func resourceWLANCreate(d *schema.ResourceData, meta interface{}) error {
|
||||
MinrateNgMgmtRateKbps: 1000,
|
||||
}
|
||||
|
||||
resp, err := c.c.CreateWLAN(site, req)
|
||||
resp, err := c.c.CreateWLAN(c.site, req)
|
||||
if err != nil {
|
||||
return err
|
||||
}
|
||||
@@ -120,10 +110,9 @@ func resourceWLANCreate(d *schema.ResourceData, meta interface{}) error {
|
||||
func resourceWLANRead(d *schema.ResourceData, meta interface{}) error {
|
||||
c := meta.(*client)
|
||||
|
||||
site := d.Get("site").(string)
|
||||
id := d.Id()
|
||||
|
||||
_, err := c.c.GetWLAN(site, id)
|
||||
_, err := c.c.GetWLAN(c.site, id)
|
||||
if _, ok := err.(*unifi.NotFoundError); ok {
|
||||
d.SetId("")
|
||||
return nil
|
||||
@@ -142,10 +131,9 @@ func resourceWLANUpdate(d *schema.ResourceData, meta interface{}) error {
|
||||
func resourceWLANDelete(d *schema.ResourceData, meta interface{}) error {
|
||||
c := meta.(*client)
|
||||
|
||||
site := d.Get("site").(string)
|
||||
id := d.Id()
|
||||
|
||||
err := c.c.DeleteWLAN(site, id)
|
||||
err := c.c.DeleteWLAN(c.site, id)
|
||||
if _, ok := err.(*unifi.NotFoundError); ok {
|
||||
return nil
|
||||
}
|
||||
|
||||
@@ -8,6 +8,7 @@ import (
|
||||
|
||||
func TestAccWLAN_basic(t *testing.T) {
|
||||
resource.ParallelTest(t, resource.TestCase{
|
||||
PreCheck: func() { preCheck(t) },
|
||||
Providers: providers,
|
||||
// TODO: CheckDestroy: ,
|
||||
Steps: []resource.TestStep{
|
||||
@@ -17,7 +18,7 @@ func TestAccWLAN_basic(t *testing.T) {
|
||||
// testCheckNetworkExists(t, "name"),
|
||||
),
|
||||
},
|
||||
// importStep("unifi_wlan.test"),
|
||||
importStep("unifi_wlan.test"),
|
||||
},
|
||||
})
|
||||
}
|
||||
|
||||
Reference in New Issue
Block a user