)]}'
{"specs/juno/oslo-config-cfgfilter.rst":[{"author":{"_account_id":2472,"name":"Doug Hellmann","email":"dhellmann@redhat.com","username":"doug-hellmann"},"change_message_id":"aa187c8fa40c6c67287e1b8fe9683125b4b2a31b","unresolved":false,"context_lines":[{"line_number":103,"context_line":"      def __init__(self, conf):"},{"line_number":104,"context_line":"          self.conf \u003d conf"},{"line_number":105,"context_line":"          self._private_conf \u003d ConfigFilter(self.conf)"},{"line_number":106,"context_line":"          self._private_conf.register_opt(StrOpt(\u0027foo\u0027))"},{"line_number":107,"context_line":""},{"line_number":108,"context_line":"      @property"},{"line_number":109,"context_line":"      def foo(self):"}],"source_content_type":"text/x-rst","patch_set":1,"id":"1ae5cdf2_e8659ab5","line":106,"updated":"2014-06-03 18:10:00.000000000","message":"This example is fine for the spec, but when we document how to use the class we need to show how to register an option as private while still making it available to the config generator.","commit_id":"2c1e27779746993d2d2b6b0fc2ea12547270b5eb"},{"author":{"_account_id":1247,"name":"Mark McLoughlin","email":"markmc@redhat.com","username":"markmc"},"change_message_id":"27ceab4d2e625be3617fe89c2b9e9757d545f9a1","unresolved":false,"context_lines":[{"line_number":103,"context_line":"      def __init__(self, conf):"},{"line_number":104,"context_line":"          self.conf \u003d conf"},{"line_number":105,"context_line":"          self._private_conf \u003d ConfigFilter(self.conf)"},{"line_number":106,"context_line":"          self._private_conf.register_opt(StrOpt(\u0027foo\u0027))"},{"line_number":107,"context_line":""},{"line_number":108,"context_line":"      @property"},{"line_number":109,"context_line":"      def foo(self):"}],"source_content_type":"text/x-rst","patch_set":1,"id":"1ae5cdf2_dbfaf841","line":106,"in_reply_to":"1ae5cdf2_e8659ab5","updated":"2014-06-06 11:46:04.000000000","message":"Excellent point Doug, this will take a little hashing out as we go, I suspect\n\nI\u0027ve updated the spec to at least mention it","commit_id":"2c1e27779746993d2d2b6b0fc2ea12547270b5eb"},{"author":{"_account_id":8041,"name":"Petr Blaho","email":"pblaho@redhat.com","username":"pblaho"},"change_message_id":"f4f1a73035514299f131a663100bba8e1139713a","unresolved":false,"context_lines":[{"line_number":42,"context_line":"However, import_opt() has a weakness - if \u0027bar\u0027 imports \u0027foo\u0027 using the import"},{"line_number":43,"context_line":"builtin and doesn\u0027t use import_opt() to import \u0027blaa\u0027, then \u0027blaa\u0027 can still be"},{"line_number":44,"context_line":"used without problems. Similarly, where multiple options are registered a"},{"line_number":45,"context_line":"module imported via importopt(), a lazy programmer can get away with only"},{"line_number":46,"context_line":"declaring a dependency on a single option."},{"line_number":47,"context_line":""},{"line_number":48,"context_line":"Private Configuration Options"}],"source_content_type":"text/x-rst","patch_set":3,"id":"1ae5cdf2_52ffcb33","line":45,"updated":"2014-06-13 12:28:49.000000000","message":"\"... imported via importopt()\" - should it be import_opt() ?","commit_id":"f79abe3c23d2a7d502a4edb77bb7143c630f1114"},{"author":{"_account_id":1247,"name":"Mark McLoughlin","email":"markmc@redhat.com","username":"markmc"},"change_message_id":"9328182826558714911b0c7fbcc946ba89256589","unresolved":false,"context_lines":[{"line_number":42,"context_line":"However, import_opt() has a weakness - if \u0027bar\u0027 imports \u0027foo\u0027 using the import"},{"line_number":43,"context_line":"builtin and doesn\u0027t use import_opt() to import \u0027blaa\u0027, then \u0027blaa\u0027 can still be"},{"line_number":44,"context_line":"used without problems. Similarly, where multiple options are registered a"},{"line_number":45,"context_line":"module imported via importopt(), a lazy programmer can get away with only"},{"line_number":46,"context_line":"declaring a dependency on a single option."},{"line_number":47,"context_line":""},{"line_number":48,"context_line":"Private Configuration Options"}],"source_content_type":"text/x-rst","patch_set":3,"id":"1ae5cdf2_48f63cd8","line":45,"in_reply_to":"1ae5cdf2_52ffcb33","updated":"2014-06-13 13:04:07.000000000","message":"Done","commit_id":"f79abe3c23d2a7d502a4edb77bb7143c630f1114"}]}
