)]}'
{"README.rst":[{"author":{"_account_id":7419,"name":"Imre Farkas","email":"farkas.imre@gmail.com","username":"ifarkas"},"change_message_id":"15fdc1dbf7aff87c4b326ffa98ef209e18cd85a6","unresolved":false,"context_lines":[{"line_number":114,"context_line":"* ``uuid`` rule UUID, will be generated, if missing."},{"line_number":115,"context_line":""},{"line_number":116,"context_line":"* ``description`` optional rule description."},{"line_number":117,"context_line":""},{"line_number":118,"context_line":"``client.rules.from_json(rule_json)``"},{"line_number":119,"context_line":""},{"line_number":120,"context_line":"* ``rule_json`` dictionary with a rule representation."}],"source_content_type":"text/x-rst","patch_set":4,"id":"ba15a1d1_e82718c8","line":117,"updated":"2015-09-16 14:44:48.000000000","message":"missing note: \tThis call is currently not represented in CLI.","commit_id":"75e36268d4fd385a3e2f1a0ccf0f34ab29b30734"},{"author":{"_account_id":10239,"name":"Dmitry Tantsur","email":"dtantsur@protonmail.com","username":"dtantsur"},"change_message_id":"310c8bd9c9737fa254427104ec1b4b327b615b1e","unresolved":false,"context_lines":[{"line_number":114,"context_line":"* ``uuid`` rule UUID, will be generated, if missing."},{"line_number":115,"context_line":""},{"line_number":116,"context_line":"* ``description`` optional rule description."},{"line_number":117,"context_line":""},{"line_number":118,"context_line":"``client.rules.from_json(rule_json)``"},{"line_number":119,"context_line":""},{"line_number":120,"context_line":"* ``rule_json`` dictionary with a rule representation."}],"source_content_type":"text/x-rst","patch_set":4,"id":"ba15a1d1_1e3ac2cc","line":117,"in_reply_to":"ba15a1d1_e82718c8","updated":"2015-09-16 14:45:53.000000000","message":"Done","commit_id":"75e36268d4fd385a3e2f1a0ccf0f34ab29b30734"},{"author":{"_account_id":6637,"name":"Sam Betts","email":"sam@code-smash.net","username":"sambetts"},"change_message_id":"c342c8b60a3514c9990df3e237a8da1210347948","unresolved":false,"context_lines":[{"line_number":123,"context_line":""},{"line_number":124,"context_line":"CLI::"},{"line_number":125,"context_line":""},{"line_number":126,"context_line":"    $ openstack baremetal introspection rule import \u003cJSON FILE\u003e"},{"line_number":127,"context_line":""},{"line_number":128,"context_line":"Listing all rules"},{"line_number":129,"context_line":"^^^^^^^^^^^^^^^^^"}],"source_content_type":"text/x-rst","patch_set":5,"id":"ba15a1d1_7acd07b6","line":126,"updated":"2015-09-16 15:51:38.000000000","message":"I know the data structure here is complicated, creating the rule in one go makes sense in code, however, this is seeing this is making me think that perhaps we should provide, \n\nopenstack baremetal introspection rule create\n\nthen\n\nopenstack baremetal introspection rule add-condition \u003cuuid\u003e --op --field etc \n\nand \n\nopenstack baremetal introspection rule add-action \u003cuuid\u003e --name --action etc \n\nThis would require changes to the rule API, which I don\u0027t think we\u0027ll want to do for L, but something to bare in mind.","commit_id":"fb9ba05d256e274b9a76f6dc2a4443d99e4dff5a"},{"author":{"_account_id":12715,"name":"John Trowbridge","email":"trown@redhat.com","username":"trown"},"change_message_id":"deef37b16e2012d12f93c4ced7ff7ab4ba5bc343","unresolved":false,"context_lines":[{"line_number":123,"context_line":""},{"line_number":124,"context_line":"CLI::"},{"line_number":125,"context_line":""},{"line_number":126,"context_line":"    $ openstack baremetal introspection rule import \u003cJSON FILE\u003e"},{"line_number":127,"context_line":""},{"line_number":128,"context_line":"Listing all rules"},{"line_number":129,"context_line":"^^^^^^^^^^^^^^^^^"}],"source_content_type":"text/x-rst","patch_set":5,"id":"ba15a1d1_797bd8f5","line":126,"updated":"2015-09-16 15:44:30.000000000","message":"It would be nice to have an example JSON file here, but I am ok with that in a follow-up.","commit_id":"fb9ba05d256e274b9a76f6dc2a4443d99e4dff5a"},{"author":{"_account_id":10239,"name":"Dmitry Tantsur","email":"dtantsur@protonmail.com","username":"dtantsur"},"change_message_id":"9477a452388304620f7f6a7aabc54132cf9f570d","unresolved":false,"context_lines":[{"line_number":123,"context_line":""},{"line_number":124,"context_line":"CLI::"},{"line_number":125,"context_line":""},{"line_number":126,"context_line":"    $ openstack baremetal introspection rule import \u003cJSON FILE\u003e"},{"line_number":127,"context_line":""},{"line_number":128,"context_line":"Listing all rules"},{"line_number":129,"context_line":"^^^^^^^^^^^^^^^^^"}],"source_content_type":"text/x-rst","patch_set":5,"id":"ba15a1d1_1d4e453b","line":126,"in_reply_to":"ba15a1d1_7acd07b6","updated":"2015-09-16 15:55:54.000000000","message":"Yeah, good improvement for M","commit_id":"fb9ba05d256e274b9a76f6dc2a4443d99e4dff5a"},{"author":{"_account_id":6637,"name":"Sam Betts","email":"sam@code-smash.net","username":"sambetts"},"change_message_id":"c342c8b60a3514c9990df3e237a8da1210347948","unresolved":false,"context_lines":[{"line_number":146,"context_line":""},{"line_number":147,"context_line":"Returns a full rule representation as a dictionary."},{"line_number":148,"context_line":""},{"line_number":149,"context_line":"This call is currently not represented in CLI."},{"line_number":150,"context_line":""},{"line_number":151,"context_line":"Deleting all rules"},{"line_number":152,"context_line":"^^^^^^^^^^^^^^^^^^"}],"source_content_type":"text/x-rst","patch_set":5,"id":"ba15a1d1_94a8f18e","line":149,"updated":"2015-09-16 15:51:38.000000000","message":"I know you mentioned this in the commit message sayings its difficult to work out how to represent this in a human readable fashion:\n\nMy thoughts would be:\n\nopenstack baremetal introspection rule show \u003cuuid\u003e\n\nshould output something like: \n\n    +-------------------------------------------------------------------------------+\n    |    Property     |                  Value                                      |\n    +-------------------------------------------------------------------------------+\n    |   uuid          |   1234172407812186241287                                    |\n    |   description   |   Rule Description goes here                                |\n    |   conditions    |   { \u0027op\u0027: \u0027gt\u0027, \u0027field\u0027: \u0027inventory.disk.*.size\u0027,           |\n    |                 |     \u0027value\u0027: \"4\"}                                           |\n    |                 |   { \u0027op\u0027: \u0027lt\u0027, \u0027field\u0027: \u0027inventory.disk.*.size\u0027,           |\n    |                 |     \u0027value\u0027: \"8\"}                                           |\n    |   actions       |   { \"action\": \"set-capability\", \"name\": \"much-memory\",      |\n    |                 |     \"value\": \"wow\"}                                         |\n    |                 |   { \"action\": \"append-attribute\", \"key\": \"/extra/matched\",   |\n    |                 |     \"value\": \"much-memory\"}                                  |\n    |                 |                                                             |\n    +-------------------------------------------------------------------------------+","commit_id":"fb9ba05d256e274b9a76f6dc2a4443d99e4dff5a"},{"author":{"_account_id":10239,"name":"Dmitry Tantsur","email":"dtantsur@protonmail.com","username":"dtantsur"},"change_message_id":"9477a452388304620f7f6a7aabc54132cf9f570d","unresolved":false,"context_lines":[{"line_number":146,"context_line":""},{"line_number":147,"context_line":"Returns a full rule representation as a dictionary."},{"line_number":148,"context_line":""},{"line_number":149,"context_line":"This call is currently not represented in CLI."},{"line_number":150,"context_line":""},{"line_number":151,"context_line":"Deleting all rules"},{"line_number":152,"context_line":"^^^^^^^^^^^^^^^^^^"}],"source_content_type":"text/x-rst","patch_set":5,"id":"ba15a1d1_fd1591e4","line":149,"in_reply_to":"ba15a1d1_94a8f18e","updated":"2015-09-16 15:55:54.000000000","message":"it requires some effort to make our JSON\u0027s look that beautiful (by default it will be a mess). Do you mind if I try something as a follow-up?","commit_id":"fb9ba05d256e274b9a76f6dc2a4443d99e4dff5a"},{"author":{"_account_id":6637,"name":"Sam Betts","email":"sam@code-smash.net","username":"sambetts"},"change_message_id":"77ef7f6351c21026cfbbb9d318202d4cb9836171","unresolved":false,"context_lines":[{"line_number":146,"context_line":""},{"line_number":147,"context_line":"Returns a full rule representation as a dictionary."},{"line_number":148,"context_line":""},{"line_number":149,"context_line":"This call is currently not represented in CLI."},{"line_number":150,"context_line":""},{"line_number":151,"context_line":"Deleting all rules"},{"line_number":152,"context_line":"^^^^^^^^^^^^^^^^^^"}],"source_content_type":"text/x-rst","patch_set":5,"id":"ba15a1d1_aed7e149","line":149,"in_reply_to":"ba15a1d1_fd1591e4","updated":"2015-09-16 16:57:08.000000000","message":"As discussed on IRC going to make it a follow up patch to include this command","commit_id":"fb9ba05d256e274b9a76f6dc2a4443d99e4dff5a"}],"test-requirements.txt":[{"author":{"_account_id":7419,"name":"Imre Farkas","email":"farkas.imre@gmail.com","username":"ifarkas"},"change_message_id":"15fdc1dbf7aff87c4b326ffa98ef209e18cd85a6","unresolved":false,"context_lines":[{"line_number":3,"context_line":"# process, which may cause wedges in the gate later."},{"line_number":4,"context_line":"coverage\u003e\u003d3.6"},{"line_number":5,"context_line":"doc8 # Apache-2.0"},{"line_number":6,"context_line":"fixtures\u003e\u003d1.3.1"},{"line_number":7,"context_line":"hacking\u003c0.11,\u003e\u003d0.10.0"},{"line_number":8,"context_line":"mock\u003e\u003d1.2"}],"source_content_type":"text/plain","patch_set":4,"id":"ba15a1d1_2389b1c9","line":6,"updated":"2015-09-16 14:44:48.000000000","message":"where is it used?","commit_id":"75e36268d4fd385a3e2f1a0ccf0f34ab29b30734"},{"author":{"_account_id":10239,"name":"Dmitry Tantsur","email":"dtantsur@protonmail.com","username":"dtantsur"},"change_message_id":"310c8bd9c9737fa254427104ec1b4b327b615b1e","unresolved":false,"context_lines":[{"line_number":3,"context_line":"# process, which may cause wedges in the gate later."},{"line_number":4,"context_line":"coverage\u003e\u003d3.6"},{"line_number":5,"context_line":"doc8 # Apache-2.0"},{"line_number":6,"context_line":"fixtures\u003e\u003d1.3.1"},{"line_number":7,"context_line":"hacking\u003c0.11,\u003e\u003d0.10.0"},{"line_number":8,"context_line":"mock\u003e\u003d1.2"}],"source_content_type":"text/plain","patch_set":4,"id":"ba15a1d1_5e235aa7","line":6,"in_reply_to":"ba15a1d1_2389b1c9","updated":"2015-09-16 14:45:53.000000000","message":"it\u0027s a dependency for openstackclient.tests.utils","commit_id":"75e36268d4fd385a3e2f1a0ccf0f34ab29b30734"}]}