backend/remote-state/gcloud: Rename Url -> URL

This commit is contained in:
Florian Forster 2017-09-07 12:56:17 +02:00 committed by James Bardin
parent 5313e15e07
commit 52ac764036
2 changed files with 17 additions and 17 deletions

View File

@ -64,7 +64,7 @@ func (b *Backend) DeleteState(name string) error {
err = client.Delete() err = client.Delete()
if err != nil { if err != nil {
return fmt.Errorf("Failed to delete state file %v: %v", client.stateFileUrl(), err) return fmt.Errorf("Failed to delete state file %v: %v", client.stateFileURL(), err)
} }
return nil return nil
@ -104,7 +104,7 @@ func (b *Backend) State(name string) (state.State, error) {
// Local helper function so we can call it multiple places // Local helper function so we can call it multiple places
lockUnlock := func(parent error) error { lockUnlock := func(parent error) error {
if err := stateMgr.Unlock(lockId); err != nil { if err := stateMgr.Unlock(lockId); err != nil {
return fmt.Errorf(strings.TrimSpace(errStateUnlock), lockId, client.lockFileUrl(), err) return fmt.Errorf(strings.TrimSpace(errStateUnlock), lockId, client.lockFileURL(), err)
} }
return parent return parent

View File

@ -22,26 +22,26 @@ type RemoteClient struct {
func (c *RemoteClient) Get() (payload *remote.Payload, err error) { func (c *RemoteClient) Get() (payload *remote.Payload, err error) {
bucket := c.storageClient.Bucket(c.bucketName) bucket := c.storageClient.Bucket(c.bucketName)
stateFile := bucket.Object(c.stateFilePath) stateFile := bucket.Object(c.stateFilePath)
stateFileUrl := c.stateFileUrl() stateFileURL := c.stateFileURL()
stateFileReader, err := stateFile.NewReader(c.storageContext) stateFileReader, err := stateFile.NewReader(c.storageContext)
if err != nil { if err != nil {
if err == storage.ErrObjectNotExist { if err == storage.ErrObjectNotExist {
return nil, nil return nil, nil
} else { } else {
return nil, fmt.Errorf("Failed to open state file at %v: %v", stateFileUrl, err) return nil, fmt.Errorf("Failed to open state file at %v: %v", stateFileURL, err)
} }
} }
defer stateFileReader.Close() defer stateFileReader.Close()
stateFileContents, err := ioutil.ReadAll(stateFileReader) stateFileContents, err := ioutil.ReadAll(stateFileReader)
if err != nil { if err != nil {
return nil, fmt.Errorf("Failed to read state file from %v: %v", stateFileUrl, err) return nil, fmt.Errorf("Failed to read state file from %v: %v", stateFileURL, err)
} }
stateFileAttrs, err := stateFile.Attrs(c.storageContext) stateFileAttrs, err := stateFile.Attrs(c.storageContext)
if err != nil { if err != nil {
return nil, fmt.Errorf("Failed to read state file attrs from %v: %v", stateFileUrl, err) return nil, fmt.Errorf("Failed to read state file attrs from %v: %v", stateFileURL, err)
} }
result := &remote.Payload{ result := &remote.Payload{
@ -62,7 +62,7 @@ func (c *RemoteClient) Put(data []byte) error {
err := stateFileWriter.Close() err := stateFileWriter.Close()
if err != nil { if err != nil {
return fmt.Errorf("Failed to upload state to %v: %v", c.stateFileUrl(), err) return fmt.Errorf("Failed to upload state to %v: %v", c.stateFileURL(), err)
} }
return nil return nil
@ -75,7 +75,7 @@ func (c *RemoteClient) Delete() error {
err := stateFile.Delete(c.storageContext) err := stateFile.Delete(c.storageContext)
if err != nil { if err != nil {
return fmt.Errorf("Failed to delete state file %v: %v", c.stateFileUrl(), err) return fmt.Errorf("Failed to delete state file %v: %v", c.stateFileURL(), err)
} }
return nil return nil
@ -91,7 +91,7 @@ func (c *RemoteClient) Lock(info *state.LockInfo) (string, error) {
info.ID = lockID info.ID = lockID
} }
info.Path = c.lockFileUrl() info.Path = c.lockFileURL()
infoJson, err := json.Marshal(info) infoJson, err := json.Marshal(info)
if err != nil { if err != nil {
@ -115,25 +115,25 @@ func (c *RemoteClient) Unlock(id string) error {
bucket := c.storageClient.Bucket(c.bucketName) bucket := c.storageClient.Bucket(c.bucketName)
lockFile := bucket.Object(c.lockFilePath) lockFile := bucket.Object(c.lockFilePath)
lockFileUrl := c.lockFileUrl() lockFileURL := c.lockFileURL()
lockFileReader, err := lockFile.NewReader(c.storageContext) lockFileReader, err := lockFile.NewReader(c.storageContext)
if err != nil { if err != nil {
lockErr.Err = fmt.Errorf("Failed to retrieve lock info (%v): %v", lockFileUrl, err) lockErr.Err = fmt.Errorf("Failed to retrieve lock info (%v): %v", lockFileURL, err)
return lockErr return lockErr
} }
defer lockFileReader.Close() defer lockFileReader.Close()
lockFileContents, err := ioutil.ReadAll(lockFileReader) lockFileContents, err := ioutil.ReadAll(lockFileReader)
if err != nil { if err != nil {
lockErr.Err = fmt.Errorf("Failed to retrieve lock info (%v): %v", lockFileUrl, err) lockErr.Err = fmt.Errorf("Failed to retrieve lock info (%v): %v", lockFileURL, err)
return lockErr return lockErr
} }
lockInfo := &state.LockInfo{} lockInfo := &state.LockInfo{}
err = json.Unmarshal(lockFileContents, lockInfo) err = json.Unmarshal(lockFileContents, lockInfo)
if err != nil { if err != nil {
lockErr.Err = fmt.Errorf("Failed to unmarshal lock info (%v): %v", lockFileUrl, err) lockErr.Err = fmt.Errorf("Failed to unmarshal lock info (%v): %v", lockFileURL, err)
return lockErr return lockErr
} }
@ -146,23 +146,23 @@ func (c *RemoteClient) Unlock(id string) error {
lockFileAttrs, err := lockFile.Attrs(c.storageContext) lockFileAttrs, err := lockFile.Attrs(c.storageContext)
if err != nil { if err != nil {
lockErr.Err = fmt.Errorf("Failed to fetch lock file attrs (%v): %v", lockFileUrl, err) lockErr.Err = fmt.Errorf("Failed to fetch lock file attrs (%v): %v", lockFileURL, err)
return lockErr return lockErr
} }
err = lockFile.If(storage.Conditions{GenerationMatch: lockFileAttrs.Generation}).Delete(c.storageContext) err = lockFile.If(storage.Conditions{GenerationMatch: lockFileAttrs.Generation}).Delete(c.storageContext)
if err != nil { if err != nil {
lockErr.Err = fmt.Errorf("Failed to delete lock file (%v): %v", lockFileUrl, err) lockErr.Err = fmt.Errorf("Failed to delete lock file (%v): %v", lockFileURL, err)
return lockErr return lockErr
} }
return nil return nil
} }
func (c *RemoteClient) stateFileUrl() string { func (c *RemoteClient) stateFileURL() string {
return fmt.Sprintf("gs://%v/%v", c.bucketName, c.stateFilePath) return fmt.Sprintf("gs://%v/%v", c.bucketName, c.stateFilePath)
} }
func (c *RemoteClient) lockFileUrl() string { func (c *RemoteClient) lockFileURL() string {
return fmt.Sprintf("gs://%v/%v", c.bucketName, c.lockFilePath) return fmt.Sprintf("gs://%v/%v", c.bucketName, c.lockFilePath)
} }