)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":3009,"name":"Vladimir Kozhukalov","email":"kozhukalov@gmail.com","username":"kozhukalov"},"change_message_id":"3a979890471fa02c09901a48c337d764d796e7f5","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":18,"id":"3b8ad0a6_89096ac3","updated":"2023-08-24 19:31:52.000000000","message":"recheck","commit_id":"3449896e379ec4b173a2c44de9a6de390dd9d5e4"},{"author":{"_account_id":35691,"name":"Sadegh Hayeri","display_name":"Sadegh Hayeri","email":"hayerisadegh@gmail.com","username":"sadeghhayeri"},"change_message_id":"716886d48e891a600ae3b48bcad6d70c2db90816","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":23,"id":"c78c5cfd_5a973b46","updated":"2023-08-29 06:51:26.000000000","message":"I wonder why somewhere we used `ingressClassName: {{ index $envAll.Values.network $backendService \"ingress\" \"classes\" \"namespace\" | quote }}` for annotations\n\nwhile somewhere else we hard coded `nginx` or `nginx-cluster`.","commit_id":"51c70e48dff173281a77d374d87af2c49caa6348"},{"author":{"_account_id":34821,"name":"Schubert Anselme","email":"sa246v@att.com","username":"sa246v"},"change_message_id":"3de45fc83cfa58e053a708c3ea3ab1fd712c686f","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":23,"id":"5109057f_a56e3ec8","updated":"2023-09-15 12:45:00.000000000","message":"recheck","commit_id":"51c70e48dff173281a77d374d87af2c49caa6348"},{"author":{"_account_id":3009,"name":"Vladimir Kozhukalov","email":"kozhukalov@gmail.com","username":"kozhukalov"},"change_message_id":"403f58ca58eb471190aa72e2fd573ff630e636a2","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":23,"id":"70faa1ab_9be34af5","updated":"2023-09-15 19:06:13.000000000","message":"recheck","commit_id":"51c70e48dff173281a77d374d87af2c49caa6348"},{"author":{"_account_id":3009,"name":"Vladimir Kozhukalov","email":"kozhukalov@gmail.com","username":"kozhukalov"},"change_message_id":"b5e731f4454e24694e09ed2f0b26e5da94b3a8b4","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":23,"id":"e979575c_76c77d9e","updated":"2023-09-16 02:47:09.000000000","message":"recheck","commit_id":"51c70e48dff173281a77d374d87af2c49caa6348"},{"author":{"_account_id":34821,"name":"Schubert Anselme","email":"sa246v@att.com","username":"sa246v"},"change_message_id":"209a06a508e668918f1d8c4d1f7442e72572aee7","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":23,"id":"d39f6f6d_5e2aa33b","in_reply_to":"c78c5cfd_5a973b46","updated":"2023-09-06 18:25:07.000000000","message":"By default, if no ingressClassName is provided when deploying the ingress chart, we create IngressClass with name nginx.\nFor osh, we use nginx-cluster for the cluster ingress (and appropriate names for namesapced IngressClass)","commit_id":"51c70e48dff173281a77d374d87af2c49caa6348"}],"helm-toolkit/templates/manifests/_ingress.tpl":[{"author":{"_account_id":3009,"name":"Vladimir Kozhukalov","email":"kozhukalov@gmail.com","username":"kozhukalov"},"change_message_id":"434b02ecb708becb0fde5e774dace1f7b325b3c4","unresolved":true,"context_lines":[{"line_number":67,"context_line":"      metadata:"},{"line_number":68,"context_line":"        name: barbican"},{"line_number":69,"context_line":"        annotations:"},{"line_number":70,"context_line":"          spec.ingressClassName: \"nginx\""},{"line_number":71,"context_line":"          nginx.ingress.kubernetes.io/rewrite-target: /"},{"line_number":72,"context_line":""},{"line_number":73,"context_line":"      spec:"}],"source_content_type":"text/x-smarty","patch_set":3,"id":"2b4a2f4f_15958ecf","line":70,"updated":"2023-08-17 22:17:33.000000000","message":"I think it should be a part of Ingress spec.\n```\napiVersion: networking.k8s.io/v1\nkind: Ingress\nmetadata:\n  name: \u003cname\u003e\nspec:\n  ingressClassName: \"nginx\"\n......\n```","commit_id":"17e1e524e0be3b6562714095ab2418061be00679"},{"author":{"_account_id":34520,"name":"Sergiy Markin","email":"smarkin@mirantis.com","username":"sm515x"},"change_message_id":"fcd42c3533e6f3918f415213c340ebc5a287ef86","unresolved":false,"context_lines":[{"line_number":67,"context_line":"      metadata:"},{"line_number":68,"context_line":"        name: barbican"},{"line_number":69,"context_line":"        annotations:"},{"line_number":70,"context_line":"          spec.ingressClassName: \"nginx\""},{"line_number":71,"context_line":"          nginx.ingress.kubernetes.io/rewrite-target: /"},{"line_number":72,"context_line":""},{"line_number":73,"context_line":"      spec:"}],"source_content_type":"text/x-smarty","patch_set":3,"id":"70adb70c_f125cde5","line":70,"in_reply_to":"2b4a2f4f_15958ecf","updated":"2023-08-17 23:30:31.000000000","message":"Done","commit_id":"17e1e524e0be3b6562714095ab2418061be00679"}]}