)]}'
{"astara/instance_manager.py":[{"author":{"_account_id":986,"name":"Rosario Di Somma","email":"rosario.disomma@gmail.com","username":"mr-rods"},"change_message_id":"a830367db13dc6278683fb6c1b4a663b934afc09","unresolved":false,"context_lines":[{"line_number":400,"context_line":""},{"line_number":401,"context_line":"        # send data about the orchestrator that owns the appliance"},{"line_number":402,"context_line":"        # this can be extended in the future as needed"},{"line_number":403,"context_line":"        config[\u0027orchestrator\u0027] \u003d worker_context.config"},{"line_number":404,"context_line":""},{"line_number":405,"context_line":"        self.log.debug(\u0027preparing to update config to %r\u0027, config)"},{"line_number":406,"context_line":""}],"source_content_type":"text/x-python","patch_set":4,"id":"5a710552_7669f824","line":403,"updated":"2015-12-15 22:14:32.000000000","message":"Since we are passing the worker_context to build_config, what about having build_config itself set the orchestrator attribute?","commit_id":"336d79f85f3828e98e36f06fcf25b41d665b2d22"},{"author":{"_account_id":1420,"name":"Adam Gandelman","email":"adamg@ubuntu.com","username":"gandelman-a"},"change_message_id":"eb0c53d11ef6d03d5a72d829b9ecda277b4fc4c6","unresolved":false,"context_lines":[{"line_number":400,"context_line":""},{"line_number":401,"context_line":"        # send data about the orchestrator that owns the appliance"},{"line_number":402,"context_line":"        # this can be extended in the future as needed"},{"line_number":403,"context_line":"        config[\u0027orchestrator\u0027] \u003d worker_context.config"},{"line_number":404,"context_line":""},{"line_number":405,"context_line":"        self.log.debug(\u0027preparing to update config to %r\u0027, config)"},{"line_number":406,"context_line":""}],"source_content_type":"text/x-python","patch_set":4,"id":"5a710552_7ccb1254","line":403,"in_reply_to":"5a710552_7669f824","updated":"2015-12-16 01:17:07.000000000","message":"Yea, I think I like that better actually. Thanks. The idea here is to tack on orchestrator-specific config bits into the config we push to appliances. I think in the future, non-astara appliances probably wont know anything about or care about orchestrator config, and its entirely specific to the astara-appliance (and really only for metadata proxy currently) .  Makes sense to isolate that into the driver, too","commit_id":"336d79f85f3828e98e36f06fcf25b41d665b2d22"}],"astara/metadata.py":[{"author":{"_account_id":2592,"name":"Mark McClain","email":"mark@mcclain.xyz","username":"markmcclain"},"change_message_id":"ed8c7e3d6cd4bebe4827bdde00a08b24392b14a4","unresolved":false,"context_lines":[{"line_number":48,"context_line":"    cfg.IntOpt(\u0027astara_metadata_port\u0027,"},{"line_number":49,"context_line":"               default\u003d8775,"},{"line_number":50,"context_line":"               help\u003d\"TCP Port used by Nova metadata server.\"),"},{"line_number":51,"context_line":"    cfg.IntOpt(\u0027nova_metadata_port\u0027,"},{"line_number":52,"context_line":"               default\u003d9697,"},{"line_number":53,"context_line":"               help\u003d\"TCP listening port used by Astara metadata proxy.\"),"},{"line_number":54,"context_line":"    cfg.StrOpt(\u0027neutron_metadata_proxy_shared_secret\u0027,"}],"source_content_type":"text/x-python","patch_set":5,"id":"3a7e1126_cec335ec","line":51,"range":{"start_line":51,"start_character":16,"end_line":51,"end_character":34},"updated":"2015-12-18 16:49:46.000000000","message":"This is confusing.  Seems like the option names are reversed. nova \u003e 8775 and astara \u003e 9697","commit_id":"5ff3d7de34508268c8ee67eb5cb81a2ad85fe44b"},{"author":{"_account_id":2592,"name":"Mark McClain","email":"mark@mcclain.xyz","username":"markmcclain"},"change_message_id":"0f90e4d2d997b2ffc38bb901124f0afbcb446d8b","unresolved":false,"context_lines":[{"line_number":15,"context_line":"# under the License."},{"line_number":16,"context_line":""},{"line_number":17,"context_line":""},{"line_number":18,"context_line":"# vim: tabstop\u003d4 shiftwidth\u003d4 softtabstop\u003d4"},{"line_number":19,"context_line":"#"},{"line_number":20,"context_line":"# Copyright 2012 New Dream Network, LLC (DreamHost)"},{"line_number":21,"context_line":"#"}],"source_content_type":"text/x-python","patch_set":11,"id":"9a68dd71_61c48390","side":"PARENT","line":18,"updated":"2016-01-25 17:06:23.000000000","message":"Actually, this is correct copyright since this file was originally written for Neutron.","commit_id":"d82458ec17907c2373c63ed830b17f9e40a31ae3"}],"astara/test/unit/common/test_linux_ip_lib.py":[{"author":{"_account_id":2592,"name":"Mark McClain","email":"mark@mcclain.xyz","username":"markmcclain"},"change_message_id":"ed8c7e3d6cd4bebe4827bdde00a08b24392b14a4","unresolved":false,"context_lines":[{"line_number":36,"context_line":""},{"line_number":37,"context_line":"import mock"},{"line_number":38,"context_line":""},{"line_number":39,"context_line":"from astara.test.unit import base"},{"line_number":40,"context_line":"from astara.common.linux import ip_lib"},{"line_number":41,"context_line":""},{"line_number":42,"context_line":""}],"source_content_type":"text/x-python","patch_set":5,"id":"3a7e1126_0e7dbdb0","line":39,"updated":"2015-12-18 16:49:46.000000000","message":"Can the changes in this file be done in a separate change?  That way we can backport this stable.","commit_id":"5ff3d7de34508268c8ee67eb5cb81a2ad85fe44b"}],"astara/test/unit/test_instance_manager.py":[{"author":{"_account_id":2592,"name":"Mark McClain","email":"mark@mcclain.xyz","username":"markmcclain"},"change_message_id":"ed8c7e3d6cd4bebe4827bdde00a08b24392b14a4","unresolved":false,"context_lines":[{"line_number":496,"context_line":""},{"line_number":497,"context_line":"    def test_configure_success(self):"},{"line_number":498,"context_line":"        fake_config_dict \u003d {\u0027fake_config\u0027: \u0027foo\u0027}"},{"line_number":499,"context_line":"        self.fake_driver.build_config.return_value \u003d copy(fake_config_dict)"},{"line_number":500,"context_line":"        self.config(astara_metadata_port\u003d4321)"},{"line_number":501,"context_line":"        self.config(host\u003d\u0027foobarhost\u0027)"},{"line_number":502,"context_line":"        with mock.patch.object(self.instance_mgr,"}],"source_content_type":"text/x-python","patch_set":5,"id":"3a7e1126_910c6c3c","line":499,"range":{"start_line":499,"start_character":53,"end_line":499,"end_character":57},"updated":"2015-12-18 16:49:46.000000000","message":"copy() isn\u0027t needed here can use dict\u0027s init to make the shallow copy.\n\ndict(fake_config_dict)","commit_id":"5ff3d7de34508268c8ee67eb5cb81a2ad85fe44b"}]}