)]}'
{"specs/kilo/extra-optional.rst":[{"author":{"_account_id":6460,"name":"Brad Topol","email":"btopol@us.ibm.com","username":"btopol"},"change_message_id":"3bedcbdaf8c18c906c7ec991b37590d7087597b2","unresolved":false,"context_lines":[{"line_number":16,"context_line":"Problem Description"},{"line_number":17,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":18,"context_line":""},{"line_number":19,"context_line":"Our SQL drivers have for a long time supported clients piggybacking on our"},{"line_number":20,"context_line":"formal entity attributes by just including extra attributes in the API call"},{"line_number":21,"context_line":"(e.g. POST) - which we then store in the \u0027extras\u0027 column. While this is used"},{"line_number":22,"context_line":"by a number of customers, it\u0027s a practice we don\u0027t want to encourage. Moreover,"}],"source_content_type":"text/x-rst","patch_set":2,"id":"da86d52c_42e913e5","line":19,"updated":"2015-02-04 01:55:12.000000000","message":"So if some OpenStack Clouds support the storing of extras and now we give the option to disable this, why have we not introduced a new potential interoperability between clouds that folks will have to worry about.  Can you provide some justification why the benefits out way this interoperability impediment?","commit_id":"1a23ea23941a4bbd1967dca9a7327a3d7c7b28bf"},{"author":{"_account_id":5707,"name":"Henry Nash","email":"henryn@linux.vnet.ibm.com","username":"henry-nash"},"change_message_id":"006b4dd627885a821a5f9027e20c0584a4e8aaa7","unresolved":false,"context_lines":[{"line_number":16,"context_line":"Problem Description"},{"line_number":17,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":18,"context_line":""},{"line_number":19,"context_line":"Our SQL drivers have for a long time supported clients piggybacking on our"},{"line_number":20,"context_line":"formal entity attributes by just including extra attributes in the API call"},{"line_number":21,"context_line":"(e.g. POST) - which we then store in the \u0027extras\u0027 column. While this is used"},{"line_number":22,"context_line":"by a number of customers, it\u0027s a practice we don\u0027t want to encourage. Moreover,"}],"source_content_type":"text/x-rst","patch_set":2,"id":"da86d52c_e4703874","line":19,"in_reply_to":"da86d52c_02d0fbc7","updated":"2015-02-04 23:22:51.000000000","message":"So I believe that we need to provide a way that clients can store data that is conceptually linked to entities in Keystone.  The recommended answer should not, however, be to stuff it into \u0027extras\u0027. That can lead to potentially both Keystone security (i.e. we\u0027re storing PII data without knowing it) and performance issues (our DB is full of user provided blobs we never knew about before). Our recommended way is to provide their own storage system, but listen to Keystone notifications so that they can keep their storage in sync with those entities in Keystone (i.e. track deletes etc.).  Hence this step is the first in getting people to transition to the new way of doing things.","commit_id":"1a23ea23941a4bbd1967dca9a7327a3d7c7b28bf"},{"author":{"_account_id":6460,"name":"Brad Topol","email":"btopol@us.ibm.com","username":"btopol"},"change_message_id":"49cdfa5e6c24fa2dfb389efbb435d9a263e80b1f","unresolved":false,"context_lines":[{"line_number":16,"context_line":"Problem Description"},{"line_number":17,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":18,"context_line":""},{"line_number":19,"context_line":"Our SQL drivers have for a long time supported clients piggybacking on our"},{"line_number":20,"context_line":"formal entity attributes by just including extra attributes in the API call"},{"line_number":21,"context_line":"(e.g. POST) - which we then store in the \u0027extras\u0027 column. While this is used"},{"line_number":22,"context_line":"by a number of customers, it\u0027s a practice we don\u0027t want to encourage. Moreover,"}],"source_content_type":"text/x-rst","patch_set":2,"id":"da86d52c_02d0fbc7","line":19,"in_reply_to":"da86d52c_42e913e5","updated":"2015-02-04 02:05:16.000000000","message":"interoperability in the 1st sentence should be interoperability impediment :-)","commit_id":"1a23ea23941a4bbd1967dca9a7327a3d7c7b28bf"},{"author":{"_account_id":6460,"name":"Brad Topol","email":"btopol@us.ibm.com","username":"btopol"},"change_message_id":"3bedcbdaf8c18c906c7ec991b37590d7087597b2","unresolved":false,"context_lines":[{"line_number":29,"context_line":"We will provide a configuration option to define whether our SQL drivers will"},{"line_number":30,"context_line":"store extra attributes. For backward compatibility this will be set,"},{"line_number":31,"context_line":"by default, to allow such storage. Disabling this option will cause any"},{"line_number":32,"context_line":"extra parameters to be silently ignored in both read and write of SQL entities."},{"line_number":33,"context_line":"If this option is disabled on a system that already has extra attributes stored"},{"line_number":34,"context_line":"this data will not be deleted - that is left as an out-of-band operation for"},{"line_number":35,"context_line":"the cloud provider."}],"source_content_type":"text/x-rst","patch_set":2,"id":"da86d52c_22a97fdf","line":32,"updated":"2015-02-04 01:55:12.000000000","message":"How silent is silent? Will a warning be logged?  For serviceability purposes I think a warning should be placed in the logs. Otherwise folks will be where is my extra stuff?","commit_id":"1a23ea23941a4bbd1967dca9a7327a3d7c7b28bf"},{"author":{"_account_id":5707,"name":"Henry Nash","email":"henryn@linux.vnet.ibm.com","username":"henry-nash"},"change_message_id":"006b4dd627885a821a5f9027e20c0584a4e8aaa7","unresolved":false,"context_lines":[{"line_number":29,"context_line":"We will provide a configuration option to define whether our SQL drivers will"},{"line_number":30,"context_line":"store extra attributes. For backward compatibility this will be set,"},{"line_number":31,"context_line":"by default, to allow such storage. Disabling this option will cause any"},{"line_number":32,"context_line":"extra parameters to be silently ignored in both read and write of SQL entities."},{"line_number":33,"context_line":"If this option is disabled on a system that already has extra attributes stored"},{"line_number":34,"context_line":"this data will not be deleted - that is left as an out-of-band operation for"},{"line_number":35,"context_line":"the cloud provider."}],"source_content_type":"text/x-rst","patch_set":2,"id":"da86d52c_4cfafd43","line":32,"in_reply_to":"da86d52c_22a97fdf","updated":"2015-02-04 23:22:51.000000000","message":"As agreed in IRC, we will provide an options to be silent (with warnings) or error.","commit_id":"1a23ea23941a4bbd1967dca9a7327a3d7c7b28bf"},{"author":{"_account_id":6460,"name":"Brad Topol","email":"btopol@us.ibm.com","username":"btopol"},"change_message_id":"3bedcbdaf8c18c906c7ec991b37590d7087597b2","unresolved":false,"context_lines":[{"line_number":37,"context_line":"As an aside, although we do support extra attributes for LDAP drivers, the"},{"line_number":38,"context_line":"storage of these is already optional by virtue of the fact that a mapping must"},{"line_number":39,"context_line":"be defined in the LDAP section of the configuration file."},{"line_number":40,"context_line":""},{"line_number":41,"context_line":"Alternatives"},{"line_number":42,"context_line":"------------"},{"line_number":43,"context_line":""}],"source_content_type":"text/x-rst","patch_set":2,"id":"da86d52c_a28baf66","line":40,"updated":"2015-02-04 01:55:12.000000000","message":"So you are arguing that the SQL driver is being brought up to function parity with the LDAP driver?  Boy is that a case of man bites dog :-)","commit_id":"1a23ea23941a4bbd1967dca9a7327a3d7c7b28bf"},{"author":{"_account_id":5707,"name":"Henry Nash","email":"henryn@linux.vnet.ibm.com","username":"henry-nash"},"change_message_id":"006b4dd627885a821a5f9027e20c0584a4e8aaa7","unresolved":false,"context_lines":[{"line_number":37,"context_line":"As an aside, although we do support extra attributes for LDAP drivers, the"},{"line_number":38,"context_line":"storage of these is already optional by virtue of the fact that a mapping must"},{"line_number":39,"context_line":"be defined in the LDAP section of the configuration file."},{"line_number":40,"context_line":""},{"line_number":41,"context_line":"Alternatives"},{"line_number":42,"context_line":"------------"},{"line_number":43,"context_line":""}],"source_content_type":"text/x-rst","patch_set":2,"id":"da86d52c_6c8ee1d1","line":40,"in_reply_to":"da86d52c_a28baf66","updated":"2015-02-04 23:22:51.000000000","message":"yep....and the fur doesn\u0027t taste so good...but a man\u0027s gotta do what a man\u0027s gotta do :-)","commit_id":"1a23ea23941a4bbd1967dca9a7327a3d7c7b28bf"},{"author":{"_account_id":6460,"name":"Brad Topol","email":"btopol@us.ibm.com","username":"btopol"},"change_message_id":"3bedcbdaf8c18c906c7ec991b37590d7087597b2","unresolved":false,"context_lines":[{"line_number":87,"context_line":"----------"},{"line_number":88,"context_line":""},{"line_number":89,"context_line":"* Create configuration option"},{"line_number":90,"context_line":"* Honor that option in the core SQL code when converts the extra attributes"},{"line_number":91,"context_line":""},{"line_number":92,"context_line":"Dependencies"},{"line_number":93,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"}],"source_content_type":"text/x-rst","patch_set":2,"id":"da86d52c_02c9dbab","line":90,"updated":"2015-02-04 01:55:12.000000000","message":"typo between when and converts","commit_id":"1a23ea23941a4bbd1967dca9a7327a3d7c7b28bf"},{"author":{"_account_id":5707,"name":"Henry Nash","email":"henryn@linux.vnet.ibm.com","username":"henry-nash"},"change_message_id":"006b4dd627885a821a5f9027e20c0584a4e8aaa7","unresolved":false,"context_lines":[{"line_number":87,"context_line":"----------"},{"line_number":88,"context_line":""},{"line_number":89,"context_line":"* Create configuration option"},{"line_number":90,"context_line":"* Honor that option in the core SQL code when converts the extra attributes"},{"line_number":91,"context_line":""},{"line_number":92,"context_line":"Dependencies"},{"line_number":93,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"}],"source_content_type":"text/x-rst","patch_set":2,"id":"da86d52c_ccb42d84","line":90,"in_reply_to":"da86d52c_02c9dbab","updated":"2015-02-04 23:22:51.000000000","message":"Done","commit_id":"1a23ea23941a4bbd1967dca9a7327a3d7c7b28bf"}]}