)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":21420,"name":"Gage Hugo","email":"gagehugo@gmail.com","username":"ghugo"},"change_message_id":"57d7d6ed4a99c1a96f222701add5ebc4208a37a1","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     Rahul Khiyani \u003crahul.khiyani@att.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2020-02-15 00:20:16 +0000"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"[WIP]Add Docker default AppArmor profile to mariadb exporter chart"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Change-Id: I6d5fcbb511f4f9cdb31727421fe320beeff1a882"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":9,"id":"3fa7e38b_19051769","line":7,"range":{"start_line":7,"start_character":0,"end_line":7,"end_character":5},"updated":"2020-02-17 16:48:31.000000000","message":"Is this still WIP?","commit_id":"320a025a3a729eb1ab7420aa43aadc2300a47303"},{"author":{"_account_id":29161,"name":"Rahul Khiyani","email":"rahul.khiyani@att.com","username":"rk0850"},"change_message_id":"ae3732fb522c7f25adb9e271fa378bccc3622dc8","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     Rahul Khiyani \u003crahul.khiyani@att.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2020-02-15 00:20:16 +0000"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"[WIP]Add Docker default AppArmor profile to mariadb exporter chart"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Change-Id: I6d5fcbb511f4f9cdb31727421fe320beeff1a882"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":9,"id":"3fa7e38b_abb6ea25","line":7,"range":{"start_line":7,"start_character":0,"end_line":7,"end_character":5},"in_reply_to":"3fa7e38b_19051769","updated":"2020-02-17 17:02:01.000000000","message":"if everything looks fine and if we can merge this. I will remove WIP.","commit_id":"320a025a3a729eb1ab7420aa43aadc2300a47303"},{"author":{"_account_id":29161,"name":"Rahul Khiyani","email":"rahul.khiyani@att.com","username":"rk0850"},"change_message_id":"07e47972b7e282752a4f258f7218e8f2dab67f92","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     Rahul Khiyani \u003crahul.khiyani@att.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2020-02-15 00:20:16 +0000"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"[WIP]Add Docker default AppArmor profile to mariadb exporter chart"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Change-Id: I6d5fcbb511f4f9cdb31727421fe320beeff1a882"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":9,"id":"3fa7e38b_6b81d2fd","line":7,"range":{"start_line":7,"start_character":0,"end_line":7,"end_character":5},"in_reply_to":"3fa7e38b_8b2a0e2c","updated":"2020-02-17 17:06:02.000000000","message":"Done","commit_id":"320a025a3a729eb1ab7420aa43aadc2300a47303"},{"author":{"_account_id":21420,"name":"Gage Hugo","email":"gagehugo@gmail.com","username":"ghugo"},"change_message_id":"1843c6caa71f8671b3ff377a71836dc8f8f79713","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     Rahul Khiyani \u003crahul.khiyani@att.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2020-02-15 00:20:16 +0000"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"[WIP]Add Docker default AppArmor profile to mariadb exporter chart"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Change-Id: I6d5fcbb511f4f9cdb31727421fe320beeff1a882"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":9,"id":"3fa7e38b_8b2a0e2c","line":7,"range":{"start_line":7,"start_character":0,"end_line":7,"end_character":5},"in_reply_to":"3fa7e38b_abb6ea25","updated":"2020-02-17 17:03:51.000000000","message":"Generally people won\u0027t review something marked as WIP, usually it implies that there\u0027s still issues with it. If you feel like it\u0027s ready for reviews then please remove it. If there\u0027s changes to be made after review, then we can get those addressed.","commit_id":"320a025a3a729eb1ab7420aa43aadc2300a47303"}],"mariadb/values_overrides/apparmor.yaml":[{"author":{"_account_id":30777,"name":"Steven Fitzpatrick","email":"steven@fitzpatrick.wtf","username":"sf280x"},"change_message_id":"13988ed27b543e9ba0dc41deba06d5c347f91ae1","unresolved":false,"context_lines":[{"line_number":7,"context_line":"      ingress: runtime/default"},{"line_number":8,"context_line":"    mariadb-server:"},{"line_number":9,"context_line":"      mariadb: runtime/default"},{"line_number":10,"context_line":"    prometheus-mysql-exporter:"},{"line_number":11,"context_line":"      mysql-exporter: runtime/default"}],"source_content_type":"text/x-yaml","patch_set":13,"id":"3fa7e38b_ba6e2e63","line":11,"range":{"start_line":10,"start_character":0,"end_line":11,"end_character":37},"updated":"2020-02-17 22:15:20.000000000","message":"If you want to test the mysql-exporter and apparmor  together, you should also provide an override to enable it:\n\nhttps://opendev.org/openstack/openstack-helm-infra/src/branch/master/mariadb/values.yaml#L404-L406","commit_id":"84d11471ba652b95d527752c23bc35599b613514"},{"author":{"_account_id":29161,"name":"Rahul Khiyani","email":"rahul.khiyani@att.com","username":"rk0850"},"change_message_id":"a40d8ff6a23d4810df882c8c2893c576c1127332","unresolved":false,"context_lines":[{"line_number":7,"context_line":"      ingress: runtime/default"},{"line_number":8,"context_line":"    mariadb-server:"},{"line_number":9,"context_line":"      mariadb: runtime/default"},{"line_number":10,"context_line":"    prometheus-mysql-exporter:"},{"line_number":11,"context_line":"      mysql-exporter: runtime/default"}],"source_content_type":"text/x-yaml","patch_set":13,"id":"3fa7e38b_dae76ad7","line":11,"range":{"start_line":10,"start_character":0,"end_line":11,"end_character":37},"in_reply_to":"3fa7e38b_ba6e2e63","updated":"2020-02-17 22:22:53.000000000","message":"Done","commit_id":"84d11471ba652b95d527752c23bc35599b613514"},{"author":{"_account_id":30777,"name":"Steven Fitzpatrick","email":"steven@fitzpatrick.wtf","username":"sf280x"},"change_message_id":"117df19aaccc9227ae475c29c62804e14f9b7fd7","unresolved":false,"context_lines":[{"line_number":7,"context_line":"      ingress: runtime/default"},{"line_number":8,"context_line":"    mariadb-server:"},{"line_number":9,"context_line":"      mariadb: runtime/default"},{"line_number":10,"context_line":"    prometheus-mysql-exporter:"},{"line_number":11,"context_line":"      mysql-exporter: runtime/default"}],"source_content_type":"text/x-yaml","patch_set":13,"id":"3fa7e38b_7a70b610","line":11,"range":{"start_line":10,"start_character":0,"end_line":11,"end_character":37},"in_reply_to":"3fa7e38b_dae76ad7","updated":"2020-02-17 22:25:31.000000000","message":"I meant here, in the values_overrides for apparmor","commit_id":"84d11471ba652b95d527752c23bc35599b613514"}],"tools/deployment/apparmor/030-mariadb.sh":[{"author":{"_account_id":21420,"name":"Gage Hugo","email":"gagehugo@gmail.com","username":"ghugo"},"change_message_id":"b9bb2635faee4e310db88b7589429f898b5b987b","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":13,"id":"3fa7e38b_3a51ded6","updated":"2020-02-17 22:13:16.000000000","message":"Could symlink to the existing mariadb script here: keystone-auth/060-mariadb.sh","commit_id":"84d11471ba652b95d527752c23bc35599b613514"},{"author":{"_account_id":29161,"name":"Rahul Khiyani","email":"rahul.khiyani@att.com","username":"rk0850"},"change_message_id":"51deb5f07fca679261658ad185db4fcaa82b4056","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":13,"id":"3fa7e38b_9ab492d5","in_reply_to":"3fa7e38b_3a51ded6","updated":"2020-02-17 22:17:07.000000000","message":"keystone-auth/060-mariadb.sh this script deploys mariadb for openstack namespace","commit_id":"84d11471ba652b95d527752c23bc35599b613514"},{"author":{"_account_id":30777,"name":"Steven Fitzpatrick","email":"steven@fitzpatrick.wtf","username":"sf280x"},"change_message_id":"93f564efe32c5180802d63b56c534aee0809eb62","unresolved":false,"context_lines":[{"line_number":29,"context_line":"    ${OSH_INFRA_EXTRA_HELM_ARGS_MARIADB}"},{"line_number":30,"context_line":""},{"line_number":31,"context_line":"#NOTE: Wait for deploy"},{"line_number":32,"context_line":"./tools/deployment/common/wait-for-pods.sh openstack"},{"line_number":33,"context_line":""},{"line_number":34,"context_line":"#NOTE: Validate Deployment info"},{"line_number":35,"context_line":"helm status mariadb"}],"source_content_type":"text/x-sh","patch_set":15,"id":"3fa7e38b_1a6bc2ee","line":32,"range":{"start_line":32,"start_character":43,"end_line":32,"end_character":52},"updated":"2020-02-17 22:49:00.000000000","message":"This does not match the namespace you\u0027re installing the chart into.","commit_id":"3308523ec2a224df9135923f6d8103c3945f7aa8"},{"author":{"_account_id":29161,"name":"Rahul Khiyani","email":"rahul.khiyani@att.com","username":"rk0850"},"change_message_id":"650e9fd6f5eda1d75ae09188de335fa23b5392db","unresolved":false,"context_lines":[{"line_number":29,"context_line":"    ${OSH_INFRA_EXTRA_HELM_ARGS_MARIADB}"},{"line_number":30,"context_line":""},{"line_number":31,"context_line":"#NOTE: Wait for deploy"},{"line_number":32,"context_line":"./tools/deployment/common/wait-for-pods.sh openstack"},{"line_number":33,"context_line":""},{"line_number":34,"context_line":"#NOTE: Validate Deployment info"},{"line_number":35,"context_line":"helm status mariadb"}],"source_content_type":"text/x-sh","patch_set":15,"id":"3fa7e38b_d5ce5b57","line":32,"range":{"start_line":32,"start_character":43,"end_line":32,"end_character":52},"in_reply_to":"3fa7e38b_1a6bc2ee","updated":"2020-02-17 23:45:25.000000000","message":"Sorry, I missed that","commit_id":"3308523ec2a224df9135923f6d8103c3945f7aa8"},{"author":{"_account_id":21420,"name":"Gage Hugo","email":"gagehugo@gmail.com","username":"ghugo"},"change_message_id":"796fa90772ca1997bd910cca4356aa6371f72d74","unresolved":false,"context_lines":[{"line_number":25,"context_line":": ${OSH_INFRA_EXTRA_HELM_ARGS:\u003d\"\"}"},{"line_number":26,"context_line":"helm upgrade --install mariadb ./mariadb \\"},{"line_number":27,"context_line":"    --namespace\u003dosh-infra \\"},{"line_number":28,"context_line":"    --set monitoring.prometheus.enabled\u003dtrue \\"},{"line_number":29,"context_line":"    ${OSH_INFRA_EXTRA_HELM_ARGS} \\"},{"line_number":30,"context_line":"    ${OSH_INFRA_EXTRA_HELM_ARGS_MARIADB}"},{"line_number":31,"context_line":""}],"source_content_type":"text/x-sh","patch_set":17,"id":"3fa7e38b_55b4ab81","line":28,"range":{"start_line":28,"start_character":0,"end_line":28,"end_character":46},"updated":"2020-02-18 00:27:52.000000000","message":"Don\u0027t need this line here, you\u0027re providing it in the apparmor.yaml file","commit_id":"f908d371c760d127f8b0dca162272bf42467fb97"},{"author":{"_account_id":29161,"name":"Rahul Khiyani","email":"rahul.khiyani@att.com","username":"rk0850"},"change_message_id":"578c65214906eae413fe192ae038e8b32f93ec2a","unresolved":false,"context_lines":[{"line_number":25,"context_line":": ${OSH_INFRA_EXTRA_HELM_ARGS:\u003d\"\"}"},{"line_number":26,"context_line":"helm upgrade --install mariadb ./mariadb \\"},{"line_number":27,"context_line":"    --namespace\u003dosh-infra \\"},{"line_number":28,"context_line":"    --set monitoring.prometheus.enabled\u003dtrue \\"},{"line_number":29,"context_line":"    ${OSH_INFRA_EXTRA_HELM_ARGS} \\"},{"line_number":30,"context_line":"    ${OSH_INFRA_EXTRA_HELM_ARGS_MARIADB}"},{"line_number":31,"context_line":""}],"source_content_type":"text/x-sh","patch_set":17,"id":"3fa7e38b_7931a158","line":28,"range":{"start_line":28,"start_character":0,"end_line":28,"end_character":46},"in_reply_to":"3fa7e38b_55b4ab81","updated":"2020-02-18 04:50:18.000000000","message":"Done","commit_id":"f908d371c760d127f8b0dca162272bf42467fb97"}]}
