)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":15197,"name":"Pierre Riteau","email":"pierre@stackhpc.com","username":"priteau","status":"StackHPC"},"change_message_id":"c5087533695c54c455cdab5a1f75e9a9a1888e18","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":2,"id":"31183e19_4d0253b4","updated":"2022-12-20 12:46:56.000000000","message":"Needs a bit more testing as it could add unrequired cafile entries when not using internal TLS.","commit_id":"aba636e1411c645f0359f8d6f658446e41dd7f71"},{"author":{"_account_id":14826,"name":"Mark Goddard","email":"markgoddard86@gmail.com","username":"mgoddard"},"change_message_id":"c00b329faa2fbb664ba488fb19bd3e857ce3a7bc","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":2,"id":"7ac2806b_38108424","in_reply_to":"31183e19_4d0253b4","updated":"2022-12-20 13:40:41.000000000","message":"It\u0027s not a problem - we do it everywhere","commit_id":"aba636e1411c645f0359f8d6f658446e41dd7f71"},{"author":{"_account_id":15197,"name":"Pierre Riteau","email":"pierre@stackhpc.com","username":"priteau","status":"StackHPC"},"change_message_id":"24d608f3013ae37da29ac79b3511004b11830ade","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"6ef66032_7488ca2f","in_reply_to":"7ac2806b_38108424","updated":"2024-05-14 07:03:33.000000000","message":"Acknowledged","commit_id":"aba636e1411c645f0359f8d6f658446e41dd7f71"},{"author":{"_account_id":22629,"name":"Michal Nasiadka","email":"mnasiadka@gmail.com","username":"mnasiadka"},"change_message_id":"876b608fc3bfbd42840c500228149a003714967d","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":7,"id":"a4002127_92a18534","updated":"2024-07-15 12:41:35.000000000","message":"been here too long","commit_id":"78e7f133f0b5462ca5449b34c9aed7745c14600b"}],"ansible/roles/cloudkitty/tasks/bootstrap.yml":[{"author":{"_account_id":32553,"name":"Sven Kieske","email":"sven_oss@posteo.de","username":"skieske"},"change_message_id":"022262b0ca1cc935ea14dcdb92ed2380c1aee8a8","unresolved":true,"context_lines":[{"line_number":45,"context_line":"      port: \"{{ influxdb_http_port }}\""},{"line_number":46,"context_line":"      ssl: \"{{ cloudkitty_influxdb_use_ssl | bool }}\""},{"line_number":47,"context_line":"      database_name: \"{{ cloudkitty_influxdb_name }}\""},{"line_number":48,"context_line":"      # The influxdb_database module and the InfluxDB 1.x Python client don\u0027t"},{"line_number":49,"context_line":"      # support specifying a CA certificate file."},{"line_number":50,"context_line":"      validate_certs: False"},{"line_number":51,"context_line":"  run_once: True"}],"source_content_type":"text/x-yaml","patch_set":7,"id":"83dd0202_622b90ac","line":48,"range":{"start_line":48,"start_character":45,"end_line":48,"end_character":58},"updated":"2024-05-14 10:10:59.000000000","message":"do we still support influxDB 1.X? the last release was in 2021, according to:\nhttps://docs.influxdata.com/influxdb/v1/about_the_project/release-notes/\n\nI\u0027m -1 on disabling cert validation - which is a security issue - for a product which seems EOL to me.","commit_id":"78e7f133f0b5462ca5449b34c9aed7745c14600b"},{"author":{"_account_id":15197,"name":"Pierre Riteau","email":"pierre@stackhpc.com","username":"priteau","status":"StackHPC"},"change_message_id":"b2251b34feec751239e61b3dbac1ac2fc2fe443a","unresolved":true,"context_lines":[{"line_number":45,"context_line":"      port: \"{{ influxdb_http_port }}\""},{"line_number":46,"context_line":"      ssl: \"{{ cloudkitty_influxdb_use_ssl | bool }}\""},{"line_number":47,"context_line":"      database_name: \"{{ cloudkitty_influxdb_name }}\""},{"line_number":48,"context_line":"      # The influxdb_database module and the InfluxDB 1.x Python client don\u0027t"},{"line_number":49,"context_line":"      # support specifying a CA certificate file."},{"line_number":50,"context_line":"      validate_certs: False"},{"line_number":51,"context_line":"  run_once: True"}],"source_content_type":"text/x-yaml","patch_set":7,"id":"962526f5_420c36b9","line":48,"range":{"start_line":48,"start_character":45,"end_line":48,"end_character":58},"in_reply_to":"83dd0202_622b90ac","updated":"2024-05-14 10:25:40.000000000","message":"Kolla still packages InfluxDB 1.x, see for example the latest master-rocky-9 image which includes influxdb 1.8.10-1: https://quay.io/repository/openstack.kolla/influxdb/manifest/sha256:2b38734a39ce59170b452c2f1ee05a2a7611bf515a1feaf9cecef736e607669e?tab\u003dpackages\n\nThis only disables SSL verification for an internal endpoint during creation of the index. CloudKitty itself will verify SSL during runtime. The alternative for people who want to use CloudKitty with InfluxDB at the moment is to completely disable internal TLS. Isn\u0027t this much worse?","commit_id":"78e7f133f0b5462ca5449b34c9aed7745c14600b"},{"author":{"_account_id":32553,"name":"Sven Kieske","email":"sven_oss@posteo.de","username":"skieske"},"change_message_id":"5b2f26b501cb55a0b66e17f3503955f0b02c13db","unresolved":false,"context_lines":[{"line_number":45,"context_line":"      port: \"{{ influxdb_http_port }}\""},{"line_number":46,"context_line":"      ssl: \"{{ cloudkitty_influxdb_use_ssl | bool }}\""},{"line_number":47,"context_line":"      database_name: \"{{ cloudkitty_influxdb_name }}\""},{"line_number":48,"context_line":"      # The influxdb_database module and the InfluxDB 1.x Python client don\u0027t"},{"line_number":49,"context_line":"      # support specifying a CA certificate file."},{"line_number":50,"context_line":"      validate_certs: False"},{"line_number":51,"context_line":"  run_once: True"}],"source_content_type":"text/x-yaml","patch_set":7,"id":"a56b61eb_9138b71a","line":48,"range":{"start_line":48,"start_character":45,"end_line":48,"end_character":58},"in_reply_to":"962526f5_420c36b9","updated":"2024-05-14 10:48:59.000000000","message":"Acknowledged","commit_id":"78e7f133f0b5462ca5449b34c9aed7745c14600b"}],"ansible/roles/cloudkitty/templates/cloudkitty.conf.j2":[{"author":{"_account_id":14826,"name":"Mark Goddard","email":"markgoddard86@gmail.com","username":"mgoddard"},"change_message_id":"629b6f8f7ba1e2846a63d6329b99484514daf64b","unresolved":true,"context_lines":[{"line_number":134,"context_line":"retention_policy \u003d {{ cloudkitty_influxdb_retention_policy }}"},{"line_number":135,"context_line":"{% endif %}"},{"line_number":136,"context_line":""},{"line_number":137,"context_line":"{% if cloudkitty_influxdb_use_ssl is defined %}"},{"line_number":138,"context_line":"use_ssl \u003d {{ cloudkitty_influxdb_use_ssl }}"},{"line_number":139,"context_line":"{% endif %}"},{"line_number":140,"context_line":""}],"source_content_type":"text/x-jinja2","patch_set":7,"id":"fe99302a_9a2d039b","line":137,"updated":"2024-05-15 16:08:38.000000000","message":"nit: This and other modified variables are now always defined.","commit_id":"78e7f133f0b5462ca5449b34c9aed7745c14600b"}]}
