Skip to content

Conversation

@r0h4n
Copy link
Contributor

@r0h4n r0h4n commented Dec 2, 2016

tendrl_definitions_add_object_listing_key

r0h4n added 2 commits December 2, 2016 12:56
tendrl_definitions_add_object_listing_key
tendrl_definitions_add_object_listing_key
@r0h4n
Copy link
Contributor Author

r0h4n commented Dec 2, 2016

@Tendrl/tendrl-core , @shtripat @anivargi @nnDarshan @anmolbabu @GowthamShanmugam please review

@mkudlej
Copy link

mkudlej commented Dec 2, 2016

Is this specification connected with future specification for tendrl-api for listing objects in etcd?

* Adds "list" key to all Objects in Tendrl namespaces

* The "list" key will point to a relative path in the central store which will contain list of all instances of that object
eg: 'clusters/xyz/objects/Volumes/' will contain list of all Volumes in that cluster
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Shouldnt it be like /clusters//Volumes rather?

Copy link
Contributor

@anivargi anivargi Dec 8, 2016

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ideally it should be /clusters/:cluster_id/volumes, but whatever you put here will be used for listing from the API

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

"objects" is required for #37

@r0h4n
Copy link
Contributor Author

r0h4n commented Dec 14, 2016

@anivargi any queries on this? I am going to merge this today

@anivargi
Copy link
Contributor

@r0h4n No.

@r0h4n r0h4n merged commit 5523c01 into master Dec 14, 2016
@r0h4n r0h4n deleted the tendrl_definitions_add_object_listing_key branch December 14, 2016 09:05
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.

5 participants