)]}'
{"inventory/service/groups.yaml":[{"author":{"_account_id":4146,"name":"Clark Boylan","email":"cboylan@sapwetik.org","username":"cboylan"},"change_message_id":"2abca9ccb25518ebb5c804c5a7c92f24c55d1e29","unresolved":true,"context_lines":[{"line_number":167,"context_line":"    - storyboard-dev[0-9]*.opendev.org"},{"line_number":168,"context_line":"    - storyboard[0-9]*.opendev.org"},{"line_number":169,"context_line":"    - translate-dev[0-9]*.open*.org"},{"line_number":170,"context_line":"    - translate[0-9]*.open*.org"},{"line_number":171,"context_line":"  zookeeper:"},{"line_number":172,"context_line":"    - zk[0-9]*.open*.org"},{"line_number":173,"context_line":"  zuul-lb:"}],"source_content_type":"text/x-yaml","patch_set":2,"id":"75cef6e2_272d8ac5","line":170,"updated":"2022-09-06 22:23:15.000000000","message":"Do we need to add tracing here to also open port 443?","commit_id":"4a2a8e2b4e7e23c09e122c7e1075880e13f71ec9"},{"author":{"_account_id":1,"name":"James E. Blair","email":"jim@acmegating.com","username":"corvus"},"change_message_id":"ede9686936c597a4f61649c9be29743b3a7f0233","unresolved":false,"context_lines":[{"line_number":167,"context_line":"    - storyboard-dev[0-9]*.opendev.org"},{"line_number":168,"context_line":"    - storyboard[0-9]*.opendev.org"},{"line_number":169,"context_line":"    - translate-dev[0-9]*.open*.org"},{"line_number":170,"context_line":"    - translate[0-9]*.open*.org"},{"line_number":171,"context_line":"  zookeeper:"},{"line_number":172,"context_line":"    - zk[0-9]*.open*.org"},{"line_number":173,"context_line":"  zuul-lb:"}],"source_content_type":"text/x-yaml","patch_set":2,"id":"6d3c7765_fa8624ef","line":170,"in_reply_to":"75cef6e2_272d8ac5","updated":"2022-09-06 22:35:58.000000000","message":"Probably so.","commit_id":"4a2a8e2b4e7e23c09e122c7e1075880e13f71ec9"}],"playbooks/roles/jaeger/templates/docker-compose.yaml.j2":[{"author":{"_account_id":4146,"name":"Clark Boylan","email":"cboylan@sapwetik.org","username":"cboylan"},"change_message_id":"2abca9ccb25518ebb5c804c5a7c92f24c55d1e29","unresolved":true,"context_lines":[{"line_number":1,"context_line":"# Version 2 is the latest that is supported by docker-compose in"},{"line_number":2,"context_line":"# Ubuntu Xenial."},{"line_number":3,"context_line":"version: \u00272\u0027"},{"line_number":4,"context_line":""}],"source_content_type":"text/x-jinja2","patch_set":2,"id":"2c4bcad4_9448e0b9","line":1,"updated":"2022-09-06 22:23:15.000000000","message":"Doesn\u0027t look like this file actually templates any info? Might want to copy it instead of writing it out as a template.","commit_id":"4a2a8e2b4e7e23c09e122c7e1075880e13f71ec9"},{"author":{"_account_id":1,"name":"James E. Blair","email":"jim@acmegating.com","username":"corvus"},"change_message_id":"ede9686936c597a4f61649c9be29743b3a7f0233","unresolved":true,"context_lines":[{"line_number":1,"context_line":"# Version 2 is the latest that is supported by docker-compose in"},{"line_number":2,"context_line":"# Ubuntu Xenial."},{"line_number":3,"context_line":"version: \u00272\u0027"},{"line_number":4,"context_line":""}],"source_content_type":"text/x-jinja2","patch_set":2,"id":"4f975196_b42af382","line":1,"in_reply_to":"2c4bcad4_9448e0b9","updated":"2022-09-06 22:35:58.000000000","message":"We could, though I kind of like the idea of always putting files that we are likely to template in templates, but that mostly comes out of previously having spent an inordinate time moving them around when we decide to template or not template a single field...\n\nI don\u0027t think this is the first non-templated file in templates.","commit_id":"4a2a8e2b4e7e23c09e122c7e1075880e13f71ec9"},{"author":{"_account_id":4146,"name":"Clark Boylan","email":"cboylan@sapwetik.org","username":"cboylan"},"change_message_id":"2abca9ccb25518ebb5c804c5a7c92f24c55d1e29","unresolved":true,"context_lines":[{"line_number":16,"context_line":"      - BADGER_SPAN_STORE_TTL\u003d30d"},{"line_number":17,"context_line":"      - COLLECTOR_GRPC_TLS_CERT\u003d/tls/certs/cert.pem"},{"line_number":18,"context_line":"      - COLLECTOR_GRPC_TLS_KEY\u003d/tls/certs/key.pem"},{"line_number":19,"context_line":"      - COLLECTOR_GRPC_TLS_CLIENT_CA\u003d/tls/certs/cacert.pem"},{"line_number":20,"context_line":"    volumes:"},{"line_number":21,"context_line":"      - /var/jaeger/badger:/badger"},{"line_number":22,"context_line":"      - /var/jaeger/tls:/tls"}],"source_content_type":"text/x-jinja2","patch_set":2,"id":"0fa3b4db_53f49480","line":19,"updated":"2022-09-06 22:23:15.000000000","message":"The CI error appears to be:\n\n{\n\"level\":\"fatal\",\n\"ts\":1662413582.347335,\n\"caller\":\"./main.go:152\",\n\"msg\":\"Failed to initialize collector\",\n\"error\":\"failed to parse gRPC server options: failed to parse gRPC TLS options: collector.grpc.tls.* options cannot be used when collector.grpc.tls.enabled is false\",\n\"stacktrace\":\n  \"main.main.func1\n    ./main.go:152\n  github.com/spf13/cobra.(*Command).execute\n    github.com/spf13/cobra@v1.5.0/command.go:872\n  github.com/spf13/cobra.(*Command).ExecuteC\n    github.com/spf13/cobra@v1.5.0/command.go:990\n  github.com/spf13/cobra.(*Command).Execute\n    github.com/spf13/cobra@v1.5.0/command.go:918\n  main.main\n    ./main.go:240\n  runtime.main\n    runtime/proc.go:250\"\n}\n\nI suspect you need to set something like COLLECTOR_GRPC_TLS_ENABLED\u003dtrue","commit_id":"4a2a8e2b4e7e23c09e122c7e1075880e13f71ec9"},{"author":{"_account_id":1,"name":"James E. Blair","email":"jim@acmegating.com","username":"corvus"},"change_message_id":"ede9686936c597a4f61649c9be29743b3a7f0233","unresolved":false,"context_lines":[{"line_number":16,"context_line":"      - BADGER_SPAN_STORE_TTL\u003d30d"},{"line_number":17,"context_line":"      - COLLECTOR_GRPC_TLS_CERT\u003d/tls/certs/cert.pem"},{"line_number":18,"context_line":"      - COLLECTOR_GRPC_TLS_KEY\u003d/tls/certs/key.pem"},{"line_number":19,"context_line":"      - COLLECTOR_GRPC_TLS_CLIENT_CA\u003d/tls/certs/cacert.pem"},{"line_number":20,"context_line":"    volumes:"},{"line_number":21,"context_line":"      - /var/jaeger/badger:/badger"},{"line_number":22,"context_line":"      - /var/jaeger/tls:/tls"}],"source_content_type":"text/x-jinja2","patch_set":2,"id":"a237585f_51597e12","line":19,"in_reply_to":"0fa3b4db_53f49480","updated":"2022-09-06 22:35:58.000000000","message":"Yep!","commit_id":"4a2a8e2b4e7e23c09e122c7e1075880e13f71ec9"}],"testinfra/test_tracing.py":[{"author":{"_account_id":7118,"name":"Ian Wienand","email":"iwienand@redhat.com","username":"iwienand"},"change_message_id":"7ee8067a6c95f3b00b68c222b15d4ce2664f593d","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":6,"id":"50ddb637_2f2e0406","line":22,"updated":"2022-09-16 01:02:34.000000000","message":"https://opendev.org/opendev/system-config/src/branch/master/testinfra/test_paste.py\n\nhas some examples of how we could actually hit the serivce to ensure the webpage returns; we also have a library to take some screenshots which is further validation (not too important here as we\u0027re not modifying the UI in ways we want to verify with that, but still handy to see what is coming up).  it might be nice to start with this being tested","commit_id":"51fa9ac75a6bc45f7161ca7fa361c61acad74e69"}],"zuul.d/system-config-run.yaml":[{"author":{"_account_id":7118,"name":"Ian Wienand","email":"iwienand@redhat.com","username":"iwienand"},"change_message_id":"da617b6cc83239e7b0a44c64d8a4522c4f8a6f0c","unresolved":true,"context_lines":[{"line_number":809,"context_line":"      nodes:"},{"line_number":810,"context_line":"        - name: bridge.openstack.org"},{"line_number":811,"context_line":"          label: ubuntu-bionic"},{"line_number":812,"context_line":"        - name: tracing01.opendev.org"},{"line_number":813,"context_line":"          label: ubuntu-focal"},{"line_number":814,"context_line":"    vars:"},{"line_number":815,"context_line":"      run_playbooks:"}],"source_content_type":"text/x-yaml","patch_set":3,"id":"c6f9a27b_9c02a2c2","line":812,"updated":"2022-09-07 22:31:31.000000000","message":"for new ones like this, could we make this \"tracing99\" instead of \"01\"?  We\u0027ve found this is a good way to find group/host problems and make sure testing is separate to production (especially now as we have a trusted test-local CA for self-signed certs setup, so \"tracing99.opendev.org\" will be valid for testing against) from all hosts.\n\nFor the LE cert var, it can be done in group vars file, with {{ inventory_hostname }} ... paste99 is an example.  Doesn\u0027t seem like this has much other content in vars to deal with.","commit_id":"5082b41866b389cd17bfb2c47a1cd9e630f3fa93"},{"author":{"_account_id":1,"name":"James E. Blair","email":"jim@acmegating.com","username":"corvus"},"change_message_id":"f23a25d2aa8a2e6119784fefa59f3b1a5023a36f","unresolved":false,"context_lines":[{"line_number":809,"context_line":"      nodes:"},{"line_number":810,"context_line":"        - name: bridge.openstack.org"},{"line_number":811,"context_line":"          label: ubuntu-bionic"},{"line_number":812,"context_line":"        - name: tracing01.opendev.org"},{"line_number":813,"context_line":"          label: ubuntu-focal"},{"line_number":814,"context_line":"    vars:"},{"line_number":815,"context_line":"      run_playbooks:"}],"source_content_type":"text/x-yaml","patch_set":3,"id":"e1063635_119bba67","line":812,"in_reply_to":"c6f9a27b_9c02a2c2","updated":"2022-09-14 18:01:44.000000000","message":"Done.","commit_id":"5082b41866b389cd17bfb2c47a1cd9e630f3fa93"}]}