)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":1,"name":"James E. Blair","email":"jim@acmegating.com","username":"corvus"},"change_message_id":"e09c666017bdb05de3043a0aa1a427532ab5bae0","unresolved":false,"context_lines":[{"line_number":10,"context_line":"which we do not use. Remove it."},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"This adds a production symlink to itself so that we can unwind hiera"},{"line_number":13,"context_line":"paths in tests."},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"Depends-On: https://review.openstack.org/592946"},{"line_number":16,"context_line":"Change-Id: I82572cc616e3c994eab38b0de8c3c72cb5ec5413"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":7,"id":"3f79a3b5_92cf97af","line":13,"updated":"2018-08-17 14:36:47.000000000","message":"I don\u0027t see that happening.","commit_id":"172e997568484119303646cb8560d005663e9a78"},{"author":{"_account_id":2,"name":"Monty Taylor","email":"mordred@inaugust.com","username":"mordred"},"change_message_id":"514215989a7692e82d72d71863163be29fd327ef","unresolved":false,"context_lines":[{"line_number":10,"context_line":"which we do not use. Remove it."},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"This adds a production symlink to itself so that we can unwind hiera"},{"line_number":13,"context_line":"paths in tests."},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"Depends-On: https://review.openstack.org/592946"},{"line_number":16,"context_line":"Change-Id: I82572cc616e3c994eab38b0de8c3c72cb5ec5413"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":7,"id":"3f79a3b5_12bb870e","line":13,"in_reply_to":"3f79a3b5_92cf97af","updated":"2018-08-17 14:37:32.000000000","message":"Oh - whoops. We came up with another solution for that issue.","commit_id":"172e997568484119303646cb8560d005663e9a78"},{"author":{"_account_id":4146,"name":"Clark Boylan","email":"cboylan@sapwetik.org","username":"cboylan"},"change_message_id":"7a62baae4e81176406f984eb682631b7acd90886","unresolved":false,"context_lines":[{"line_number":7,"context_line":"Move /opt/system-config/production to /opt/system-config"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"The production directory is a relic from the puppet environment concept,"},{"line_number":10,"context_line":"which we do not use. Remove it."},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"The puppet apply tests run puppet locally, where the production"},{"line_number":13,"context_line":"environment is still needed, so don\u0027t update the paths in the"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":8,"id":"3f79a3b5_7bcad74f","line":10,"range":{"start_line":10,"start_character":0,"end_line":10,"end_character":19},"updated":"2018-08-17 16:20:41.000000000","message":"I\u0027m not sure this assertion is correct. I believe we use it occasionally for launch node in particular to bring up a new instance with special or new config that needs further testing.\n\nI want to say fungi made use of it recently for this purpose.\n\nThat said we don\u0027t use it often and there are likely other methods that can be used to achieve this.","commit_id":"1a8c2f66daf8ad9bd220d0cc8cfcc8cfac333a90"},{"author":{"_account_id":2,"name":"Monty Taylor","email":"mordred@inaugust.com","username":"mordred"},"change_message_id":"7af9346093ae51b6858e4c17361037d8460f4924","unresolved":false,"context_lines":[{"line_number":7,"context_line":"Move /opt/system-config/production to /opt/system-config"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"The production directory is a relic from the puppet environment concept,"},{"line_number":10,"context_line":"which we do not use. Remove it."},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"The puppet apply tests run puppet locally, where the production"},{"line_number":13,"context_line":"environment is still needed, so don\u0027t update the paths in the"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":8,"id":"3f79a3b5_5b2a7b90","line":10,"range":{"start_line":10,"start_character":0,"end_line":10,"end_character":19},"in_reply_to":"3f79a3b5_7bcad74f","updated":"2018-08-17 16:38:57.000000000","message":"Yeah - with ansible we can just run the ansible from a different directory, so the structural \u0027environment\u0027 concept which is actually a thing in puppet config is a thing we\u0027re not really making use of. (we used to back in puppetmaster days)","commit_id":"1a8c2f66daf8ad9bd220d0cc8cfcc8cfac333a90"}],"doc/source/afs.rst":[{"author":{"_account_id":4146,"name":"Clark Boylan","email":"cboylan@sapwetik.org","username":"cboylan"},"change_message_id":"7a62baae4e81176406f984eb682631b7acd90886","unresolved":false,"context_lines":[{"line_number":368,"context_line":""},{"line_number":369,"context_line":"  .. code-block:: console"},{"line_number":370,"context_line":""},{"line_number":371,"context_line":"    root@puppetmaster:~# /opt/system-config/tools/hieraedit.py \\"},{"line_number":372,"context_line":"      --yaml /etc/puppet/hieradata/production/fqdn/mirror-update.openstack.org.yaml \\"},{"line_number":373,"context_line":"      -f /path/to/foo.keytab KEYNAME"},{"line_number":374,"context_line":""}],"source_content_type":"text/x-rst","patch_set":8,"id":"3f79a3b5_5bc59b5a","line":371,"range":{"start_line":371,"start_character":9,"end_line":371,"end_character":21},"updated":"2018-08-17 16:20:41.000000000","message":"Should be updated to bridge but we can update that in a followup.","commit_id":"1a8c2f66daf8ad9bd220d0cc8cfcc8cfac333a90"},{"author":{"_account_id":2,"name":"Monty Taylor","email":"mordred@inaugust.com","username":"mordred"},"change_message_id":"7af9346093ae51b6858e4c17361037d8460f4924","unresolved":false,"context_lines":[{"line_number":368,"context_line":""},{"line_number":369,"context_line":"  .. code-block:: console"},{"line_number":370,"context_line":""},{"line_number":371,"context_line":"    root@puppetmaster:~# /opt/system-config/tools/hieraedit.py \\"},{"line_number":372,"context_line":"      --yaml /etc/puppet/hieradata/production/fqdn/mirror-update.openstack.org.yaml \\"},{"line_number":373,"context_line":"      -f /path/to/foo.keytab KEYNAME"},{"line_number":374,"context_line":""}],"source_content_type":"text/x-rst","patch_set":8,"id":"3f79a3b5_1e03910f","line":371,"range":{"start_line":371,"start_character":9,"end_line":371,"end_character":21},"in_reply_to":"3f79a3b5_5bc59b5a","updated":"2018-08-17 16:38:57.000000000","message":"Done","commit_id":"1a8c2f66daf8ad9bd220d0cc8cfcc8cfac333a90"}],"doc/source/certificate_authority.rst":[{"author":{"_account_id":4146,"name":"Clark Boylan","email":"cboylan@sapwetik.org","username":"cboylan"},"change_message_id":"7a62baae4e81176406f984eb682631b7acd90886","unresolved":false,"context_lines":[{"line_number":244,"context_line":".. code-block:: bash"},{"line_number":245,"context_line":""},{"line_number":246,"context_line":"    root@puppetmaster:~# cd /etc/zuul-ca"},{"line_number":247,"context_line":"    root@puppetmaster:/etc/zuul-ca# /opt/system-config/tools/hieraedit.py \\"},{"line_number":248,"context_line":"    \u003e --yaml /etc/puppet/hieradata/production/group/gearman.yaml \\"},{"line_number":249,"context_line":"    \u003e -f newreq.pem gearman_ssl_key"},{"line_number":250,"context_line":"    root@puppetmaster:/etc/zuul-ca# /opt/system-config/tools/hieraedit.py \\"}],"source_content_type":"text/x-rst","patch_set":8,"id":"3f79a3b5_1ba5e3b5","line":247,"range":{"start_line":247,"start_character":9,"end_line":247,"end_character":21},"updated":"2018-08-17 16:20:41.000000000","message":"Should be updated to bridge but we can update that in a followup.","commit_id":"1a8c2f66daf8ad9bd220d0cc8cfcc8cfac333a90"},{"author":{"_account_id":2,"name":"Monty Taylor","email":"mordred@inaugust.com","username":"mordred"},"change_message_id":"7af9346093ae51b6858e4c17361037d8460f4924","unresolved":false,"context_lines":[{"line_number":244,"context_line":".. code-block:: bash"},{"line_number":245,"context_line":""},{"line_number":246,"context_line":"    root@puppetmaster:~# cd /etc/zuul-ca"},{"line_number":247,"context_line":"    root@puppetmaster:/etc/zuul-ca# /opt/system-config/tools/hieraedit.py \\"},{"line_number":248,"context_line":"    \u003e --yaml /etc/puppet/hieradata/production/group/gearman.yaml \\"},{"line_number":249,"context_line":"    \u003e -f newreq.pem gearman_ssl_key"},{"line_number":250,"context_line":"    root@puppetmaster:/etc/zuul-ca# /opt/system-config/tools/hieraedit.py \\"}],"source_content_type":"text/x-rst","patch_set":8,"id":"3f79a3b5_3e49ad5f","line":247,"range":{"start_line":247,"start_character":9,"end_line":247,"end_character":21},"in_reply_to":"3f79a3b5_1ba5e3b5","updated":"2018-08-17 16:38:57.000000000","message":"Done","commit_id":"1a8c2f66daf8ad9bd220d0cc8cfcc8cfac333a90"}]}