)]}'
{"jenkins/jobs/nova.yaml":[{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"9b16b61f386a02ad21514167f46372ab8f97cf16","unresolved":false,"context_lines":[{"line_number":64,"context_line":"      - devstack-checkout"},{"line_number":65,"context_line":"      - shell: |"},{"line_number":66,"context_line":"          #!/bin/bash -xe"},{"line_number":67,"context_line":"          export ENABLED_SERVICES\u003dplacement-api,n-novnc"},{"line_number":68,"context_line":"          export PYTHONUNBUFFERED\u003dtrue"},{"line_number":69,"context_line":"          export DEVSTACK_GATE_TEMPEST\u003d1"},{"line_number":70,"context_line":"          export DEVSTACK_GATE_TEMPEST_FULL\u003d1"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"3a71b18c_90d33e38","line":67,"updated":"2016-12-01 23:52:37.000000000","message":"If we do this patch:\n\nhttps://review.openstack.org/#/c/345626/\n\nThen you get the n-novnc service in all jobs that use nova.","commit_id":"6f15e9ef0cdb94cb686c9bbafb34903dd88f2c25"},{"author":{"_account_id":13252,"name":"Dr. Jens Harbott","display_name":"Jens Harbott (frickler)","email":"frickler@offenerstapel.de","username":"jrosenboom"},"change_message_id":"9d8009e43b810b739355252fc161632102c05822","unresolved":false,"context_lines":[{"line_number":80,"context_line":"          export DEVSTACK_GATE_NEUTRON\u003d1"},{"line_number":81,"context_line":"          # NOTE(mriedem): cells v2 is required starting in Ocata."},{"line_number":82,"context_line":"          export DEVSTACK_LOCAL_CONFIG\u003d\"NOVA_CONFIGURE_CELLSV2\u003dTrue\""},{"line_number":83,"context_line":"          # Enable TLS between the noVNC proxy \u0026 compute nodes"},{"line_number":84,"context_line":"          export DEVSTACK_LOCAL_CONFIG+\u003d$\u0027\\n\u0027\"NOVA_CONSOLE_PROXY_COMPUTE_TLS\u003dTrue\""},{"line_number":85,"context_line":"          export BRANCH_OVERRIDE\u003d{branch-override}"},{"line_number":86,"context_line":"          if [ \"$BRANCH_OVERRIDE\" !\u003d \"default\" ] ; then"}],"source_content_type":"text/x-yaml","patch_set":2,"id":"dfeb2761_931779a4","line":83,"updated":"2017-04-03 13:28:19.000000000","message":"This sounds strange to me, as the job being changed is single-node, is this really the intended scenario?","commit_id":"2dfe54f02e32eeb8d17bf1f2cec6918172ff4190"},{"author":{"_account_id":6804,"name":"bruce-benjamin","email":"bruce.benjamin@jhuapl.edu","username":"bruce-benjamin"},"change_message_id":"59a18059b1d4482f33e588e1232871278732802f","unresolved":false,"context_lines":[{"line_number":80,"context_line":"          export DEVSTACK_GATE_NEUTRON\u003d1"},{"line_number":81,"context_line":"          # NOTE(mriedem): cells v2 is required starting in Ocata."},{"line_number":82,"context_line":"          export DEVSTACK_LOCAL_CONFIG\u003d\"NOVA_CONFIGURE_CELLSV2\u003dTrue\""},{"line_number":83,"context_line":"          # Enable TLS between the noVNC proxy \u0026 compute nodes"},{"line_number":84,"context_line":"          export DEVSTACK_LOCAL_CONFIG+\u003d$\u0027\\n\u0027\"NOVA_CONSOLE_PROXY_COMPUTE_TLS\u003dTrue\""},{"line_number":85,"context_line":"          export BRANCH_OVERRIDE\u003d{branch-override}"},{"line_number":86,"context_line":"          if [ \"$BRANCH_OVERRIDE\" !\u003d \"default\" ] ; then"}],"source_content_type":"text/x-yaml","patch_set":2,"id":"bff0334d_6e8de6d1","line":83,"in_reply_to":"dfeb2761_931779a4","updated":"2017-04-10 16:11:54.000000000","message":"If I understand this correctly, I believe that ssh is solely used for a multi-node use case using devstack https://docs.openstack.org/developer/devstack/guides/multinode-lab.html. If that\u0027s so, then I think this patch is still valid if it includes a comment to say that TLS is only supported for a single node at this time.","commit_id":"2dfe54f02e32eeb8d17bf1f2cec6918172ff4190"}]}