From 4f590cb4fd558c9c2112ca822c0edf8cbb34c966 Mon Sep 17 00:00:00 2001 From: Sean Chittenden Date: Wed, 10 May 2017 14:54:06 -0700 Subject: [PATCH] Update to the latest github.com/joyent/triton-go --- .../github.com/joyent/triton-go/accounts.go | 13 +-- vendor/github.com/joyent/triton-go/client.go | 51 ++++----- vendor/github.com/joyent/triton-go/config.go | 12 ++- .../joyent/triton-go/datacenters.go | 9 +- vendor/github.com/joyent/triton-go/fabrics.go | 37 +++---- .../github.com/joyent/triton-go/firewall.go | 37 ++++--- vendor/github.com/joyent/triton-go/images.go | 25 ++--- vendor/github.com/joyent/triton-go/keys.go | 20 ++-- .../github.com/joyent/triton-go/machines.go | 100 +++++++++--------- .../github.com/joyent/triton-go/networks.go | 11 +- .../github.com/joyent/triton-go/packages.go | 10 +- vendor/github.com/joyent/triton-go/roles.go | 33 +++--- .../github.com/joyent/triton-go/services.go | 6 +- vendor/vendor.json | 6 +- 14 files changed, 192 insertions(+), 178 deletions(-) diff --git a/vendor/github.com/joyent/triton-go/accounts.go b/vendor/github.com/joyent/triton-go/accounts.go index 474cce000..88e7bbf12 100644 --- a/vendor/github.com/joyent/triton-go/accounts.go +++ b/vendor/github.com/joyent/triton-go/accounts.go @@ -1,12 +1,12 @@ package triton import ( + "context" "encoding/json" + "fmt" "net/http" "time" - "fmt" - "github.com/hashicorp/errwrap" ) @@ -40,9 +40,9 @@ type Account struct { type GetAccountInput struct{} -func (client *AccountsClient) GetAccount(input *GetAccountInput) (*Account, error) { +func (client *AccountsClient) GetAccount(ctx context.Context, input *GetAccountInput) (*Account, error) { path := fmt.Sprintf("/%s", client.accountName) - respReader, err := client.executeRequest(http.MethodGet, path, nil) + respReader, err := client.executeRequest(ctx, http.MethodGet, path, nil) if respReader != nil { defer respReader.Close() } @@ -75,8 +75,9 @@ type UpdateAccountInput struct { // UpdateAccount updates your account details with the given parameters. // TODO(jen20) Work out a safe way to test this -func (client *AccountsClient) UpdateAccount(input *UpdateAccountInput) (*Account, error) { - respReader, err := client.executeRequest(http.MethodPost, fmt.Sprintf("/%s", client.accountName), input) +func (client *AccountsClient) UpdateAccount(ctx context.Context, input *UpdateAccountInput) (*Account, error) { + path := fmt.Sprintf("/%s", client.accountName) + respReader, err := client.executeRequest(ctx, http.MethodPost, path, input) if respReader != nil { defer respReader.Close() } diff --git a/vendor/github.com/joyent/triton-go/client.go b/vendor/github.com/joyent/triton-go/client.go index 5ee1f0be7..c0ecc5fd0 100644 --- a/vendor/github.com/joyent/triton-go/client.go +++ b/vendor/github.com/joyent/triton-go/client.go @@ -2,25 +2,25 @@ package triton import ( "bytes" + "context" "crypto/tls" "encoding/json" - "fmt" + "errors" "io" - "log" "net" "net/http" "net/url" - "os" "time" "github.com/hashicorp/errwrap" - "github.com/hashicorp/go-retryablehttp" "github.com/joyent/triton-go/authentication" ) +const nilContext = "nil context" + // Client represents a connection to the Triton API. type Client struct { - client *retryablehttp.Client + client *http.Client authorizer []authentication.Signer apiURL url.URL accountName string @@ -32,17 +32,13 @@ type Client struct { // At least one signer must be provided - example signers include // authentication.PrivateKeySigner and authentication.SSHAgentSigner. func NewClient(endpoint string, accountName string, signers ...authentication.Signer) (*Client, error) { - defaultRetryWaitMin := 1 * time.Second - defaultRetryWaitMax := 5 * time.Minute - defaultRetryMax := 32 - apiURL, err := url.Parse(endpoint) if err != nil { return nil, errwrap.Wrapf("invalid endpoint: {{err}}", err) } if accountName == "" { - return nil, fmt.Errorf("account name can not be empty") + return nil, errors.New("account name can not be empty") } httpClient := &http.Client{ @@ -50,17 +46,8 @@ func NewClient(endpoint string, accountName string, signers ...authentication.Si CheckRedirect: doNotFollowRedirects, } - retryableClient := &retryablehttp.Client{ - HTTPClient: httpClient, - Logger: log.New(os.Stderr, "", log.LstdFlags), - RetryWaitMin: defaultRetryWaitMin, - RetryWaitMax: defaultRetryWaitMax, - RetryMax: defaultRetryMax, - CheckRetry: retryablehttp.DefaultRetryPolicy, - } - return &Client{ - client: retryableClient, + client: httpClient, authorizer: signers, apiURL: *apiURL, accountName: accountName, @@ -76,7 +63,7 @@ func (c *Client) InsecureSkipTLSVerify() { return } - c.client.HTTPClient.Transport = httpTransport(true) + c.client.Transport = httpTransport(true) } func httpTransport(insecureSkipTLSVerify bool) *http.Transport { @@ -99,7 +86,7 @@ func doNotFollowRedirects(*http.Request, []*http.Request) error { return http.ErrUseLastResponse } -func (c *Client) executeRequestURIParams(method, path string, body interface{}, query *url.Values) (io.ReadCloser, error) { +func (c *Client) executeRequestURIParams(ctx context.Context, method, path string, body interface{}, query *url.Values) (io.ReadCloser, error) { var requestBody io.ReadSeeker if body != nil { marshaled, err := json.MarshalIndent(body, "", " ") @@ -115,7 +102,7 @@ func (c *Client) executeRequestURIParams(method, path string, body interface{}, endpoint.RawQuery = query.Encode() } - req, err := retryablehttp.NewRequest(method, endpoint.String(), requestBody) + req, err := http.NewRequest(method, endpoint.String(), requestBody) if err != nil { return nil, errwrap.Wrapf("Error constructing HTTP request: {{err}}", err) } @@ -136,7 +123,7 @@ func (c *Client) executeRequestURIParams(method, path string, body interface{}, req.Header.Set("Content-Type", "application/json") } - resp, err := c.client.Do(req) + resp, err := c.client.Do(req.WithContext(ctx)) if err != nil { return nil, errwrap.Wrapf("Error executing HTTP request: {{err}}", err) } @@ -149,23 +136,23 @@ func (c *Client) executeRequestURIParams(method, path string, body interface{}, } func (c *Client) decodeError(statusCode int, body io.Reader) error { - tritonError := &TritonError{ + err := &TritonError{ StatusCode: statusCode, } errorDecoder := json.NewDecoder(body) - if err := errorDecoder.Decode(tritonError); err != nil { + if err := errorDecoder.Decode(err); err != nil { return errwrap.Wrapf("Error decoding error response: {{err}}", err) } - return tritonError + return err } -func (c *Client) executeRequest(method, path string, body interface{}) (io.ReadCloser, error) { - return c.executeRequestURIParams(method, path, body, nil) +func (c *Client) executeRequest(ctx context.Context, method, path string, body interface{}) (io.ReadCloser, error) { + return c.executeRequestURIParams(ctx, method, path, body, nil) } -func (c *Client) executeRequestRaw(method, path string, body interface{}) (*http.Response, error) { +func (c *Client) executeRequestRaw(ctx context.Context, method, path string, body interface{}) (*http.Response, error) { var requestBody io.ReadSeeker if body != nil { marshaled, err := json.MarshalIndent(body, "", " ") @@ -178,7 +165,7 @@ func (c *Client) executeRequestRaw(method, path string, body interface{}) (*http endpoint := c.apiURL endpoint.Path = path - req, err := retryablehttp.NewRequest(method, endpoint.String(), requestBody) + req, err := http.NewRequest(method, endpoint.String(), requestBody) if err != nil { return nil, errwrap.Wrapf("Error constructing HTTP request: {{err}}", err) } @@ -199,7 +186,7 @@ func (c *Client) executeRequestRaw(method, path string, body interface{}) (*http req.Header.Set("Content-Type", "application/json") } - resp, err := c.client.Do(req) + resp, err := c.client.Do(req.WithContext(ctx)) if err != nil { return nil, errwrap.Wrapf("Error executing HTTP request: {{err}}", err) } diff --git a/vendor/github.com/joyent/triton-go/config.go b/vendor/github.com/joyent/triton-go/config.go index 83d27790a..b4f20e0a8 100644 --- a/vendor/github.com/joyent/triton-go/config.go +++ b/vendor/github.com/joyent/triton-go/config.go @@ -1,6 +1,7 @@ package triton import ( + "context" "encoding/json" "fmt" "net/http" @@ -27,8 +28,9 @@ type Config struct { type GetConfigInput struct{} // GetConfig outputs configuration for your account. -func (client *ConfigClient) GetConfig(input *GetConfigInput) (*Config, error) { - respReader, err := client.executeRequest(http.MethodGet, fmt.Sprintf("/%s/config", client.accountName), nil) +func (client *ConfigClient) GetConfig(ctx context.Context, input *GetConfigInput) (*Config, error) { + path := fmt.Sprintf("/%s/config", client.accountName) + respReader, err := client.executeRequest(ctx, http.MethodGet, path, nil) if respReader != nil { defer respReader.Close() } @@ -51,9 +53,9 @@ type UpdateConfigInput struct { } // UpdateConfig updates configuration values for your account. -// TODO(jen20) Work out a safe way to test this (after networks c implemented) -func (client *ConfigClient) UpdateConfig(input *UpdateConfigInput) (*Config, error) { - respReader, err := client.executeRequest(http.MethodPut, fmt.Sprintf("/%s/config", client.accountName), input) +func (client *ConfigClient) UpdateConfig(ctx context.Context, input *UpdateConfigInput) (*Config, error) { + path := fmt.Sprintf("/%s/config", client.accountName) + respReader, err := client.executeRequest(ctx, http.MethodPut, path, input) if respReader != nil { defer respReader.Close() } diff --git a/vendor/github.com/joyent/triton-go/datacenters.go b/vendor/github.com/joyent/triton-go/datacenters.go index 72ccbd6d9..2834f77c2 100644 --- a/vendor/github.com/joyent/triton-go/datacenters.go +++ b/vendor/github.com/joyent/triton-go/datacenters.go @@ -7,6 +7,7 @@ import ( "net/http" "sort" + "context" "github.com/hashicorp/errwrap" ) @@ -27,9 +28,9 @@ type DataCenter struct { type ListDataCentersInput struct{} -func (client *DataCentersClient) ListDataCenters(*ListDataCentersInput) ([]*DataCenter, error) { +func (client *DataCentersClient) ListDataCenters(ctx context.Context, _ *ListDataCentersInput) ([]*DataCenter, error) { path := fmt.Sprintf("/%s/datacenters", client.accountName) - respReader, err := client.executeRequest(http.MethodGet, path, nil) + respReader, err := client.executeRequest(ctx, http.MethodGet, path, nil) if respReader != nil { defer respReader.Close() } @@ -68,9 +69,9 @@ type GetDataCenterInput struct { Name string } -func (client *DataCentersClient) GetDataCenter(input *GetDataCenterInput) (*DataCenter, error) { +func (client *DataCentersClient) GetDataCenter(ctx context.Context, input *GetDataCenterInput) (*DataCenter, error) { path := fmt.Sprintf("/%s/datacenters/%s", client.accountName, input.Name) - resp, err := client.executeRequestRaw(http.MethodGet, path, nil) + resp, err := client.executeRequestRaw(ctx, http.MethodGet, path, nil) if err != nil { return nil, errwrap.Wrapf("Error executing GetDatacenter request: {{err}}", err) } diff --git a/vendor/github.com/joyent/triton-go/fabrics.go b/vendor/github.com/joyent/triton-go/fabrics.go index 5b1caa521..1d32b83ba 100644 --- a/vendor/github.com/joyent/triton-go/fabrics.go +++ b/vendor/github.com/joyent/triton-go/fabrics.go @@ -5,6 +5,7 @@ import ( "fmt" "net/http" + "context" "github.com/hashicorp/errwrap" ) @@ -26,9 +27,9 @@ type FabricVLAN struct { type ListFabricVLANsInput struct{} -func (client *FabricsClient) ListFabricVLANs(*ListFabricVLANsInput) ([]*FabricVLAN, error) { +func (client *FabricsClient) ListFabricVLANs(ctx context.Context, _ *ListFabricVLANsInput) ([]*FabricVLAN, error) { path := fmt.Sprintf("/%s/fabrics/default/vlans", client.accountName) - respReader, err := client.executeRequest(http.MethodGet, path, nil) + respReader, err := client.executeRequest(ctx, http.MethodGet, path, nil) if respReader != nil { defer respReader.Close() } @@ -51,9 +52,9 @@ type CreateFabricVLANInput struct { Description string `json:"description"` } -func (client *FabricsClient) CreateFabricVLAN(input *CreateFabricVLANInput) (*FabricVLAN, error) { +func (client *FabricsClient) CreateFabricVLAN(ctx context.Context, input *CreateFabricVLANInput) (*FabricVLAN, error) { path := fmt.Sprintf("/%s/fabrics/default/vlans", client.accountName) - respReader, err := client.executeRequest(http.MethodPost, path, input) + respReader, err := client.executeRequest(ctx, http.MethodPost, path, input) if respReader != nil { defer respReader.Close() } @@ -76,9 +77,9 @@ type UpdateFabricVLANInput struct { Description string `json:"description"` } -func (client *FabricsClient) UpdateFabricVLAN(input *UpdateFabricVLANInput) (*FabricVLAN, error) { +func (client *FabricsClient) UpdateFabricVLAN(ctx context.Context, input *UpdateFabricVLANInput) (*FabricVLAN, error) { path := fmt.Sprintf("/%s/fabrics/default/vlans/%d", client.accountName, input.ID) - respReader, err := client.executeRequest(http.MethodPut, path, input) + respReader, err := client.executeRequest(ctx, http.MethodPut, path, input) if respReader != nil { defer respReader.Close() } @@ -99,9 +100,9 @@ type GetFabricVLANInput struct { ID int `json:"-"` } -func (client *FabricsClient) GetFabricVLAN(input *GetFabricVLANInput) (*FabricVLAN, error) { +func (client *FabricsClient) GetFabricVLAN(ctx context.Context, input *GetFabricVLANInput) (*FabricVLAN, error) { path := fmt.Sprintf("/%s/fabrics/default/vlans/%d", client.accountName, input.ID) - respReader, err := client.executeRequest(http.MethodGet, path, nil) + respReader, err := client.executeRequest(ctx, http.MethodGet, path, nil) if respReader != nil { defer respReader.Close() } @@ -122,9 +123,9 @@ type DeleteFabricVLANInput struct { ID int `json:"-"` } -func (client *FabricsClient) DeleteFabricVLAN(input *DeleteFabricVLANInput) error { +func (client *FabricsClient) DeleteFabricVLAN(ctx context.Context, input *DeleteFabricVLANInput) error { path := fmt.Sprintf("/%s/fabrics/default/vlans/%d", client.accountName, input.ID) - respReader, err := client.executeRequest(http.MethodDelete, path, nil) + respReader, err := client.executeRequest(ctx, http.MethodDelete, path, nil) if respReader != nil { defer respReader.Close() } @@ -139,9 +140,9 @@ type ListFabricNetworksInput struct { FabricVLANID int `json:"-"` } -func (client *FabricsClient) ListFabricNetworks(input *ListFabricNetworksInput) ([]*Network, error) { +func (client *FabricsClient) ListFabricNetworks(ctx context.Context, input *ListFabricNetworksInput) ([]*Network, error) { path := fmt.Sprintf("/%s/fabrics/default/vlans/%d/networks", client.accountName, input.FabricVLANID) - respReader, err := client.executeRequest(http.MethodGet, path, nil) + respReader, err := client.executeRequest(ctx, http.MethodGet, path, nil) if respReader != nil { defer respReader.Close() } @@ -171,9 +172,9 @@ type CreateFabricNetworkInput struct { InternetNAT bool `json:"internet_nat"` } -func (client *FabricsClient) CreateFabricNetwork(input *CreateFabricNetworkInput) (*Network, error) { +func (client *FabricsClient) CreateFabricNetwork(ctx context.Context, input *CreateFabricNetworkInput) (*Network, error) { path := fmt.Sprintf("/%s/fabrics/default/vlans/%d/networks", client.accountName, input.FabricVLANID) - respReader, err := client.executeRequest(http.MethodPost, path, input) + respReader, err := client.executeRequest(ctx, http.MethodPost, path, input) if respReader != nil { defer respReader.Close() } @@ -195,9 +196,9 @@ type GetFabricNetworkInput struct { NetworkID string `json:"-"` } -func (client *FabricsClient) GetFabricNetwork(input *GetFabricNetworkInput) (*Network, error) { +func (client *FabricsClient) GetFabricNetwork(ctx context.Context, input *GetFabricNetworkInput) (*Network, error) { path := fmt.Sprintf("/%s/fabrics/default/vlans/%d/networks/%s", client.accountName, input.FabricVLANID, input.NetworkID) - respReader, err := client.executeRequest(http.MethodGet, path, nil) + respReader, err := client.executeRequest(ctx, http.MethodGet, path, nil) if respReader != nil { defer respReader.Close() } @@ -219,9 +220,9 @@ type DeleteFabricNetworkInput struct { NetworkID string `json:"-"` } -func (client *FabricsClient) DeleteFabricNetwork(input *DeleteFabricNetworkInput) error { +func (client *FabricsClient) DeleteFabricNetwork(ctx context.Context, input *DeleteFabricNetworkInput) error { path := fmt.Sprintf("/%s/fabrics/default/vlans/%d/networks/%s", client.accountName, input.FabricVLANID, input.NetworkID) - respReader, err := client.executeRequest(http.MethodDelete, path, nil) + respReader, err := client.executeRequest(ctx, http.MethodDelete, path, nil) if respReader != nil { defer respReader.Close() } diff --git a/vendor/github.com/joyent/triton-go/firewall.go b/vendor/github.com/joyent/triton-go/firewall.go index e9d57a80e..a3de9ccf6 100644 --- a/vendor/github.com/joyent/triton-go/firewall.go +++ b/vendor/github.com/joyent/triton-go/firewall.go @@ -1,6 +1,7 @@ package triton import ( + "context" "encoding/json" "fmt" "net/http" @@ -38,9 +39,9 @@ type FirewallRule struct { type ListFirewallRulesInput struct{} -func (client *FirewallClient) ListFirewallRules(*ListFirewallRulesInput) ([]*FirewallRule, error) { +func (client *FirewallClient) ListFirewallRules(ctx context.Context, _ *ListFirewallRulesInput) ([]*FirewallRule, error) { path := fmt.Sprintf("/%s/fwrules", client.accountName) - respReader, err := client.executeRequest(http.MethodGet, path, nil) + respReader, err := client.executeRequest(ctx, http.MethodGet, path, nil) if respReader != nil { defer respReader.Close() } @@ -61,9 +62,9 @@ type GetFirewallRuleInput struct { ID string } -func (client *FirewallClient) GetFirewallRule(input *GetFirewallRuleInput) (*FirewallRule, error) { +func (client *FirewallClient) GetFirewallRule(ctx context.Context, input *GetFirewallRuleInput) (*FirewallRule, error) { path := fmt.Sprintf("/%s/fwrules/%s", client.accountName, input.ID) - respReader, err := client.executeRequest(http.MethodGet, path, nil) + respReader, err := client.executeRequest(ctx, http.MethodGet, path, nil) if respReader != nil { defer respReader.Close() } @@ -86,8 +87,9 @@ type CreateFirewallRuleInput struct { Description string `json:"description"` } -func (client *FirewallClient) CreateFirewallRule(input *CreateFirewallRuleInput) (*FirewallRule, error) { - respReader, err := client.executeRequest(http.MethodPost, fmt.Sprintf("/%s/fwrules", client.accountName), input) +func (client *FirewallClient) CreateFirewallRule(ctx context.Context, input *CreateFirewallRuleInput) (*FirewallRule, error) { + path := fmt.Sprintf("/%s/fwrules", client.accountName) + respReader, err := client.executeRequest(ctx, http.MethodPost, path, input) if respReader != nil { defer respReader.Close() } @@ -111,8 +113,9 @@ type UpdateFirewallRuleInput struct { Description string `json:"description"` } -func (client *FirewallClient) UpdateFirewallRule(input *UpdateFirewallRuleInput) (*FirewallRule, error) { - respReader, err := client.executeRequest(http.MethodPost, fmt.Sprintf("/%s/fwrules/%s", client.accountName, input.ID), input) +func (client *FirewallClient) UpdateFirewallRule(ctx context.Context, input *UpdateFirewallRuleInput) (*FirewallRule, error) { + path := fmt.Sprintf("/%s/fwrules/%s", client.accountName, input.ID) + respReader, err := client.executeRequest(ctx, http.MethodPost, path, input) if respReader != nil { defer respReader.Close() } @@ -133,8 +136,9 @@ type EnableFirewallRuleInput struct { ID string `json:"-"` } -func (client *FirewallClient) EnableFirewallRule(input *EnableFirewallRuleInput) (*FirewallRule, error) { - respReader, err := client.executeRequest(http.MethodPost, fmt.Sprintf("/%s/fwrules/%s/enable", client.accountName, input.ID), input) +func (client *FirewallClient) EnableFirewallRule(ctx context.Context, input *EnableFirewallRuleInput) (*FirewallRule, error) { + path := fmt.Sprintf("/%s/fwrules/%s/enable", client.accountName, input.ID) + respReader, err := client.executeRequest(ctx, http.MethodPost, path, input) if respReader != nil { defer respReader.Close() } @@ -155,8 +159,9 @@ type DisableFirewallRuleInput struct { ID string `json:"-"` } -func (client *FirewallClient) DisableFirewallRule(input *DisableFirewallRuleInput) (*FirewallRule, error) { - respReader, err := client.executeRequest(http.MethodPost, fmt.Sprintf("/%s/fwrules/%s/disable", client.accountName, input.ID), input) +func (client *FirewallClient) DisableFirewallRule(ctx context.Context, input *DisableFirewallRuleInput) (*FirewallRule, error) { + path := fmt.Sprintf("/%s/fwrules/%s/disable", client.accountName, input.ID) + respReader, err := client.executeRequest(ctx, http.MethodPost, path, input) if respReader != nil { defer respReader.Close() } @@ -177,9 +182,9 @@ type DeleteFirewallRuleInput struct { ID string } -func (client *FirewallClient) DeleteFirewallRule(input *DeleteFirewallRuleInput) error { +func (client *FirewallClient) DeleteFirewallRule(ctx context.Context, input *DeleteFirewallRuleInput) error { path := fmt.Sprintf("/%s/fwrules/%s", client.accountName, input.ID) - respReader, err := client.executeRequest(http.MethodDelete, path, nil) + respReader, err := client.executeRequest(ctx, http.MethodDelete, path, nil) if respReader != nil { defer respReader.Close() } @@ -194,9 +199,9 @@ type ListMachineFirewallRulesInput struct { MachineID string } -func (client *FirewallClient) ListMachineFirewallRules(input *ListMachineFirewallRulesInput) ([]*FirewallRule, error) { +func (client *FirewallClient) ListMachineFirewallRules(ctx context.Context, input *ListMachineFirewallRulesInput) ([]*FirewallRule, error) { path := fmt.Sprintf("/%s/machines/%s/firewallrules", client.accountName, input.MachineID) - respReader, err := client.executeRequest(http.MethodGet, path, nil) + respReader, err := client.executeRequest(ctx, http.MethodGet, path, nil) if respReader != nil { defer respReader.Close() } diff --git a/vendor/github.com/joyent/triton-go/images.go b/vendor/github.com/joyent/triton-go/images.go index 01d6ea96d..f6ab1fce9 100644 --- a/vendor/github.com/joyent/triton-go/images.go +++ b/vendor/github.com/joyent/triton-go/images.go @@ -1,6 +1,7 @@ package triton import ( + "context" "encoding/json" "fmt" "net/http" @@ -48,9 +49,9 @@ type Image struct { type ListImagesInput struct{} -func (client *ImagesClient) ListImages(*ListImagesInput) ([]*Image, error) { +func (client *ImagesClient) ListImages(ctx context.Context, _ *ListImagesInput) ([]*Image, error) { path := fmt.Sprintf("/%s/images", client.accountName) - respReader, err := client.executeRequest(http.MethodGet, path, nil) + respReader, err := client.executeRequest(ctx, http.MethodGet, path, nil) if respReader != nil { defer respReader.Close() } @@ -71,9 +72,9 @@ type GetImageInput struct { ImageID string } -func (client *ImagesClient) GetImage(input *GetImageInput) (*Image, error) { +func (client *ImagesClient) GetImage(ctx context.Context, input *GetImageInput) (*Image, error) { path := fmt.Sprintf("/%s/images/%s", client.accountName, input.ImageID) - respReader, err := client.executeRequest(http.MethodGet, path, nil) + respReader, err := client.executeRequest(ctx, http.MethodGet, path, nil) if respReader != nil { defer respReader.Close() } @@ -94,9 +95,9 @@ type DeleteImageInput struct { ImageID string } -func (client *ImagesClient) DeleteImage(input *DeleteImageInput) error { +func (client *ImagesClient) DeleteImage(ctx context.Context, input *DeleteImageInput) error { path := fmt.Sprintf("/%s/images/%s", client.accountName, input.ImageID) - respReader, err := client.executeRequest(http.MethodDelete, path, nil) + respReader, err := client.executeRequest(ctx, http.MethodDelete, path, nil) if respReader != nil { defer respReader.Close() } @@ -118,13 +119,13 @@ type MantaLocation struct { ManifestPath string `json:"manifest_path"` } -func (client *ImagesClient) ExportImage(input *ExportImageInput) (*MantaLocation, error) { +func (client *ImagesClient) ExportImage(ctx context.Context, input *ExportImageInput) (*MantaLocation, error) { path := fmt.Sprintf("/%s/images/%s", client.accountName, input.ImageID) query := &url.Values{} query.Set("action", "export") query.Set("manta_path", input.MantaPath) - respReader, err := client.executeRequestURIParams(http.MethodGet, path, nil, query) + respReader, err := client.executeRequestURIParams(ctx, http.MethodGet, path, nil, query) if respReader != nil { defer respReader.Close() } @@ -152,9 +153,9 @@ type CreateImageFromMachineInput struct { Tags map[string]string `json:"tags,omitempty"` } -func (client *ImagesClient) CreateImageFromMachine(input *CreateImageFromMachineInput) (*Image, error) { +func (client *ImagesClient) CreateImageFromMachine(ctx context.Context, input *CreateImageFromMachineInput) (*Image, error) { path := fmt.Sprintf("/%s/images", client.accountName) - respReader, err := client.executeRequest(http.MethodPost, path, input) + respReader, err := client.executeRequest(ctx, http.MethodPost, path, input) if respReader != nil { defer respReader.Close() } @@ -182,12 +183,12 @@ type UpdateImageInput struct { Tags map[string]string `json:"tags,omitempty"` } -func (client *ImagesClient) UpdateImage(input *UpdateImageInput) (*Image, error) { +func (client *ImagesClient) UpdateImage(ctx context.Context, input *UpdateImageInput) (*Image, error) { path := fmt.Sprintf("/%s/images/%s", client.accountName, input.ImageID) query := &url.Values{} query.Set("action", "update") - respReader, err := client.executeRequestURIParams(http.MethodPost, path, input, query) + respReader, err := client.executeRequestURIParams(ctx, http.MethodPost, path, input, query) if respReader != nil { defer respReader.Close() } diff --git a/vendor/github.com/joyent/triton-go/keys.go b/vendor/github.com/joyent/triton-go/keys.go index e63302349..001f020ee 100644 --- a/vendor/github.com/joyent/triton-go/keys.go +++ b/vendor/github.com/joyent/triton-go/keys.go @@ -1,6 +1,7 @@ package triton import ( + "context" "encoding/json" "fmt" "net/http" @@ -34,9 +35,9 @@ type ListKeysInput struct{} // ListKeys lists all public keys we have on record for the specified // account. -func (client *KeysClient) ListKeys(*ListKeysInput) ([]*Key, error) { - path := fmt.Sprintf("/%s/keys") - respReader, err := client.executeRequest(http.MethodGet, path, nil) +func (client *KeysClient) ListKeys(ctx context.Context, _ *ListKeysInput) ([]*Key, error) { + path := fmt.Sprintf("/%s/keys", client.accountName) + respReader, err := client.executeRequest(ctx, http.MethodGet, path, nil) if respReader != nil { defer respReader.Close() } @@ -57,9 +58,9 @@ type GetKeyInput struct { KeyName string } -func (client *KeysClient) GetKey(input *GetKeyInput) (*Key, error) { +func (client *KeysClient) GetKey(ctx context.Context, input *GetKeyInput) (*Key, error) { path := fmt.Sprintf("/%s/keys/%s", client.accountName, input.KeyName) - respReader, err := client.executeRequest(http.MethodGet, path, nil) + respReader, err := client.executeRequest(ctx, http.MethodGet, path, nil) if respReader != nil { defer respReader.Close() } @@ -80,9 +81,9 @@ type DeleteKeyInput struct { KeyName string } -func (client *KeysClient) DeleteKey(input *DeleteKeyInput) error { +func (client *KeysClient) DeleteKey(ctx context.Context, input *DeleteKeyInput) error { path := fmt.Sprintf("/%s/keys/%s", client.accountName, input.KeyName) - respReader, err := client.executeRequest(http.MethodDelete, path, nil) + respReader, err := client.executeRequest(ctx, http.MethodDelete, path, nil) if respReader != nil { defer respReader.Close() } @@ -104,8 +105,9 @@ type CreateKeyInput struct { } // CreateKey uploads a new OpenSSH key to Triton for use in HTTP signing and SSH. -func (client *KeysClient) CreateKey(input *CreateKeyInput) (*Key, error) { - respReader, err := client.executeRequest(http.MethodPost, fmt.Sprintf("/%s/keys", client.accountName), input) +func (client *KeysClient) CreateKey(ctx context.Context, input *CreateKeyInput) (*Key, error) { + path := fmt.Sprintf("/%s/keys", client.accountName) + respReader, err := client.executeRequest(ctx, http.MethodPost, path, input) if respReader != nil { defer respReader.Close() } diff --git a/vendor/github.com/joyent/triton-go/machines.go b/vendor/github.com/joyent/triton-go/machines.go index 287c706c7..aba5a984a 100644 --- a/vendor/github.com/joyent/triton-go/machines.go +++ b/vendor/github.com/joyent/triton-go/machines.go @@ -1,14 +1,14 @@ package triton import ( + "context" "encoding/json" "fmt" "net/http" + "net/url" "strings" "time" - "net/url" - "github.com/hashicorp/errwrap" ) @@ -62,7 +62,7 @@ type Machine struct { } // _Machine is a private facade over Machine that handles the necessary API -// overrides from vmapi's machine endpoint(s). +// overrides from VMAPI's machine endpoint(s). type _Machine struct { Machine Tags map[string]interface{} `json:"tags"` @@ -90,19 +90,20 @@ func (gmi *GetMachineInput) Validate() error { return nil } -func (client *MachinesClient) GetMachine(input *GetMachineInput) (*Machine, error) { +func (client *MachinesClient) GetMachine(ctx context.Context, input *GetMachineInput) (*Machine, error) { if err := input.Validate(); err != nil { return nil, errwrap.Wrapf("unable to get machine: {{err}}", err) } path := fmt.Sprintf("/%s/machines/%s", client.accountName, input.ID) - response, err := client.executeRequestRaw(http.MethodGet, path, nil) + response, err := client.executeRequestRaw(ctx, http.MethodGet, path, nil) if response != nil { defer response.Body.Close() } if response.StatusCode == http.StatusNotFound || response.StatusCode == http.StatusGone { return nil, &TritonError{ - Code: "ResourceNotFound", + StatusCode: response.StatusCode, + Code: "ResourceNotFound", } } if err != nil { @@ -124,26 +125,29 @@ func (client *MachinesClient) GetMachine(input *GetMachineInput) (*Machine, erro return native, nil } -func (client *MachinesClient) GetMachines() ([]*Machine, error) { +type ListMachinesInput struct{} + +func (client *MachinesClient) ListMachines(ctx context.Context, _ *ListMachinesInput) ([]*Machine, error) { path := fmt.Sprintf("/%s/machines", client.accountName) - response, err := client.executeRequestRaw(http.MethodGet, path, nil) + response, err := client.executeRequestRaw(ctx, http.MethodGet, path, nil) if response != nil { defer response.Body.Close() } if response.StatusCode == http.StatusNotFound { return nil, &TritonError{ - Code: "ResourceNotFound", + StatusCode: response.StatusCode, + Code: "ResourceNotFound", } } if err != nil { - return nil, errwrap.Wrapf("Error executing GetMachines request: {{err}}", + return nil, errwrap.Wrapf("Error executing ListMachines request: {{err}}", client.decodeError(response.StatusCode, response.Body)) } var results []*_Machine decoder := json.NewDecoder(response.Body) if err = decoder.Decode(&results); err != nil { - return nil, errwrap.Wrapf("Error decoding GetMachines response: {{err}}", err) + return nil, errwrap.Wrapf("Error decoding ListMachines response: {{err}}", err) } machines := make([]*Machine, 0, len(results)) @@ -218,9 +222,9 @@ func (input *CreateMachineInput) toAPI() map[string]interface{} { return result } -func (client *MachinesClient) CreateMachine(input *CreateMachineInput) (*Machine, error) { +func (client *MachinesClient) CreateMachine(ctx context.Context, input *CreateMachineInput) (*Machine, error) { path := fmt.Sprintf("/%s/machines", client.accountName) - respReader, err := client.executeRequest(http.MethodPost, path, input.toAPI()) + respReader, err := client.executeRequest(ctx, http.MethodPost, path, input.toAPI()) if respReader != nil { defer respReader.Close() } @@ -241,13 +245,13 @@ type DeleteMachineInput struct { ID string } -func (client *MachinesClient) DeleteMachine(input *DeleteMachineInput) error { +func (client *MachinesClient) DeleteMachine(ctx context.Context, input *DeleteMachineInput) error { path := fmt.Sprintf("/%s/machines/%s", client.accountName, input.ID) - response, err := client.executeRequestRaw(http.MethodDelete, path, nil) + response, err := client.executeRequestRaw(ctx, http.MethodDelete, path, nil) if response.Body != nil { defer response.Body.Close() } - if response.StatusCode == http.StatusNotFound { + if response.StatusCode == http.StatusNotFound || response.StatusCode == http.StatusGone { return nil } if err != nil { @@ -262,9 +266,9 @@ type DeleteMachineTagsInput struct { ID string } -func (client *MachinesClient) DeleteMachineTags(input *DeleteMachineTagsInput) error { +func (client *MachinesClient) DeleteMachineTags(ctx context.Context, input *DeleteMachineTagsInput) error { path := fmt.Sprintf("/%s/machines/%s/tags", client.accountName, input.ID) - response, err := client.executeRequestRaw(http.MethodDelete, path, nil) + response, err := client.executeRequestRaw(ctx, http.MethodDelete, path, nil) if response.Body != nil { defer response.Body.Close() } @@ -284,9 +288,9 @@ type DeleteMachineTagInput struct { Key string } -func (client *MachinesClient) DeleteMachineTag(input *DeleteMachineTagInput) error { +func (client *MachinesClient) DeleteMachineTag(ctx context.Context, input *DeleteMachineTagInput) error { path := fmt.Sprintf("/%s/machines/%s/tags/%s", client.accountName, input.ID, input.Key) - response, err := client.executeRequestRaw(http.MethodDelete, path, nil) + response, err := client.executeRequestRaw(ctx, http.MethodDelete, path, nil) if response.Body != nil { defer response.Body.Close() } @@ -306,14 +310,14 @@ type RenameMachineInput struct { Name string } -func (client *MachinesClient) RenameMachine(input *RenameMachineInput) error { +func (client *MachinesClient) RenameMachine(ctx context.Context, input *RenameMachineInput) error { path := fmt.Sprintf("/%s/machines/%s", client.accountName, input.ID) params := &url.Values{} params.Set("action", "rename") params.Set("name", input.Name) - respReader, err := client.executeRequestURIParams(http.MethodPost, path, nil, params) + respReader, err := client.executeRequestURIParams(ctx, http.MethodPost, path, nil, params) if respReader != nil { defer respReader.Close() } @@ -329,9 +333,9 @@ type ReplaceMachineTagsInput struct { Tags map[string]string } -func (client *MachinesClient) ReplaceMachineTags(input *ReplaceMachineTagsInput) error { +func (client *MachinesClient) ReplaceMachineTags(ctx context.Context, input *ReplaceMachineTagsInput) error { path := fmt.Sprintf("/%s/machines/%s/tags", client.accountName, input.ID) - respReader, err := client.executeRequest(http.MethodPut, path, input.Tags) + respReader, err := client.executeRequest(ctx, http.MethodPut, path, input.Tags) if respReader != nil { defer respReader.Close() } @@ -347,9 +351,9 @@ type AddMachineTagsInput struct { Tags map[string]string } -func (client *MachinesClient) AddMachineTags(input *AddMachineTagsInput) error { +func (client *MachinesClient) AddMachineTags(ctx context.Context, input *AddMachineTagsInput) error { path := fmt.Sprintf("/%s/machines/%s/tags", client.accountName, input.ID) - respReader, err := client.executeRequest(http.MethodPost, path, input.Tags) + respReader, err := client.executeRequest(ctx, http.MethodPost, path, input.Tags) if respReader != nil { defer respReader.Close() } @@ -365,9 +369,9 @@ type GetMachineTagInput struct { Key string } -func (client *MachinesClient) GetMachineTag(input *GetMachineTagInput) (string, error) { +func (client *MachinesClient) GetMachineTag(ctx context.Context, input *GetMachineTagInput) (string, error) { path := fmt.Sprintf("/%s/machines/%s/tags/%s", client.accountName, input.ID, input.Key) - respReader, err := client.executeRequest(http.MethodGet, path, nil) + respReader, err := client.executeRequest(ctx, http.MethodGet, path, nil) if respReader != nil { defer respReader.Close() } @@ -388,9 +392,9 @@ type ListMachineTagsInput struct { ID string } -func (client *MachinesClient) ListMachineTags(input *ListMachineTagsInput) (map[string]string, error) { +func (client *MachinesClient) ListMachineTags(ctx context.Context, input *ListMachineTagsInput) (map[string]string, error) { path := fmt.Sprintf("/%s/machines/%s/tags", client.accountName, input.ID) - respReader, err := client.executeRequest(http.MethodGet, path, nil) + respReader, err := client.executeRequest(ctx, http.MethodGet, path, nil) if respReader != nil { defer respReader.Close() } @@ -413,9 +417,9 @@ type UpdateMachineMetadataInput struct { Metadata map[string]string } -func (client *MachinesClient) UpdateMachineMetadata(input *UpdateMachineMetadataInput) (map[string]string, error) { +func (client *MachinesClient) UpdateMachineMetadata(ctx context.Context, input *UpdateMachineMetadataInput) (map[string]string, error) { path := fmt.Sprintf("/%s/machines/%s/tags", client.accountName, input.ID) - respReader, err := client.executeRequest(http.MethodPost, path, input.Metadata) + respReader, err := client.executeRequest(ctx, http.MethodPost, path, input.Metadata) if respReader != nil { defer respReader.Close() } @@ -437,14 +441,14 @@ type ResizeMachineInput struct { Package string } -func (client *MachinesClient) ResizeMachine(input *ResizeMachineInput) error { +func (client *MachinesClient) ResizeMachine(ctx context.Context, input *ResizeMachineInput) error { path := fmt.Sprintf("/%s/machines/%s", client.accountName, input.ID) params := &url.Values{} params.Set("action", "resize") params.Set("package", input.Package) - respReader, err := client.executeRequestURIParams(http.MethodPost, path, nil, params) + respReader, err := client.executeRequestURIParams(ctx, http.MethodPost, path, nil, params) if respReader != nil { defer respReader.Close() } @@ -459,13 +463,13 @@ type EnableMachineFirewallInput struct { ID string } -func (client *MachinesClient) EnableMachineFirewall(input *EnableMachineFirewallInput) error { +func (client *MachinesClient) EnableMachineFirewall(ctx context.Context, input *EnableMachineFirewallInput) error { path := fmt.Sprintf("/%s/machines/%s", client.accountName, input.ID) params := &url.Values{} params.Set("action", "enable_firewall") - respReader, err := client.executeRequestURIParams(http.MethodPost, path, nil, params) + respReader, err := client.executeRequestURIParams(ctx, http.MethodPost, path, nil, params) if respReader != nil { defer respReader.Close() } @@ -480,13 +484,13 @@ type DisableMachineFirewallInput struct { ID string } -func (client *MachinesClient) DisableMachineFirewall(input *DisableMachineFirewallInput) error { +func (client *MachinesClient) DisableMachineFirewall(ctx context.Context, input *DisableMachineFirewallInput) error { path := fmt.Sprintf("/%s/machines/%s", client.accountName, input.ID) params := &url.Values{} params.Set("action", "disable_firewall") - respReader, err := client.executeRequestURIParams(http.MethodPost, path, nil, params) + respReader, err := client.executeRequestURIParams(ctx, http.MethodPost, path, nil, params) if respReader != nil { defer respReader.Close() } @@ -501,9 +505,9 @@ type ListNICsInput struct { MachineID string } -func (client *MachinesClient) ListNICs(input *ListNICsInput) ([]*NIC, error) { +func (client *MachinesClient) ListNICs(ctx context.Context, input *ListNICsInput) ([]*NIC, error) { path := fmt.Sprintf("/%s/machines/%s/nics", client.accountName, input.MachineID) - respReader, err := client.executeRequest(http.MethodGet, path, nil) + respReader, err := client.executeRequest(ctx, http.MethodGet, path, nil) if respReader != nil { defer respReader.Close() } @@ -525,9 +529,9 @@ type AddNICInput struct { Network string `json:"network"` } -func (client *MachinesClient) AddNIC(input *AddNICInput) (*NIC, error) { +func (client *MachinesClient) AddNIC(ctx context.Context, input *AddNICInput) (*NIC, error) { path := fmt.Sprintf("/%s/machines/%s/nics", client.accountName, input.MachineID) - respReader, err := client.executeRequest(http.MethodPost, path, input) + respReader, err := client.executeRequest(ctx, http.MethodPost, path, input) if respReader != nil { defer respReader.Close() } @@ -549,9 +553,9 @@ type RemoveNICInput struct { MAC string } -func (client *MachinesClient) RemoveNIC(input *RemoveNICInput) error { +func (client *MachinesClient) RemoveNIC(ctx context.Context, input *RemoveNICInput) error { path := fmt.Sprintf("/%s/machines/%s/nics/%s", client.accountName, input.MachineID, input.MAC) - respReader, err := client.executeRequest(http.MethodDelete, path, nil) + respReader, err := client.executeRequest(ctx, http.MethodDelete, path, nil) if respReader != nil { defer respReader.Close() } @@ -566,13 +570,13 @@ type StopMachineInput struct { MachineID string } -func (client *MachinesClient) StopMachine(input *StopMachineInput) error { +func (client *MachinesClient) StopMachine(ctx context.Context, input *StopMachineInput) error { path := fmt.Sprintf("/%s/machines/%s", client.accountName, input.MachineID) params := &url.Values{} params.Set("action", "stop") - respReader, err := client.executeRequestURIParams(http.MethodPost, path, nil, params) + respReader, err := client.executeRequestURIParams(ctx, http.MethodPost, path, nil, params) if respReader != nil { defer respReader.Close() } @@ -587,13 +591,13 @@ type StartMachineInput struct { MachineID string } -func (client *MachinesClient) StartMachine(input *StartMachineInput) error { +func (client *MachinesClient) StartMachine(ctx context.Context, input *StartMachineInput) error { path := fmt.Sprintf("/%s/machines/%s", client.accountName, input.MachineID) params := &url.Values{} params.Set("action", "start") - respReader, err := client.executeRequestURIParams(http.MethodPost, path, nil, params) + respReader, err := client.executeRequestURIParams(ctx, http.MethodPost, path, nil, params) if respReader != nil { defer respReader.Close() } diff --git a/vendor/github.com/joyent/triton-go/networks.go b/vendor/github.com/joyent/triton-go/networks.go index 440e3bc87..585a65e7c 100644 --- a/vendor/github.com/joyent/triton-go/networks.go +++ b/vendor/github.com/joyent/triton-go/networks.go @@ -2,9 +2,10 @@ package triton import ( "encoding/json" + "fmt" "net/http" - "fmt" + "context" "github.com/hashicorp/errwrap" ) @@ -35,9 +36,9 @@ type Network struct { type ListNetworksInput struct{} -func (client *NetworksClient) ListNetworks(*ListNetworksInput) ([]*Network, error) { +func (client *NetworksClient) ListNetworks(ctx context.Context, _ *ListNetworksInput) ([]*Network, error) { path := fmt.Sprintf("/%s/networks", client.accountName) - respReader, err := client.executeRequest(http.MethodGet, path, nil) + respReader, err := client.executeRequest(ctx, http.MethodGet, path, nil) if respReader != nil { defer respReader.Close() } @@ -58,9 +59,9 @@ type GetNetworkInput struct { ID string } -func (client *NetworksClient) GetNetwork(input *GetNetworkInput) (*Network, error) { +func (client *NetworksClient) GetNetwork(ctx context.Context, input *GetNetworkInput) (*Network, error) { path := fmt.Sprintf("/%s/networks/%s", client.accountName, input.ID) - respReader, err := client.executeRequest(http.MethodGet, path, nil) + respReader, err := client.executeRequest(ctx, http.MethodGet, path, nil) if respReader != nil { defer respReader.Close() } diff --git a/vendor/github.com/joyent/triton-go/packages.go b/vendor/github.com/joyent/triton-go/packages.go index 2e525acfe..e8a4adbbe 100644 --- a/vendor/github.com/joyent/triton-go/packages.go +++ b/vendor/github.com/joyent/triton-go/packages.go @@ -1,6 +1,7 @@ package triton import ( + "context" "encoding/json" "fmt" "net/http" @@ -43,8 +44,9 @@ type ListPackagesInput struct { Group string `json:"group"` } -func (client *PackagesClient) ListPackages(input *ListPackagesInput) ([]*Package, error) { - respReader, err := client.executeRequest(http.MethodGet, fmt.Sprintf("/%s/packages", client.accountName), input) +func (client *PackagesClient) ListPackages(ctx context.Context, input *ListPackagesInput) ([]*Package, error) { + path := fmt.Sprintf("/%s/packages", client.accountName) + respReader, err := client.executeRequest(ctx, http.MethodGet, path, input) if respReader != nil { defer respReader.Close() } @@ -65,9 +67,9 @@ type GetPackageInput struct { ID string } -func (client *PackagesClient) GetPackage(input *GetPackageInput) (*Package, error) { +func (client *PackagesClient) GetPackage(ctx context.Context, input *GetPackageInput) (*Package, error) { path := fmt.Sprintf("/%s/packages/%s", client.accountName, input.ID) - respReader, err := client.executeRequest(http.MethodGet, path, nil) + respReader, err := client.executeRequest(ctx, http.MethodGet, path, nil) if respReader != nil { defer respReader.Close() } diff --git a/vendor/github.com/joyent/triton-go/roles.go b/vendor/github.com/joyent/triton-go/roles.go index 81bbc44e7..a01fba6a9 100644 --- a/vendor/github.com/joyent/triton-go/roles.go +++ b/vendor/github.com/joyent/triton-go/roles.go @@ -1,10 +1,12 @@ package triton import ( - "fmt" - "github.com/hashicorp/errwrap" - "net/http" + "context" "encoding/json" + "fmt" + "net/http" + + "github.com/hashicorp/errwrap" ) type RolesClient struct { @@ -27,8 +29,9 @@ type Role struct { type ListRolesInput struct{} -func (client *RolesClient) ListRoles(*ListRolesInput) ([]*Role, error) { - respReader, err := client.executeRequest(http.MethodGet, fmt.Sprintf("/%s/roles", client.accountName), nil) +func (client *RolesClient) ListRoles(ctx context.Context, _ *ListRolesInput) ([]*Role, error) { + path := fmt.Sprintf("/%s/roles", client.accountName) + respReader, err := client.executeRequest(ctx, http.MethodGet, path, nil) if respReader != nil { defer respReader.Close() } @@ -45,13 +48,13 @@ func (client *RolesClient) ListRoles(*ListRolesInput) ([]*Role, error) { return result, nil } -type GetRoleInput struct{ +type GetRoleInput struct { RoleID string } -func (client *RolesClient) GetRole(input *GetRoleInput) (*Role, error) { +func (client *RolesClient) GetRole(ctx context.Context, input *GetRoleInput) (*Role, error) { path := fmt.Sprintf("/%s/roles/%s", client.accountName, input.RoleID) - respReader, err := client.executeRequest(http.MethodGet, path, nil) + respReader, err := client.executeRequest(ctx, http.MethodGet, path, nil) if respReader != nil { defer respReader.Close() } @@ -85,8 +88,9 @@ type CreateRoleInput struct { DefaultMembers []string `json:"default_members,omitempty"` } -func (client *RolesClient) CreateRole(input *CreateRoleInput) (*Role, error) { - respReader, err := client.executeRequest(http.MethodPost, fmt.Sprintf("/%s/roles", client.accountName), input) +func (client *RolesClient) CreateRole(ctx context.Context, input *CreateRoleInput) (*Role, error) { + path := fmt.Sprintf("/%s/roles", client.accountName) + respReader, err := client.executeRequest(ctx, http.MethodPost, path, input) if respReader != nil { defer respReader.Close() } @@ -123,8 +127,9 @@ type UpdateRoleInput struct { DefaultMembers []string `json:"default_members,omitempty"` } -func (client *RolesClient) UpdateRole(input *UpdateRoleInput) (*Role, error) { - respReader, err := client.executeRequest(http.MethodPost, fmt.Sprintf("/%s/roles/%s", client.accountName, input.RoleID), input) +func (client *RolesClient) UpdateRole(ctx context.Context, input *UpdateRoleInput) (*Role, error) { + path := fmt.Sprintf("/%s/roles/%s", client.accountName, input.RoleID) + respReader, err := client.executeRequest(ctx, http.MethodPost, path, input) if respReader != nil { defer respReader.Close() } @@ -145,9 +150,9 @@ type DeleteRoleInput struct { RoleID string } -func (client *RolesClient) DeleteRoles(input *DeleteRoleInput) error { +func (client *RolesClient) DeleteRoles(ctx context.Context, input *DeleteRoleInput) error { path := fmt.Sprintf("/%s/roles/%s", client.accountName, input.RoleID) - respReader, err := client.executeRequest(http.MethodDelete, path, nil) + respReader, err := client.executeRequest(ctx, http.MethodDelete, path, nil) if respReader != nil { defer respReader.Close() } diff --git a/vendor/github.com/joyent/triton-go/services.go b/vendor/github.com/joyent/triton-go/services.go index 8b2d53191..e220b2699 100644 --- a/vendor/github.com/joyent/triton-go/services.go +++ b/vendor/github.com/joyent/triton-go/services.go @@ -1,6 +1,7 @@ package triton import ( + "context" "encoding/json" "fmt" "net/http" @@ -26,8 +27,9 @@ type Service struct { type ListServicesInput struct{} -func (client *ServicesClient) ListServices(*ListServicesInput) ([]*Service, error) { - respReader, err := client.executeRequest(http.MethodGet, fmt.Sprintf("/%s/services", client.accountName), nil) +func (client *ServicesClient) ListServices(ctx context.Context, _ *ListServicesInput) ([]*Service, error) { + path := fmt.Sprintf("/%s/services", client.accountName) + respReader, err := client.executeRequest(ctx, http.MethodGet, path, nil) if respReader != nil { defer respReader.Close() } diff --git a/vendor/vendor.json b/vendor/vendor.json index d851a4387..268e12f5b 100644 --- a/vendor/vendor.json +++ b/vendor/vendor.json @@ -2349,10 +2349,10 @@ "revisionTime": "2016-06-16T18:50:15Z" }, { - "checksumSHA1": "XsjyaC6eTHUy/n0iuR46TZcgAK8=", + "checksumSHA1": "o8jaSD36Zq42PMnmUaiB+vq+QNA=", "path": "github.com/joyent/triton-go", - "revision": "c73729fd38522591909a371c8180ca7090a59ab9", - "revisionTime": "2017-04-28T18:47:44Z" + "revision": "97ccd9f6c0c0652cf87997bcb01955e0329cd37e", + "revisionTime": "2017-05-09T20:29:43Z" }, { "checksumSHA1": "QzUqkCSn/ZHyIK346xb9V6EBw9U=",