)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":935,"name":"James Page","email":"ringo.page@gmail.com","username":"james-page"},"change_message_id":"d894e58b9ac37e1c7047dbde0ab9d4bea2b54ba5","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":16,"id":"3f040764_27166041","updated":"2023-01-12 16:46:21.000000000","message":"+1 but please can you tidy the commented out lines in the test-requirements.txt","commit_id":"a5fe3264750a7e6467ed78f45ebd81ef7ab52f8a"},{"author":{"_account_id":10366,"name":"Hemanth N","email":"hemanth.nakkina@canonical.com","username":"Hemanth"},"change_message_id":"9c5234cae7b4e7fc34d60bb6a3c30df250afa5d4","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":16,"id":"454f1200_003c0924","in_reply_to":"3f040764_27166041","updated":"2023-01-13 02:29:25.000000000","message":"Done","commit_id":"a5fe3264750a7e6467ed78f45ebd81ef7ab52f8a"}],"test-requirements.txt":[{"author":{"_account_id":935,"name":"James Page","email":"ringo.page@gmail.com","username":"james-page"},"change_message_id":"39d69c676770e2d9c26048305c1384e6042184cc","unresolved":true,"context_lines":[{"line_number":1,"context_line":"# coverage\u003e\u003d3.6"},{"line_number":2,"context_line":"# mock\u003e\u003d1.2"},{"line_number":3,"context_line":"# stestr\u003e\u003d2.2.0"},{"line_number":4,"context_line":"# requests\u003e\u003d2.18.4"},{"line_number":5,"context_line":""},{"line_number":6,"context_line":"coverage"},{"line_number":7,"context_line":"mock"}],"source_content_type":"text/plain","patch_set":16,"id":"81507fea_89aea5ad","line":4,"updated":"2023-01-12 13:47:58.000000000","message":"Any reason for dropping the minimum versions?","commit_id":"a5fe3264750a7e6467ed78f45ebd81ef7ab52f8a"},{"author":{"_account_id":10366,"name":"Hemanth N","email":"hemanth.nakkina@canonical.com","username":"Hemanth"},"change_message_id":"6381f19a644eab601b22ad2df9f59fd6a8f558b6","unresolved":true,"context_lines":[{"line_number":1,"context_line":"# coverage\u003e\u003d3.6"},{"line_number":2,"context_line":"# mock\u003e\u003d1.2"},{"line_number":3,"context_line":"# stestr\u003e\u003d2.2.0"},{"line_number":4,"context_line":"# requests\u003e\u003d2.18.4"},{"line_number":5,"context_line":""},{"line_number":6,"context_line":"coverage"},{"line_number":7,"context_line":"mock"}],"source_content_type":"text/plain","patch_set":16,"id":"e5db8a4a_aa28dfc9","line":4,"in_reply_to":"81507fea_89aea5ad","updated":"2023-01-12 14:49:30.000000000","message":"The minimum versions mentioned above are pretty much old package versions which might not support python 3.8, python 3.10. So removed them, we need to revisit minimum package versions.","commit_id":"a5fe3264750a7e6467ed78f45ebd81ef7ab52f8a"},{"author":{"_account_id":935,"name":"James Page","email":"ringo.page@gmail.com","username":"james-page"},"change_message_id":"39d69c676770e2d9c26048305c1384e6042184cc","unresolved":true,"context_lines":[{"line_number":12,"context_line":"# oslo.i18n dropped py35 support"},{"line_number":13,"context_line":"# oslo.i18n\u003c4.0.0"},{"line_number":14,"context_line":"# pytz    # workaround for 14.04 pip/tox"},{"line_number":15,"context_line":"# pyudev  # for ceph-* charm unit tests (not mocked?)"}],"source_content_type":"text/plain","patch_set":16,"id":"8651f4cf_c0e43189","line":15,"updated":"2023-01-12 13:47:58.000000000","message":"Are these dropped due to the switch to the new interface? I know the old interface had depends on charmhelpers etc?\n\nIf so lets just delete them :)","commit_id":"a5fe3264750a7e6467ed78f45ebd81ef7ab52f8a"},{"author":{"_account_id":10366,"name":"Hemanth N","email":"hemanth.nakkina@canonical.com","username":"Hemanth"},"change_message_id":"6381f19a644eab601b22ad2df9f59fd6a8f558b6","unresolved":true,"context_lines":[{"line_number":12,"context_line":"# oslo.i18n dropped py35 support"},{"line_number":13,"context_line":"# oslo.i18n\u003c4.0.0"},{"line_number":14,"context_line":"# pytz    # workaround for 14.04 pip/tox"},{"line_number":15,"context_line":"# pyudev  # for ceph-* charm unit tests (not mocked?)"}],"source_content_type":"text/plain","patch_set":16,"id":"44ee88bb_a1676024","line":15,"in_reply_to":"8651f4cf_c0e43189","updated":"2023-01-12 14:49:30.000000000","message":"Yes these are not required for running charm-ops-sunbeam tests.","commit_id":"a5fe3264750a7e6467ed78f45ebd81ef7ab52f8a"}]}
