)]}'
{"openstack/compute/v2/server.py":[{"author":{"_account_id":8257,"name":"Brian Curtin","email":"brian@python.org","username":"briancurtin"},"change_message_id":"4acbff64eb8e9b05c7cde3e6a2b452c9cb2fa2ea","unresolved":false,"context_lines":[{"line_number":61,"context_line":"        body \u003d {\u0027reboot\u0027: {\u0027type\u0027: reboot_type}}"},{"line_number":62,"context_line":"        return self.action(session, body)"},{"line_number":63,"context_line":""},{"line_number":64,"context_line":"    def rebuild(self, session, **kwargs):"},{"line_number":65,"context_line":"        \"\"\"Rebuild the server with the given arguments."},{"line_number":66,"context_line":""},{"line_number":67,"context_line":"        The arguments may be for example:"}],"source_content_type":"text/x-python","patch_set":1,"id":"baada198_3c0dce40","line":64,"updated":"2014-08-11 17:47:21.000000000","message":"I think we should be doing some validation on arguments here, as name, imageRef, and adminPass are required. Those three could be named arguments, with kwargs making up the rest of the optional arguments the rebuild call could take.\n\nIn a similar fashion, `resize` does this","commit_id":"69876721988e907ec8e9e1cede68b9e65f3dedad"},{"author":{"_account_id":8257,"name":"Brian Curtin","email":"brian@python.org","username":"briancurtin"},"change_message_id":"4acbff64eb8e9b05c7cde3e6a2b452c9cb2fa2ea","unresolved":false,"context_lines":[{"line_number":68,"context_line":"        * imageRef"},{"line_number":69,"context_line":"        * name"},{"line_number":70,"context_line":"        * adminPass"},{"line_number":71,"context_line":"        * accessIPv4"},{"line_number":72,"context_line":"        * accessIPv6"},{"line_number":73,"context_line":"        * metadata"},{"line_number":74,"context_line":"        * personality"}],"source_content_type":"text/x-python","patch_set":1,"id":"baada198_82bd94d3","line":71,"updated":"2014-08-11 17:47:21.000000000","message":"This is a change we\u0027d handle elsewhere, but a few of these names will end up as underscore_separated properties on the resource instead of their expected camelCase names. For example, I might have Server.access_IPv4 set to some value and I want to send it right through to this rebuild call. Do we need to maintain some mapping between server-expected names and internal names that we swap between right before sending? Something to think about.","commit_id":"69876721988e907ec8e9e1cede68b9e65f3dedad"},{"author":{"_account_id":8257,"name":"Brian Curtin","email":"brian@python.org","username":"briancurtin"},"change_message_id":"4acbff64eb8e9b05c7cde3e6a2b452c9cb2fa2ea","unresolved":false,"context_lines":[{"line_number":92,"context_line":"        body \u003d {\u0027revertResize\u0027: None}"},{"line_number":93,"context_line":"        return self.action(session, body)"},{"line_number":94,"context_line":""},{"line_number":95,"context_line":"    def create_image(self, session, name, metadata):"},{"line_number":96,"context_line":"        \"\"\"Create image from server.\"\"\""},{"line_number":97,"context_line":"        body \u003d {\u0027createImage\u0027: {\u0027name\u0027: name, \u0027metadata\u0027: metadata}}"},{"line_number":98,"context_line":"        return self.action(session, body)"}],"source_content_type":"text/x-python","patch_set":1,"id":"baada198_e2e1f0cf","line":95,"updated":"2014-08-11 17:47:21.000000000","message":"metadata isn\u0027t required, so perhaps it should default to None","commit_id":"69876721988e907ec8e9e1cede68b9e65f3dedad"}]}