)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":25701,"name":"Yasufumi Ogawa","email":"yasufum.o@gmail.com","username":"yasufum"},"change_message_id":"7872ef7d67accfbb8ca887bd8bad50408c2826df","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"6e1c370d_ab5cbfed","updated":"2021-11-17 16:50:35.000000000","message":"Let\u0027s me recheck to confirm it can be compiled on the latest zuul because running tox on my local environment is failed.","commit_id":"13a49db4bcda8f2134fbdee6f62b849a73658d89"},{"author":{"_account_id":31857,"name":"Ayumu Ueha","email":"ueha.ayumu@fujitsu.com","username":"ueha"},"change_message_id":"2f22d44f5ca6819c773721f7353c250a9407197b","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"11cd48b6_2af8ea90","updated":"2021-12-15 00:48:20.000000000","message":"Thanks for patch, please find my comments.","commit_id":"13a49db4bcda8f2134fbdee6f62b849a73658d89"},{"author":{"_account_id":25701,"name":"Yasufumi Ogawa","email":"yasufum.o@gmail.com","username":"yasufum"},"change_message_id":"d5682d556d0b1fede673555ac5b7879d37f04ba8","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"f099da49_a6c88ba2","updated":"2021-11-17 16:50:44.000000000","message":"recheck","commit_id":"13a49db4bcda8f2134fbdee6f62b849a73658d89"},{"author":{"_account_id":33292,"name":"Masaki UENO","email":"masaki.ueno.up@hco.ntt.co.jp","username":"masaki-ueno"},"change_message_id":"13fb127bcdd5654c814a76bcb24f4eca4c7c85e6","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"56558695_305da2bb","updated":"2021-12-20 08:45:38.000000000","message":"Thank you for your comments. I\u0027ve uploaded Patchset 2,","commit_id":"a33c1bb2579589763ceeec16aeee9de2c675edb3"},{"author":{"_account_id":31857,"name":"Ayumu Ueha","email":"ueha.ayumu@fujitsu.com","username":"ueha"},"change_message_id":"d615a2838d787f5c5ab461330f3a8de9e1bacd43","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"a5e9d64c_b0c6838c","updated":"2021-12-20 09:53:51.000000000","message":"Thanks for your rework, but there seems to be one mistake. Please find additional comment.","commit_id":"a33c1bb2579589763ceeec16aeee9de2c675edb3"},{"author":{"_account_id":33292,"name":"Masaki UENO","email":"masaki.ueno.up@hco.ntt.co.jp","username":"masaki-ueno"},"change_message_id":"7e8c61e35536adb3cd42aa45afef81662f140686","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"477ee4ff_95bd6813","in_reply_to":"56558695_305da2bb","updated":"2021-12-20 08:46:38.000000000","message":"Sorry, not Patchset 2. The patch number is 3.","commit_id":"a33c1bb2579589763ceeec16aeee9de2c675edb3"},{"author":{"_account_id":33292,"name":"Masaki UENO","email":"masaki.ueno.up@hco.ntt.co.jp","username":"masaki-ueno"},"change_message_id":"8e963879dffd5339157c1143632d5a642437b2c4","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"1ff2554e_5aa69c27","updated":"2021-12-22 02:07:42.000000000","message":"Thank you for your comments. I\u0027ve fixed them in PS4.","commit_id":"2866442fb7d530226a1d3924eda50456eb925dde"},{"author":{"_account_id":31857,"name":"Ayumu Ueha","email":"ueha.ayumu@fujitsu.com","username":"ueha"},"change_message_id":"73fb2c60b153871772f4c79803a7fc377200e3cf","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"334c5850_010745f6","updated":"2021-12-23 08:18:22.000000000","message":"Thanks for quick rework! LGTM.","commit_id":"3b67e46e8b4c18ecb02e65d7032006b7ac526865"}],"specs/yoga/prometheus-plugin-heal.rst":[{"author":{"_account_id":31857,"name":"Ayumu Ueha","email":"ueha.ayumu@fujitsu.com","username":"ueha"},"change_message_id":"2f22d44f5ca6819c773721f7353c250a9407197b","unresolved":true,"context_lines":[{"line_number":57,"context_line":""},{"line_number":58,"context_line":"The following changes are needed:"},{"line_number":59,"context_line":""},{"line_number":60,"context_line":"#. Added support for Fault Management interface specified in"},{"line_number":61,"context_line":"   SOL003 [#SOL003_v3.3.1]_"},{"line_number":62,"context_line":""},{"line_number":63,"context_line":"   + Add VNF Fault Management interface:"}],"source_content_type":"text/x-rst","patch_set":2,"id":"ae8a7a05_fab0a1e2","line":60,"range":{"start_line":60,"start_character":3,"end_line":60,"end_character":16},"updated":"2021-12-15 00:48:20.000000000","message":"Why \"Added support ~\"? I think \"Add support ~\" or just \"Support ~\" is better.","commit_id":"13a49db4bcda8f2134fbdee6f62b849a73658d89"},{"author":{"_account_id":31857,"name":"Ayumu Ueha","email":"ueha.ayumu@fujitsu.com","username":"ueha"},"change_message_id":"d615a2838d787f5c5ab461330f3a8de9e1bacd43","unresolved":false,"context_lines":[{"line_number":57,"context_line":""},{"line_number":58,"context_line":"The following changes are needed:"},{"line_number":59,"context_line":""},{"line_number":60,"context_line":"#. Added support for Fault Management interface specified in"},{"line_number":61,"context_line":"   SOL003 [#SOL003_v3.3.1]_"},{"line_number":62,"context_line":""},{"line_number":63,"context_line":"   + Add VNF Fault Management interface:"}],"source_content_type":"text/x-rst","patch_set":2,"id":"6b542211_c43276bd","line":60,"range":{"start_line":60,"start_character":3,"end_line":60,"end_character":16},"in_reply_to":"699fe627_a5255c03","updated":"2021-12-20 09:53:51.000000000","message":"Ack","commit_id":"13a49db4bcda8f2134fbdee6f62b849a73658d89"},{"author":{"_account_id":33292,"name":"Masaki UENO","email":"masaki.ueno.up@hco.ntt.co.jp","username":"masaki-ueno"},"change_message_id":"13fb127bcdd5654c814a76bcb24f4eca4c7c85e6","unresolved":true,"context_lines":[{"line_number":57,"context_line":""},{"line_number":58,"context_line":"The following changes are needed:"},{"line_number":59,"context_line":""},{"line_number":60,"context_line":"#. Added support for Fault Management interface specified in"},{"line_number":61,"context_line":"   SOL003 [#SOL003_v3.3.1]_"},{"line_number":62,"context_line":""},{"line_number":63,"context_line":"   + Add VNF Fault Management interface:"}],"source_content_type":"text/x-rst","patch_set":2,"id":"699fe627_a5255c03","line":60,"range":{"start_line":60,"start_character":3,"end_line":60,"end_character":16},"in_reply_to":"ae8a7a05_fab0a1e2","updated":"2021-12-20 08:45:38.000000000","message":"I\u0027ve fixed expression in Patchset 2.","commit_id":"13a49db4bcda8f2134fbdee6f62b849a73658d89"},{"author":{"_account_id":31857,"name":"Ayumu Ueha","email":"ueha.ayumu@fujitsu.com","username":"ueha"},"change_message_id":"2f22d44f5ca6819c773721f7353c250a9407197b","unresolved":true,"context_lines":[{"line_number":94,"context_line":"     + GET \u003cClient URI from subscriptions\u003e"},{"line_number":95,"context_line":"       to confirm that the URI of Client is correct."},{"line_number":96,"context_line":""},{"line_number":97,"context_line":"#. Added support for Modify Restful API for communications between"},{"line_number":98,"context_line":"   Tacker and Alertmanager"},{"line_number":99,"context_line":""},{"line_number":100,"context_line":"   + Modify POST /alert/vnf_instances/{vnf_instance_id}"}],"source_content_type":"text/x-rst","patch_set":2,"id":"78932a20_5a64ef7c","line":97,"range":{"start_line":97,"start_character":3,"end_line":97,"end_character":16},"updated":"2021-12-15 00:48:20.000000000","message":"ditto.","commit_id":"13a49db4bcda8f2134fbdee6f62b849a73658d89"},{"author":{"_account_id":33292,"name":"Masaki UENO","email":"masaki.ueno.up@hco.ntt.co.jp","username":"masaki-ueno"},"change_message_id":"13fb127bcdd5654c814a76bcb24f4eca4c7c85e6","unresolved":true,"context_lines":[{"line_number":94,"context_line":"     + GET \u003cClient URI from subscriptions\u003e"},{"line_number":95,"context_line":"       to confirm that the URI of Client is correct."},{"line_number":96,"context_line":""},{"line_number":97,"context_line":"#. Added support for Modify Restful API for communications between"},{"line_number":98,"context_line":"   Tacker and Alertmanager"},{"line_number":99,"context_line":""},{"line_number":100,"context_line":"   + Modify POST /alert/vnf_instances/{vnf_instance_id}"}],"source_content_type":"text/x-rst","patch_set":2,"id":"e426fc14_6c7d7ac2","line":97,"range":{"start_line":97,"start_character":3,"end_line":97,"end_character":16},"in_reply_to":"78932a20_5a64ef7c","updated":"2021-12-20 08:45:38.000000000","message":"I\u0027ve fixed expression in Patchset 2.","commit_id":"13a49db4bcda8f2134fbdee6f62b849a73658d89"},{"author":{"_account_id":31857,"name":"Ayumu Ueha","email":"ueha.ayumu@fujitsu.com","username":"ueha"},"change_message_id":"d615a2838d787f5c5ab461330f3a8de9e1bacd43","unresolved":false,"context_lines":[{"line_number":94,"context_line":"     + GET \u003cClient URI from subscriptions\u003e"},{"line_number":95,"context_line":"       to confirm that the URI of Client is correct."},{"line_number":96,"context_line":""},{"line_number":97,"context_line":"#. Added support for Modify Restful API for communications between"},{"line_number":98,"context_line":"   Tacker and Alertmanager"},{"line_number":99,"context_line":""},{"line_number":100,"context_line":"   + Modify POST /alert/vnf_instances/{vnf_instance_id}"}],"source_content_type":"text/x-rst","patch_set":2,"id":"b5f29c8b_893d83d6","line":97,"range":{"start_line":97,"start_character":3,"end_line":97,"end_character":16},"in_reply_to":"e426fc14_6c7d7ac2","updated":"2021-12-20 09:53:51.000000000","message":"Ack","commit_id":"13a49db4bcda8f2134fbdee6f62b849a73658d89"},{"author":{"_account_id":31857,"name":"Ayumu Ueha","email":"ueha.ayumu@fujitsu.com","username":"ueha"},"change_message_id":"2f22d44f5ca6819c773721f7353c250a9407197b","unresolved":true,"context_lines":[{"line_number":132,"context_line":""},{"line_number":133,"context_line":"       + Delete monitor/alert rules in Prometheus."},{"line_number":134,"context_line":""},{"line_number":135,"context_line":"   + scale_start"},{"line_number":136,"context_line":""},{"line_number":137,"context_line":"     + Remove monitoring target of Worker nodes."},{"line_number":138,"context_line":"     + Delete the hostname corresponding to the VM in the"},{"line_number":139,"context_line":"       ``InstantiatedVnfInfo.vnfcResourceInfo.metadata`` field."},{"line_number":140,"context_line":""},{"line_number":141,"context_line":"   + scale_end"},{"line_number":142,"context_line":""}],"source_content_type":"text/x-rst","patch_set":2,"id":"dfa1e50a_e7e4f3a3","line":139,"range":{"start_line":135,"start_character":0,"end_line":139,"end_character":63},"updated":"2021-12-15 00:48:20.000000000","message":"Why don\u0027t you make it clear that this is a scale-in case?","commit_id":"13a49db4bcda8f2134fbdee6f62b849a73658d89"},{"author":{"_account_id":31857,"name":"Ayumu Ueha","email":"ueha.ayumu@fujitsu.com","username":"ueha"},"change_message_id":"d615a2838d787f5c5ab461330f3a8de9e1bacd43","unresolved":false,"context_lines":[{"line_number":132,"context_line":""},{"line_number":133,"context_line":"       + Delete monitor/alert rules in Prometheus."},{"line_number":134,"context_line":""},{"line_number":135,"context_line":"   + scale_start"},{"line_number":136,"context_line":""},{"line_number":137,"context_line":"     + Remove monitoring target of Worker nodes."},{"line_number":138,"context_line":"     + Delete the hostname corresponding to the VM in the"},{"line_number":139,"context_line":"       ``InstantiatedVnfInfo.vnfcResourceInfo.metadata`` field."},{"line_number":140,"context_line":""},{"line_number":141,"context_line":"   + scale_end"},{"line_number":142,"context_line":""}],"source_content_type":"text/x-rst","patch_set":2,"id":"400501a4_78d764b8","line":139,"range":{"start_line":135,"start_character":0,"end_line":139,"end_character":63},"in_reply_to":"41689f3f_eb8ea9dc","updated":"2021-12-20 09:53:51.000000000","message":"Ack","commit_id":"13a49db4bcda8f2134fbdee6f62b849a73658d89"},{"author":{"_account_id":33292,"name":"Masaki UENO","email":"masaki.ueno.up@hco.ntt.co.jp","username":"masaki-ueno"},"change_message_id":"13fb127bcdd5654c814a76bcb24f4eca4c7c85e6","unresolved":true,"context_lines":[{"line_number":132,"context_line":""},{"line_number":133,"context_line":"       + Delete monitor/alert rules in Prometheus."},{"line_number":134,"context_line":""},{"line_number":135,"context_line":"   + scale_start"},{"line_number":136,"context_line":""},{"line_number":137,"context_line":"     + Remove monitoring target of Worker nodes."},{"line_number":138,"context_line":"     + Delete the hostname corresponding to the VM in the"},{"line_number":139,"context_line":"       ``InstantiatedVnfInfo.vnfcResourceInfo.metadata`` field."},{"line_number":140,"context_line":""},{"line_number":141,"context_line":"   + scale_end"},{"line_number":142,"context_line":""}],"source_content_type":"text/x-rst","patch_set":2,"id":"41689f3f_eb8ea9dc","line":139,"range":{"start_line":135,"start_character":0,"end_line":139,"end_character":63},"in_reply_to":"dfa1e50a_e7e4f3a3","updated":"2021-12-20 08:45:38.000000000","message":"I\u0027ve added descriptions in Patchset 2.","commit_id":"13a49db4bcda8f2134fbdee6f62b849a73658d89"},{"author":{"_account_id":31857,"name":"Ayumu Ueha","email":"ueha.ayumu@fujitsu.com","username":"ueha"},"change_message_id":"2f22d44f5ca6819c773721f7353c250a9407197b","unresolved":true,"context_lines":[{"line_number":138,"context_line":"     + Delete the hostname corresponding to the VM in the"},{"line_number":139,"context_line":"       ``InstantiatedVnfInfo.vnfcResourceInfo.metadata`` field."},{"line_number":140,"context_line":""},{"line_number":141,"context_line":"   + scale_end"},{"line_number":142,"context_line":""},{"line_number":143,"context_line":"     + Set up Prometheus in new Worker nodes."},{"line_number":144,"context_line":"     + Add new monitoring target of Worker nodes."},{"line_number":145,"context_line":"     + Add the hostname corresponding to the VM in the"},{"line_number":146,"context_line":"       ``InstantiatedVnfInfo.vnfcResourceInfo.metadata`` field."},{"line_number":147,"context_line":""},{"line_number":148,"context_line":"   + heal_start"},{"line_number":149,"context_line":""}],"source_content_type":"text/x-rst","patch_set":2,"id":"3975a584_b35a85b1","line":146,"range":{"start_line":141,"start_character":0,"end_line":146,"end_character":63},"updated":"2021-12-15 00:48:20.000000000","message":"Why don\u0027t you make it clear that this is a scale-out case?","commit_id":"13a49db4bcda8f2134fbdee6f62b849a73658d89"},{"author":{"_account_id":33292,"name":"Masaki UENO","email":"masaki.ueno.up@hco.ntt.co.jp","username":"masaki-ueno"},"change_message_id":"13fb127bcdd5654c814a76bcb24f4eca4c7c85e6","unresolved":true,"context_lines":[{"line_number":138,"context_line":"     + Delete the hostname corresponding to the VM in the"},{"line_number":139,"context_line":"       ``InstantiatedVnfInfo.vnfcResourceInfo.metadata`` field."},{"line_number":140,"context_line":""},{"line_number":141,"context_line":"   + scale_end"},{"line_number":142,"context_line":""},{"line_number":143,"context_line":"     + Set up Prometheus in new Worker nodes."},{"line_number":144,"context_line":"     + Add new monitoring target of Worker nodes."},{"line_number":145,"context_line":"     + Add the hostname corresponding to the VM in the"},{"line_number":146,"context_line":"       ``InstantiatedVnfInfo.vnfcResourceInfo.metadata`` field."},{"line_number":147,"context_line":""},{"line_number":148,"context_line":"   + heal_start"},{"line_number":149,"context_line":""}],"source_content_type":"text/x-rst","patch_set":2,"id":"cb8e943f_0422c9c2","line":146,"range":{"start_line":141,"start_character":0,"end_line":146,"end_character":63},"in_reply_to":"3975a584_b35a85b1","updated":"2021-12-20 08:45:38.000000000","message":"I\u0027ve added descriptions in Patchset 2.","commit_id":"13a49db4bcda8f2134fbdee6f62b849a73658d89"},{"author":{"_account_id":31857,"name":"Ayumu Ueha","email":"ueha.ayumu@fujitsu.com","username":"ueha"},"change_message_id":"d615a2838d787f5c5ab461330f3a8de9e1bacd43","unresolved":false,"context_lines":[{"line_number":138,"context_line":"     + Delete the hostname corresponding to the VM in the"},{"line_number":139,"context_line":"       ``InstantiatedVnfInfo.vnfcResourceInfo.metadata`` field."},{"line_number":140,"context_line":""},{"line_number":141,"context_line":"   + scale_end"},{"line_number":142,"context_line":""},{"line_number":143,"context_line":"     + Set up Prometheus in new Worker nodes."},{"line_number":144,"context_line":"     + Add new monitoring target of Worker nodes."},{"line_number":145,"context_line":"     + Add the hostname corresponding to the VM in the"},{"line_number":146,"context_line":"       ``InstantiatedVnfInfo.vnfcResourceInfo.metadata`` field."},{"line_number":147,"context_line":""},{"line_number":148,"context_line":"   + heal_start"},{"line_number":149,"context_line":""}],"source_content_type":"text/x-rst","patch_set":2,"id":"156978dd_682af6e3","line":146,"range":{"start_line":141,"start_character":0,"end_line":146,"end_character":63},"in_reply_to":"cb8e943f_0422c9c2","updated":"2021-12-20 09:53:51.000000000","message":"Ack","commit_id":"13a49db4bcda8f2134fbdee6f62b849a73658d89"},{"author":{"_account_id":31857,"name":"Ayumu Ueha","email":"ueha.ayumu@fujitsu.com","username":"ueha"},"change_message_id":"2f22d44f5ca6819c773721f7353c250a9407197b","unresolved":true,"context_lines":[{"line_number":165,"context_line":"      to be changed."},{"line_number":166,"context_line":""},{"line_number":167,"context_line":"Tacker Alert Server"},{"line_number":168,"context_line":"--------------------"},{"line_number":169,"context_line":""},{"line_number":170,"context_line":"The Prometheus Plugin is an optional feature. Tacker will decide whether"},{"line_number":171,"context_line":"to enable it according to the content of the configuration file."}],"source_content_type":"text/x-rst","patch_set":2,"id":"036c8b2b_cdfe7281","line":168,"range":{"start_line":168,"start_character":19,"end_line":168,"end_character":20},"updated":"2021-12-15 00:48:20.000000000","message":"nit: Could you delete the excess characters?","commit_id":"13a49db4bcda8f2134fbdee6f62b849a73658d89"},{"author":{"_account_id":33292,"name":"Masaki UENO","email":"masaki.ueno.up@hco.ntt.co.jp","username":"masaki-ueno"},"change_message_id":"13fb127bcdd5654c814a76bcb24f4eca4c7c85e6","unresolved":true,"context_lines":[{"line_number":165,"context_line":"      to be changed."},{"line_number":166,"context_line":""},{"line_number":167,"context_line":"Tacker Alert Server"},{"line_number":168,"context_line":"--------------------"},{"line_number":169,"context_line":""},{"line_number":170,"context_line":"The Prometheus Plugin is an optional feature. Tacker will decide whether"},{"line_number":171,"context_line":"to enable it according to the content of the configuration file."}],"source_content_type":"text/x-rst","patch_set":2,"id":"f1df26d7_0e75719a","line":168,"range":{"start_line":168,"start_character":19,"end_line":168,"end_character":20},"in_reply_to":"036c8b2b_cdfe7281","updated":"2021-12-20 08:45:38.000000000","message":"I\u0027ve deleted trailing hyphens in Patchset 2.","commit_id":"13a49db4bcda8f2134fbdee6f62b849a73658d89"},{"author":{"_account_id":31857,"name":"Ayumu Ueha","email":"ueha.ayumu@fujitsu.com","username":"ueha"},"change_message_id":"d615a2838d787f5c5ab461330f3a8de9e1bacd43","unresolved":false,"context_lines":[{"line_number":165,"context_line":"      to be changed."},{"line_number":166,"context_line":""},{"line_number":167,"context_line":"Tacker Alert Server"},{"line_number":168,"context_line":"--------------------"},{"line_number":169,"context_line":""},{"line_number":170,"context_line":"The Prometheus Plugin is an optional feature. Tacker will decide whether"},{"line_number":171,"context_line":"to enable it according to the content of the configuration file."}],"source_content_type":"text/x-rst","patch_set":2,"id":"a6e5ffa3_7d4623d7","line":168,"range":{"start_line":168,"start_character":19,"end_line":168,"end_character":20},"in_reply_to":"f1df26d7_0e75719a","updated":"2021-12-20 09:53:51.000000000","message":"Ack","commit_id":"13a49db4bcda8f2134fbdee6f62b849a73658d89"},{"author":{"_account_id":31857,"name":"Ayumu Ueha","email":"ueha.ayumu@fujitsu.com","username":"ueha"},"change_message_id":"2f22d44f5ca6819c773721f7353c250a9407197b","unresolved":true,"context_lines":[{"line_number":243,"context_line":""},{"line_number":244,"context_line":""},{"line_number":245,"context_line":"AutoHeal Worker nodes on alert trigger"},{"line_number":246,"context_line":"----------------------------------------"},{"line_number":247,"context_line":""},{"line_number":248,"context_line":"When the Prometheus server detects that the Worker node of Kubernetes"},{"line_number":249,"context_line":"Cluster has failed, it will send alert messages to Tacker."}],"source_content_type":"text/x-rst","patch_set":2,"id":"5764ae30_87e21d06","line":246,"range":{"start_line":246,"start_character":38,"end_line":246,"end_character":40},"updated":"2021-12-15 00:48:20.000000000","message":"ditto","commit_id":"13a49db4bcda8f2134fbdee6f62b849a73658d89"},{"author":{"_account_id":31857,"name":"Ayumu Ueha","email":"ueha.ayumu@fujitsu.com","username":"ueha"},"change_message_id":"d615a2838d787f5c5ab461330f3a8de9e1bacd43","unresolved":false,"context_lines":[{"line_number":243,"context_line":""},{"line_number":244,"context_line":""},{"line_number":245,"context_line":"AutoHeal Worker nodes on alert trigger"},{"line_number":246,"context_line":"----------------------------------------"},{"line_number":247,"context_line":""},{"line_number":248,"context_line":"When the Prometheus server detects that the Worker node of Kubernetes"},{"line_number":249,"context_line":"Cluster has failed, it will send alert messages to Tacker."}],"source_content_type":"text/x-rst","patch_set":2,"id":"e5291fae_b78cc6c0","line":246,"range":{"start_line":246,"start_character":38,"end_line":246,"end_character":40},"in_reply_to":"37e1156b_6d86b8ca","updated":"2021-12-20 09:53:51.000000000","message":"Ack","commit_id":"13a49db4bcda8f2134fbdee6f62b849a73658d89"},{"author":{"_account_id":33292,"name":"Masaki UENO","email":"masaki.ueno.up@hco.ntt.co.jp","username":"masaki-ueno"},"change_message_id":"13fb127bcdd5654c814a76bcb24f4eca4c7c85e6","unresolved":true,"context_lines":[{"line_number":243,"context_line":""},{"line_number":244,"context_line":""},{"line_number":245,"context_line":"AutoHeal Worker nodes on alert trigger"},{"line_number":246,"context_line":"----------------------------------------"},{"line_number":247,"context_line":""},{"line_number":248,"context_line":"When the Prometheus server detects that the Worker node of Kubernetes"},{"line_number":249,"context_line":"Cluster has failed, it will send alert messages to Tacker."}],"source_content_type":"text/x-rst","patch_set":2,"id":"37e1156b_6d86b8ca","line":246,"range":{"start_line":246,"start_character":38,"end_line":246,"end_character":40},"in_reply_to":"5764ae30_87e21d06","updated":"2021-12-20 08:45:38.000000000","message":"I\u0027ve deleted trailing hyphens in Patchset 2.","commit_id":"13a49db4bcda8f2134fbdee6f62b849a73658d89"},{"author":{"_account_id":31857,"name":"Ayumu Ueha","email":"ueha.ayumu@fujitsu.com","username":"ueha"},"change_message_id":"2f22d44f5ca6819c773721f7353c250a9407197b","unresolved":true,"context_lines":[{"line_number":250,"context_line":"After Tacker receives the alert, it will convert the alert to alarm and"},{"line_number":251,"context_line":"store it in the DB. At this time, according to"},{"line_number":252,"context_line":"ETSI NFV-SOL 003 v3.3.1 [#SOL003_v3.3.1]_ 7. VNF Fault Management interface,"},{"line_number":253,"context_line":"it supports two AutoHeal modes, Polling and Notify."},{"line_number":254,"context_line":""},{"line_number":255,"context_line":"Design of heal operation in Polling mode"},{"line_number":256,"context_line":"~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~"}],"source_content_type":"text/x-rst","patch_set":2,"id":"7585a973_317052e5","line":253,"range":{"start_line":253,"start_character":0,"end_line":253,"end_character":51},"updated":"2021-12-15 00:48:20.000000000","message":"Just a question.\nHow does Tacker know which mode to use, polling or notify?\nIf is subscription created or not?","commit_id":"13a49db4bcda8f2134fbdee6f62b849a73658d89"},{"author":{"_account_id":31857,"name":"Ayumu Ueha","email":"ueha.ayumu@fujitsu.com","username":"ueha"},"change_message_id":"d615a2838d787f5c5ab461330f3a8de9e1bacd43","unresolved":false,"context_lines":[{"line_number":250,"context_line":"After Tacker receives the alert, it will convert the alert to alarm and"},{"line_number":251,"context_line":"store it in the DB. At this time, according to"},{"line_number":252,"context_line":"ETSI NFV-SOL 003 v3.3.1 [#SOL003_v3.3.1]_ 7. VNF Fault Management interface,"},{"line_number":253,"context_line":"it supports two AutoHeal modes, Polling and Notify."},{"line_number":254,"context_line":""},{"line_number":255,"context_line":"Design of heal operation in Polling mode"},{"line_number":256,"context_line":"~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~"}],"source_content_type":"text/x-rst","patch_set":2,"id":"de97ade0_ecd6c562","line":253,"range":{"start_line":253,"start_character":0,"end_line":253,"end_character":51},"in_reply_to":"64c0519b_021aed6f","updated":"2021-12-20 09:53:51.000000000","message":"Thanks for answer, I got it!","commit_id":"13a49db4bcda8f2134fbdee6f62b849a73658d89"},{"author":{"_account_id":33292,"name":"Masaki UENO","email":"masaki.ueno.up@hco.ntt.co.jp","username":"masaki-ueno"},"change_message_id":"13fb127bcdd5654c814a76bcb24f4eca4c7c85e6","unresolved":true,"context_lines":[{"line_number":250,"context_line":"After Tacker receives the alert, it will convert the alert to alarm and"},{"line_number":251,"context_line":"store it in the DB. At this time, according to"},{"line_number":252,"context_line":"ETSI NFV-SOL 003 v3.3.1 [#SOL003_v3.3.1]_ 7. VNF Fault Management interface,"},{"line_number":253,"context_line":"it supports two AutoHeal modes, Polling and Notify."},{"line_number":254,"context_line":""},{"line_number":255,"context_line":"Design of heal operation in Polling mode"},{"line_number":256,"context_line":"~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~"}],"source_content_type":"text/x-rst","patch_set":2,"id":"64c0519b_021aed6f","line":253,"range":{"start_line":253,"start_character":0,"end_line":253,"end_character":51},"in_reply_to":"7585a973_317052e5","updated":"2021-12-20 08:45:38.000000000","message":"Yes, Tacker will send notification for subscriptions, thus creating subscriptions will be a trigger of Notify mode.","commit_id":"13a49db4bcda8f2134fbdee6f62b849a73658d89"},{"author":{"_account_id":31857,"name":"Ayumu Ueha","email":"ueha.ayumu@fujitsu.com","username":"ueha"},"change_message_id":"2f22d44f5ca6819c773721f7353c250a9407197b","unresolved":true,"context_lines":[{"line_number":372,"context_line":"    \"Prometheus-Plugin\" -\u003e \"Prometheus-Plugin\""},{"line_number":373,"context_line":"      [label \u003d \"5. Convert received alert to alarm\"];"},{"line_number":374,"context_line":"    \"Prometheus-Plugin\" -\u003e \"VnfFmDriver\""},{"line_number":375,"context_line":"      [label \u003d \"6. execute VnfFmDriver\"];"},{"line_number":376,"context_line":"    \"VnfFmDriver\" -\u003e \"Tacker DB\""},{"line_number":377,"context_line":"      [label \u003d \"7. Save alarm to DB\"];"},{"line_number":378,"context_line":"    \"VnfFmDriver\" \u003c-- \"Tacker DB\""}],"source_content_type":"text/x-rst","patch_set":2,"id":"6cc63728_f8f81138","line":375,"range":{"start_line":375,"start_character":19,"end_line":375,"end_character":20},"updated":"2021-12-15 00:48:20.000000000","message":"nit: Why don\u0027t you change it to capital letters?","commit_id":"13a49db4bcda8f2134fbdee6f62b849a73658d89"},{"author":{"_account_id":33292,"name":"Masaki UENO","email":"masaki.ueno.up@hco.ntt.co.jp","username":"masaki-ueno"},"change_message_id":"13fb127bcdd5654c814a76bcb24f4eca4c7c85e6","unresolved":true,"context_lines":[{"line_number":372,"context_line":"    \"Prometheus-Plugin\" -\u003e \"Prometheus-Plugin\""},{"line_number":373,"context_line":"      [label \u003d \"5. Convert received alert to alarm\"];"},{"line_number":374,"context_line":"    \"Prometheus-Plugin\" -\u003e \"VnfFmDriver\""},{"line_number":375,"context_line":"      [label \u003d \"6. execute VnfFmDriver\"];"},{"line_number":376,"context_line":"    \"VnfFmDriver\" -\u003e \"Tacker DB\""},{"line_number":377,"context_line":"      [label \u003d \"7. Save alarm to DB\"];"},{"line_number":378,"context_line":"    \"VnfFmDriver\" \u003c-- \"Tacker DB\""}],"source_content_type":"text/x-rst","patch_set":2,"id":"ebb353cd_9bb13c9d","line":375,"range":{"start_line":375,"start_character":19,"end_line":375,"end_character":20},"in_reply_to":"6cc63728_f8f81138","updated":"2021-12-20 08:45:38.000000000","message":"I\u0027ve fixed it in Patchset 2.","commit_id":"13a49db4bcda8f2134fbdee6f62b849a73658d89"},{"author":{"_account_id":31857,"name":"Ayumu Ueha","email":"ueha.ayumu@fujitsu.com","username":"ueha"},"change_message_id":"d615a2838d787f5c5ab461330f3a8de9e1bacd43","unresolved":false,"context_lines":[{"line_number":372,"context_line":"    \"Prometheus-Plugin\" -\u003e \"Prometheus-Plugin\""},{"line_number":373,"context_line":"      [label \u003d \"5. Convert received alert to alarm\"];"},{"line_number":374,"context_line":"    \"Prometheus-Plugin\" -\u003e \"VnfFmDriver\""},{"line_number":375,"context_line":"      [label \u003d \"6. execute VnfFmDriver\"];"},{"line_number":376,"context_line":"    \"VnfFmDriver\" -\u003e \"Tacker DB\""},{"line_number":377,"context_line":"      [label \u003d \"7. Save alarm to DB\"];"},{"line_number":378,"context_line":"    \"VnfFmDriver\" \u003c-- \"Tacker DB\""}],"source_content_type":"text/x-rst","patch_set":2,"id":"6bf433b9_64fcbb0f","line":375,"range":{"start_line":375,"start_character":19,"end_line":375,"end_character":20},"in_reply_to":"ebb353cd_9bb13c9d","updated":"2021-12-20 09:53:51.000000000","message":"Ack","commit_id":"13a49db4bcda8f2134fbdee6f62b849a73658d89"},{"author":{"_account_id":31857,"name":"Ayumu Ueha","email":"ueha.ayumu@fujitsu.com","username":"ueha"},"change_message_id":"2f22d44f5ca6819c773721f7353c250a9407197b","unresolved":true,"context_lines":[{"line_number":470,"context_line":"    * The only difference is that for"},{"line_number":471,"context_line":"      ``InstantiatedVnfInfo.vnfcResourceInfo``, in ``heal_end``,"},{"line_number":472,"context_line":"      update the hostname of the VM in the metadata field."},{"line_number":473,"context_line":""},{"line_number":474,"context_line":""},{"line_number":475,"context_line":""},{"line_number":476,"context_line":""},{"line_number":477,"context_line":"Design of heal operation in Notify mode"},{"line_number":478,"context_line":"~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~"},{"line_number":479,"context_line":""}],"source_content_type":"text/x-rst","patch_set":2,"id":"f442ea9e_867e6c4c","line":476,"range":{"start_line":473,"start_character":0,"end_line":476,"end_character":0},"updated":"2021-12-15 00:48:20.000000000","message":"nit: too many blank lines here.","commit_id":"13a49db4bcda8f2134fbdee6f62b849a73658d89"},{"author":{"_account_id":31857,"name":"Ayumu Ueha","email":"ueha.ayumu@fujitsu.com","username":"ueha"},"change_message_id":"d615a2838d787f5c5ab461330f3a8de9e1bacd43","unresolved":false,"context_lines":[{"line_number":470,"context_line":"    * The only difference is that for"},{"line_number":471,"context_line":"      ``InstantiatedVnfInfo.vnfcResourceInfo``, in ``heal_end``,"},{"line_number":472,"context_line":"      update the hostname of the VM in the metadata field."},{"line_number":473,"context_line":""},{"line_number":474,"context_line":""},{"line_number":475,"context_line":""},{"line_number":476,"context_line":""},{"line_number":477,"context_line":"Design of heal operation in Notify mode"},{"line_number":478,"context_line":"~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~"},{"line_number":479,"context_line":""}],"source_content_type":"text/x-rst","patch_set":2,"id":"dcf506af_13bd0754","line":476,"range":{"start_line":473,"start_character":0,"end_line":476,"end_character":0},"in_reply_to":"61618948_c956580c","updated":"2021-12-20 09:53:51.000000000","message":"Ack","commit_id":"13a49db4bcda8f2134fbdee6f62b849a73658d89"},{"author":{"_account_id":33292,"name":"Masaki UENO","email":"masaki.ueno.up@hco.ntt.co.jp","username":"masaki-ueno"},"change_message_id":"13fb127bcdd5654c814a76bcb24f4eca4c7c85e6","unresolved":true,"context_lines":[{"line_number":470,"context_line":"    * The only difference is that for"},{"line_number":471,"context_line":"      ``InstantiatedVnfInfo.vnfcResourceInfo``, in ``heal_end``,"},{"line_number":472,"context_line":"      update the hostname of the VM in the metadata field."},{"line_number":473,"context_line":""},{"line_number":474,"context_line":""},{"line_number":475,"context_line":""},{"line_number":476,"context_line":""},{"line_number":477,"context_line":"Design of heal operation in Notify mode"},{"line_number":478,"context_line":"~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~"},{"line_number":479,"context_line":""}],"source_content_type":"text/x-rst","patch_set":2,"id":"61618948_c956580c","line":476,"range":{"start_line":473,"start_character":0,"end_line":476,"end_character":0},"in_reply_to":"f442ea9e_867e6c4c","updated":"2021-12-20 08:45:38.000000000","message":"I\u0027ve removed trailing blank lines in Patchset 2.","commit_id":"13a49db4bcda8f2134fbdee6f62b849a73658d89"},{"author":{"_account_id":31857,"name":"Ayumu Ueha","email":"ueha.ayumu@fujitsu.com","username":"ueha"},"change_message_id":"2f22d44f5ca6819c773721f7353c250a9407197b","unresolved":true,"context_lines":[{"line_number":655,"context_line":""},{"line_number":656,"context_line":""},{"line_number":657,"context_line":"Scale For Kubernetes cluster"},{"line_number":658,"context_line":"--------------------------------"},{"line_number":659,"context_line":""},{"line_number":660,"context_line":"The process is almost the same with"},{"line_number":661,"context_line":"\"Scale-out/in Worker nodes of Kubernetes cluster VNF\""}],"source_content_type":"text/x-rst","patch_set":2,"id":"538bc44c_53fcb5a6","line":658,"range":{"start_line":658,"start_character":28,"end_line":658,"end_character":32},"updated":"2021-12-15 00:48:20.000000000","message":"nit: Could you delete the excess characters?","commit_id":"13a49db4bcda8f2134fbdee6f62b849a73658d89"},{"author":{"_account_id":31857,"name":"Ayumu Ueha","email":"ueha.ayumu@fujitsu.com","username":"ueha"},"change_message_id":"d615a2838d787f5c5ab461330f3a8de9e1bacd43","unresolved":false,"context_lines":[{"line_number":655,"context_line":""},{"line_number":656,"context_line":""},{"line_number":657,"context_line":"Scale For Kubernetes cluster"},{"line_number":658,"context_line":"--------------------------------"},{"line_number":659,"context_line":""},{"line_number":660,"context_line":"The process is almost the same with"},{"line_number":661,"context_line":"\"Scale-out/in Worker nodes of Kubernetes cluster VNF\""}],"source_content_type":"text/x-rst","patch_set":2,"id":"c7761aee_30ba6b6e","line":658,"range":{"start_line":658,"start_character":28,"end_line":658,"end_character":32},"in_reply_to":"40814782_4970531b","updated":"2021-12-20 09:53:51.000000000","message":"Ack","commit_id":"13a49db4bcda8f2134fbdee6f62b849a73658d89"},{"author":{"_account_id":33292,"name":"Masaki UENO","email":"masaki.ueno.up@hco.ntt.co.jp","username":"masaki-ueno"},"change_message_id":"13fb127bcdd5654c814a76bcb24f4eca4c7c85e6","unresolved":true,"context_lines":[{"line_number":655,"context_line":""},{"line_number":656,"context_line":""},{"line_number":657,"context_line":"Scale For Kubernetes cluster"},{"line_number":658,"context_line":"--------------------------------"},{"line_number":659,"context_line":""},{"line_number":660,"context_line":"The process is almost the same with"},{"line_number":661,"context_line":"\"Scale-out/in Worker nodes of Kubernetes cluster VNF\""}],"source_content_type":"text/x-rst","patch_set":2,"id":"40814782_4970531b","line":658,"range":{"start_line":658,"start_character":28,"end_line":658,"end_character":32},"in_reply_to":"538bc44c_53fcb5a6","updated":"2021-12-20 08:45:38.000000000","message":"I\u0027ve deleted trailing hyphens in Patchset 2.","commit_id":"13a49db4bcda8f2134fbdee6f62b849a73658d89"},{"author":{"_account_id":31857,"name":"Ayumu Ueha","email":"ueha.ayumu@fujitsu.com","username":"ueha"},"change_message_id":"2f22d44f5ca6819c773721f7353c250a9407197b","unresolved":true,"context_lines":[{"line_number":693,"context_line":""},{"line_number":694,"context_line":".. code-block:: console"},{"line_number":695,"context_line":""},{"line_number":696,"context_line":"  `id` varchar(36) Pri, auto_increment"},{"line_number":697,"context_line":""},{"line_number":698,"context_line":"  `managed_object_id` varchar(36) NOT NULL"},{"line_number":699,"context_line":""}],"source_content_type":"text/x-rst","patch_set":2,"id":"32e2c822_7b6f4a3d","line":696,"range":{"start_line":696,"start_character":2,"end_line":696,"end_character":38},"updated":"2021-12-15 00:48:20.000000000","message":"Does \"auto_increment\" mean id is not UUID that generated randomly?\nDoes it mean id increase from 0 to ffffffffffffffffffffffffffffffffffff?\n(In the example below it appears to be in UUID form, so 00000000-0000-0000-0000-000000000000 to ffffffff-ffff-ffff-ffff-ffffffffffff?)\n\nAnd is type \"varchar\" correct? If you use \"auto_increment\", don\u0027t you need to use integer?","commit_id":"13a49db4bcda8f2134fbdee6f62b849a73658d89"},{"author":{"_account_id":33292,"name":"Masaki UENO","email":"masaki.ueno.up@hco.ntt.co.jp","username":"masaki-ueno"},"change_message_id":"13fb127bcdd5654c814a76bcb24f4eca4c7c85e6","unresolved":true,"context_lines":[{"line_number":693,"context_line":""},{"line_number":694,"context_line":".. code-block:: console"},{"line_number":695,"context_line":""},{"line_number":696,"context_line":"  `id` varchar(36) Pri, auto_increment"},{"line_number":697,"context_line":""},{"line_number":698,"context_line":"  `managed_object_id` varchar(36) NOT NULL"},{"line_number":699,"context_line":""}],"source_content_type":"text/x-rst","patch_set":2,"id":"a5fdffcf_7b7f3419","line":696,"range":{"start_line":696,"start_character":2,"end_line":696,"end_character":38},"in_reply_to":"213be114_bda25ff5","updated":"2021-12-20 08:45:38.000000000","message":"As you pointed out, this parameter is responsible for `Type: Alarm` data model defined in SOL003 v3.3.1, and data type is `Identifier`. `auto_increment` will automatically generate unique id for each alarm, but I don\u0027t think this is a good solution, because there\u0027s no need for using auto increment values for those identifiers.\n\nI\u0027ve removed `auto_increment` from `id` in vnf_fm_alarms in Patchset 2.","commit_id":"13a49db4bcda8f2134fbdee6f62b849a73658d89"},{"author":{"_account_id":31857,"name":"Ayumu Ueha","email":"ueha.ayumu@fujitsu.com","username":"ueha"},"change_message_id":"b40f6f1822000dafeae2a246490b01ca0a7aee03","unresolved":true,"context_lines":[{"line_number":693,"context_line":""},{"line_number":694,"context_line":".. code-block:: console"},{"line_number":695,"context_line":""},{"line_number":696,"context_line":"  `id` varchar(36) Pri, auto_increment"},{"line_number":697,"context_line":""},{"line_number":698,"context_line":"  `managed_object_id` varchar(36) NOT NULL"},{"line_number":699,"context_line":""}],"source_content_type":"text/x-rst","patch_set":2,"id":"213be114_bda25ff5","line":696,"range":{"start_line":696,"start_character":2,"end_line":696,"end_character":38},"in_reply_to":"32e2c822_7b6f4a3d","updated":"2021-12-15 12:02:04.000000000","message":"Sorry, I found this id\u0027s data type is defined as `Identifier` in SOL003 v3.3.1 \"7.5.2.4 Type: Alarm\".\nSo \"varchar(36)\" may correct, but still I wonder if \"auto_increment\" is correct.\nCould you confirm about it?","commit_id":"13a49db4bcda8f2134fbdee6f62b849a73658d89"},{"author":{"_account_id":31857,"name":"Ayumu Ueha","email":"ueha.ayumu@fujitsu.com","username":"ueha"},"change_message_id":"d615a2838d787f5c5ab461330f3a8de9e1bacd43","unresolved":false,"context_lines":[{"line_number":693,"context_line":""},{"line_number":694,"context_line":".. code-block:: console"},{"line_number":695,"context_line":""},{"line_number":696,"context_line":"  `id` varchar(36) Pri, auto_increment"},{"line_number":697,"context_line":""},{"line_number":698,"context_line":"  `managed_object_id` varchar(36) NOT NULL"},{"line_number":699,"context_line":""}],"source_content_type":"text/x-rst","patch_set":2,"id":"d5a5df9f_9b790e1a","line":696,"range":{"start_line":696,"start_character":2,"end_line":696,"end_character":38},"in_reply_to":"a5fdffcf_7b7f3419","updated":"2021-12-20 09:53:51.000000000","message":"I\u0027ve confirmed in PS3, thanks.","commit_id":"13a49db4bcda8f2134fbdee6f62b849a73658d89"},{"author":{"_account_id":31857,"name":"Ayumu Ueha","email":"ueha.ayumu@fujitsu.com","username":"ueha"},"change_message_id":"2f22d44f5ca6819c773721f7353c250a9407197b","unresolved":true,"context_lines":[{"line_number":737,"context_line":""},{"line_number":738,"context_line":".. code-block:: console"},{"line_number":739,"context_line":""},{"line_number":740,"context_line":"  `id` varchar(36) Pri, auto_increment"},{"line_number":741,"context_line":""},{"line_number":742,"context_line":"  `fm_notifications_filter` JSON NULL"},{"line_number":743,"context_line":""}],"source_content_type":"text/x-rst","patch_set":2,"id":"375027a1_949ff73a","line":740,"range":{"start_line":740,"start_character":2,"end_line":740,"end_character":38},"updated":"2021-12-15 00:48:20.000000000","message":"I can see that id for vnf_fm_alarms is incremented in the order of notification, however I wonder if the id for vnf_fm_subscriptions should also be \"auto_increment\"?\n\nOr shouldn\u0027t it be an integer if \"auto_increment\" is needed as in the above comment?","commit_id":"13a49db4bcda8f2134fbdee6f62b849a73658d89"},{"author":{"_account_id":33292,"name":"Masaki UENO","email":"masaki.ueno.up@hco.ntt.co.jp","username":"masaki-ueno"},"change_message_id":"13fb127bcdd5654c814a76bcb24f4eca4c7c85e6","unresolved":true,"context_lines":[{"line_number":737,"context_line":""},{"line_number":738,"context_line":".. code-block:: console"},{"line_number":739,"context_line":""},{"line_number":740,"context_line":"  `id` varchar(36) Pri, auto_increment"},{"line_number":741,"context_line":""},{"line_number":742,"context_line":"  `fm_notifications_filter` JSON NULL"},{"line_number":743,"context_line":""}],"source_content_type":"text/x-rst","patch_set":2,"id":"8ea0c19f_45a8ac3f","line":740,"range":{"start_line":740,"start_character":2,"end_line":740,"end_character":38},"in_reply_to":"375027a1_949ff73a","updated":"2021-12-20 08:45:38.000000000","message":"ditto. I\u0027ve removed `auto_increment` from `id` field in vnf_fm_subscriptions.","commit_id":"13a49db4bcda8f2134fbdee6f62b849a73658d89"},{"author":{"_account_id":31857,"name":"Ayumu Ueha","email":"ueha.ayumu@fujitsu.com","username":"ueha"},"change_message_id":"d615a2838d787f5c5ab461330f3a8de9e1bacd43","unresolved":false,"context_lines":[{"line_number":737,"context_line":""},{"line_number":738,"context_line":".. code-block:: console"},{"line_number":739,"context_line":""},{"line_number":740,"context_line":"  `id` varchar(36) Pri, auto_increment"},{"line_number":741,"context_line":""},{"line_number":742,"context_line":"  `fm_notifications_filter` JSON NULL"},{"line_number":743,"context_line":""}],"source_content_type":"text/x-rst","patch_set":2,"id":"3256a782_7dbc19f9","line":740,"range":{"start_line":740,"start_character":2,"end_line":740,"end_character":38},"in_reply_to":"8ea0c19f_45a8ac3f","updated":"2021-12-20 09:53:51.000000000","message":"I\u0027ve confirmed in PS3, thanks.","commit_id":"13a49db4bcda8f2134fbdee6f62b849a73658d89"},{"author":{"_account_id":31857,"name":"Ayumu Ueha","email":"ueha.ayumu@fujitsu.com","username":"ueha"},"change_message_id":"2f22d44f5ca6819c773721f7353c250a9407197b","unresolved":true,"context_lines":[{"line_number":1311,"context_line":"         \"subscriptionId\": \"fb782658-af96-47e7-9faa-90ba8416e426\","},{"line_number":1312,"context_line":"         \"timeStamp\": \"2021-09-03 10:21:03\","},{"line_number":1313,"context_line":"         \"alarmId\": \"78a39661-60a8-4824-b989-88c1b0c3534a\","},{"line_number":1314,"context_line":"         \"alarmClearedTime\": \"2021-09-03 10:36:03\""},{"line_number":1315,"context_line":"         \"_links\": {"},{"line_number":1316,"context_line":"             \"subscription\": {"},{"line_number":1317,"context_line":"                 \"href\": \"/vnffm/v1/subscriptions/fb782658-af96-47e7-9faa-90ba8416e426\""}],"source_content_type":"text/x-rst","patch_set":2,"id":"476b4e3d_6d03cdcb","line":1314,"updated":"2021-12-15 00:48:20.000000000","message":"comma character is missing at end of line.","commit_id":"13a49db4bcda8f2134fbdee6f62b849a73658d89"},{"author":{"_account_id":33292,"name":"Masaki UENO","email":"masaki.ueno.up@hco.ntt.co.jp","username":"masaki-ueno"},"change_message_id":"13fb127bcdd5654c814a76bcb24f4eca4c7c85e6","unresolved":true,"context_lines":[{"line_number":1311,"context_line":"         \"subscriptionId\": \"fb782658-af96-47e7-9faa-90ba8416e426\","},{"line_number":1312,"context_line":"         \"timeStamp\": \"2021-09-03 10:21:03\","},{"line_number":1313,"context_line":"         \"alarmId\": \"78a39661-60a8-4824-b989-88c1b0c3534a\","},{"line_number":1314,"context_line":"         \"alarmClearedTime\": \"2021-09-03 10:36:03\""},{"line_number":1315,"context_line":"         \"_links\": {"},{"line_number":1316,"context_line":"             \"subscription\": {"},{"line_number":1317,"context_line":"                 \"href\": \"/vnffm/v1/subscriptions/fb782658-af96-47e7-9faa-90ba8416e426\""}],"source_content_type":"text/x-rst","patch_set":2,"id":"c224ff80_4d9a4ed7","line":1314,"in_reply_to":"476b4e3d_6d03cdcb","updated":"2021-12-20 08:45:38.000000000","message":"I\u0027ve added missing comma in Patchset 2.","commit_id":"13a49db4bcda8f2134fbdee6f62b849a73658d89"},{"author":{"_account_id":31857,"name":"Ayumu Ueha","email":"ueha.ayumu@fujitsu.com","username":"ueha"},"change_message_id":"d615a2838d787f5c5ab461330f3a8de9e1bacd43","unresolved":false,"context_lines":[{"line_number":1311,"context_line":"         \"subscriptionId\": \"fb782658-af96-47e7-9faa-90ba8416e426\","},{"line_number":1312,"context_line":"         \"timeStamp\": \"2021-09-03 10:21:03\","},{"line_number":1313,"context_line":"         \"alarmId\": \"78a39661-60a8-4824-b989-88c1b0c3534a\","},{"line_number":1314,"context_line":"         \"alarmClearedTime\": \"2021-09-03 10:36:03\""},{"line_number":1315,"context_line":"         \"_links\": {"},{"line_number":1316,"context_line":"             \"subscription\": {"},{"line_number":1317,"context_line":"                 \"href\": \"/vnffm/v1/subscriptions/fb782658-af96-47e7-9faa-90ba8416e426\""}],"source_content_type":"text/x-rst","patch_set":2,"id":"e002e396_4d511419","line":1314,"in_reply_to":"c224ff80_4d9a4ed7","updated":"2021-12-20 09:53:51.000000000","message":"Ack","commit_id":"13a49db4bcda8f2134fbdee6f62b849a73658d89"},{"author":{"_account_id":31857,"name":"Ayumu Ueha","email":"ueha.ayumu@fujitsu.com","username":"ueha"},"change_message_id":"2f22d44f5ca6819c773721f7353c250a9407197b","unresolved":true,"context_lines":[{"line_number":1392,"context_line":"         \"version\": \"\","},{"line_number":1393,"context_line":"         \"groupKey\": \"{}:{}\""},{"line_number":1394,"context_line":"     }"},{"line_number":1395,"context_line":""},{"line_number":1396,"context_line":""},{"line_number":1397,"context_line":""},{"line_number":1398,"context_line":""},{"line_number":1399,"context_line":"Security impact"},{"line_number":1400,"context_line":"---------------"},{"line_number":1401,"context_line":""}],"source_content_type":"text/x-rst","patch_set":2,"id":"7c9501f5_d3ce73f4","line":1398,"range":{"start_line":1395,"start_character":0,"end_line":1398,"end_character":0},"updated":"2021-12-15 00:48:20.000000000","message":"nit: too many blank lines here.","commit_id":"13a49db4bcda8f2134fbdee6f62b849a73658d89"},{"author":{"_account_id":33292,"name":"Masaki UENO","email":"masaki.ueno.up@hco.ntt.co.jp","username":"masaki-ueno"},"change_message_id":"13fb127bcdd5654c814a76bcb24f4eca4c7c85e6","unresolved":true,"context_lines":[{"line_number":1392,"context_line":"         \"version\": \"\","},{"line_number":1393,"context_line":"         \"groupKey\": \"{}:{}\""},{"line_number":1394,"context_line":"     }"},{"line_number":1395,"context_line":""},{"line_number":1396,"context_line":""},{"line_number":1397,"context_line":""},{"line_number":1398,"context_line":""},{"line_number":1399,"context_line":"Security impact"},{"line_number":1400,"context_line":"---------------"},{"line_number":1401,"context_line":""}],"source_content_type":"text/x-rst","patch_set":2,"id":"a01c5c55_bf4cee94","line":1398,"range":{"start_line":1395,"start_character":0,"end_line":1398,"end_character":0},"in_reply_to":"7c9501f5_d3ce73f4","updated":"2021-12-20 08:45:38.000000000","message":"I\u0027ve deleted trailing blank lines in Patchset 2.","commit_id":"13a49db4bcda8f2134fbdee6f62b849a73658d89"},{"author":{"_account_id":31857,"name":"Ayumu Ueha","email":"ueha.ayumu@fujitsu.com","username":"ueha"},"change_message_id":"d615a2838d787f5c5ab461330f3a8de9e1bacd43","unresolved":false,"context_lines":[{"line_number":1392,"context_line":"         \"version\": \"\","},{"line_number":1393,"context_line":"         \"groupKey\": \"{}:{}\""},{"line_number":1394,"context_line":"     }"},{"line_number":1395,"context_line":""},{"line_number":1396,"context_line":""},{"line_number":1397,"context_line":""},{"line_number":1398,"context_line":""},{"line_number":1399,"context_line":"Security impact"},{"line_number":1400,"context_line":"---------------"},{"line_number":1401,"context_line":""}],"source_content_type":"text/x-rst","patch_set":2,"id":"022a4409_7a8825ce","line":1398,"range":{"start_line":1395,"start_character":0,"end_line":1398,"end_character":0},"in_reply_to":"a01c5c55_bf4cee94","updated":"2021-12-20 09:53:51.000000000","message":"Ack","commit_id":"13a49db4bcda8f2134fbdee6f62b849a73658d89"},{"author":{"_account_id":25701,"name":"Yasufumi Ogawa","email":"yasufum.o@gmail.com","username":"yasufum"},"change_message_id":"118718b27835b1253d08fceb055c6f1cf3d51841","unresolved":true,"context_lines":[{"line_number":173,"context_line":""},{"line_number":174,"context_line":"The Prometheus Plugin is an optional feature. Tacker will decide whether"},{"line_number":175,"context_line":"to enable it according to the content of the configuration file."},{"line_number":176,"context_line":"Modify the configuration file please refer to the description in"},{"line_number":177,"context_line":"[#Alert_server_for_Prometheus_with_Kubernetes_cluster_VNF_sample]_."},{"line_number":178,"context_line":""},{"line_number":179,"context_line":"Instantiate Kubernetes cluster VNF"}],"source_content_type":"text/x-rst","patch_set":3,"id":"71f9f625_4895467a","line":176,"range":{"start_line":176,"start_character":0,"end_line":176,"end_character":46},"updated":"2021-12-21 07:59:25.000000000","message":"nit: looks incorrect grammatically.","commit_id":"a33c1bb2579589763ceeec16aeee9de2c675edb3"},{"author":{"_account_id":33292,"name":"Masaki UENO","email":"masaki.ueno.up@hco.ntt.co.jp","username":"masaki-ueno"},"change_message_id":"8e963879dffd5339157c1143632d5a642437b2c4","unresolved":true,"context_lines":[{"line_number":173,"context_line":""},{"line_number":174,"context_line":"The Prometheus Plugin is an optional feature. Tacker will decide whether"},{"line_number":175,"context_line":"to enable it according to the content of the configuration file."},{"line_number":176,"context_line":"Modify the configuration file please refer to the description in"},{"line_number":177,"context_line":"[#Alert_server_for_Prometheus_with_Kubernetes_cluster_VNF_sample]_."},{"line_number":178,"context_line":""},{"line_number":179,"context_line":"Instantiate Kubernetes cluster VNF"}],"source_content_type":"text/x-rst","patch_set":3,"id":"dc208303_8f65f80d","line":176,"range":{"start_line":176,"start_character":0,"end_line":176,"end_character":46},"in_reply_to":"71f9f625_4895467a","updated":"2021-12-22 02:07:42.000000000","message":"I\u0027ve corrected the expression in PS4.","commit_id":"a33c1bb2579589763ceeec16aeee9de2c675edb3"},{"author":{"_account_id":31857,"name":"Ayumu Ueha","email":"ueha.ayumu@fujitsu.com","username":"ueha"},"change_message_id":"d615a2838d787f5c5ab461330f3a8de9e1bacd43","unresolved":true,"context_lines":[{"line_number":268,"context_line":"                                  |   Client (NFVO etc.)   |"},{"line_number":269,"context_line":"                                  |                        |"},{"line_number":270,"context_line":"                                  +--+--------^+-----------+"},{"line_number":271,"context_line":"                             1. Heal |        || 5. Query Alarms and return result"},{"line_number":272,"context_line":"                              +------|--------||------------------------------------------------------------+"},{"line_number":273,"context_line":"                              |      |        ||                                                       VNFM |"},{"line_number":274,"context_line":"                              |  +---|--------||---------+ +---------------------------------+              |"}],"source_content_type":"text/x-rst","patch_set":3,"id":"1081cfae_93a2fb3f","line":271,"range":{"start_line":271,"start_character":29,"end_line":271,"end_character":30},"updated":"2021-12-20 09:53:51.000000000","message":"Why did you change this number to 1 on the PS3?","commit_id":"a33c1bb2579589763ceeec16aeee9de2c675edb3"},{"author":{"_account_id":25701,"name":"Yasufumi Ogawa","email":"yasufum.o@gmail.com","username":"yasufum"},"change_message_id":"118718b27835b1253d08fceb055c6f1cf3d51841","unresolved":true,"context_lines":[{"line_number":268,"context_line":"                                  |   Client (NFVO etc.)   |"},{"line_number":269,"context_line":"                                  |                        |"},{"line_number":270,"context_line":"                                  +--+--------^+-----------+"},{"line_number":271,"context_line":"                             1. Heal |        || 5. Query Alarms and return result"},{"line_number":272,"context_line":"                              +------|--------||------------------------------------------------------------+"},{"line_number":273,"context_line":"                              |      |        ||                                                       VNFM |"},{"line_number":274,"context_line":"                              |  +---|--------||---------+ +---------------------------------+              |"}],"source_content_type":"text/x-rst","patch_set":3,"id":"21df56fb_6740d6e5","line":271,"range":{"start_line":271,"start_character":29,"end_line":271,"end_character":30},"in_reply_to":"1081cfae_93a2fb3f","updated":"2021-12-21 07:59:25.000000000","message":"It might be just a mistake, simply correct it.","commit_id":"a33c1bb2579589763ceeec16aeee9de2c675edb3"},{"author":{"_account_id":33292,"name":"Masaki UENO","email":"masaki.ueno.up@hco.ntt.co.jp","username":"masaki-ueno"},"change_message_id":"8e963879dffd5339157c1143632d5a642437b2c4","unresolved":true,"context_lines":[{"line_number":268,"context_line":"                                  |   Client (NFVO etc.)   |"},{"line_number":269,"context_line":"                                  |                        |"},{"line_number":270,"context_line":"                                  +--+--------^+-----------+"},{"line_number":271,"context_line":"                             1. Heal |        || 5. Query Alarms and return result"},{"line_number":272,"context_line":"                              +------|--------||------------------------------------------------------------+"},{"line_number":273,"context_line":"                              |      |        ||                                                       VNFM |"},{"line_number":274,"context_line":"                              |  +---|--------||---------+ +---------------------------------+              |"}],"source_content_type":"text/x-rst","patch_set":3,"id":"74aea3df_864f98f0","line":271,"range":{"start_line":271,"start_character":29,"end_line":271,"end_character":30},"in_reply_to":"21df56fb_6740d6e5","updated":"2021-12-22 02:07:42.000000000","message":"Sorry, this is just a mistake. I\u0027ve corrected it in PS4.","commit_id":"a33c1bb2579589763ceeec16aeee9de2c675edb3"},{"author":{"_account_id":31857,"name":"Ayumu Ueha","email":"ueha.ayumu@fujitsu.com","username":"ueha"},"change_message_id":"95cc83b7ca219cb5996065cd88d7ec93a7147932","unresolved":true,"context_lines":[{"line_number":268,"context_line":"                                  |   Client (NFVO etc.)   |"},{"line_number":269,"context_line":"                                  |                        |"},{"line_number":270,"context_line":"                                  +--+--------^+-----------+"},{"line_number":271,"context_line":"                             1. Heal |        || 5. Query Alarms and return result"},{"line_number":272,"context_line":"                              +------|--------||------------------------------------------------------------+"},{"line_number":273,"context_line":"                              |      |        ||                                                       VNFM |"},{"line_number":274,"context_line":"                              |  +---|--------||---------+ +---------------------------------+              |"}],"source_content_type":"text/x-rst","patch_set":3,"id":"ab87b729_504bae78","line":271,"range":{"start_line":271,"start_character":29,"end_line":271,"end_character":30},"in_reply_to":"74aea3df_864f98f0","updated":"2021-12-23 06:59:20.000000000","message":"It doesn\u0027t seem to be fixed in PS4.","commit_id":"a33c1bb2579589763ceeec16aeee9de2c675edb3"},{"author":{"_account_id":33292,"name":"Masaki UENO","email":"masaki.ueno.up@hco.ntt.co.jp","username":"masaki-ueno"},"change_message_id":"f003fd591ac5b04847b59106436b2af1d609da11","unresolved":true,"context_lines":[{"line_number":268,"context_line":"                                  |   Client (NFVO etc.)   |"},{"line_number":269,"context_line":"                                  |                        |"},{"line_number":270,"context_line":"                                  +--+--------^+-----------+"},{"line_number":271,"context_line":"                             1. Heal |        || 5. Query Alarms and return result"},{"line_number":272,"context_line":"                              +------|--------||------------------------------------------------------------+"},{"line_number":273,"context_line":"                              |      |        ||                                                       VNFM |"},{"line_number":274,"context_line":"                              |  +---|--------||---------+ +---------------------------------+              |"}],"source_content_type":"text/x-rst","patch_set":3,"id":"c6a393fb_655d5496","line":271,"range":{"start_line":271,"start_character":29,"end_line":271,"end_character":30},"in_reply_to":"ab87b729_504bae78","updated":"2021-12-23 07:59:58.000000000","message":"Sorry, it\u0027s my mistake. Could you confirm it in PS5?","commit_id":"a33c1bb2579589763ceeec16aeee9de2c675edb3"},{"author":{"_account_id":31857,"name":"Ayumu Ueha","email":"ueha.ayumu@fujitsu.com","username":"ueha"},"change_message_id":"1fb5c0376a0c9684ff1bc431e6a300487cf48488","unresolved":false,"context_lines":[{"line_number":268,"context_line":"                                  |   Client (NFVO etc.)   |"},{"line_number":269,"context_line":"                                  |                        |"},{"line_number":270,"context_line":"                                  +--+--------^+-----------+"},{"line_number":271,"context_line":"                             1. Heal |        || 5. Query Alarms and return result"},{"line_number":272,"context_line":"                              +------|--------||------------------------------------------------------------+"},{"line_number":273,"context_line":"                              |      |        ||                                                       VNFM |"},{"line_number":274,"context_line":"                              |  +---|--------||---------+ +---------------------------------+              |"}],"source_content_type":"text/x-rst","patch_set":3,"id":"9293327f_05a72040","line":271,"range":{"start_line":271,"start_character":29,"end_line":271,"end_character":30},"in_reply_to":"c6a393fb_655d5496","updated":"2021-12-23 08:18:39.000000000","message":"Ack","commit_id":"a33c1bb2579589763ceeec16aeee9de2c675edb3"}]}