Skip to content

Address a few issues#25

Merged
displague merged 5 commits intolinode:masterfrom
displague:feature/no-require-region
Apr 14, 2019
Merged

Address a few issues#25
displague merged 5 commits intolinode:masterfrom
displague:feature/no-require-region

Conversation

@displague
Copy link
Contributor

Addresses a few issues

@displague displague merged commit 2f1cfcf into linode:master Apr 14, 2019
@displague
Copy link
Contributor Author

displague commented Apr 16, 2019

By fetching the region from the instance, instead of taking it as a user parameter, we also avoid the potential issue of a user supplied region being an alias region, which the volume list filter-by-region poll would never resolve:

if jsonFilter, err = json.Marshal(map[string]string{"label": volumeLabel, "region": driver.region}); err != nil {

I encountered this a few times when using us-south which is an alias for us-central (Dallas, TX).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Must provide a region or a Linode ID Remove docker volume without removing the physical volume Add ability to specify filesystem type

1 participant