)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":21776,"name":"Jerry Sun","email":"jerry.sun@windriver.com","username":"jerrysun"},"change_message_id":"1ebd9f3e8c39c5763c909d841c49081dba039d92","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"486c935a_8b47e2fc","updated":"2022-06-17 18:02:14.000000000","message":"i tested this too. lgtm","commit_id":"0dacd737b2c7dd22aef4dfd053d51a5030de3ea3"},{"author":{"_account_id":31646,"name":"Carmen Rata","email":"carmen.rata@windriver.com","username":"crata"},"change_message_id":"242dedb68fdf08e87986d5f08b48a6761b4a871f","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"dd1e1f32_72b19777","in_reply_to":"486c935a_8b47e2fc","updated":"2022-06-17 18:09:02.000000000","message":"Thank you Jerry","commit_id":"0dacd737b2c7dd22aef4dfd053d51a5030de3ea3"}],"controllerconfig/controllerconfig/upgrade-scripts/68-pod-security-admission-controller-labels.py":[{"author":{"_account_id":15435,"name":"Al Bailey","email":"albailey1974@gmail.com","username":"albailey"},"change_message_id":"d2040e87080fa27f41fe22e74c3c7352c272a9ca","unresolved":true,"context_lines":[{"line_number":57,"context_line":"            continue"},{"line_number":58,"context_line":""},{"line_number":59,"context_line":"        security_version \u003d \u0027latest\u0027"},{"line_number":60,"context_line":"        if namespace in common.PRIVILEGED_NS:"},{"line_number":61,"context_line":"            security_level \u003d \u0027privileged\u0027"},{"line_number":62,"context_line":""},{"line_number":63,"context_line":"        try:"}],"source_content_type":"text/x-python","patch_set":2,"id":"dc2f2531_0c7ea666","line":60,"updated":"2022-06-17 19:24:24.000000000","message":"why do we need this line .  based on line line 56 above, wont this always be true?\nthe old code needed this because of the AND statement that has now been removed from line 56","commit_id":"0dacd737b2c7dd22aef4dfd053d51a5030de3ea3"},{"author":{"_account_id":31646,"name":"Carmen Rata","email":"carmen.rata@windriver.com","username":"crata"},"change_message_id":"491c07901d88d57a961e686eb967f3badb804823","unresolved":false,"context_lines":[{"line_number":57,"context_line":"            continue"},{"line_number":58,"context_line":""},{"line_number":59,"context_line":"        security_version \u003d \u0027latest\u0027"},{"line_number":60,"context_line":"        if namespace in common.PRIVILEGED_NS:"},{"line_number":61,"context_line":"            security_level \u003d \u0027privileged\u0027"},{"line_number":62,"context_line":""},{"line_number":63,"context_line":"        try:"}],"source_content_type":"text/x-python","patch_set":2,"id":"a1de9334_d9958392","line":60,"in_reply_to":"dc2f2531_0c7ea666","updated":"2022-06-18 00:31:53.000000000","message":"Scenario for upgrades is special because deals with all namespaces, application or not. With this commit we accommodate a transitioning to full support of PSA feature coming later this year.","commit_id":"0dacd737b2c7dd22aef4dfd053d51a5030de3ea3"}],"sysinv/sysinv/sysinv/sysinv/helm/lifecycle_utils.py":[{"author":{"_account_id":9926,"name":"John Kung","email":"john.kung@windriver.com","username":"jkung"},"change_message_id":"8d94cf6de855ed142b8b8254cb2201ee74ae3e51","unresolved":true,"context_lines":[{"line_number":221,"context_line":""},{"line_number":222,"context_line":"        if ns in common.PRIVILEGED_NS:"},{"line_number":223,"context_line":"            security_level \u003d \u0027privileged\u0027"},{"line_number":224,"context_line":""},{"line_number":225,"context_line":"        body \u003d {"},{"line_number":226,"context_line":"            \"metadata\": {"},{"line_number":227,"context_line":"                \"labels\": {"}],"source_content_type":"text/x-python","patch_set":2,"id":"63c4f867_57c47002","line":224,"updated":"2022-06-17 18:46:52.000000000","message":"\u0027security_level\u0027 should have a default and be reset if ns is not in common.PRIVILEGED_NS ?","commit_id":"0dacd737b2c7dd22aef4dfd053d51a5030de3ea3"},{"author":{"_account_id":31646,"name":"Carmen Rata","email":"carmen.rata@windriver.com","username":"crata"},"change_message_id":"491c07901d88d57a961e686eb967f3badb804823","unresolved":false,"context_lines":[{"line_number":221,"context_line":""},{"line_number":222,"context_line":"        if ns in common.PRIVILEGED_NS:"},{"line_number":223,"context_line":"            security_level \u003d \u0027privileged\u0027"},{"line_number":224,"context_line":""},{"line_number":225,"context_line":"        body \u003d {"},{"line_number":226,"context_line":"            \"metadata\": {"},{"line_number":227,"context_line":"                \"labels\": {"}],"source_content_type":"text/x-python","patch_set":2,"id":"58fcb517_1d54bbf6","line":224,"in_reply_to":"1e666d37_26b6290b","updated":"2022-06-18 00:31:53.000000000","message":"Done","commit_id":"0dacd737b2c7dd22aef4dfd053d51a5030de3ea3"},{"author":{"_account_id":21776,"name":"Jerry Sun","email":"jerry.sun@windriver.com","username":"jerrysun"},"change_message_id":"db851b2c278e182fa71b3b6ad068e9ded3d53a98","unresolved":true,"context_lines":[{"line_number":221,"context_line":""},{"line_number":222,"context_line":"        if ns in common.PRIVILEGED_NS:"},{"line_number":223,"context_line":"            security_level \u003d \u0027privileged\u0027"},{"line_number":224,"context_line":""},{"line_number":225,"context_line":"        body \u003d {"},{"line_number":226,"context_line":"            \"metadata\": {"},{"line_number":227,"context_line":"                \"labels\": {"}],"source_content_type":"text/x-python","patch_set":2,"id":"1e666d37_26b6290b","line":224,"in_reply_to":"63c4f867_57c47002","updated":"2022-06-17 18:51:47.000000000","message":"i think we should just get rid of the \"if ns in common.PRIVILEGED_NS:\" check. We want the default to be privileged","commit_id":"0dacd737b2c7dd22aef4dfd053d51a5030de3ea3"}]}