)]}'
{"gertty/sync.py":[{"author":{"_account_id":1,"name":"James E. Blair","email":"jim@acmegating.com","username":"corvus"},"change_message_id":"1aec4ae809a4d4ab52a782610e2f28887f5c990f","unresolved":false,"context_lines":[{"line_number":781,"context_line":"        # If the user has specified a git url, use it instead"},{"line_number":782,"context_line":"        # of what Gerrit supplies."},{"line_number":783,"context_line":"        if sync.app.config.git_url_supplied:"},{"line_number":784,"context_line":"            url \u003d sync.app.config.git_url + self.project.name"},{"line_number":785,"context_line":"        self.log.debug(\"git fetch %s %s\" % (url, self.refs))"},{"line_number":786,"context_line":"        repo \u003d sync.app.getRepo(self.project_name)"},{"line_number":787,"context_line":"        refs \u003d [\u0027+%(ref)s:%(ref)s\u0027 % dict(ref\u003dref) for ref in self.refs]"}],"source_content_type":"text/x-python","patch_set":2,"id":"baa041b7_3a4611c0","line":784,"updated":"2015-04-01 15:45:02.000000000","message":"It occurs to me that while we now support a lot of fetch methods when syncing the change, we no longer keep enough information about them in the db for this process of fetching missing refs to work.  For instance, if a user _does not_ configure a git-url, then the sync task will work with an ssh url because gerrit supplies it, but we don\u0027t record the ssh url, so this task will fail if it needs to run.\n\nI think we could go with Cristoph\u0027s approach and have the autoconfiguration of the git url work only when using http (if you want another method to work, you must configure git-url), or we could save more information to the database.","commit_id":"faf7fa31f9edff570d9b2fad075d803457a82bd7"}]}