)]}'
{"jenkins/jobs/manila.yaml":[{"author":{"_account_id":7118,"name":"Ian Wienand","email":"iwienand@redhat.com","username":"iwienand"},"change_message_id":"f42534342277f1235ea92e1a122c1e31c9a73062","unresolved":false,"context_lines":[{"line_number":169,"context_line":"              fi"},{"line_number":170,"context_line":"              if [ \"{node}\" \u003d\u003d \"centos-7\" ]; then"},{"line_number":171,"context_line":"                 # Set LVM to use the host IPv4 and IPv6 addresses"},{"line_number":172,"context_line":"                 export MANILA_LVM_SHARE_EXPORT_IP\u003d$HOST_IP,$HOST_IPV6"},{"line_number":173,"context_line":"              fi"},{"line_number":174,"context_line":"          fi"},{"line_number":175,"context_line":"          export OVERRIDE_ENABLED_SERVICES"}],"source_content_type":"text/x-yaml","patch_set":2,"id":"5f201791_a75592c8","line":172,"updated":"2017-06-23 05:15:57.000000000","message":"I don\u0027t believe HOST_IPV6 is actually set here?","commit_id":"e2afeb26ed25989b4f5b414ae759ef49522e5cf8"},{"author":{"_account_id":15100,"name":"zhongjun","email":"jun.zhongjun2@gmail.com","username":"zhongjun2"},"change_message_id":"01597e11663749a6f36051aa1ec41542f66e811e","unresolved":false,"context_lines":[{"line_number":169,"context_line":"              fi"},{"line_number":170,"context_line":"              if [ \"{node}\" \u003d\u003d \"centos-7\" ]; then"},{"line_number":171,"context_line":"                 # Set LVM to use the host IPv4 and IPv6 addresses"},{"line_number":172,"context_line":"                 export MANILA_LVM_SHARE_EXPORT_IP\u003d$HOST_IP,$HOST_IPV6"},{"line_number":173,"context_line":"              fi"},{"line_number":174,"context_line":"          fi"},{"line_number":175,"context_line":"          export OVERRIDE_ENABLED_SERVICES"}],"source_content_type":"text/x-yaml","patch_set":2,"id":"5f201791_9569a08c","line":172,"in_reply_to":"5f201791_a75592c8","updated":"2017-06-23 06:56:09.000000000","message":"Hi Ian Wienand, we will support both ipv6 and ipv4 in lvm driver in centos-7, after we merge Allow 2 or more export IPs for LVM driver patch( change MANILA_LVM_SHARE_EXPORT_IP to MANILA_LVM_SHARE_EXPORT_IPS )\nhttps://review.openstack.org/#/c/444479/ \n\nBefore we try to separate it to either ipv6 and ipv4, now we want to support both. please take a look at this link.\nhttps://review.openstack.org/#/c/443737/","commit_id":"e2afeb26ed25989b4f5b414ae759ef49522e5cf8"},{"author":{"_account_id":6547,"name":"Andreas Jaeger","email":"jaegerandi@gmail.com","username":"jaegerandi"},"change_message_id":"cb2192197596cb442803535f2f36287a74989390","unresolved":false,"context_lines":[{"line_number":169,"context_line":"              fi"},{"line_number":170,"context_line":"              if [ \"{node}\" \u003d\u003d \"centos-7\" ]; then"},{"line_number":171,"context_line":"                 # Set LVM to use the host IPv4 and IPv6 addresses"},{"line_number":172,"context_line":"                 export MANILA_LVM_SHARE_EXPORT_IPS\u003d$HOST_IP,$HOST_IPV6"},{"line_number":173,"context_line":"              fi"},{"line_number":174,"context_line":"          fi"},{"line_number":175,"context_line":"          export OVERRIDE_ENABLED_SERVICES"}],"source_content_type":"text/x-yaml","patch_set":3,"id":"5f201791_fec1c3e5","line":172,"updated":"2017-06-23 16:55:09.000000000","message":"I doubt that this will be past through, you might need to put it into DEVSTACK_LOCAL_CONFIG.\n\nBUT having distro specific differences is IMHO a really bad idea.","commit_id":"c957fdb286ec7436fe4d2cf61b654a43590e72e2"}]}