)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":6796,"name":"Giulio Fidente","email":"gfidente@redhat.com","username":"gfidente"},"change_message_id":"a3528201ccb40cbaa2ee46e8734ed006e131d009","unresolved":true,"context_lines":[{"line_number":4,"context_line":"Commit:     Giulio Fidente \u003cgfidente@redhat.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2022-07-15 10:38:55 +0200"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Add config option for algo to use in temp_url tests"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"This is so to configure which hashlib algorithm should be used for"},{"line_number":10,"context_line":"the temp_url tests, defaulting to sha256 but preserving the"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"a3c2d700_e897880e","line":7,"updated":"2022-07-15 09:47:06.000000000","message":"fwiw, I like more the other implementation at [1] because it doesn\u0027t need a config option and just tries to use a stronger algo when available\n\nour purpose is not to see if swift implemented correctly digest with all available algos, but just to find a good one to run the tests\n\n1. https://review.opendev.org/c/openstack/tempest/+/849857","commit_id":"95758b165dd13893250d68d4617cc51095aabfb7"}],"/PATCHSET_LEVEL":[{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"89b1a52ab69bfe59dce306d16f61eaa623eb617b","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"6910bd94_6ed282ee","updated":"2022-07-14 19:55:54.000000000","message":"also add a release note about this new config. change lgtm","commit_id":"ec4a5d0f40b16d37bb01495e29f5dfbb213d1887"},{"author":{"_account_id":16137,"name":"Tobias Urdin","email":"tobias.urdin@binero.com","username":"tobasco"},"change_message_id":"7c8ac38db4628d21388d7fbdb2f735e38921def3","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"64b65065_f99ec4f0","updated":"2023-03-31 07:44:52.000000000","message":"Thanks for also adding a config opt for backward compatibility, this hit Ceph which meant we could start track adding SHA256 as a new feature but still keep CI running with SHA1 by setting the config option 😊","commit_id":"95758b165dd13893250d68d4617cc51095aabfb7"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"4fb06de30cdd48f88f66becf331d1f20a3810a51","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"1949a4ad_63a7d0a1","updated":"2022-07-15 13:54:11.000000000","message":"thanks","commit_id":"95758b165dd13893250d68d4617cc51095aabfb7"}],"tempest/config.py":[{"author":{"_account_id":11075,"name":"Benny Kopilov","email":"bkopilov@redhat.com","username":"bkopilov"},"change_message_id":"0e3201e85da322708396b9967a74e39cd3455d1f","unresolved":true,"context_lines":[{"line_number":1166,"context_line":"                help\u003d\"Execute discoverability tests\"),"},{"line_number":1167,"context_line":"    cfg.StrOpt(\u0027tempurl_digest_hashlib\u0027,"},{"line_number":1168,"context_line":"               default\u003d\u0027sha256\u0027,"},{"line_number":1169,"context_line":"               help\u003d\"Hashing algorithm to use for the temp_url tests\"),"},{"line_number":1170,"context_line":"]"},{"line_number":1171,"context_line":""},{"line_number":1172,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"14c3bc26_005bab53","line":1169,"updated":"2022-07-14 16:50:41.000000000","message":"i would add here known options because it use getattr(...) ,\n[\u0027sha1\u0027, \u0027sha256\u0027, \u0027sha512\u0027]","commit_id":"ec4a5d0f40b16d37bb01495e29f5dfbb213d1887"},{"author":{"_account_id":6796,"name":"Giulio Fidente","email":"gfidente@redhat.com","username":"gfidente"},"change_message_id":"dbad3d6bee0a5681f7a2d84427c30e7a338f5c3b","unresolved":false,"context_lines":[{"line_number":1166,"context_line":"                help\u003d\"Execute discoverability tests\"),"},{"line_number":1167,"context_line":"    cfg.StrOpt(\u0027tempurl_digest_hashlib\u0027,"},{"line_number":1168,"context_line":"               default\u003d\u0027sha256\u0027,"},{"line_number":1169,"context_line":"               help\u003d\"Hashing algorithm to use for the temp_url tests\"),"},{"line_number":1170,"context_line":"]"},{"line_number":1171,"context_line":""},{"line_number":1172,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"24547f50_43fbf708","line":1169,"in_reply_to":"021bf342_fce010c6","updated":"2022-07-15 09:45:20.000000000","message":"Done","commit_id":"ec4a5d0f40b16d37bb01495e29f5dfbb213d1887"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"89b1a52ab69bfe59dce306d16f61eaa623eb617b","unresolved":true,"context_lines":[{"line_number":1166,"context_line":"                help\u003d\"Execute discoverability tests\"),"},{"line_number":1167,"context_line":"    cfg.StrOpt(\u0027tempurl_digest_hashlib\u0027,"},{"line_number":1168,"context_line":"               default\u003d\u0027sha256\u0027,"},{"line_number":1169,"context_line":"               help\u003d\"Hashing algorithm to use for the temp_url tests\"),"},{"line_number":1170,"context_line":"]"},{"line_number":1171,"context_line":""},{"line_number":1172,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"021bf342_fce010c6","line":1169,"in_reply_to":"14c3bc26_005bab53","updated":"2022-07-14 19:55:54.000000000","message":"+1","commit_id":"ec4a5d0f40b16d37bb01495e29f5dfbb213d1887"}]}