)]}'
{"shade/operatorcloud.py":[{"author":{"_account_id":2,"name":"Monty Taylor","email":"mordred@inaugust.com","username":"mordred"},"change_message_id":"584e2c52204d5e209648e70205217508e2bb0bfe","unresolved":false,"context_lines":[{"line_number":56,"context_line":"        return self._normalize_machines("},{"line_number":57,"context_line":"            self.manager.submit_task(_tasks.MachineNodeList()))"},{"line_number":58,"context_line":""},{"line_number":59,"context_line":"    def get_machine(self, name_or_id, version\u003d\u00271.6\u0027):"},{"line_number":60,"context_line":"        \"\"\"Get Machine by name or uuid"},{"line_number":61,"context_line":""},{"line_number":62,"context_line":"        Search the baremetal host out by utilizing the supplied id value"}],"source_content_type":"text/x-python","patch_set":3,"id":"ff82abbf_28267d69","line":59,"updated":"2017-11-29 21:49:38.000000000","message":"For now (can do this in a followup before we release) let\u0027s remove the version argument and just keep it as a local variable. Reasoning is - I\u0027d like (if we can manage it) to keep the data model shade returns the same no matter what version the cloud has... so as we add support for later microversions I *think* what we should do is add entries to the data model and then if we\u0027re using an earlier microversion those fields will just have None in them.\n\nI say *think* because this is new and exciting territory, and what you have here with a parameter might be what we wind up doing ... I think we should play with it a little bit before growing an API parameter we can\u0027t walk back.","commit_id":"f1cd4019cd6190821a0898551b2905a572fa3792"},{"author":{"_account_id":11655,"name":"Julia Kreger","email":"juliaashleykreger@gmail.com","username":"jkreger","status":"Flying to the moon with a Jetpack!"},"change_message_id":"a7da85efae2530ec5bfaa9965bf3e05e716349d6","unresolved":false,"context_lines":[{"line_number":56,"context_line":"        return self._normalize_machines("},{"line_number":57,"context_line":"            self.manager.submit_task(_tasks.MachineNodeList()))"},{"line_number":58,"context_line":""},{"line_number":59,"context_line":"    def get_machine(self, name_or_id, version\u003d\u00271.6\u0027):"},{"line_number":60,"context_line":"        \"\"\"Get Machine by name or uuid"},{"line_number":61,"context_line":""},{"line_number":62,"context_line":"        Search the baremetal host out by utilizing the supplied id value"}],"source_content_type":"text/x-python","patch_set":3,"id":"ff82abbf_a875ad33","line":59,"in_reply_to":"ff82abbf_28267d69","updated":"2017-11-29 21:52:54.000000000","message":"Agreed, I\u0027ll take care of it in the morning.  I think that is what ironic\u0027s API already does, with a ?single? documented exception, which makes things very easy for that.","commit_id":"f1cd4019cd6190821a0898551b2905a572fa3792"}]}