)]}'
{"specs/kilo/ceilometer_ceph_integration.rst":[{"author":{"_account_id":7052,"name":"Fabio Giannetti","email":"fgiannet@cisco.com","username":"fabgia"},"change_message_id":"99d859919e06990dabbae3208947f536291443e1","unresolved":false,"context_lines":[{"line_number":12,"context_line":""},{"line_number":13,"context_line":"This spec proposes to add ceph object storage metering to Ceilometer by polling"},{"line_number":14,"context_line":"mechanism, using the ceph radosgw (rados gateway) APIs, when the ceph is used"},{"line_number":15,"context_line":"as object storage back-end, instead of swift. "},{"line_number":16,"context_line":""},{"line_number":17,"context_line":""},{"line_number":18,"context_line":"Problem description"}],"source_content_type":"text/x-rst","patch_set":4,"id":"5a890539_9de334d7","line":15,"updated":"2014-12-03 18:37:14.000000000","message":"NIT: Please get rid of these extra spaces.","commit_id":"fff9e7c0123c60e2f707a44e0cafd420eeddc63b"},{"author":{"_account_id":7052,"name":"Fabio Giannetti","email":"fgiannet@cisco.com","username":"fabgia"},"change_message_id":"99d859919e06990dabbae3208947f536291443e1","unresolved":false,"context_lines":[{"line_number":24,"context_line":"not there with ceilometer. Ceph object storage provide REST APIs to get the"},{"line_number":25,"context_line":"meters like object/container list, object/container size, etc. "},{"line_number":26,"context_line":""},{"line_number":27,"context_line":"A polling machanisum is required, in Ceilometer, to get the meter details from"},{"line_number":28,"context_line":"the ceph object storage. This polling method could be similar to swift polling"},{"line_number":29,"context_line":"machanisum."},{"line_number":30,"context_line":""}],"source_content_type":"text/x-rst","patch_set":4,"id":"5a890539_3deb20a5","line":27,"updated":"2014-12-03 18:37:14.000000000","message":"A polling mechanism","commit_id":"fff9e7c0123c60e2f707a44e0cafd420eeddc63b"},{"author":{"_account_id":7052,"name":"Fabio Giannetti","email":"fgiannet@cisco.com","username":"fabgia"},"change_message_id":"99d859919e06990dabbae3208947f536291443e1","unresolved":false,"context_lines":[{"line_number":26,"context_line":""},{"line_number":27,"context_line":"A polling machanisum is required, in Ceilometer, to get the meter details from"},{"line_number":28,"context_line":"the ceph object storage. This polling method could be similar to swift polling"},{"line_number":29,"context_line":"machanisum."},{"line_number":30,"context_line":""},{"line_number":31,"context_line":"Proposed change"},{"line_number":32,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"}],"source_content_type":"text/x-rst","patch_set":4,"id":"5a890539_dd3e9c28","line":29,"updated":"2014-12-03 18:37:14.000000000","message":"mechanism","commit_id":"fff9e7c0123c60e2f707a44e0cafd420eeddc63b"},{"author":{"_account_id":7052,"name":"Fabio Giannetti","email":"fgiannet@cisco.com","username":"fabgia"},"change_message_id":"99d859919e06990dabbae3208947f536291443e1","unresolved":false,"context_lines":[{"line_number":31,"context_line":"Proposed change"},{"line_number":32,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":33,"context_line":""},{"line_number":34,"context_line":"Add a new poller classes for ceph object storage to get object storage meter"},{"line_number":35,"context_line":"information into samples, using existing ``storage.objects`` implementations"},{"line_number":36,"context_line":"as a model with swift object storage. These poller include objects and "},{"line_number":37,"context_line":"containers of the ceph object storage. "}],"source_content_type":"text/x-rst","patch_set":4,"id":"5a890539_1d4584b3","line":34,"updated":"2014-12-03 18:37:14.000000000","message":"pollster","commit_id":"fff9e7c0123c60e2f707a44e0cafd420eeddc63b"},{"author":{"_account_id":7052,"name":"Fabio Giannetti","email":"fgiannet@cisco.com","username":"fabgia"},"change_message_id":"99d859919e06990dabbae3208947f536291443e1","unresolved":false,"context_lines":[{"line_number":45,"context_line":"* ceph.storage.containers.objects.size"},{"line_number":46,"context_line":"* ceph.storage.api.request"},{"line_number":47,"context_line":""},{"line_number":48,"context_line":"For this implementation we will call the ceph object storage (radosgw) APIs."},{"line_number":49,"context_line":""},{"line_number":50,"context_line":"Alternatives"},{"line_number":51,"context_line":"------------"}],"source_content_type":"text/x-rst","patch_set":4,"id":"5a890539_1d1ca4c1","line":48,"updated":"2014-12-03 18:37:14.000000000","message":"Usually the pollsters in Ceilometer interface directly with the python client. Is that going to be case as well?\nDoes Ceph has a python client ?","commit_id":"fff9e7c0123c60e2f707a44e0cafd420eeddc63b"},{"author":{"_account_id":11224,"name":"Swami Reddy","email":"swamireddy@gmail.com","username":"swamireddy"},"change_message_id":"a82f34b9b782b7a3f03dc8e409cef4e9799ecdc8","unresolved":false,"context_lines":[{"line_number":45,"context_line":"* ceph.storage.containers.objects.size"},{"line_number":46,"context_line":"* ceph.storage.api.request"},{"line_number":47,"context_line":""},{"line_number":48,"context_line":"For this implementation we will call the ceph object storage (radosgw) APIs."},{"line_number":49,"context_line":""},{"line_number":50,"context_line":"Alternatives"},{"line_number":51,"context_line":"------------"}],"source_content_type":"text/x-rst","patch_set":4,"id":"5a890539_22691825","line":48,"in_reply_to":"5a890539_1d1ca4c1","updated":"2014-12-04 11:46:02.000000000","message":"Ceph radosgw doesn\u0027t have the python client, but we can use the swiftclient to send the REST APIs to radosgw.","commit_id":"fff9e7c0123c60e2f707a44e0cafd420eeddc63b"},{"author":{"_account_id":7052,"name":"Fabio Giannetti","email":"fgiannet@cisco.com","username":"fabgia"},"change_message_id":"99d859919e06990dabbae3208947f536291443e1","unresolved":false,"context_lines":[{"line_number":50,"context_line":"Alternatives"},{"line_number":51,"context_line":"------------"},{"line_number":52,"context_line":""},{"line_number":53,"context_line":"None."},{"line_number":54,"context_line":""},{"line_number":55,"context_line":"Data model impact"},{"line_number":56,"context_line":"-----------------"}],"source_content_type":"text/x-rst","patch_set":4,"id":"5a890539_9d56d45c","line":53,"updated":"2014-12-03 18:37:14.000000000","message":"Alternatively (or complementary) it is possible to have Ceph sending notifications. Clearly this is a Ceph change but I think it is worth mentioning it.","commit_id":"fff9e7c0123c60e2f707a44e0cafd420eeddc63b"},{"author":{"_account_id":11224,"name":"Swami Reddy","email":"swamireddy@gmail.com","username":"swamireddy"},"change_message_id":"a82f34b9b782b7a3f03dc8e409cef4e9799ecdc8","unresolved":false,"context_lines":[{"line_number":50,"context_line":"Alternatives"},{"line_number":51,"context_line":"------------"},{"line_number":52,"context_line":""},{"line_number":53,"context_line":"None."},{"line_number":54,"context_line":""},{"line_number":55,"context_line":"Data model impact"},{"line_number":56,"context_line":"-----------------"}],"source_content_type":"text/x-rst","patch_set":4,"id":"5a890539_8261ec3a","line":53,"in_reply_to":"5a890539_9d56d45c","updated":"2014-12-04 11:46:02.000000000","message":"Currently ceph doesn\u0027t support notifications, so adding the push notification services in ceph may be complex ATM. For the above listed 6 meters, the pollster method works, because, ceph radosgw support the REST APIs. For ex: the ceph radosgw support Swift object storage APIs.","commit_id":"fff9e7c0123c60e2f707a44e0cafd420eeddc63b"},{"author":{"_account_id":2284,"name":"Eoghan Glynn","email":"eglynn@redhat.com","username":"eglynn"},"change_message_id":"ae2195d1af2a11bd95c0b691e30944b9852626c0","unresolved":false,"context_lines":[{"line_number":33,"context_line":""},{"line_number":34,"context_line":"In Ceilometer - polling mechanism is required, to get the meter details from"},{"line_number":35,"context_line":"the ceph object storage. This polling method could be similar to swift polling"},{"line_number":36,"context_line":"mechanism."},{"line_number":37,"context_line":""},{"line_number":38,"context_line":""},{"line_number":39,"context_line":"Add new pollster classes for ceph object storage to get object storage meter"}],"source_content_type":"text/x-rst","patch_set":5,"id":"3a961159_450dc93d","line":36,"updated":"2014-12-10 11:45:21.000000000","message":"The swift polling mechanism includes support for per-tenant splitting among scaled out central agents.\n\nWould a similar concept be required for ceph polling?","commit_id":"cc6017299f79bf36cee0a54f965bb6666f5fa4cb"},{"author":{"_account_id":11224,"name":"Swami Reddy","email":"swamireddy@gmail.com","username":"swamireddy"},"change_message_id":"9fbb331ef5fb3a6baa9af287dec63ff9a63bb74c","unresolved":false,"context_lines":[{"line_number":33,"context_line":""},{"line_number":34,"context_line":"In Ceilometer - polling mechanism is required, to get the meter details from"},{"line_number":35,"context_line":"the ceph object storage. This polling method could be similar to swift polling"},{"line_number":36,"context_line":"mechanism."},{"line_number":37,"context_line":""},{"line_number":38,"context_line":""},{"line_number":39,"context_line":"Add new pollster classes for ceph object storage to get object storage meter"}],"source_content_type":"text/x-rst","patch_set":5,"id":"3a961159_f42f41f4","line":36,"in_reply_to":"3a961159_1e46ae7c","updated":"2014-12-10 14:03:20.000000000","message":"Currently, the swiftclient use the TenantDiscovery internally in its pollster classes.","commit_id":"cc6017299f79bf36cee0a54f965bb6666f5fa4cb"},{"author":{"_account_id":11224,"name":"Swami Reddy","email":"swamireddy@gmail.com","username":"swamireddy"},"change_message_id":"7e06405ec3ee0778a120e406284a5848de0e8dd8","unresolved":false,"context_lines":[{"line_number":33,"context_line":""},{"line_number":34,"context_line":"In Ceilometer - polling mechanism is required, to get the meter details from"},{"line_number":35,"context_line":"the ceph object storage. This polling method could be similar to swift polling"},{"line_number":36,"context_line":"mechanism."},{"line_number":37,"context_line":""},{"line_number":38,"context_line":""},{"line_number":39,"context_line":"Add new pollster classes for ceph object storage to get object storage meter"}],"source_content_type":"text/x-rst","patch_set":5,"id":"3a961159_88e6e8b2","line":36,"in_reply_to":"3a961159_450dc93d","updated":"2014-12-10 12:29:37.000000000","message":"Yes, ceph also required similar concept for polling.","commit_id":"cc6017299f79bf36cee0a54f965bb6666f5fa4cb"},{"author":{"_account_id":2284,"name":"Eoghan Glynn","email":"eglynn@redhat.com","username":"eglynn"},"change_message_id":"7122b25de6c16d839f740c0d82ec8ed9c600f5b6","unresolved":false,"context_lines":[{"line_number":33,"context_line":""},{"line_number":34,"context_line":"In Ceilometer - polling mechanism is required, to get the meter details from"},{"line_number":35,"context_line":"the ceph object storage. This polling method could be similar to swift polling"},{"line_number":36,"context_line":"mechanism."},{"line_number":37,"context_line":""},{"line_number":38,"context_line":""},{"line_number":39,"context_line":"Add new pollster classes for ceph object storage to get object storage meter"}],"source_content_type":"text/x-rst","patch_set":5,"id":"3a961159_1e46ae7c","line":36,"in_reply_to":"3a961159_88e6e8b2","updated":"2014-12-10 13:14:02.000000000","message":"A similar concept of containers/buckets being associated with individual tenants?\n\nIn that case, it would make sense to explicitly call that you\u0027ll be re-using the TenantDiscovery[1] in the ceph pollsters.\n\n[1] https://github.com/openstack/ceilometer/blob/master/ceilometer/central/discovery.py#L52","commit_id":"cc6017299f79bf36cee0a54f965bb6666f5fa4cb"},{"author":{"_account_id":11224,"name":"Swami Reddy","email":"swamireddy@gmail.com","username":"swamireddy"},"change_message_id":"67ec2edb8519bd344eb08511d6088ce258407422","unresolved":false,"context_lines":[{"line_number":33,"context_line":""},{"line_number":34,"context_line":"In Ceilometer - polling mechanism is required, to get the meter details from"},{"line_number":35,"context_line":"the ceph object storage. This polling method could be similar to swift polling"},{"line_number":36,"context_line":"mechanism."},{"line_number":37,"context_line":""},{"line_number":38,"context_line":""},{"line_number":39,"context_line":"Add new pollster classes for ceph object storage to get object storage meter"}],"source_content_type":"text/x-rst","patch_set":5,"id":"3a961159_48e135fd","line":36,"in_reply_to":"3a961159_d1b95416","updated":"2014-12-11 12:03:27.000000000","message":"Yep. That\u0027s correct. will be doing the same.","commit_id":"cc6017299f79bf36cee0a54f965bb6666f5fa4cb"},{"author":{"_account_id":2284,"name":"Eoghan Glynn","email":"eglynn@redhat.com","username":"eglynn"},"change_message_id":"d349a29648128f276861b5c3df94299f944cc3b6","unresolved":false,"context_lines":[{"line_number":33,"context_line":""},{"line_number":34,"context_line":"In Ceilometer - polling mechanism is required, to get the meter details from"},{"line_number":35,"context_line":"the ceph object storage. This polling method could be similar to swift polling"},{"line_number":36,"context_line":"mechanism."},{"line_number":37,"context_line":""},{"line_number":38,"context_line":""},{"line_number":39,"context_line":"Add new pollster classes for ceph object storage to get object storage meter"}],"source_content_type":"text/x-rst","patch_set":5,"id":"3a961159_d1b95416","line":36,"in_reply_to":"3a961159_f42f41f4","updated":"2014-12-11 09:22:48.000000000","message":"Yep, TenantDiscovery is generic and may be re-used by any pollsters (including the new ceph pollsters that you\u0027ll writing) where it makes sense to partition the resources along tenant lines (i.e. for allocation across multiple central agents).","commit_id":"cc6017299f79bf36cee0a54f965bb6666f5fa4cb"},{"author":{"_account_id":2284,"name":"Eoghan Glynn","email":"eglynn@redhat.com","username":"eglynn"},"change_message_id":"ae2195d1af2a11bd95c0b691e30944b9852626c0","unresolved":false,"context_lines":[{"line_number":123,"context_line":"  metering information."},{"line_number":124,"context_line":""},{"line_number":125,"context_line":"* Implement the ceph object storage (radosgw) APIs calls for appropriately"},{"line_number":126,"context_line":"  for each metering item."},{"line_number":127,"context_line":""},{"line_number":128,"context_line":"* Integrate the keystone authentication method for users validation."},{"line_number":129,"context_line":""}],"source_content_type":"text/x-rst","patch_set":5,"id":"3a961159_c532990f","line":126,"updated":"2014-12-10 11:45:21.000000000","message":"What python client library are you intending to use for calling the radosgw?\n\nIs this already in the global requirements?\n\nApparently the rados module is not currently on pypi, amiright?","commit_id":"cc6017299f79bf36cee0a54f965bb6666f5fa4cb"},{"author":{"_account_id":11224,"name":"Swami Reddy","email":"swamireddy@gmail.com","username":"swamireddy"},"change_message_id":"9fbb331ef5fb3a6baa9af287dec63ff9a63bb74c","unresolved":false,"context_lines":[{"line_number":123,"context_line":"  metering information."},{"line_number":124,"context_line":""},{"line_number":125,"context_line":"* Implement the ceph object storage (radosgw) APIs calls for appropriately"},{"line_number":126,"context_line":"  for each metering item."},{"line_number":127,"context_line":""},{"line_number":128,"context_line":"* Integrate the keystone authentication method for users validation."},{"line_number":129,"context_line":""}],"source_content_type":"text/x-rst","patch_set":5,"id":"3a961159_996d589b","line":126,"in_reply_to":"3a961159_5e5f5641","updated":"2014-12-10 14:03:20.000000000","message":"Sure, update the spec appropriately.","commit_id":"cc6017299f79bf36cee0a54f965bb6666f5fa4cb"},{"author":{"_account_id":11224,"name":"Swami Reddy","email":"swamireddy@gmail.com","username":"swamireddy"},"change_message_id":"7e06405ec3ee0778a120e406284a5848de0e8dd8","unresolved":false,"context_lines":[{"line_number":123,"context_line":"  metering information."},{"line_number":124,"context_line":""},{"line_number":125,"context_line":"* Implement the ceph object storage (radosgw) APIs calls for appropriately"},{"line_number":126,"context_line":"  for each metering item."},{"line_number":127,"context_line":""},{"line_number":128,"context_line":"* Integrate the keystone authentication method for users validation."},{"line_number":129,"context_line":""}],"source_content_type":"text/x-rst","patch_set":5,"id":"3a961159_e3d14d58","line":126,"in_reply_to":"3a961159_c532990f","updated":"2014-12-10 12:29:37.000000000","message":"Yes, the rados module is not pypi and here, we won\u0027t be using the rados.\nWe use only the radosgw REST APIs only with swiftclient, because radosgw support Swift.","commit_id":"cc6017299f79bf36cee0a54f965bb6666f5fa4cb"},{"author":{"_account_id":2284,"name":"Eoghan Glynn","email":"eglynn@redhat.com","username":"eglynn"},"change_message_id":"7122b25de6c16d839f740c0d82ec8ed9c600f5b6","unresolved":false,"context_lines":[{"line_number":123,"context_line":"  metering information."},{"line_number":124,"context_line":""},{"line_number":125,"context_line":"* Implement the ceph object storage (radosgw) APIs calls for appropriately"},{"line_number":126,"context_line":"  for each metering item."},{"line_number":127,"context_line":""},{"line_number":128,"context_line":"* Integrate the keystone authentication method for users validation."},{"line_number":129,"context_line":""}],"source_content_type":"text/x-rst","patch_set":5,"id":"3a961159_5e5f5641","line":126,"in_reply_to":"3a961159_e3d14d58","updated":"2014-12-10 13:14:02.000000000","message":"A-ha, OK, can you state that explicitly in the spec that only the swiftclient is required here?","commit_id":"cc6017299f79bf36cee0a54f965bb6666f5fa4cb"},{"author":{"_account_id":2284,"name":"Eoghan Glynn","email":"eglynn@redhat.com","username":"eglynn"},"change_message_id":"ae2195d1af2a11bd95c0b691e30944b9852626c0","unresolved":false,"context_lines":[{"line_number":125,"context_line":"* Implement the ceph object storage (radosgw) APIs calls for appropriately"},{"line_number":126,"context_line":"  for each metering item."},{"line_number":127,"context_line":""},{"line_number":128,"context_line":"* Integrate the keystone authentication method for users validation."},{"line_number":129,"context_line":""},{"line_number":130,"context_line":"* Update/modify the ceph radosgw REST APIs appropriately, based on ceilometer"},{"line_number":131,"context_line":"  requirements."}],"source_content_type":"text/x-rst","patch_set":5,"id":"3a961159_a5f82d4a","line":128,"updated":"2014-12-10 11:45:21.000000000","message":"Where would this keystone integration be required?\n\nCan ceph itself already be deployed with the keystone auth middelware?","commit_id":"cc6017299f79bf36cee0a54f965bb6666f5fa4cb"},{"author":{"_account_id":11224,"name":"Swami Reddy","email":"swamireddy@gmail.com","username":"swamireddy"},"change_message_id":"9fbb331ef5fb3a6baa9af287dec63ff9a63bb74c","unresolved":false,"context_lines":[{"line_number":125,"context_line":"* Implement the ceph object storage (radosgw) APIs calls for appropriately"},{"line_number":126,"context_line":"  for each metering item."},{"line_number":127,"context_line":""},{"line_number":128,"context_line":"* Integrate the keystone authentication method for users validation."},{"line_number":129,"context_line":""},{"line_number":130,"context_line":"* Update/modify the ceph radosgw REST APIs appropriately, based on ceilometer"},{"line_number":131,"context_line":"  requirements."}],"source_content_type":"text/x-rst","patch_set":5,"id":"3a961159_79ca6c6f","line":128,"in_reply_to":"3a961159_3e40ca55","updated":"2014-12-10 14:03:20.000000000","message":"Ceph supports the keystone auth and its tested with Swift, but not tested with ceilometer, so I have added a work list for keystone integration here.","commit_id":"cc6017299f79bf36cee0a54f965bb6666f5fa4cb"},{"author":{"_account_id":11224,"name":"Swami Reddy","email":"swamireddy@gmail.com","username":"swamireddy"},"change_message_id":"67ec2edb8519bd344eb08511d6088ce258407422","unresolved":false,"context_lines":[{"line_number":125,"context_line":"* Implement the ceph object storage (radosgw) APIs calls for appropriately"},{"line_number":126,"context_line":"  for each metering item."},{"line_number":127,"context_line":""},{"line_number":128,"context_line":"* Integrate the keystone authentication method for users validation."},{"line_number":129,"context_line":""},{"line_number":130,"context_line":"* Update/modify the ceph radosgw REST APIs appropriately, based on ceilometer"},{"line_number":131,"context_line":"  requirements."}],"source_content_type":"text/x-rst","patch_set":5,"id":"3a961159_d77ce4b7","line":128,"in_reply_to":"3a961159_5113a40c","updated":"2014-12-11 12:03:27.000000000","message":"That\u0027s correct. I have rephrased this work item appropriately in the next patch set (#6).","commit_id":"cc6017299f79bf36cee0a54f965bb6666f5fa4cb"},{"author":{"_account_id":2284,"name":"Eoghan Glynn","email":"eglynn@redhat.com","username":"eglynn"},"change_message_id":"d349a29648128f276861b5c3df94299f944cc3b6","unresolved":false,"context_lines":[{"line_number":125,"context_line":"* Implement the ceph object storage (radosgw) APIs calls for appropriately"},{"line_number":126,"context_line":"  for each metering item."},{"line_number":127,"context_line":""},{"line_number":128,"context_line":"* Integrate the keystone authentication method for users validation."},{"line_number":129,"context_line":""},{"line_number":130,"context_line":"* Update/modify the ceph radosgw REST APIs appropriately, based on ceilometer"},{"line_number":131,"context_line":"  requirements."}],"source_content_type":"text/x-rst","patch_set":5,"id":"3a961159_5113a40c","line":128,"in_reply_to":"3a961159_79ca6c6f","updated":"2014-12-11 09:22:48.000000000","message":"OK, so it\u0027s not so much an integration task, as just a check that a call out to the radosgw from ceilometer (with the standard service_credentials configured in the ceilometer.conf) will be properly authenticated?\n\nSince keystone auth should be completely generic in the radosgw, this seems to mainly amount to a check that the tenant/role assignment for the ceilometer user is sufficient for the typical ceph RBAC policy setup.","commit_id":"cc6017299f79bf36cee0a54f965bb6666f5fa4cb"},{"author":{"_account_id":11224,"name":"Swami Reddy","email":"swamireddy@gmail.com","username":"swamireddy"},"change_message_id":"7e06405ec3ee0778a120e406284a5848de0e8dd8","unresolved":false,"context_lines":[{"line_number":125,"context_line":"* Implement the ceph object storage (radosgw) APIs calls for appropriately"},{"line_number":126,"context_line":"  for each metering item."},{"line_number":127,"context_line":""},{"line_number":128,"context_line":"* Integrate the keystone authentication method for users validation."},{"line_number":129,"context_line":""},{"line_number":130,"context_line":"* Update/modify the ceph radosgw REST APIs appropriately, based on ceilometer"},{"line_number":131,"context_line":"  requirements."}],"source_content_type":"text/x-rst","patch_set":5,"id":"3a961159_c351b1d9","line":128,"in_reply_to":"3a961159_a5f82d4a","updated":"2014-12-10 12:29:37.000000000","message":"Yes, ceph support keystone authentication. We are already using the ceph with keystone.","commit_id":"cc6017299f79bf36cee0a54f965bb6666f5fa4cb"},{"author":{"_account_id":2284,"name":"Eoghan Glynn","email":"eglynn@redhat.com","username":"eglynn"},"change_message_id":"7122b25de6c16d839f740c0d82ec8ed9c600f5b6","unresolved":false,"context_lines":[{"line_number":125,"context_line":"* Implement the ceph object storage (radosgw) APIs calls for appropriately"},{"line_number":126,"context_line":"  for each metering item."},{"line_number":127,"context_line":""},{"line_number":128,"context_line":"* Integrate the keystone authentication method for users validation."},{"line_number":129,"context_line":""},{"line_number":130,"context_line":"* Update/modify the ceph radosgw REST APIs appropriately, based on ceilometer"},{"line_number":131,"context_line":"  requirements."}],"source_content_type":"text/x-rst","patch_set":5,"id":"3a961159_3e40ca55","line":128,"in_reply_to":"3a961159_c351b1d9","updated":"2014-12-10 13:14:02.000000000","message":"In that case, I don\u0027t understand the need for this work item in the blueprint.","commit_id":"cc6017299f79bf36cee0a54f965bb6666f5fa4cb"},{"author":{"_account_id":2284,"name":"Eoghan Glynn","email":"eglynn@redhat.com","username":"eglynn"},"change_message_id":"ae2195d1af2a11bd95c0b691e30944b9852626c0","unresolved":false,"context_lines":[{"line_number":128,"context_line":"* Integrate the keystone authentication method for users validation."},{"line_number":129,"context_line":""},{"line_number":130,"context_line":"* Update/modify the ceph radosgw REST APIs appropriately, based on ceilometer"},{"line_number":131,"context_line":"  requirements."},{"line_number":132,"context_line":""},{"line_number":133,"context_line":"* Test coverage for the above pollster classes and sample validation."},{"line_number":134,"context_line":""}],"source_content_type":"text/x-rst","patch_set":5,"id":"3a961159_457869c4","line":131,"updated":"2014-12-10 11:45:21.000000000","message":"Can you give more detail on those changes to the radosgw REST API?\n\nWould landing these changes on the ceph side block progress on the ceilometer changes?","commit_id":"cc6017299f79bf36cee0a54f965bb6666f5fa4cb"},{"author":{"_account_id":2284,"name":"Eoghan Glynn","email":"eglynn@redhat.com","username":"eglynn"},"change_message_id":"7122b25de6c16d839f740c0d82ec8ed9c600f5b6","unresolved":false,"context_lines":[{"line_number":128,"context_line":"* Integrate the keystone authentication method for users validation."},{"line_number":129,"context_line":""},{"line_number":130,"context_line":"* Update/modify the ceph radosgw REST APIs appropriately, based on ceilometer"},{"line_number":131,"context_line":"  requirements."},{"line_number":132,"context_line":""},{"line_number":133,"context_line":"* Test coverage for the above pollster classes and sample validation."},{"line_number":134,"context_line":""}],"source_content_type":"text/x-rst","patch_set":5,"id":"3a961159_5e0d963d","line":131,"in_reply_to":"3a961159_0331f9a1","updated":"2014-12-10 13:14:02.000000000","message":"Again, it would be good to state explicitly that no specific radosgw changes are currently envisaged.","commit_id":"cc6017299f79bf36cee0a54f965bb6666f5fa4cb"},{"author":{"_account_id":11224,"name":"Swami Reddy","email":"swamireddy@gmail.com","username":"swamireddy"},"change_message_id":"7e06405ec3ee0778a120e406284a5848de0e8dd8","unresolved":false,"context_lines":[{"line_number":128,"context_line":"* Integrate the keystone authentication method for users validation."},{"line_number":129,"context_line":""},{"line_number":130,"context_line":"* Update/modify the ceph radosgw REST APIs appropriately, based on ceilometer"},{"line_number":131,"context_line":"  requirements."},{"line_number":132,"context_line":""},{"line_number":133,"context_line":"* Test coverage for the above pollster classes and sample validation."},{"line_number":134,"context_line":""}],"source_content_type":"text/x-rst","patch_set":5,"id":"3a961159_0331f9a1","line":131,"in_reply_to":"3a961159_457869c4","updated":"2014-12-10 12:29:37.000000000","message":"At present, plan to use the currently supported radosgw REST APIs as it is. If any thing spl. needed for ceilometer,  we will add them in radosgw also.","commit_id":"cc6017299f79bf36cee0a54f965bb6666f5fa4cb"},{"author":{"_account_id":11224,"name":"Swami Reddy","email":"swamireddy@gmail.com","username":"swamireddy"},"change_message_id":"9fbb331ef5fb3a6baa9af287dec63ff9a63bb74c","unresolved":false,"context_lines":[{"line_number":128,"context_line":"* Integrate the keystone authentication method for users validation."},{"line_number":129,"context_line":""},{"line_number":130,"context_line":"* Update/modify the ceph radosgw REST APIs appropriately, based on ceilometer"},{"line_number":131,"context_line":"  requirements."},{"line_number":132,"context_line":""},{"line_number":133,"context_line":"* Test coverage for the above pollster classes and sample validation."},{"line_number":134,"context_line":""}],"source_content_type":"text/x-rst","patch_set":5,"id":"3a961159_195a48f3","line":131,"in_reply_to":"3a961159_5e0d963d","updated":"2014-12-10 14:03:20.000000000","message":"Sure, update the spec as you suggested above.","commit_id":"cc6017299f79bf36cee0a54f965bb6666f5fa4cb"},{"author":{"_account_id":2284,"name":"Eoghan Glynn","email":"eglynn@redhat.com","username":"eglynn"},"change_message_id":"ae2195d1af2a11bd95c0b691e30944b9852626c0","unresolved":false,"context_lines":[{"line_number":157,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":158,"context_line":""},{"line_number":159,"context_line":"Unit and integration Tests will be added to cover the necessary pollster"},{"line_number":160,"context_line":"classes and validate samples generated."},{"line_number":161,"context_line":""},{"line_number":162,"context_line":"Documentation Impact"},{"line_number":163,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"}],"source_content_type":"text/x-rst","patch_set":5,"id":"3a961159_85e6d1b2","line":160,"updated":"2014-12-10 11:45:21.000000000","message":"What about integration testing?\n\nIs ceph installed on the nodes used for upstream CI?\n\nIf not, should we consider a 3rd-party CI approach?","commit_id":"cc6017299f79bf36cee0a54f965bb6666f5fa4cb"},{"author":{"_account_id":2284,"name":"Eoghan Glynn","email":"eglynn@redhat.com","username":"eglynn"},"change_message_id":"d349a29648128f276861b5c3df94299f944cc3b6","unresolved":false,"context_lines":[{"line_number":157,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":158,"context_line":""},{"line_number":159,"context_line":"Unit and integration Tests will be added to cover the necessary pollster"},{"line_number":160,"context_line":"classes and validate samples generated."},{"line_number":161,"context_line":""},{"line_number":162,"context_line":"Documentation Impact"},{"line_number":163,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"}],"source_content_type":"text/x-rst","patch_set":5,"id":"3a961159_b118481f","line":160,"in_reply_to":"3a961159_190ec88d","updated":"2014-12-11 09:22:48.000000000","message":"Can both swift *and* ceph storage options be installed on the same CI node?\n\nIn that case, would two separate endpoints be exposed in the keystone service catalog?\n\nOr is the idea instead that swift and ceph are mutually exclusive, and the same endpoint is exposed in the catalog in either case?","commit_id":"cc6017299f79bf36cee0a54f965bb6666f5fa4cb"},{"author":{"_account_id":11224,"name":"Swami Reddy","email":"swamireddy@gmail.com","username":"swamireddy"},"change_message_id":"9fbb331ef5fb3a6baa9af287dec63ff9a63bb74c","unresolved":false,"context_lines":[{"line_number":157,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":158,"context_line":""},{"line_number":159,"context_line":"Unit and integration Tests will be added to cover the necessary pollster"},{"line_number":160,"context_line":"classes and validate samples generated."},{"line_number":161,"context_line":""},{"line_number":162,"context_line":"Documentation Impact"},{"line_number":163,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"}],"source_content_type":"text/x-rst","patch_set":5,"id":"3a961159_190ec88d","line":160,"in_reply_to":"3a961159_390e2435","updated":"2014-12-10 14:03:20.000000000","message":"Not sure about agreement with infra team. If the ceph is used as objectstorage, then the ceilometer should support the metering. By default swift is used objectstorage and all meters works as expected from swift object storage. Please  advice, if need to discuss with infra team on this.","commit_id":"cc6017299f79bf36cee0a54f965bb6666f5fa4cb"},{"author":{"_account_id":2284,"name":"Eoghan Glynn","email":"eglynn@redhat.com","username":"eglynn"},"change_message_id":"7122b25de6c16d839f740c0d82ec8ed9c600f5b6","unresolved":false,"context_lines":[{"line_number":157,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":158,"context_line":""},{"line_number":159,"context_line":"Unit and integration Tests will be added to cover the necessary pollster"},{"line_number":160,"context_line":"classes and validate samples generated."},{"line_number":161,"context_line":""},{"line_number":162,"context_line":"Documentation Impact"},{"line_number":163,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"}],"source_content_type":"text/x-rst","patch_set":5,"id":"3a961159_390e2435","line":160,"in_reply_to":"3a961159_83e32904","updated":"2014-12-10 13:14:02.000000000","message":"So you\u0027ll need to get agreement from infra on adding ceph to the packages that are installed on CI nodes. That sounds like a non-trivial work item that should be called out.","commit_id":"cc6017299f79bf36cee0a54f965bb6666f5fa4cb"},{"author":{"_account_id":11224,"name":"Swami Reddy","email":"swamireddy@gmail.com","username":"swamireddy"},"change_message_id":"7e06405ec3ee0778a120e406284a5848de0e8dd8","unresolved":false,"context_lines":[{"line_number":157,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":158,"context_line":""},{"line_number":159,"context_line":"Unit and integration Tests will be added to cover the necessary pollster"},{"line_number":160,"context_line":"classes and validate samples generated."},{"line_number":161,"context_line":""},{"line_number":162,"context_line":"Documentation Impact"},{"line_number":163,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"}],"source_content_type":"text/x-rst","patch_set":5,"id":"3a961159_83e32904","line":160,"in_reply_to":"3a961159_85e6d1b2","updated":"2014-12-10 12:29:37.000000000","message":"Ceph is not part of CI. Plan to use the latest stable version of ceph release packages installed on the nodes.","commit_id":"cc6017299f79bf36cee0a54f965bb6666f5fa4cb"},{"author":{"_account_id":11224,"name":"Swami Reddy","email":"swamireddy@gmail.com","username":"swamireddy"},"change_message_id":"67ec2edb8519bd344eb08511d6088ce258407422","unresolved":false,"context_lines":[{"line_number":157,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":158,"context_line":""},{"line_number":159,"context_line":"Unit and integration Tests will be added to cover the necessary pollster"},{"line_number":160,"context_line":"classes and validate samples generated."},{"line_number":161,"context_line":""},{"line_number":162,"context_line":"Documentation Impact"},{"line_number":163,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"}],"source_content_type":"text/x-rst","patch_set":5,"id":"3a961159_226de8cc","line":160,"in_reply_to":"3a961159_b118481f","updated":"2014-12-11 12:03:27.000000000","message":"Yep, both swift and ceph can be installed on the same CI node. And these will be exposed to by separate endpoints.\n\nBut, I/We are planning to use only Ceph (not swift) and the CI setup accordingly.","commit_id":"cc6017299f79bf36cee0a54f965bb6666f5fa4cb"},{"author":{"_account_id":2284,"name":"Eoghan Glynn","email":"eglynn@redhat.com","username":"eglynn"},"change_message_id":"1a3411ec615e2cc9572a037826095e06b00356ad","unresolved":false,"context_lines":[{"line_number":119,"context_line":""},{"line_number":120,"context_line":"* Configure the setup (i.e setup.cfg) file based on the object storage used"},{"line_number":121,"context_line":"  to get the metering information. If swift used, use swift collectors to get"},{"line_number":122,"context_line":"  the metering information. If ceph used, use ceph collectors to get the"},{"line_number":123,"context_line":"  metering information."},{"line_number":124,"context_line":""},{"line_number":125,"context_line":"* Implement the ceph object storage (radosgw) APIs calls for appropriately"}],"source_content_type":"text/x-rst","patch_set":6,"id":"3a961159_33c23c53","line":122,"updated":"2014-12-15 11:23:47.000000000","message":"Non-blocking nitpick (only fix if you\u0027re uploading a new patchset anyway) ...\n\nThe term \"collector\" means something different in the ceilometer architecture, I think you meant: \"swift pollsters\" and \"ceph pollsters\".","commit_id":"808d2b052de9deed183ccddc8eff446463dc3fed"},{"author":{"_account_id":11224,"name":"Swami Reddy","email":"swamireddy@gmail.com","username":"swamireddy"},"change_message_id":"c8109050dcf1a23cca03ae19888c49931a231195","unresolved":false,"context_lines":[{"line_number":119,"context_line":""},{"line_number":120,"context_line":"* Configure the setup (i.e setup.cfg) file based on the object storage used"},{"line_number":121,"context_line":"  to get the metering information. If swift used, use swift collectors to get"},{"line_number":122,"context_line":"  the metering information. If ceph used, use ceph collectors to get the"},{"line_number":123,"context_line":"  metering information."},{"line_number":124,"context_line":""},{"line_number":125,"context_line":"* Implement the ceph object storage (radosgw) APIs calls for appropriately"}],"source_content_type":"text/x-rst","patch_set":6,"id":"3a961159_b3a1cc47","line":122,"in_reply_to":"3a961159_33c23c53","updated":"2014-12-15 11:39:35.000000000","message":"That\u0027s correct. The swift collectors and ceph collectors meant  -  swift pollsters and ceph pollsters only.","commit_id":"808d2b052de9deed183ccddc8eff446463dc3fed"}]}