)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":28619,"name":"Dmitriy Rabotyagov","email":"noonedeadpunk@gmail.com","username":"noonedeadpunk"},"change_message_id":"f2ecffd8f8c4a8e375378a2b5f89822cd98a6afd","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"7ea484fc_f35dc3b3","updated":"2026-08-24 15:02:18.000000000","message":"Hey!\n\nDo we need to also update pymongo requirement for the project?\nhttps://opendev.org/openstack/freezer/src/branch/master/requirements.txt#L26","commit_id":"998ebe07b7c30b6ebcc02dfbadb2246156059196"},{"author":{"_account_id":39376,"name":"Jitendra Mahari","display_name":"jitendra","email":"jitendra.mahari@rackspace.com","username":"jitenubuntu1988"},"change_message_id":"f71991fe94ea66d642fbcfb0e0ef15e2db2b2e92","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"a4666e19_0100f075","in_reply_to":"7ea484fc_f35dc3b3","updated":"2026-08-24 16:17:39.000000000","message":"We don\u0027t need to change the dependency because PyMongo 4.x is already allowed by the current requirements, and this code fix makes the MongoDB connection work correctly without changing which PyMongo versions Freezer supports.","commit_id":"998ebe07b7c30b6ebcc02dfbadb2246156059196"},{"author":{"_account_id":28619,"name":"Dmitriy Rabotyagov","email":"noonedeadpunk@gmail.com","username":"noonedeadpunk"},"change_message_id":"fa45f3fc74832ea40793d1f5cd7c88f8f616f593","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"305631a6_44a7b28d","in_reply_to":"a4666e19_0100f075","updated":"2026-08-25 07:57:02.000000000","message":"I was more wondering if it still works correctly for PyMongo\u003c4.0 or it breaks PyMongo\u003d\u003d3.0.2 for instance.\n\nBut checking through the docs, it looks like being already supported, according to examples:\nhttps://github.com/mongodb/mongo-python-driver/blob/3.0.1/doc/examples/authentication.rst","commit_id":"998ebe07b7c30b6ebcc02dfbadb2246156059196"},{"author":{"_account_id":39376,"name":"Jitendra Mahari","display_name":"jitendra","email":"jitendra.mahari@rackspace.com","username":"jitenubuntu1988"},"change_message_id":"a44d4d72be9c977e486fd617697420cb0c17e3c1","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"8239fa8d_3131b486","updated":"2026-08-26 09:00:07.000000000","message":"Hi @noonedeadpunk@gmail.com,just following up on this patch. Could you please add Workflow +1 if it looks ready to merge or is it required review from other team members as well? \n\nThanks!","commit_id":"64645a3fb473ec50e6fe37938f50adb927d1927d"},{"author":{"_account_id":28619,"name":"Dmitriy Rabotyagov","email":"noonedeadpunk@gmail.com","username":"noonedeadpunk"},"change_message_id":"fa45f3fc74832ea40793d1f5cd7c88f8f616f593","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"3145301f_5fb49632","updated":"2026-08-25 07:57:02.000000000","message":"lgtm","commit_id":"64645a3fb473ec50e6fe37938f50adb927d1927d"},{"author":{"_account_id":28619,"name":"Dmitriy Rabotyagov","email":"noonedeadpunk@gmail.com","username":"noonedeadpunk"},"change_message_id":"690910465841c4fb32f8ade2bac80bf7a1f6c645","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"eceb6c06_2a82eea2","in_reply_to":"8239fa8d_3131b486","updated":"2026-08-27 08:13:37.000000000","message":"it would be great to have another review. We\u0027re somehow short on reviewers and I am trying to onboard new people, so will wait for couple of days first to give them an opportunity.","commit_id":"64645a3fb473ec50e6fe37938f50adb927d1927d"}],"freezer/mode/mongo.py":[{"author":{"_account_id":28619,"name":"Dmitriy Rabotyagov","email":"noonedeadpunk@gmail.com","username":"noonedeadpunk"},"change_message_id":"c4abd1db9932c9bea483686aab2cfcf26061738c","unresolved":true,"context_lines":[{"line_number":49,"context_line":"        # todo unhardcode this"},{"line_number":50,"context_line":"        mongodb_port \u003d \u002727017\u0027"},{"line_number":51,"context_line":"        local_hostname \u003d conf.hostname"},{"line_number":52,"context_line":"        db_host_port \u003d \u0027{0}:{1}\u0027.format(local_hostname, mongodb_port)"},{"line_number":53,"context_line":"        # Use a valid MongoDB connection URI. pymongo 4.x rejects a bare"},{"line_number":54,"context_line":"        # \"host:port\" string with \"[Errno 22] Invalid argument\"; the"},{"line_number":55,"context_line":"        # \"mongodb://\" scheme is required for it to be parsed correctly."}],"source_content_type":"text/x-python","patch_set":1,"id":"ebd5fada_f3a73454","line":52,"updated":"2026-08-24 15:04:21.000000000","message":"any reason not to add `mongodb://` here and instead do another formatting?\nI think it would be more efficient to combine a string once?","commit_id":"998ebe07b7c30b6ebcc02dfbadb2246156059196"},{"author":{"_account_id":28619,"name":"Dmitriy Rabotyagov","email":"noonedeadpunk@gmail.com","username":"noonedeadpunk"},"change_message_id":"fa45f3fc74832ea40793d1f5cd7c88f8f616f593","unresolved":false,"context_lines":[{"line_number":49,"context_line":"        # todo unhardcode this"},{"line_number":50,"context_line":"        mongodb_port \u003d \u002727017\u0027"},{"line_number":51,"context_line":"        local_hostname \u003d conf.hostname"},{"line_number":52,"context_line":"        db_host_port \u003d \u0027{0}:{1}\u0027.format(local_hostname, mongodb_port)"},{"line_number":53,"context_line":"        # Use a valid MongoDB connection URI. pymongo 4.x rejects a bare"},{"line_number":54,"context_line":"        # \"host:port\" string with \"[Errno 22] Invalid argument\"; the"},{"line_number":55,"context_line":"        # \"mongodb://\" scheme is required for it to be parsed correctly."}],"source_content_type":"text/x-python","patch_set":1,"id":"4c8bebd9_af49bd4e","line":52,"in_reply_to":"661f544f_04faa7e7","updated":"2026-08-25 07:57:02.000000000","message":"Done","commit_id":"998ebe07b7c30b6ebcc02dfbadb2246156059196"},{"author":{"_account_id":39376,"name":"Jitendra Mahari","display_name":"jitendra","email":"jitendra.mahari@rackspace.com","username":"jitenubuntu1988"},"change_message_id":"f71991fe94ea66d642fbcfb0e0ef15e2db2b2e92","unresolved":true,"context_lines":[{"line_number":49,"context_line":"        # todo unhardcode this"},{"line_number":50,"context_line":"        mongodb_port \u003d \u002727017\u0027"},{"line_number":51,"context_line":"        local_hostname \u003d conf.hostname"},{"line_number":52,"context_line":"        db_host_port \u003d \u0027{0}:{1}\u0027.format(local_hostname, mongodb_port)"},{"line_number":53,"context_line":"        # Use a valid MongoDB connection URI. pymongo 4.x rejects a bare"},{"line_number":54,"context_line":"        # \"host:port\" string with \"[Errno 22] Invalid argument\"; the"},{"line_number":55,"context_line":"        # \"mongodb://\" scheme is required for it to be parsed correctly."}],"source_content_type":"text/x-python","patch_set":1,"id":"661f544f_04faa7e7","line":52,"in_reply_to":"ebd5fada_f3a73454","updated":"2026-08-24 16:17:39.000000000","message":"Agreed. Updated it to construct the MongoDB URI in a single format operation and pass it directly to MongoClient.","commit_id":"998ebe07b7c30b6ebcc02dfbadb2246156059196"}],"freezer/tests/unit/mode/test_mongo.py":[{"author":{"_account_id":28619,"name":"Dmitriy Rabotyagov","email":"noonedeadpunk@gmail.com","username":"noonedeadpunk"},"change_message_id":"c4abd1db9932c9bea483686aab2cfcf26061738c","unresolved":true,"context_lines":[{"line_number":36,"context_line":"            \u0027primary\u0027: \u0027test-host:27017\u0027,"},{"line_number":37,"context_line":"        }"},{"line_number":38,"context_line":""},{"line_number":39,"context_line":"        from freezer.mode import mongo"},{"line_number":40,"context_line":"        mongo.MongoMode(FakeConf())"},{"line_number":41,"context_line":""},{"line_number":42,"context_line":"        # MongoClient must be called once with a mongodb:// URI."}],"source_content_type":"text/x-python","patch_set":1,"id":"c7e1a8ef_3f14966a","line":39,"updated":"2026-08-24 15:04:21.000000000","message":"I think we should do the import in the beginning of the file","commit_id":"998ebe07b7c30b6ebcc02dfbadb2246156059196"},{"author":{"_account_id":39376,"name":"Jitendra Mahari","display_name":"jitendra","email":"jitendra.mahari@rackspace.com","username":"jitenubuntu1988"},"change_message_id":"f71991fe94ea66d642fbcfb0e0ef15e2db2b2e92","unresolved":true,"context_lines":[{"line_number":36,"context_line":"            \u0027primary\u0027: \u0027test-host:27017\u0027,"},{"line_number":37,"context_line":"        }"},{"line_number":38,"context_line":""},{"line_number":39,"context_line":"        from freezer.mode import mongo"},{"line_number":40,"context_line":"        mongo.MongoMode(FakeConf())"},{"line_number":41,"context_line":""},{"line_number":42,"context_line":"        # MongoClient must be called once with a mongodb:// URI."}],"source_content_type":"text/x-python","patch_set":1,"id":"f21626ef_c59b47ea","line":39,"in_reply_to":"c7e1a8ef_3f14966a","updated":"2026-08-24 16:17:39.000000000","message":"Agreed. Moved the mongo import to the module-level imports.","commit_id":"998ebe07b7c30b6ebcc02dfbadb2246156059196"},{"author":{"_account_id":28619,"name":"Dmitriy Rabotyagov","email":"noonedeadpunk@gmail.com","username":"noonedeadpunk"},"change_message_id":"fa45f3fc74832ea40793d1f5cd7c88f8f616f593","unresolved":false,"context_lines":[{"line_number":36,"context_line":"            \u0027primary\u0027: \u0027test-host:27017\u0027,"},{"line_number":37,"context_line":"        }"},{"line_number":38,"context_line":""},{"line_number":39,"context_line":"        from freezer.mode import mongo"},{"line_number":40,"context_line":"        mongo.MongoMode(FakeConf())"},{"line_number":41,"context_line":""},{"line_number":42,"context_line":"        # MongoClient must be called once with a mongodb:// URI."}],"source_content_type":"text/x-python","patch_set":1,"id":"c1e511a3_d2229703","line":39,"in_reply_to":"f21626ef_c59b47ea","updated":"2026-08-25 07:57:02.000000000","message":"Done","commit_id":"998ebe07b7c30b6ebcc02dfbadb2246156059196"}]}