names: add StorageTag #37

Merged
merged 2 commits into from Jan 20, 2015

Conversation

Projects
None yet
2 participants
Member

axw commented Jan 20, 2015

A new kind of tag is added for referring to storage instances;
these are instances of charm storage. The format is "storage/N",
where "storage" is the name defined in charm storage metadata
and "N" is a unique sequence number.

names: add StorageTag
A new kind of tag is added for referring to storage instances;
these are instances of charm storage. The format is "storage/N",
where "storage" is the name defined in charm storage metadata
and "N" is a unique sequence number.
storage_test.go
+}
+
+func (s *storageSuite) TestStorageNameValidity(c *gc.C) {
+ assertStorageNameValid(c, "store/0")
@dimitern

dimitern Jan 20, 2015

Contributor

I'd add a couple of valid cases (with actual names you plan to use) to illustrate "store" is not required, as it got me confused initially.

@axw

axw Jan 20, 2015

Member

Done.

Contributor

dimitern commented Jan 20, 2015

LGTM

axw added a commit that referenced this pull request Jan 20, 2015

@axw axw merged commit f2761fa into juju:master Jan 20, 2015

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