)]}'
{"rally/common/utils.py":[{"author":{"_account_id":12395,"name":"Roman Vasylets","email":"pomeo92@gmail.com","username":"rvasilets"},"change_message_id":"4f33d4d62a90557837e810019a7441da18d11b0e","unresolved":false,"context_lines":[{"line_number":268,"context_line":""},{"line_number":269,"context_line":"        if len(task_id_part) \u003c length:"},{"line_number":270,"context_line":"            LOG.debug(\"Task ID %(task_id)s cannot be included in a random \""},{"line_number":271,"context_line":"                      \"name because it is too short. Format: %(format)s\" %"},{"line_number":272,"context_line":"                      {\"task_id\": task_id,"},{"line_number":273,"context_line":"                       \"format\": cls.RESOURCE_NAME_FORMAT})"},{"line_number":274,"context_line":"        elif any(char not in cls.RESOURCE_NAME_ALLOWED_CHARACTERS"}],"source_content_type":"text/x-python","patch_set":4,"id":"3a57f1b5_7c2452ae","line":271,"updated":"2016-02-11 13:05:39.000000000","message":"Is it possible? uuid to sqla models as an attribute and its length always the same https://github.com/openstack/rally/blob/master/rally/common/db/sqlalchemy/models.py#L148","commit_id":"ead35de3b9a19bff11fc12c058bc968f84a807d3"},{"author":{"_account_id":11748,"name":"Chris St. Pierre","email":"cstpierr@cisco.com","username":"stpierre"},"change_message_id":"0332df21a2cd5372d8aab4c5a224519aef7f9dda","unresolved":false,"context_lines":[{"line_number":268,"context_line":""},{"line_number":269,"context_line":"        if len(task_id_part) \u003c length:"},{"line_number":270,"context_line":"            LOG.debug(\"Task ID %(task_id)s cannot be included in a random \""},{"line_number":271,"context_line":"                      \"name because it is too short. Format: %(format)s\" %"},{"line_number":272,"context_line":"                      {\"task_id\": task_id,"},{"line_number":273,"context_line":"                       \"format\": cls.RESOURCE_NAME_FORMAT})"},{"line_number":274,"context_line":"        elif any(char not in cls.RESOURCE_NAME_ALLOWED_CHARACTERS"}],"source_content_type":"text/x-python","patch_set":4,"id":"3a57f1b5_7fed1a24","line":271,"in_reply_to":"3a57f1b5_7c2452ae","updated":"2016-02-11 15:11:04.000000000","message":"Apparently it is: https://bugs.launchpad.net/rally/+bug/1509402\n\nI don\u0027t understand how, but perhaps Alexander does.","commit_id":"ead35de3b9a19bff11fc12c058bc968f84a807d3"},{"author":{"_account_id":12395,"name":"Roman Vasylets","email":"pomeo92@gmail.com","username":"rvasilets"},"change_message_id":"4f33d4d62a90557837e810019a7441da18d11b0e","unresolved":false,"context_lines":[{"line_number":275,"context_line":"                 for char in task_id_part):"},{"line_number":276,"context_line":"            LOG.debug(\"Task ID %(task_id)s cannot be included in a random \""},{"line_number":277,"context_line":"                      \"name because it includes disallowed characters. \""},{"line_number":278,"context_line":"                      \"Allowed characters: %(chars)s\" %"},{"line_number":279,"context_line":"                      {\"task_id\": task_id,"},{"line_number":280,"context_line":"                       \"chars\": cls.RESOURCE_NAME_ALLOWED_CHARACTERS})"},{"line_number":281,"context_line":"        else:"}],"source_content_type":"text/x-python","patch_set":4,"id":"3a57f1b5_a22f04f5","line":278,"updated":"2016-02-11 13:05:39.000000000","message":"Maybe you missed a verb? \"Allowed characters are:\"","commit_id":"ead35de3b9a19bff11fc12c058bc968f84a807d3"},{"author":{"_account_id":11748,"name":"Chris St. Pierre","email":"cstpierr@cisco.com","username":"stpierre"},"change_message_id":"0332df21a2cd5372d8aab4c5a224519aef7f9dda","unresolved":false,"context_lines":[{"line_number":275,"context_line":"                 for char in task_id_part):"},{"line_number":276,"context_line":"            LOG.debug(\"Task ID %(task_id)s cannot be included in a random \""},{"line_number":277,"context_line":"                      \"name because it includes disallowed characters. \""},{"line_number":278,"context_line":"                      \"Allowed characters: %(chars)s\" %"},{"line_number":279,"context_line":"                      {\"task_id\": task_id,"},{"line_number":280,"context_line":"                       \"chars\": cls.RESOURCE_NAME_ALLOWED_CHARACTERS})"},{"line_number":281,"context_line":"        else:"}],"source_content_type":"text/x-python","patch_set":4,"id":"3a57f1b5_fff58aca","line":278,"in_reply_to":"3a57f1b5_a22f04f5","updated":"2016-02-11 15:11:04.000000000","message":"This is fine as-is, but if adding the verb makes it clearer for you I\u0027ll add it. Done.","commit_id":"ead35de3b9a19bff11fc12c058bc968f84a807d3"},{"author":{"_account_id":12395,"name":"Roman Vasylets","email":"pomeo92@gmail.com","username":"rvasilets"},"change_message_id":"4f33d4d62a90557837e810019a7441da18d11b0e","unresolved":false,"context_lines":[{"line_number":326,"context_line":"            parts[\"suffix\"]])"},{"line_number":327,"context_line":""},{"line_number":328,"context_line":"    @classmethod"},{"line_number":329,"context_line":"    def name_matches_object(cls, name, task_id\u003dNone):"},{"line_number":330,"context_line":"        \"\"\"Determine if a resource name could have been created by this class."},{"line_number":331,"context_line":""},{"line_number":332,"context_line":"        :param name: The resource name to check against this class\u0027s"}],"source_content_type":"text/x-python","patch_set":4,"id":"3a57f1b5_685b2d2f","line":329,"updated":"2016-02-11 13:05:39.000000000","message":"Possibly better to rename this function. Looks like it does not what we expect from the name. As for me\u003d)","commit_id":"ead35de3b9a19bff11fc12c058bc968f84a807d3"},{"author":{"_account_id":11748,"name":"Chris St. Pierre","email":"cstpierr@cisco.com","username":"stpierre"},"change_message_id":"0332df21a2cd5372d8aab4c5a224519aef7f9dda","unresolved":false,"context_lines":[{"line_number":326,"context_line":"            parts[\"suffix\"]])"},{"line_number":327,"context_line":""},{"line_number":328,"context_line":"    @classmethod"},{"line_number":329,"context_line":"    def name_matches_object(cls, name, task_id\u003dNone):"},{"line_number":330,"context_line":"        \"\"\"Determine if a resource name could have been created by this class."},{"line_number":331,"context_line":""},{"line_number":332,"context_line":"        :param name: The resource name to check against this class\u0027s"}],"source_content_type":"text/x-python","patch_set":4,"id":"3a57f1b5_5fde1e12","line":329,"in_reply_to":"3a57f1b5_685b2d2f","updated":"2016-02-11 15:11:04.000000000","message":"Bearing in mind that rally.common.utils.name_matches_object() already exists, what would you suggest?","commit_id":"ead35de3b9a19bff11fc12c058bc968f84a807d3"},{"author":{"_account_id":6172,"name":"Boris Pavlovic","email":"boris@pavlovic.me","username":"boris-42"},"change_message_id":"e07c34d9d23b4dab58dd48b10b1fd0ffbee179cd","unresolved":false,"context_lines":[{"line_number":266,"context_line":"        # subset of the task ID"},{"line_number":267,"context_line":"        task_id_part \u003d task_id.replace(\"-\", \"\")[0:length]"},{"line_number":268,"context_line":""},{"line_number":269,"context_line":"        if len(task_id_part) \u003c length:"},{"line_number":270,"context_line":"            LOG.debug(\"Task ID %(task_id)s cannot be included in a random \""},{"line_number":271,"context_line":"                      \"name because it is too short. Format: %(format)s\" %"},{"line_number":272,"context_line":"                      {\"task_id\": task_id,"}],"source_content_type":"text/x-python","patch_set":10,"id":"5aef4532_33c0252a","line":269,"updated":"2016-03-09 16:11:00.000000000","message":"Where can this can happen? Except the wrong written tests?","commit_id":"c948375b2e97f2b8853a413e4b58d35684e53aa1"}]}
