)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":28459,"name":"Bob Church","email":"robert.church@windriver.com","username":"rchurch"},"change_message_id":"f0bcdda4ae3f6cc146bd39a80f43662094874011","unresolved":true,"context_lines":[{"line_number":10,"context_line":"supported and evolving set of provisioners based on"},{"line_number":11,"context_line":"https://github.com/ceph/ceph-csi version 3.6.2."},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"Test Plan:"},{"line_number":14,"context_line":"PASS: AIO-SX app upload/apply/remove/delete"},{"line_number":15,"context_line":"PASS: AIO-DX app upload/apply/remove/delete"},{"line_number":16,"context_line":"PASS: Storage 2+2+2 app upload/apply/remove/delete"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":8,"id":"4f478509_e6afc222","line":13,"updated":"2022-11-03 09:11:25.000000000","message":"What about support for application-update? Will this break platform upgrades when updating from the previous version in stx.7.0? If so we need a temporary upgrade activation script that does a remove/delete/upload/apply to allow upgrade testing to not be impacted.","commit_id":"6c316bdffdd9896847ba06a3c38cde2cf98a9b71"},{"author":{"_account_id":34926,"name":"Hediberto Cavalcante da Silva","display_name":"Hediberto C Silva","email":"hediberto.cavalcantedasilva@windriver.com","username":"hcavalca"},"change_message_id":"5f82221d387d6cce8a513f5695d8c6dec75e1b9b","unresolved":true,"context_lines":[{"line_number":10,"context_line":"supported and evolving set of provisioners based on"},{"line_number":11,"context_line":"https://github.com/ceph/ceph-csi version 3.6.2."},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"Test Plan:"},{"line_number":14,"context_line":"PASS: AIO-SX app upload/apply/remove/delete"},{"line_number":15,"context_line":"PASS: AIO-DX app upload/apply/remove/delete"},{"line_number":16,"context_line":"PASS: Storage 2+2+2 app upload/apply/remove/delete"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":8,"id":"bd415f05_0e59f9e0","line":13,"in_reply_to":"4f478509_e6afc222","updated":"2022-11-04 23:39:20.000000000","message":"I still testing the application-update. This feature has an already identified issue that will be fixed by story 2009966. I will try these suggested fixes ASAP.\nAbout platform upgrade, the test is in progress.","commit_id":"6c316bdffdd9896847ba06a3c38cde2cf98a9b71"},{"author":{"_account_id":34926,"name":"Hediberto Cavalcante da Silva","display_name":"Hediberto C Silva","email":"hediberto.cavalcantedasilva@windriver.com","username":"hcavalca"},"change_message_id":"12d460794d699550f6a84d52f7f8b11a5fde66b0","unresolved":false,"context_lines":[{"line_number":10,"context_line":"supported and evolving set of provisioners based on"},{"line_number":11,"context_line":"https://github.com/ceph/ceph-csi version 3.6.2."},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"Test Plan:"},{"line_number":14,"context_line":"PASS: AIO-SX app upload/apply/remove/delete"},{"line_number":15,"context_line":"PASS: AIO-DX app upload/apply/remove/delete"},{"line_number":16,"context_line":"PASS: Storage 2+2+2 app upload/apply/remove/delete"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":8,"id":"a8cc8181_c4c5dd72","line":13,"in_reply_to":"bd415f05_0e59f9e0","updated":"2022-11-07 22:38:20.000000000","message":"The upgrade activation script was required. Story: 2009987. Task: 46752.","commit_id":"6c316bdffdd9896847ba06a3c38cde2cf98a9b71"}],"/PATCHSET_LEVEL":[{"author":{"_account_id":33443,"name":"Felipe Sanches Zanoni","email":"Felipe.SanchesZanoni@windriver.com","username":"fsanches"},"change_message_id":"3530e9fdf3d0e3198e03a53ba8f6c22232677ae8","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"3c6f2fb0_b1eb4786","updated":"2022-10-27 18:56:34.000000000","message":"recheck","commit_id":"76d1fc65617cf35706d6452cfb78b0949d0f78f1"},{"author":{"_account_id":28459,"name":"Bob Church","email":"robert.church@windriver.com","username":"rchurch"},"change_message_id":"31104db08385fa46b0f3ab6622d7ad73e5b07bec","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":8,"id":"0f94bc62_ee1d371d","updated":"2022-11-04 18:54:03.000000000","message":"The packages with the tarball and the plugins should have the same versioning\n\nThe plugins package needs to have the following changes\n\n modified   python-k8sapp-platform/debian/deb_folder/python3-k8sapp-platform.install\n @@ -1,2 +1,2 @@\n -usr/lib/python3/dist-packages/k8sapp_platform-1.0.0.egg-info/*\n +usr/lib/python3/dist-packages/k8sapp_platform-1.0.*.egg-info/*\n  usr/lib/python3/dist-packages/k8sapp_platform/*\n modified   python-k8sapp-platform/debian/deb_folder/rules\n @@ -2,7 +2,12 @@\n  # export DH_VERBOSE \u003d 1\n  \n  export APP_NAME\u003dplatform-integ-apps\n -export PBR_VERSION\u003d1.0.0\n +\n +export DEB_VERSION \u003d $(shell dpkg-parsechangelog | egrep \u0027^Version:\u0027 | cut -f 2 -d \u0027 \u0027)\n +export MAJOR \u003d $(shell echo $(DEB_VERSION) | cut -f 1 -d \u0027-\u0027)\n +export MINOR_PATCH \u003d $(shell echo $(DEB_VERSION) | cut -f 4 -d \u0027.\u0027)\n +export PBR_VERSION\u003d$(MAJOR).$(MINOR_PATCH)\n +\n  export PYBUILD_NAME\u003dk8sapp-platform\n  export SKIP_PIP_INSTALL\u003d1\n  export ROOT\u003ddebian/tmp\n \n modified   python-k8sapp-platform/debian/meta_data.yaml\n @@ -4,4 +4,6 @@ debver: 1.0-1\n  src_path: k8sapp_platform\n  revision:\n    dist: $STX_DIST\n -  PKG_GITREVCOUNT: true\n +  GITREVCOUNT:\n +    BASE_SRCREV: c608f2aaa92064b712e7076e4141a162b78fe995\n +    SRC_DIR: ${MY_REPO}/stx/platform-armada-app\n\nThis should produce:\n\n python3-k8sapp-platform/python3-k8sapp-platform_1.0-1.stx.94_amd64.deb:\n -rw-r--r-- root/root        90 2022-11-04 03:50 ./usr/lib/python3/dist-packages/k8sapp_platform/__init__.py\n -rw-r--r-- root/root        90 2022-11-04 03:50 ./usr/lib/python3/dist-packages/k8sapp_platform/common/__init__.py\n -rw-r--r-- root/root      1393 2022-11-04 03:50 ./usr/lib/python3/dist-packages/k8sapp_platform/common/constants.py\n -rw-r--r-- root/root       384 2022-11-04 03:50 ./usr/lib/python3/dist-packages/k8sapp_platform/helm/__init__.py\n -rw-r--r-- root/root      8363 2022-11-04 03:50 ./usr/lib/python3/dist-packages/k8sapp_platform/helm/ceph_fs_provisioner.py\n -rw-r--r-- root/root      3571 2022-11-04 03:50 ./usr/lib/python3/dist-packages/k8sapp_platform/helm/ceph_pools_audit.py\n -rw-r--r-- root/root      4767 2022-11-04 03:50 ./usr/lib/python3/dist-packages/k8sapp_platform/helm/rbd_provisioner.py\n -rw-r--r-- root/root       384 2022-11-04 03:50 ./usr/lib/python3/dist-packages/k8sapp_platform/kustomize/__init__.py\n -rw-r--r-- root/root       723 2022-11-04 03:50 ./usr/lib/python3/dist-packages/k8sapp_platform/kustomize/kustomize_platform.py\n -rw-r--r-- root/root        90 2022-11-04 03:50 ./usr/lib/python3/dist-packages/k8sapp_platform/lifecycle/__init__.py\n -rw-r--r-- root/root      4409 2022-11-04 03:50 ./usr/lib/python3/dist-packages/k8sapp_platform/lifecycle/lifecycle_platform.py\n -rw-r--r-- root/root        90 2022-11-04 03:50 ./usr/lib/python3/dist-packages/k8sapp_platform/tests/__init__.py\n -rw-r--r-- root/root      1210 2022-11-04 03:50 ./usr/lib/python3/dist-packages/k8sapp_platform/tests/test_plugins.py\n -rw-r--r-- root/root      1220 2022-11-04 03:50 ./usr/lib/python3/dist-packages/k8sapp_platform-1.0.94.egg-info/PKG-INFO\n -rw-r--r-- root/root       804 2022-11-04 03:50 ./usr/lib/python3/dist-packages/k8sapp_platform-1.0.94.egg-info/SOURCES.txt\n -rw-r--r-- root/root         1 2022-11-04 03:50 ./usr/lib/python3/dist-packages/k8sapp_platform-1.0.94.egg-info/dependency_links.txt\n -rw-r--r-- root/root       659 2022-11-04 03:50 ./usr/lib/python3/dist-packages/k8sapp_platform-1.0.94.egg-info/entry_points.txt\n -rw-r--r-- root/root         1 2022-11-04 03:50 ./usr/lib/python3/dist-packages/k8sapp_platform-1.0.94.egg-info/not-zip-safe\n -rw-r--r-- root/root         0 2022-11-04 03:50 ./usr/lib/python3/dist-packages/k8sapp_platform-1.0.94.egg-info/requires.txt\n -rw-r--r-- root/root        16 2022-11-04 03:50 ./usr/lib/python3/dist-packages/k8sapp_platform-1.0.94.egg-info/top_level.txt\n -rw-r--r-- root/root       247 2022-11-04 03:50 ./usr/share/doc/python3-k8sapp-platform/changelog.Debian.gz\n -rw-r--r-- root/root      1802 2022-11-04 03:50 ./usr/share/doc/python3-k8sapp-platform/copyright\n\n\n","commit_id":"6c316bdffdd9896847ba06a3c38cde2cf98a9b71"},{"author":{"_account_id":28459,"name":"Bob Church","email":"robert.church@windriver.com","username":"rchurch"},"change_message_id":"f0bcdda4ae3f6cc146bd39a80f43662094874011","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":8,"id":"4473acc9_fccb5d96","updated":"2022-11-03 09:11:25.000000000","message":"We need to:\n- drop the helm-toolkit dependencies here so that we simplify the build dependencies and allows us to decouple from openstack-helm\n- make sure that the new charts are versioned correctly\n- cleanup the CentOS builds as this change would break if someone tries to build the app in CentOS.\n- drop the Armada related plugin code.","commit_id":"6c316bdffdd9896847ba06a3c38cde2cf98a9b71"},{"author":{"_account_id":28459,"name":"Bob Church","email":"robert.church@windriver.com","username":"rchurch"},"change_message_id":"261a9b5e339541e88808a07c20c6ef8e5e6aba71","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":8,"id":"b3a9c052_7679b1d5","in_reply_to":"0f94bc62_ee1d371d","updated":"2022-11-05 18:19:25.000000000","message":"Missing these changes still.","commit_id":"6c316bdffdd9896847ba06a3c38cde2cf98a9b71"},{"author":{"_account_id":34926,"name":"Hediberto Cavalcante da Silva","display_name":"Hediberto C Silva","email":"hediberto.cavalcantedasilva@windriver.com","username":"hcavalca"},"change_message_id":"4eebd0560ab01cc6c62e93497ed9d024d5a7922a","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":8,"id":"45eb499c_8b53f004","in_reply_to":"b3a9c052_7679b1d5","updated":"2022-11-05 19:41:17.000000000","message":"Done","commit_id":"6c316bdffdd9896847ba06a3c38cde2cf98a9b71"},{"author":{"_account_id":34926,"name":"Hediberto Cavalcante da Silva","display_name":"Hediberto C Silva","email":"hediberto.cavalcantedasilva@windriver.com","username":"hcavalca"},"change_message_id":"437f7ceffb6434d76f707df7b42993b5d7881efd","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":10,"id":"b3a43445_fde770ed","updated":"2022-11-04 23:46:23.000000000","message":"I removed the centos_build_layer.cfg, centos_iso_image.inc and centos_pkg_dirs files as it was asked to remove the centos build files from python-k8sapp-platform and stx-platform-helm. Please, let me know if it was still needed.","commit_id":"ff78686ac8d87d875fa022a0721f4d09379a17c3"},{"author":{"_account_id":28459,"name":"Bob Church","email":"robert.church@windriver.com","username":"rchurch"},"change_message_id":"261a9b5e339541e88808a07c20c6ef8e5e6aba71","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":10,"id":"9a6f7544_37d48535","updated":"2022-11-05 18:19:25.000000000","message":"Thanks for the changes. A few minor ones remaining.","commit_id":"ff78686ac8d87d875fa022a0721f4d09379a17c3"},{"author":{"_account_id":34926,"name":"Hediberto Cavalcante da Silva","display_name":"Hediberto C Silva","email":"hediberto.cavalcantedasilva@windriver.com","username":"hcavalca"},"change_message_id":"4eebd0560ab01cc6c62e93497ed9d024d5a7922a","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":10,"id":"a43ea701_dc8ab0d4","in_reply_to":"b3a43445_fde770ed","updated":"2022-11-05 19:41:17.000000000","message":"I removed the python-k8sapp-platform/k8sapp_platform/k8sapp_platform/helm/helm_toolkit.py file too and the helm_toolkit was removed from python-k8sapp-platform/k8sapp_platform/setup.cfg","commit_id":"ff78686ac8d87d875fa022a0721f4d09379a17c3"},{"author":{"_account_id":28459,"name":"Bob Church","email":"robert.church@windriver.com","username":"rchurch"},"change_message_id":"261a9b5e339541e88808a07c20c6ef8e5e6aba71","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":10,"id":"e4bf6c11_62b95308","in_reply_to":"b3a43445_fde770ed","updated":"2022-11-05 18:19:25.000000000","message":"Nope. Not needed. Thanks.","commit_id":"ff78686ac8d87d875fa022a0721f4d09379a17c3"},{"author":{"_account_id":28459,"name":"Bob Church","email":"robert.church@windriver.com","username":"rchurch"},"change_message_id":"05015fb2917f0127225879e27d66463c300c2b7d","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":11,"id":"09965afc_870c0a59","updated":"2022-11-06 16:47:06.000000000","message":"I think the cephfs and rbd provisioner charts need their ServiceAccounts patched to include \n\nimagePullSecrets:\n  - name: default-registry-key\n\nor you will run into image pull issues on a controller-1, depending on what is in the local image cache","commit_id":"05907fe190c1761eed38f668b60471b65ccad206"},{"author":{"_account_id":34926,"name":"Hediberto Cavalcante da Silva","display_name":"Hediberto C Silva","email":"hediberto.cavalcantedasilva@windriver.com","username":"hcavalca"},"change_message_id":"2b60e1c5abaec6080b5c711b3cb708ba01b7f785","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":11,"id":"93243ea4_15e761eb","in_reply_to":"09965afc_870c0a59","updated":"2022-11-06 23:07:48.000000000","message":"Yes. It was missing.","commit_id":"05907fe190c1761eed38f668b60471b65ccad206"},{"author":{"_account_id":28459,"name":"Bob Church","email":"robert.church@windriver.com","username":"rchurch"},"change_message_id":"a7e16b0649854e32e717acf4b340c2dafcef2702","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":13,"id":"95ae2555_2bebe891","updated":"2022-11-09 10:38:39.000000000","message":"For me, adding the following deployment annotations allowed the app to be upgraded from the stx 7.0 FluxCD version to this new CSI version:\n\n modified   charts/ceph-csi-cephfs/templates/provisioner-deployment.yaml\n @@ -9,6 +9,9 @@ metadata:\n      component: {{ .Values.provisioner.name }}\n      release: {{ .Release.Name }}\n      heritage: {{ .Release.Service }}\n +  annotations:\n +    \"helm.sh/hook\": \"pre-upgrade, pre-install\"\n +    \"helm.sh/hook-delete-policy\": \"before-hook-creation\"\n  spec:\n    replicas: {{ .Values.provisioner.replicaCount }}\n    strategy:\n modified   charts/ceph-csi-rbd/templates/provisioner-deployment.yaml\n @@ -9,6 +9,9 @@ metadata:\n      component: {{ .Values.provisioner.name }}\n      release: {{ .Release.Name }}\n      heritage: {{ .Release.Service }}\n +  annotations:\n +    \"helm.sh/hook\": \"pre-upgrade, pre-install\"\n +    \"helm.sh/hook-delete-policy\": \"before-hook-creation\"\n  spec:\n    replicas: {{ .Values.provisioner.replicaCount }}\n    strategy:\n\nPlease add and confirm. With this, a special upgrade script should not be needed as the existing 65-k8s-app-upgrade.sh script should orchestrate the app upgrade during platform activation","commit_id":"bbdcf24f1a3896ce1093e6c801226980a3c5d659"},{"author":{"_account_id":34926,"name":"Hediberto Cavalcante da Silva","display_name":"Hediberto C Silva","email":"hediberto.cavalcantedasilva@windriver.com","username":"hcavalca"},"change_message_id":"2a2dad4321d0c7734f5349d3c6ec1201f4272188","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":13,"id":"b46ab70c_e4277d05","in_reply_to":"95ae2555_2bebe891","updated":"2022-11-10 01:27:51.000000000","message":"As discussed we arrived at this solution:\n\nrbd/cephfs: provisioner-deployment \n  \"helm.sh/hook\": \"post-upgrade, post-install\"\n  \"helm.sh/hook-delete-policy\": \"before-hook-creation\"\n\nfluxcd: storageclass\n  \"helm.sh/hook\": \"post-upgrade, post-install\",\n  \"helm.sh/hook-delete-policy\": \"before-hook-creation\"\n\nPassed in my tests:\n\n- app update from 22.06 -\u003e new CSI\n- fresh upload/apply\n- system helm-override + reapply\n- manifest change + version bump + app update","commit_id":"bbdcf24f1a3896ce1093e6c801226980a3c5d659"},{"author":{"_account_id":28459,"name":"Bob Church","email":"robert.church@windriver.com","username":"rchurch"},"change_message_id":"9473f1dbcc1f3839183723600d302a11daca4997","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":15,"id":"c632a769_d6093d93","updated":"2022-11-10 02:21:10.000000000","message":"As a side note... This removes CentOS support for this app as build support is ending soon. If the provisioner is needed for some testing under CentOS the stx 7.0 provisioner can be copied to the install and manually uploaded/applied OR this commit can be reverted for a designer build.","commit_id":"69c37e9978d45dd928537523eda905e34e034b73"},{"author":{"_account_id":28459,"name":"Bob Church","email":"robert.church@windriver.com","username":"rchurch"},"change_message_id":"2f8cd40a8647a8d8ca32e35852f3028197164463","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":15,"id":"bd5be2a6_533fb1a7","updated":"2022-11-10 01:58:43.000000000","message":"Thanks for the updates.","commit_id":"69c37e9978d45dd928537523eda905e34e034b73"}],"platform-helm/debian/deb_folder/control":[{"author":{"_account_id":28459,"name":"Bob Church","email":"robert.church@windriver.com","username":"rchurch"},"change_message_id":"f0bcdda4ae3f6cc146bd39a80f43662094874011","unresolved":true,"context_lines":[{"line_number":3,"context_line":"Priority: optional"},{"line_number":4,"context_line":"Maintainer: StarlingX Developers \u003cstarlingx-discuss@lists.starlingx.io\u003e"},{"line_number":5,"context_line":"Build-Depends: debhelper-compat (\u003d 13),"},{"line_number":6,"context_line":" chartmuseum,"},{"line_number":7,"context_line":" helm,"},{"line_number":8,"context_line":" openstack-helm-infra,"},{"line_number":9,"context_line":" procps"},{"line_number":10,"context_line":"Standards-Version: 4.5.1"},{"line_number":11,"context_line":"Homepage: https://www.starlingx.io"},{"line_number":12,"context_line":""}],"source_content_type":"application/octet-stream","patch_set":8,"id":"b8d339a0_1362dd52","line":9,"range":{"start_line":6,"start_character":0,"end_line":9,"end_character":7},"updated":"2022-11-03 09:11:25.000000000","message":"Remove the dependencies on the helm-toolkit and all we need here is the helm dependency.","commit_id":"6c316bdffdd9896847ba06a3c38cde2cf98a9b71"},{"author":{"_account_id":34926,"name":"Hediberto Cavalcante da Silva","display_name":"Hediberto C Silva","email":"hediberto.cavalcantedasilva@windriver.com","username":"hcavalca"},"change_message_id":"bb7f02154ff167ca8eb59b35cc9da113a9df5f6d","unresolved":false,"context_lines":[{"line_number":3,"context_line":"Priority: optional"},{"line_number":4,"context_line":"Maintainer: StarlingX Developers \u003cstarlingx-discuss@lists.starlingx.io\u003e"},{"line_number":5,"context_line":"Build-Depends: debhelper-compat (\u003d 13),"},{"line_number":6,"context_line":" chartmuseum,"},{"line_number":7,"context_line":" helm,"},{"line_number":8,"context_line":" openstack-helm-infra,"},{"line_number":9,"context_line":" procps"},{"line_number":10,"context_line":"Standards-Version: 4.5.1"},{"line_number":11,"context_line":"Homepage: https://www.starlingx.io"},{"line_number":12,"context_line":""}],"source_content_type":"application/octet-stream","patch_set":8,"id":"3d406ab7_9b4b933d","line":9,"range":{"start_line":6,"start_character":0,"end_line":9,"end_character":7},"in_reply_to":"b8d339a0_1362dd52","updated":"2022-11-04 23:29:54.000000000","message":"Done","commit_id":"6c316bdffdd9896847ba06a3c38cde2cf98a9b71"}],"platform-helm/debian/deb_folder/copyright":[{"author":{"_account_id":28459,"name":"Bob Church","email":"robert.church@windriver.com","username":"rchurch"},"change_message_id":"f0bcdda4ae3f6cc146bd39a80f43662094874011","unresolved":true,"context_lines":[{"line_number":3,"context_line":"Source: https://opendev.org/starlingx/platform-armada-app/"},{"line_number":4,"context_line":""},{"line_number":5,"context_line":"Files: *"},{"line_number":6,"context_line":"Copyright: (c) 2013-2022 Wind River Systems, Inc"},{"line_number":7,"context_line":"License: Apache-2"},{"line_number":8,"context_line":" Licensed under the Apache License, Version 2.0 (the \"License\");"},{"line_number":9,"context_line":" you may not use this file except in compliance with the License."}],"source_content_type":"application/octet-stream","patch_set":8,"id":"11d894d5_7c45578e","line":6,"range":{"start_line":6,"start_character":15,"end_line":6,"end_character":24},"updated":"2022-11-03 09:11:25.000000000","message":"As this is new, should this just be 2022?","commit_id":"6c316bdffdd9896847ba06a3c38cde2cf98a9b71"},{"author":{"_account_id":34926,"name":"Hediberto Cavalcante da Silva","display_name":"Hediberto C Silva","email":"hediberto.cavalcantedasilva@windriver.com","username":"hcavalca"},"change_message_id":"bb7f02154ff167ca8eb59b35cc9da113a9df5f6d","unresolved":false,"context_lines":[{"line_number":3,"context_line":"Source: https://opendev.org/starlingx/platform-armada-app/"},{"line_number":4,"context_line":""},{"line_number":5,"context_line":"Files: *"},{"line_number":6,"context_line":"Copyright: (c) 2013-2022 Wind River Systems, Inc"},{"line_number":7,"context_line":"License: Apache-2"},{"line_number":8,"context_line":" Licensed under the Apache License, Version 2.0 (the \"License\");"},{"line_number":9,"context_line":" you may not use this file except in compliance with the License."}],"source_content_type":"application/octet-stream","patch_set":8,"id":"1536f8b5_c79b10e7","line":6,"range":{"start_line":6,"start_character":15,"end_line":6,"end_character":24},"in_reply_to":"11d894d5_7c45578e","updated":"2022-11-04 23:29:54.000000000","message":"Done","commit_id":"6c316bdffdd9896847ba06a3c38cde2cf98a9b71"}],"platform-helm/debian/deb_folder/patches/0001-ceph-csi-rbd-add-default-fields-to-values.yaml.patch":[{"author":{"_account_id":28459,"name":"Bob Church","email":"robert.church@windriver.com","username":"rchurch"},"change_message_id":"f0bcdda4ae3f6cc146bd39a80f43662094874011","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":8,"id":"98e5dc2f_32cfd87b","updated":"2022-11-03 09:11:25.000000000","message":"appVersion/version in Chart.yaml needs to be patched so that the it reflects 3.6.2 instead of \"canary\"","commit_id":"6c316bdffdd9896847ba06a3c38cde2cf98a9b71"},{"author":{"_account_id":34926,"name":"Hediberto Cavalcante da Silva","display_name":"Hediberto C Silva","email":"hediberto.cavalcantedasilva@windriver.com","username":"hcavalca"},"change_message_id":"bb7f02154ff167ca8eb59b35cc9da113a9df5f6d","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":8,"id":"8a6f2684_ecb07c7c","in_reply_to":"98e5dc2f_32cfd87b","updated":"2022-11-04 23:29:54.000000000","message":"Done","commit_id":"6c316bdffdd9896847ba06a3c38cde2cf98a9b71"},{"author":{"_account_id":15435,"name":"Al Bailey","email":"albailey1974@gmail.com","username":"albailey"},"change_message_id":"bf65fdc8b98bd389cf09fa657f196917a7225de9","unresolved":true,"context_lines":[{"line_number":64,"context_line":"+    csi_provisioner: k8s.gcr.io/sig-storage/csi-provisioner:v3.1.0"},{"line_number":65,"context_line":"+    csi_snapshotter: k8s.gcr.io/sig-storage/csi-snapshotter:v4.2.0"},{"line_number":66,"context_line":"+    csi_attacher: k8s.gcr.io/sig-storage/csi-attacher:v3.4.0"},{"line_number":67,"context_line":"+    csi_resizer: k8s.gcr.io/sig-storage/csi-resizer:v1.4.0"},{"line_number":68,"context_line":"+    csi_cephcsi: quay.io/cephcsi/cephcsi:v3.6.2"},{"line_number":69,"context_line":"+    csi_registrar: k8s.gcr.io/sig-storage/csi-node-driver-registrar:v2.4.0"},{"line_number":70,"context_line":"+    rbd_provisioner_storage_init: docker.io/openstackhelm/ceph-config-helper:ubuntu_bionic-20220802"}],"source_content_type":"text/x-diff","patch_set":8,"id":"5ed3326e_081f7060","line":67,"range":{"start_line":67,"start_character":4,"end_line":67,"end_character":59},"updated":"2022-11-02 20:56:01.000000000","message":"Do we eventually need to align with the versions we are using in ansible\nex:\nhttps://github.com/starlingx/ansible-playbooks/blob/master/playbookconfig/src/playbooks/host_vars/netapp/default.yml#L51","commit_id":"6c316bdffdd9896847ba06a3c38cde2cf98a9b71"},{"author":{"_account_id":34926,"name":"Hediberto Cavalcante da Silva","display_name":"Hediberto C Silva","email":"hediberto.cavalcantedasilva@windriver.com","username":"hcavalca"},"change_message_id":"bb7f02154ff167ca8eb59b35cc9da113a9df5f6d","unresolved":false,"context_lines":[{"line_number":64,"context_line":"+    csi_provisioner: k8s.gcr.io/sig-storage/csi-provisioner:v3.1.0"},{"line_number":65,"context_line":"+    csi_snapshotter: k8s.gcr.io/sig-storage/csi-snapshotter:v4.2.0"},{"line_number":66,"context_line":"+    csi_attacher: k8s.gcr.io/sig-storage/csi-attacher:v3.4.0"},{"line_number":67,"context_line":"+    csi_resizer: k8s.gcr.io/sig-storage/csi-resizer:v1.4.0"},{"line_number":68,"context_line":"+    csi_cephcsi: quay.io/cephcsi/cephcsi:v3.6.2"},{"line_number":69,"context_line":"+    csi_registrar: k8s.gcr.io/sig-storage/csi-node-driver-registrar:v2.4.0"},{"line_number":70,"context_line":"+    rbd_provisioner_storage_init: docker.io/openstackhelm/ceph-config-helper:ubuntu_bionic-20220802"}],"source_content_type":"text/x-diff","patch_set":8,"id":"a1b88b91_7eed6436","line":67,"range":{"start_line":67,"start_character":4,"end_line":67,"end_character":59},"in_reply_to":"18675cd4_95fbc15c","updated":"2022-11-04 23:29:54.000000000","message":"The rook app will be updated to the same version as ceph-csi.\nThe netapp/trident app will be updated to a newer version required.","commit_id":"6c316bdffdd9896847ba06a3c38cde2cf98a9b71"},{"author":{"_account_id":15435,"name":"Al Bailey","email":"albailey1974@gmail.com","username":"albailey"},"change_message_id":"8192c6e7e9f80559cbce713c9097f1164f9d0345","unresolved":true,"context_lines":[{"line_number":64,"context_line":"+    csi_provisioner: k8s.gcr.io/sig-storage/csi-provisioner:v3.1.0"},{"line_number":65,"context_line":"+    csi_snapshotter: k8s.gcr.io/sig-storage/csi-snapshotter:v4.2.0"},{"line_number":66,"context_line":"+    csi_attacher: k8s.gcr.io/sig-storage/csi-attacher:v3.4.0"},{"line_number":67,"context_line":"+    csi_resizer: k8s.gcr.io/sig-storage/csi-resizer:v1.4.0"},{"line_number":68,"context_line":"+    csi_cephcsi: quay.io/cephcsi/cephcsi:v3.6.2"},{"line_number":69,"context_line":"+    csi_registrar: k8s.gcr.io/sig-storage/csi-node-driver-registrar:v2.4.0"},{"line_number":70,"context_line":"+    rbd_provisioner_storage_init: docker.io/openstackhelm/ceph-config-helper:ubuntu_bionic-20220802"}],"source_content_type":"text/x-diff","patch_set":8,"id":"18675cd4_95fbc15c","line":67,"range":{"start_line":67,"start_character":4,"end_line":67,"end_character":59},"in_reply_to":"5ed3326e_081f7060","updated":"2022-11-02 20:59:16.000000000","message":"Maybe it doesnt matter.\nrook app still uses 0.4.0 version","commit_id":"6c316bdffdd9896847ba06a3c38cde2cf98a9b71"}],"platform-helm/debian/deb_folder/patches/0002-ceph-csi-cephfs-add-default-fields-to-values.yaml.patch":[{"author":{"_account_id":28459,"name":"Bob Church","email":"robert.church@windriver.com","username":"rchurch"},"change_message_id":"f0bcdda4ae3f6cc146bd39a80f43662094874011","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":8,"id":"4b9729b2_ecae1c2f","updated":"2022-11-03 09:11:25.000000000","message":"appVersion/version in Chart.yaml needs to be patched so that the it reflects 3.6.2 instead of \"canary\"","commit_id":"6c316bdffdd9896847ba06a3c38cde2cf98a9b71"},{"author":{"_account_id":34926,"name":"Hediberto Cavalcante da Silva","display_name":"Hediberto C Silva","email":"hediberto.cavalcantedasilva@windriver.com","username":"hcavalca"},"change_message_id":"bb7f02154ff167ca8eb59b35cc9da113a9df5f6d","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":8,"id":"e968c316_06811d8a","in_reply_to":"4b9729b2_ecae1c2f","updated":"2022-11-04 23:29:54.000000000","message":"Done","commit_id":"6c316bdffdd9896847ba06a3c38cde2cf98a9b71"},{"author":{"_account_id":8768,"name":"Chris Friesen","email":"chris.friesen@windriver.com","username":"cbf123"},"change_message_id":"8fee8cb58fc2fb1cdf25bb558967507a2340142b","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":15,"id":"a6bb45c2_9f793d80","updated":"2022-11-10 14:51:11.000000000","message":"Just wondering why we\u0027re setting the defaults here rather than in the manifest file (which wouldn\u0027t require a patch).  And won\u0027t things like the monitor IP address get set using sysinv data anyway?","commit_id":"69c37e9978d45dd928537523eda905e34e034b73"},{"author":{"_account_id":34926,"name":"Hediberto Cavalcante da Silva","display_name":"Hediberto C Silva","email":"hediberto.cavalcantedasilva@windriver.com","username":"hcavalca"},"change_message_id":"43aaa4dff1899e20c3ee34f98510faa6af353569","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":15,"id":"0d018e3c_ec753861","in_reply_to":"a6bb45c2_9f793d80","updated":"2022-11-10 15:05:37.000000000","message":"When I tested fluxcd it asked for the field to already exist to apply the static settings.","commit_id":"69c37e9978d45dd928537523eda905e34e034b73"}],"platform-helm/debian/deb_folder/rules":[{"author":{"_account_id":28459,"name":"Bob Church","email":"robert.church@windriver.com","username":"rchurch"},"change_message_id":"f0bcdda4ae3f6cc146bd39a80f43662094874011","unresolved":true,"context_lines":[{"line_number":10,"context_line":"\tdh $@"},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"override_dh_auto_build:"},{"line_number":13,"context_line":"\t# Stage helm-toolkit in the local repo."},{"line_number":14,"context_line":"\tcp $(HELM_FOLDER)/helm-toolkit-$(TOOLKIT_VERSION).tgz ."},{"line_number":15,"context_line":""},{"line_number":16,"context_line":"\t# Host a server for the helm charts."},{"line_number":17,"context_line":"\tchartmuseum --debug --port\u003d8879 --context-path\u003d\u0027/charts\u0027 --storage\u003d\"local\" \\"},{"line_number":18,"context_line":"\t\t--storage-local-rootdir\u003d\".\" \u0026"},{"line_number":19,"context_line":"\tsleep 2"},{"line_number":20,"context_line":"\thelm repo add local http://localhost:8879/charts"},{"line_number":21,"context_line":""},{"line_number":22,"context_line":"\tmkdir -p ceph-csi"},{"line_number":23,"context_line":""}],"source_content_type":"application/octet-stream","patch_set":8,"id":"d18c980a_45f73439","line":20,"range":{"start_line":13,"start_character":0,"end_line":20,"end_character":49},"updated":"2022-11-03 09:11:25.000000000","message":"remove the dependency on the helm-toolkit and this can be removed","commit_id":"6c316bdffdd9896847ba06a3c38cde2cf98a9b71"},{"author":{"_account_id":34926,"name":"Hediberto Cavalcante da Silva","display_name":"Hediberto C Silva","email":"hediberto.cavalcantedasilva@windriver.com","username":"hcavalca"},"change_message_id":"bb7f02154ff167ca8eb59b35cc9da113a9df5f6d","unresolved":false,"context_lines":[{"line_number":10,"context_line":"\tdh $@"},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"override_dh_auto_build:"},{"line_number":13,"context_line":"\t# Stage helm-toolkit in the local repo."},{"line_number":14,"context_line":"\tcp $(HELM_FOLDER)/helm-toolkit-$(TOOLKIT_VERSION).tgz ."},{"line_number":15,"context_line":""},{"line_number":16,"context_line":"\t# Host a server for the helm charts."},{"line_number":17,"context_line":"\tchartmuseum --debug --port\u003d8879 --context-path\u003d\u0027/charts\u0027 --storage\u003d\"local\" \\"},{"line_number":18,"context_line":"\t\t--storage-local-rootdir\u003d\".\" \u0026"},{"line_number":19,"context_line":"\tsleep 2"},{"line_number":20,"context_line":"\thelm repo add local http://localhost:8879/charts"},{"line_number":21,"context_line":""},{"line_number":22,"context_line":"\tmkdir -p ceph-csi"},{"line_number":23,"context_line":""}],"source_content_type":"application/octet-stream","patch_set":8,"id":"7a5f137c_08bd48c6","line":20,"range":{"start_line":13,"start_character":0,"end_line":20,"end_character":49},"in_reply_to":"d18c980a_45f73439","updated":"2022-11-04 23:29:54.000000000","message":"Done","commit_id":"6c316bdffdd9896847ba06a3c38cde2cf98a9b71"},{"author":{"_account_id":28459,"name":"Bob Church","email":"robert.church@windriver.com","username":"rchurch"},"change_message_id":"f0bcdda4ae3f6cc146bd39a80f43662094874011","unresolved":true,"context_lines":[{"line_number":26,"context_line":""},{"line_number":27,"context_line":"\t# Copy build files"},{"line_number":28,"context_line":"\tcp Makefile ceph-csi"},{"line_number":29,"context_line":"\tcp requirements.yaml ceph-csi/ceph-csi-rbd"},{"line_number":30,"context_line":"\tcp requirements.yaml ceph-csi/ceph-csi-cephfs"},{"line_number":31,"context_line":""},{"line_number":32,"context_line":"\t# Copy pre-install scripts"},{"line_number":33,"context_line":"\tcp -r ceph-csi-rbd/* ceph-csi/ceph-csi-rbd/templates"}],"source_content_type":"application/octet-stream","patch_set":8,"id":"cac9ed73_377568f3","line":30,"range":{"start_line":29,"start_character":0,"end_line":30,"end_character":46},"updated":"2022-11-03 09:11:25.000000000","message":"same comment as above","commit_id":"6c316bdffdd9896847ba06a3c38cde2cf98a9b71"},{"author":{"_account_id":34926,"name":"Hediberto Cavalcante da Silva","display_name":"Hediberto C Silva","email":"hediberto.cavalcantedasilva@windriver.com","username":"hcavalca"},"change_message_id":"bb7f02154ff167ca8eb59b35cc9da113a9df5f6d","unresolved":false,"context_lines":[{"line_number":26,"context_line":""},{"line_number":27,"context_line":"\t# Copy build files"},{"line_number":28,"context_line":"\tcp Makefile ceph-csi"},{"line_number":29,"context_line":"\tcp requirements.yaml ceph-csi/ceph-csi-rbd"},{"line_number":30,"context_line":"\tcp requirements.yaml ceph-csi/ceph-csi-cephfs"},{"line_number":31,"context_line":""},{"line_number":32,"context_line":"\t# Copy pre-install scripts"},{"line_number":33,"context_line":"\tcp -r ceph-csi-rbd/* ceph-csi/ceph-csi-rbd/templates"}],"source_content_type":"application/octet-stream","patch_set":8,"id":"3db038a2_fddf0e73","line":30,"range":{"start_line":29,"start_character":0,"end_line":30,"end_character":46},"in_reply_to":"cac9ed73_377568f3","updated":"2022-11-04 23:29:54.000000000","message":"Done","commit_id":"6c316bdffdd9896847ba06a3c38cde2cf98a9b71"},{"author":{"_account_id":28459,"name":"Bob Church","email":"robert.church@windriver.com","username":"rchurch"},"change_message_id":"f0bcdda4ae3f6cc146bd39a80f43662094874011","unresolved":true,"context_lines":[{"line_number":29,"context_line":"\tcp requirements.yaml ceph-csi/ceph-csi-rbd"},{"line_number":30,"context_line":"\tcp requirements.yaml ceph-csi/ceph-csi-cephfs"},{"line_number":31,"context_line":""},{"line_number":32,"context_line":"\t# Copy pre-install scripts"},{"line_number":33,"context_line":"\tcp -r ceph-csi-rbd/* ceph-csi/ceph-csi-rbd/templates"},{"line_number":34,"context_line":"\tcp -r ceph-csi-cephfs/* ceph-csi/ceph-csi-cephfs/templates"},{"line_number":35,"context_line":""},{"line_number":36,"context_line":"\tcd ceph-csi \u0026\u0026 make ceph-csi-rbd"},{"line_number":37,"context_line":"\tcd ceph-csi \u0026\u0026 make ceph-csi-cephfs"},{"line_number":38,"context_line":""}],"source_content_type":"application/octet-stream","patch_set":8,"id":"30ce4a7e_a7462024","line":35,"range":{"start_line":32,"start_character":0,"end_line":35,"end_character":0},"updated":"2022-11-03 09:11:25.000000000","message":"these should be patches to the charts","commit_id":"6c316bdffdd9896847ba06a3c38cde2cf98a9b71"},{"author":{"_account_id":34926,"name":"Hediberto Cavalcante da Silva","display_name":"Hediberto C Silva","email":"hediberto.cavalcantedasilva@windriver.com","username":"hcavalca"},"change_message_id":"bb7f02154ff167ca8eb59b35cc9da113a9df5f6d","unresolved":false,"context_lines":[{"line_number":29,"context_line":"\tcp requirements.yaml ceph-csi/ceph-csi-rbd"},{"line_number":30,"context_line":"\tcp requirements.yaml ceph-csi/ceph-csi-cephfs"},{"line_number":31,"context_line":""},{"line_number":32,"context_line":"\t# Copy pre-install scripts"},{"line_number":33,"context_line":"\tcp -r ceph-csi-rbd/* ceph-csi/ceph-csi-rbd/templates"},{"line_number":34,"context_line":"\tcp -r ceph-csi-cephfs/* ceph-csi/ceph-csi-cephfs/templates"},{"line_number":35,"context_line":""},{"line_number":36,"context_line":"\tcd ceph-csi \u0026\u0026 make ceph-csi-rbd"},{"line_number":37,"context_line":"\tcd ceph-csi \u0026\u0026 make ceph-csi-cephfs"},{"line_number":38,"context_line":""}],"source_content_type":"application/octet-stream","patch_set":8,"id":"423de2c8_fb720a73","line":35,"range":{"start_line":32,"start_character":0,"end_line":35,"end_character":0},"in_reply_to":"30ce4a7e_a7462024","updated":"2022-11-04 23:29:54.000000000","message":"Done","commit_id":"6c316bdffdd9896847ba06a3c38cde2cf98a9b71"}],"platform-helm/debian/meta_data.yaml":[{"author":{"_account_id":28459,"name":"Bob Church","email":"robert.church@windriver.com","username":"rchurch"},"change_message_id":"f0bcdda4ae3f6cc146bd39a80f43662094874011","unresolved":true,"context_lines":[{"line_number":3,"context_line":"debver: 1.0-1"},{"line_number":4,"context_line":"dl_path:"},{"line_number":5,"context_line":"  name: ceph-csi-3.6.2.tar.gz"},{"line_number":6,"context_line":"  url: https://github.com/ceph/ceph-csi/archive/v3.6.2.tar.gz"},{"line_number":7,"context_line":"  md5sum: a5fd6785c521faf0cb7df008a1012381"},{"line_number":8,"context_line":"src_files:"},{"line_number":9,"context_line":"  - platform-helm/files/index.yaml"}],"source_content_type":"text/x-yaml","patch_set":8,"id":"2f3ca9ed_febfb9d1","line":6,"range":{"start_line":6,"start_character":49,"end_line":6,"end_character":55},"updated":"2022-11-03 09:11:25.000000000","message":"This version needs to be reflected in the charts instead of \"canary\"","commit_id":"6c316bdffdd9896847ba06a3c38cde2cf98a9b71"},{"author":{"_account_id":34926,"name":"Hediberto Cavalcante da Silva","display_name":"Hediberto C Silva","email":"hediberto.cavalcantedasilva@windriver.com","username":"hcavalca"},"change_message_id":"bb7f02154ff167ca8eb59b35cc9da113a9df5f6d","unresolved":false,"context_lines":[{"line_number":3,"context_line":"debver: 1.0-1"},{"line_number":4,"context_line":"dl_path:"},{"line_number":5,"context_line":"  name: ceph-csi-3.6.2.tar.gz"},{"line_number":6,"context_line":"  url: https://github.com/ceph/ceph-csi/archive/v3.6.2.tar.gz"},{"line_number":7,"context_line":"  md5sum: a5fd6785c521faf0cb7df008a1012381"},{"line_number":8,"context_line":"src_files:"},{"line_number":9,"context_line":"  - platform-helm/files/index.yaml"}],"source_content_type":"text/x-yaml","patch_set":8,"id":"d23edef4_7879221a","line":6,"range":{"start_line":6,"start_character":49,"end_line":6,"end_character":55},"in_reply_to":"2f3ca9ed_febfb9d1","updated":"2022-11-04 23:29:54.000000000","message":"Done","commit_id":"6c316bdffdd9896847ba06a3c38cde2cf98a9b71"},{"author":{"_account_id":28459,"name":"Bob Church","email":"robert.church@windriver.com","username":"rchurch"},"change_message_id":"f0bcdda4ae3f6cc146bd39a80f43662094874011","unresolved":true,"context_lines":[{"line_number":6,"context_line":"  url: https://github.com/ceph/ceph-csi/archive/v3.6.2.tar.gz"},{"line_number":7,"context_line":"  md5sum: a5fd6785c521faf0cb7df008a1012381"},{"line_number":8,"context_line":"src_files:"},{"line_number":9,"context_line":"  - platform-helm/files/index.yaml"},{"line_number":10,"context_line":"  - platform-helm/files/Makefile"},{"line_number":11,"context_line":"  - platform-helm/files/metadata.yaml"},{"line_number":12,"context_line":"  - platform-helm/files/repositories.yaml"},{"line_number":13,"context_line":"  - platform-helm/files/requirements.yaml"},{"line_number":14,"context_line":"  - platform-helm/helm-charts/ceph-csi-rbd"},{"line_number":15,"context_line":"  - platform-helm/helm-charts/ceph-csi-cephfs"},{"line_number":16,"context_line":"revision:"},{"line_number":17,"context_line":"  dist: $STX_DIST"},{"line_number":18,"context_line":"  PKG_GITREVCOUNT: true"}],"source_content_type":"text/x-yaml","patch_set":8,"id":"ea2ccb83_a56a34ba","line":15,"range":{"start_line":9,"start_character":0,"end_line":15,"end_character":45},"updated":"2022-11-03 09:11:25.000000000","message":"Only the Makefile should be needed if we patch the charts and drop the helm-toolkit dependency","commit_id":"6c316bdffdd9896847ba06a3c38cde2cf98a9b71"},{"author":{"_account_id":34926,"name":"Hediberto Cavalcante da Silva","display_name":"Hediberto C Silva","email":"hediberto.cavalcantedasilva@windriver.com","username":"hcavalca"},"change_message_id":"bb7f02154ff167ca8eb59b35cc9da113a9df5f6d","unresolved":false,"context_lines":[{"line_number":6,"context_line":"  url: https://github.com/ceph/ceph-csi/archive/v3.6.2.tar.gz"},{"line_number":7,"context_line":"  md5sum: a5fd6785c521faf0cb7df008a1012381"},{"line_number":8,"context_line":"src_files:"},{"line_number":9,"context_line":"  - platform-helm/files/index.yaml"},{"line_number":10,"context_line":"  - platform-helm/files/Makefile"},{"line_number":11,"context_line":"  - platform-helm/files/metadata.yaml"},{"line_number":12,"context_line":"  - platform-helm/files/repositories.yaml"},{"line_number":13,"context_line":"  - platform-helm/files/requirements.yaml"},{"line_number":14,"context_line":"  - platform-helm/helm-charts/ceph-csi-rbd"},{"line_number":15,"context_line":"  - platform-helm/helm-charts/ceph-csi-cephfs"},{"line_number":16,"context_line":"revision:"},{"line_number":17,"context_line":"  dist: $STX_DIST"},{"line_number":18,"context_line":"  PKG_GITREVCOUNT: true"}],"source_content_type":"text/x-yaml","patch_set":8,"id":"53d42fbd_6bd67c92","line":15,"range":{"start_line":9,"start_character":0,"end_line":15,"end_character":45},"in_reply_to":"ea2ccb83_a56a34ba","updated":"2022-11-04 23:29:54.000000000","message":"Done","commit_id":"6c316bdffdd9896847ba06a3c38cde2cf98a9b71"}],"platform-helm/platform-helm/files/index.yaml":[{"author":{"_account_id":28459,"name":"Bob Church","email":"robert.church@windriver.com","username":"rchurch"},"change_message_id":"f0bcdda4ae3f6cc146bd39a80f43662094874011","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":8,"id":"233a249c_cdb97942","updated":"2022-11-03 09:11:25.000000000","message":"Not needed if we drop the helm-toolkit","commit_id":"6c316bdffdd9896847ba06a3c38cde2cf98a9b71"},{"author":{"_account_id":34926,"name":"Hediberto Cavalcante da Silva","display_name":"Hediberto C Silva","email":"hediberto.cavalcantedasilva@windriver.com","username":"hcavalca"},"change_message_id":"bb7f02154ff167ca8eb59b35cc9da113a9df5f6d","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":8,"id":"81fb2a6b_c5459ac8","in_reply_to":"233a249c_cdb97942","updated":"2022-11-04 23:29:54.000000000","message":"Done","commit_id":"6c316bdffdd9896847ba06a3c38cde2cf98a9b71"}],"platform-helm/platform-helm/files/metadata.yaml":[{"author":{"_account_id":28459,"name":"Bob Church","email":"robert.church@windriver.com","username":"rchurch"},"change_message_id":"f0bcdda4ae3f6cc146bd39a80f43662094874011","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":8,"id":"b299f298_5866286e","updated":"2022-11-03 09:11:25.000000000","message":"Not needed. remove.","commit_id":"6c316bdffdd9896847ba06a3c38cde2cf98a9b71"},{"author":{"_account_id":34926,"name":"Hediberto Cavalcante da Silva","display_name":"Hediberto C Silva","email":"hediberto.cavalcantedasilva@windriver.com","username":"hcavalca"},"change_message_id":"bb7f02154ff167ca8eb59b35cc9da113a9df5f6d","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":8,"id":"f7c9e753_7bc73bc5","in_reply_to":"b299f298_5866286e","updated":"2022-11-04 23:29:54.000000000","message":"Done","commit_id":"6c316bdffdd9896847ba06a3c38cde2cf98a9b71"}],"platform-helm/platform-helm/files/repositories.yaml":[{"author":{"_account_id":28459,"name":"Bob Church","email":"robert.church@windriver.com","username":"rchurch"},"change_message_id":"f0bcdda4ae3f6cc146bd39a80f43662094874011","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":8,"id":"3a0949e0_ed91ff69","updated":"2022-11-03 09:11:25.000000000","message":"Not needed if we drop the helm-toolkit","commit_id":"6c316bdffdd9896847ba06a3c38cde2cf98a9b71"},{"author":{"_account_id":34926,"name":"Hediberto Cavalcante da Silva","display_name":"Hediberto C Silva","email":"hediberto.cavalcantedasilva@windriver.com","username":"hcavalca"},"change_message_id":"bb7f02154ff167ca8eb59b35cc9da113a9df5f6d","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":8,"id":"67076a58_60c2a8b1","in_reply_to":"3a0949e0_ed91ff69","updated":"2022-11-04 23:29:54.000000000","message":"Done","commit_id":"6c316bdffdd9896847ba06a3c38cde2cf98a9b71"}],"platform-helm/platform-helm/files/requirements.yaml":[{"author":{"_account_id":28459,"name":"Bob Church","email":"robert.church@windriver.com","username":"rchurch"},"change_message_id":"f0bcdda4ae3f6cc146bd39a80f43662094874011","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":8,"id":"05c4a8ea_e5eecfd0","updated":"2022-11-03 09:11:25.000000000","message":"Not needed if we drop the helm-toolkit","commit_id":"6c316bdffdd9896847ba06a3c38cde2cf98a9b71"},{"author":{"_account_id":34926,"name":"Hediberto Cavalcante da Silva","display_name":"Hediberto C Silva","email":"hediberto.cavalcantedasilva@windriver.com","username":"hcavalca"},"change_message_id":"bb7f02154ff167ca8eb59b35cc9da113a9df5f6d","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":8,"id":"e0cad350_ed0237bd","in_reply_to":"05c4a8ea_e5eecfd0","updated":"2022-11-04 23:29:54.000000000","message":"Done","commit_id":"6c316bdffdd9896847ba06a3c38cde2cf98a9b71"}],"platform-helm/platform-helm/helm-charts/ceph-csi-cephfs/bin/_storage_init.sh.tpl":[{"author":{"_account_id":28459,"name":"Bob Church","email":"robert.church@windriver.com","username":"rchurch"},"change_message_id":"f0bcdda4ae3f6cc146bd39a80f43662094874011","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":8,"id":"ce77c3aa_c88bdb64","updated":"2022-11-03 09:11:25.000000000","message":"Embed this directly in the cephfs-storage-init.yaml and drop the toolkit dependency","commit_id":"6c316bdffdd9896847ba06a3c38cde2cf98a9b71"},{"author":{"_account_id":34926,"name":"Hediberto Cavalcante da Silva","display_name":"Hediberto C Silva","email":"hediberto.cavalcantedasilva@windriver.com","username":"hcavalca"},"change_message_id":"bb7f02154ff167ca8eb59b35cc9da113a9df5f6d","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":8,"id":"082596e5_2d8b3827","in_reply_to":"ce77c3aa_c88bdb64","updated":"2022-11-04 23:29:54.000000000","message":"Done","commit_id":"6c316bdffdd9896847ba06a3c38cde2cf98a9b71"}],"platform-helm/platform-helm/helm-charts/ceph-csi-cephfs/cephfs-storage-init.yaml":[{"author":{"_account_id":28459,"name":"Bob Church","email":"robert.church@windriver.com","username":"rchurch"},"change_message_id":"f0bcdda4ae3f6cc146bd39a80f43662094874011","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":8,"id":"86908b74_2d969295","updated":"2022-11-03 09:11:25.000000000","message":"Turn this into a patch for the chart","commit_id":"6c316bdffdd9896847ba06a3c38cde2cf98a9b71"},{"author":{"_account_id":34926,"name":"Hediberto Cavalcante da Silva","display_name":"Hediberto C Silva","email":"hediberto.cavalcantedasilva@windriver.com","username":"hcavalca"},"change_message_id":"bb7f02154ff167ca8eb59b35cc9da113a9df5f6d","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":8,"id":"c1630e36_39d3c548","in_reply_to":"86908b74_2d969295","updated":"2022-11-04 23:29:54.000000000","message":"Done","commit_id":"6c316bdffdd9896847ba06a3c38cde2cf98a9b71"},{"author":{"_account_id":28459,"name":"Bob Church","email":"robert.church@windriver.com","username":"rchurch"},"change_message_id":"f0bcdda4ae3f6cc146bd39a80f43662094874011","unresolved":true,"context_lines":[{"line_number":75,"context_line":"    \"helm.sh/hook-delete-policy\": \"before-hook-creation\""},{"line_number":76,"context_line":"data:"},{"line_number":77,"context_line":"  ceph.conf: |"},{"line_number":78,"context_line":"{{ tuple \"conf/_ceph-conf.tpl\" . | include \"helm-toolkit.utils.template\" | indent 4 }}"},{"line_number":79,"context_line":""},{"line_number":80,"context_line":"  storage-init.sh: |"},{"line_number":81,"context_line":"{{ tuple \"bin/_storage_init.sh.tpl\" . | include \"helm-toolkit.utils.template\" | indent 4 }}"}],"source_content_type":"text/x-yaml","patch_set":8,"id":"25331dcb_18ed266d","line":78,"updated":"2022-11-03 09:11:25.000000000","message":"Drop this dependency. Embed the content here","commit_id":"6c316bdffdd9896847ba06a3c38cde2cf98a9b71"},{"author":{"_account_id":34926,"name":"Hediberto Cavalcante da Silva","display_name":"Hediberto C Silva","email":"hediberto.cavalcantedasilva@windriver.com","username":"hcavalca"},"change_message_id":"bb7f02154ff167ca8eb59b35cc9da113a9df5f6d","unresolved":false,"context_lines":[{"line_number":75,"context_line":"    \"helm.sh/hook-delete-policy\": \"before-hook-creation\""},{"line_number":76,"context_line":"data:"},{"line_number":77,"context_line":"  ceph.conf: |"},{"line_number":78,"context_line":"{{ tuple \"conf/_ceph-conf.tpl\" . | include \"helm-toolkit.utils.template\" | indent 4 }}"},{"line_number":79,"context_line":""},{"line_number":80,"context_line":"  storage-init.sh: |"},{"line_number":81,"context_line":"{{ tuple \"bin/_storage_init.sh.tpl\" . | include \"helm-toolkit.utils.template\" | indent 4 }}"}],"source_content_type":"text/x-yaml","patch_set":8,"id":"78f8a268_9e063e78","line":78,"in_reply_to":"25331dcb_18ed266d","updated":"2022-11-04 23:29:54.000000000","message":"Done","commit_id":"6c316bdffdd9896847ba06a3c38cde2cf98a9b71"},{"author":{"_account_id":28459,"name":"Bob Church","email":"robert.church@windriver.com","username":"rchurch"},"change_message_id":"f0bcdda4ae3f6cc146bd39a80f43662094874011","unresolved":true,"context_lines":[{"line_number":78,"context_line":"{{ tuple \"conf/_ceph-conf.tpl\" . | include \"helm-toolkit.utils.template\" | indent 4 }}"},{"line_number":79,"context_line":""},{"line_number":80,"context_line":"  storage-init.sh: |"},{"line_number":81,"context_line":"{{ tuple \"bin/_storage_init.sh.tpl\" . | include \"helm-toolkit.utils.template\" | indent 4 }}"},{"line_number":82,"context_line":""},{"line_number":83,"context_line":"---"},{"line_number":84,"context_line":""}],"source_content_type":"text/x-yaml","patch_set":8,"id":"54aa00cd_a088ee51","line":81,"updated":"2022-11-03 09:11:25.000000000","message":"Drop this dependency. Embed the content here","commit_id":"6c316bdffdd9896847ba06a3c38cde2cf98a9b71"},{"author":{"_account_id":34926,"name":"Hediberto Cavalcante da Silva","display_name":"Hediberto C Silva","email":"hediberto.cavalcantedasilva@windriver.com","username":"hcavalca"},"change_message_id":"bb7f02154ff167ca8eb59b35cc9da113a9df5f6d","unresolved":false,"context_lines":[{"line_number":78,"context_line":"{{ tuple \"conf/_ceph-conf.tpl\" . | include \"helm-toolkit.utils.template\" | indent 4 }}"},{"line_number":79,"context_line":""},{"line_number":80,"context_line":"  storage-init.sh: |"},{"line_number":81,"context_line":"{{ tuple \"bin/_storage_init.sh.tpl\" . | include \"helm-toolkit.utils.template\" | indent 4 }}"},{"line_number":82,"context_line":""},{"line_number":83,"context_line":"---"},{"line_number":84,"context_line":""}],"source_content_type":"text/x-yaml","patch_set":8,"id":"de47e936_0df928e9","line":81,"in_reply_to":"54aa00cd_a088ee51","updated":"2022-11-04 23:29:54.000000000","message":"Done","commit_id":"6c316bdffdd9896847ba06a3c38cde2cf98a9b71"},{"author":{"_account_id":28459,"name":"Bob Church","email":"robert.church@windriver.com","username":"rchurch"},"change_message_id":"f0bcdda4ae3f6cc146bd39a80f43662094874011","unresolved":true,"context_lines":[{"line_number":116,"context_line":"            - name: NAMESPACE"},{"line_number":117,"context_line":"              value: {{ .Values.classDefaults.cephFSNamespace }}"},{"line_number":118,"context_line":"            - name: ADDITIONAL_NAMESPACES"},{"line_number":119,"context_line":"              value: {{ include \"helm-toolkit.utils.joinListWithComma\" .Values.storageClass.additionalNamespaces | quote }}"},{"line_number":120,"context_line":"            - name: CEPH_USER_SECRET"},{"line_number":121,"context_line":"              value: {{ .Values.storageClass.userSecretName }}"},{"line_number":122,"context_line":"            - name: USER_ID"}],"source_content_type":"text/x-yaml","patch_set":8,"id":"cb8deea9_fb198cc3","line":119,"updated":"2022-11-03 09:11:25.000000000","message":"Drop the dependency and use\n\n{{ join \",\" .Values.storageClass.additionalNamespaces | quote }}\n\nshould work.","commit_id":"6c316bdffdd9896847ba06a3c38cde2cf98a9b71"},{"author":{"_account_id":34926,"name":"Hediberto Cavalcante da Silva","display_name":"Hediberto C Silva","email":"hediberto.cavalcantedasilva@windriver.com","username":"hcavalca"},"change_message_id":"bb7f02154ff167ca8eb59b35cc9da113a9df5f6d","unresolved":false,"context_lines":[{"line_number":116,"context_line":"            - name: NAMESPACE"},{"line_number":117,"context_line":"              value: {{ .Values.classDefaults.cephFSNamespace }}"},{"line_number":118,"context_line":"            - name: ADDITIONAL_NAMESPACES"},{"line_number":119,"context_line":"              value: {{ include \"helm-toolkit.utils.joinListWithComma\" .Values.storageClass.additionalNamespaces | quote }}"},{"line_number":120,"context_line":"            - name: CEPH_USER_SECRET"},{"line_number":121,"context_line":"              value: {{ .Values.storageClass.userSecretName }}"},{"line_number":122,"context_line":"            - name: USER_ID"}],"source_content_type":"text/x-yaml","patch_set":8,"id":"02e60348_a5e016bc","line":119,"in_reply_to":"cb8deea9_fb198cc3","updated":"2022-11-04 23:29:54.000000000","message":"Done","commit_id":"6c316bdffdd9896847ba06a3c38cde2cf98a9b71"}],"platform-helm/platform-helm/helm-charts/ceph-csi-cephfs/conf/_ceph-conf.tpl":[{"author":{"_account_id":28459,"name":"Bob Church","email":"robert.church@windriver.com","username":"rchurch"},"change_message_id":"f0bcdda4ae3f6cc146bd39a80f43662094874011","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":8,"id":"803d49e3_87cf041f","updated":"2022-11-03 09:11:25.000000000","message":"Embed this directly in the cephfs-storage-init.yaml and drop the toolkit dependency there.","commit_id":"6c316bdffdd9896847ba06a3c38cde2cf98a9b71"},{"author":{"_account_id":34926,"name":"Hediberto Cavalcante da Silva","display_name":"Hediberto C Silva","email":"hediberto.cavalcantedasilva@windriver.com","username":"hcavalca"},"change_message_id":"bb7f02154ff167ca8eb59b35cc9da113a9df5f6d","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":8,"id":"5538f626_97023b70","in_reply_to":"803d49e3_87cf041f","updated":"2022-11-04 23:29:54.000000000","message":"Done","commit_id":"6c316bdffdd9896847ba06a3c38cde2cf98a9b71"}],"platform-helm/platform-helm/helm-charts/ceph-csi-rbd/bin/_storage_init.sh.tpl":[{"author":{"_account_id":28459,"name":"Bob Church","email":"robert.church@windriver.com","username":"rchurch"},"change_message_id":"f0bcdda4ae3f6cc146bd39a80f43662094874011","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":8,"id":"6c909133_97a8411f","updated":"2022-11-03 09:11:25.000000000","message":"Embed this directly in the rbd-storage-init.yaml and drop the toolkit dependency","commit_id":"6c316bdffdd9896847ba06a3c38cde2cf98a9b71"},{"author":{"_account_id":34926,"name":"Hediberto Cavalcante da Silva","display_name":"Hediberto C Silva","email":"hediberto.cavalcantedasilva@windriver.com","username":"hcavalca"},"change_message_id":"bb7f02154ff167ca8eb59b35cc9da113a9df5f6d","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":8,"id":"41825524_d2ff395d","in_reply_to":"6c909133_97a8411f","updated":"2022-11-04 23:29:54.000000000","message":"Done","commit_id":"6c316bdffdd9896847ba06a3c38cde2cf98a9b71"}],"platform-helm/platform-helm/helm-charts/ceph-csi-rbd/conf/_ceph-conf.tpl":[{"author":{"_account_id":28459,"name":"Bob Church","email":"robert.church@windriver.com","username":"rchurch"},"change_message_id":"f0bcdda4ae3f6cc146bd39a80f43662094874011","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":8,"id":"be2c7a8b_1eafb565","updated":"2022-11-03 09:11:25.000000000","message":"Embed this directly in the rbd-storage-init.yaml and drop the toolkit dependency","commit_id":"6c316bdffdd9896847ba06a3c38cde2cf98a9b71"},{"author":{"_account_id":34926,"name":"Hediberto Cavalcante da Silva","display_name":"Hediberto C Silva","email":"hediberto.cavalcantedasilva@windriver.com","username":"hcavalca"},"change_message_id":"bb7f02154ff167ca8eb59b35cc9da113a9df5f6d","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":8,"id":"82bdf6dc_55135494","in_reply_to":"be2c7a8b_1eafb565","updated":"2022-11-04 23:29:54.000000000","message":"Done","commit_id":"6c316bdffdd9896847ba06a3c38cde2cf98a9b71"}],"platform-helm/platform-helm/helm-charts/ceph-csi-rbd/rbd-storage-init.yaml":[{"author":{"_account_id":28459,"name":"Bob Church","email":"robert.church@windriver.com","username":"rchurch"},"change_message_id":"f0bcdda4ae3f6cc146bd39a80f43662094874011","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":8,"id":"d0372a8b_54e27fdd","updated":"2022-11-03 09:11:25.000000000","message":"Turn this into a patch for the chart","commit_id":"6c316bdffdd9896847ba06a3c38cde2cf98a9b71"},{"author":{"_account_id":34926,"name":"Hediberto Cavalcante da Silva","display_name":"Hediberto C Silva","email":"hediberto.cavalcantedasilva@windriver.com","username":"hcavalca"},"change_message_id":"bb7f02154ff167ca8eb59b35cc9da113a9df5f6d","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":8,"id":"34e651ce_5bdc2fe3","in_reply_to":"d0372a8b_54e27fdd","updated":"2022-11-04 23:29:54.000000000","message":"Done","commit_id":"6c316bdffdd9896847ba06a3c38cde2cf98a9b71"},{"author":{"_account_id":28459,"name":"Bob Church","email":"robert.church@windriver.com","username":"rchurch"},"change_message_id":"f0bcdda4ae3f6cc146bd39a80f43662094874011","unresolved":true,"context_lines":[{"line_number":75,"context_line":"    \"helm.sh/hook-delete-policy\": \"before-hook-creation\""},{"line_number":76,"context_line":"data:"},{"line_number":77,"context_line":"  ceph.conf: |"},{"line_number":78,"context_line":"{{ tuple \"conf/_ceph-conf.tpl\" . | include \"helm-toolkit.utils.template\" | indent 4 }}"},{"line_number":79,"context_line":""},{"line_number":80,"context_line":"  storage-init.sh: |"},{"line_number":81,"context_line":"{{ tuple \"bin/_storage_init.sh.tpl\" . | include \"helm-toolkit.utils.template\" | indent 4 }}"}],"source_content_type":"text/x-yaml","patch_set":8,"id":"ba2fb291_f9e0a309","line":78,"updated":"2022-11-03 09:11:25.000000000","message":"Embed the content here and drop the helm-toolkit dependency","commit_id":"6c316bdffdd9896847ba06a3c38cde2cf98a9b71"},{"author":{"_account_id":34926,"name":"Hediberto Cavalcante da Silva","display_name":"Hediberto C Silva","email":"hediberto.cavalcantedasilva@windriver.com","username":"hcavalca"},"change_message_id":"bb7f02154ff167ca8eb59b35cc9da113a9df5f6d","unresolved":false,"context_lines":[{"line_number":75,"context_line":"    \"helm.sh/hook-delete-policy\": \"before-hook-creation\""},{"line_number":76,"context_line":"data:"},{"line_number":77,"context_line":"  ceph.conf: |"},{"line_number":78,"context_line":"{{ tuple \"conf/_ceph-conf.tpl\" . | include \"helm-toolkit.utils.template\" | indent 4 }}"},{"line_number":79,"context_line":""},{"line_number":80,"context_line":"  storage-init.sh: |"},{"line_number":81,"context_line":"{{ tuple \"bin/_storage_init.sh.tpl\" . | include \"helm-toolkit.utils.template\" | indent 4 }}"}],"source_content_type":"text/x-yaml","patch_set":8,"id":"69b24186_ebb11b51","line":78,"in_reply_to":"ba2fb291_f9e0a309","updated":"2022-11-04 23:29:54.000000000","message":"Done","commit_id":"6c316bdffdd9896847ba06a3c38cde2cf98a9b71"},{"author":{"_account_id":28459,"name":"Bob Church","email":"robert.church@windriver.com","username":"rchurch"},"change_message_id":"f0bcdda4ae3f6cc146bd39a80f43662094874011","unresolved":true,"context_lines":[{"line_number":78,"context_line":"{{ tuple \"conf/_ceph-conf.tpl\" . | include \"helm-toolkit.utils.template\" | indent 4 }}"},{"line_number":79,"context_line":""},{"line_number":80,"context_line":"  storage-init.sh: |"},{"line_number":81,"context_line":"{{ tuple \"bin/_storage_init.sh.tpl\" . | include \"helm-toolkit.utils.template\" | indent 4 }}"},{"line_number":82,"context_line":""},{"line_number":83,"context_line":"---"},{"line_number":84,"context_line":""}],"source_content_type":"text/x-yaml","patch_set":8,"id":"93f96401_61daacb5","line":81,"updated":"2022-11-03 09:11:25.000000000","message":"Embed the content here and drop the helm-toolkit dependency","commit_id":"6c316bdffdd9896847ba06a3c38cde2cf98a9b71"},{"author":{"_account_id":34926,"name":"Hediberto Cavalcante da Silva","display_name":"Hediberto C Silva","email":"hediberto.cavalcantedasilva@windriver.com","username":"hcavalca"},"change_message_id":"bb7f02154ff167ca8eb59b35cc9da113a9df5f6d","unresolved":false,"context_lines":[{"line_number":78,"context_line":"{{ tuple \"conf/_ceph-conf.tpl\" . | include \"helm-toolkit.utils.template\" | indent 4 }}"},{"line_number":79,"context_line":""},{"line_number":80,"context_line":"  storage-init.sh: |"},{"line_number":81,"context_line":"{{ tuple \"bin/_storage_init.sh.tpl\" . | include \"helm-toolkit.utils.template\" | indent 4 }}"},{"line_number":82,"context_line":""},{"line_number":83,"context_line":"---"},{"line_number":84,"context_line":""}],"source_content_type":"text/x-yaml","patch_set":8,"id":"c195e80c_30776242","line":81,"in_reply_to":"93f96401_61daacb5","updated":"2022-11-04 23:29:54.000000000","message":"Done","commit_id":"6c316bdffdd9896847ba06a3c38cde2cf98a9b71"},{"author":{"_account_id":28459,"name":"Bob Church","email":"robert.church@windriver.com","username":"rchurch"},"change_message_id":"f0bcdda4ae3f6cc146bd39a80f43662094874011","unresolved":true,"context_lines":[{"line_number":124,"context_line":"            - name: NAMESPACE"},{"line_number":125,"context_line":"              value: {{ .Release.Namespace }}"},{"line_number":126,"context_line":"            - name: ADDITIONAL_NAMESPACES"},{"line_number":127,"context_line":"              value: {{ include \"helm-toolkit.utils.joinListWithComma\" .Values.storageClass.additionalNamespaces | quote }}"},{"line_number":128,"context_line":"            - name: CEPH_ADMIN_SECRET"},{"line_number":129,"context_line":"              value: {{ .Values.classDefaults.adminSecretName }}"},{"line_number":130,"context_line":"            - name: CEPH_USER_SECRET"}],"source_content_type":"text/x-yaml","patch_set":8,"id":"0a3dd7ae_c7288f91","line":127,"updated":"2022-11-03 09:11:25.000000000","message":"drop the helm-toolkit dependency and use\n\n{{ join \",\" .Values.storageClass.additionalNamespaces | quote }}\n\nshould work.","commit_id":"6c316bdffdd9896847ba06a3c38cde2cf98a9b71"},{"author":{"_account_id":34926,"name":"Hediberto Cavalcante da Silva","display_name":"Hediberto C Silva","email":"hediberto.cavalcantedasilva@windriver.com","username":"hcavalca"},"change_message_id":"bb7f02154ff167ca8eb59b35cc9da113a9df5f6d","unresolved":false,"context_lines":[{"line_number":124,"context_line":"            - name: NAMESPACE"},{"line_number":125,"context_line":"              value: {{ .Release.Namespace }}"},{"line_number":126,"context_line":"            - name: ADDITIONAL_NAMESPACES"},{"line_number":127,"context_line":"              value: {{ include \"helm-toolkit.utils.joinListWithComma\" .Values.storageClass.additionalNamespaces | quote }}"},{"line_number":128,"context_line":"            - name: CEPH_ADMIN_SECRET"},{"line_number":129,"context_line":"              value: {{ .Values.classDefaults.adminSecretName }}"},{"line_number":130,"context_line":"            - name: CEPH_USER_SECRET"}],"source_content_type":"text/x-yaml","patch_set":8,"id":"868cf7f2_c450ef7c","line":127,"in_reply_to":"0a3dd7ae_c7288f91","updated":"2022-11-04 23:29:54.000000000","message":"Done","commit_id":"6c316bdffdd9896847ba06a3c38cde2cf98a9b71"}],"python-k8sapp-platform/k8sapp_platform/k8sapp_platform/helm/ceph_fs_provisioner.py":[{"author":{"_account_id":28459,"name":"Bob Church","email":"robert.church@windriver.com","username":"rchurch"},"change_message_id":"f0bcdda4ae3f6cc146bd39a80f43662094874011","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":8,"id":"085d4d2b_a01c7bb9","updated":"2022-11-03 09:11:25.000000000","message":"For this package:\n- remove the Centos build since we don\u0027t have a CentOS package for the new charts\n- remove the k8sapp_platform/armada directory, k8sapp_platform/armada/manifest_platform.py, armada_apply_retry from lifecycle_platform.py, systemconfig.armada.manifest_ops from k8sapp_platform/setup.cfg.","commit_id":"6c316bdffdd9896847ba06a3c38cde2cf98a9b71"},{"author":{"_account_id":34926,"name":"Hediberto Cavalcante da Silva","display_name":"Hediberto C Silva","email":"hediberto.cavalcantedasilva@windriver.com","username":"hcavalca"},"change_message_id":"bb7f02154ff167ca8eb59b35cc9da113a9df5f6d","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":8,"id":"8d52a8f6_643d9d94","in_reply_to":"085d4d2b_a01c7bb9","updated":"2022-11-04 23:29:54.000000000","message":"Done","commit_id":"6c316bdffdd9896847ba06a3c38cde2cf98a9b71"}],"stx-platform-helm/debian/deb_folder/control":[{"author":{"_account_id":28459,"name":"Bob Church","email":"robert.church@windriver.com","username":"rchurch"},"change_message_id":"f0bcdda4ae3f6cc146bd39a80f43662094874011","unresolved":true,"context_lines":[{"line_number":3,"context_line":"Priority: optional"},{"line_number":4,"context_line":"Maintainer: StarlingX Developers \u003cstarlingx-discuss@lists.starlingx.io\u003e"},{"line_number":5,"context_line":"Build-Depends: debhelper-compat (\u003d 13),"},{"line_number":6,"context_line":" chartmuseum,"},{"line_number":7,"context_line":" helm,"},{"line_number":8,"context_line":" openstack-helm-infra,"},{"line_number":9,"context_line":" procps,"},{"line_number":10,"context_line":" python3-k8sapp-platform,"},{"line_number":11,"context_line":" python3-k8sapp-platform-wheels,"},{"line_number":12,"context_line":" platform-helm"}],"source_content_type":"application/octet-stream","patch_set":8,"id":"f2110785_5874d55b","line":9,"range":{"start_line":6,"start_character":0,"end_line":9,"end_character":8},"updated":"2022-11-03 09:11:25.000000000","message":"only helm should be needed if we drop the helm toolkit dependency","commit_id":"6c316bdffdd9896847ba06a3c38cde2cf98a9b71"},{"author":{"_account_id":34926,"name":"Hediberto Cavalcante da Silva","display_name":"Hediberto C Silva","email":"hediberto.cavalcantedasilva@windriver.com","username":"hcavalca"},"change_message_id":"bb7f02154ff167ca8eb59b35cc9da113a9df5f6d","unresolved":false,"context_lines":[{"line_number":3,"context_line":"Priority: optional"},{"line_number":4,"context_line":"Maintainer: StarlingX Developers \u003cstarlingx-discuss@lists.starlingx.io\u003e"},{"line_number":5,"context_line":"Build-Depends: debhelper-compat (\u003d 13),"},{"line_number":6,"context_line":" chartmuseum,"},{"line_number":7,"context_line":" helm,"},{"line_number":8,"context_line":" openstack-helm-infra,"},{"line_number":9,"context_line":" procps,"},{"line_number":10,"context_line":" python3-k8sapp-platform,"},{"line_number":11,"context_line":" python3-k8sapp-platform-wheels,"},{"line_number":12,"context_line":" platform-helm"}],"source_content_type":"application/octet-stream","patch_set":8,"id":"3f54773d_558a89de","line":9,"range":{"start_line":6,"start_character":0,"end_line":9,"end_character":8},"in_reply_to":"f2110785_5874d55b","updated":"2022-11-04 23:29:54.000000000","message":"Done","commit_id":"6c316bdffdd9896847ba06a3c38cde2cf98a9b71"},{"author":{"_account_id":28459,"name":"Bob Church","email":"robert.church@windriver.com","username":"rchurch"},"change_message_id":"f0bcdda4ae3f6cc146bd39a80f43662094874011","unresolved":true,"context_lines":[{"line_number":18,"context_line":"Architecture: any"},{"line_number":19,"context_line":"Depends: ${misc:Depends}"},{"line_number":20,"context_line":"Description: StarlingX Platform FluxCD Helm Charts"},{"line_number":21,"context_line":" This package contains FluxCD helm charts for the platform armada"},{"line_number":22,"context_line":" application."}],"source_content_type":"application/octet-stream","patch_set":8,"id":"d78621bd_96b16e66","line":21,"updated":"2022-11-03 09:11:25.000000000","message":"remove the armada reference","commit_id":"6c316bdffdd9896847ba06a3c38cde2cf98a9b71"},{"author":{"_account_id":34926,"name":"Hediberto Cavalcante da Silva","display_name":"Hediberto C Silva","email":"hediberto.cavalcantedasilva@windriver.com","username":"hcavalca"},"change_message_id":"bb7f02154ff167ca8eb59b35cc9da113a9df5f6d","unresolved":false,"context_lines":[{"line_number":18,"context_line":"Architecture: any"},{"line_number":19,"context_line":"Depends: ${misc:Depends}"},{"line_number":20,"context_line":"Description: StarlingX Platform FluxCD Helm Charts"},{"line_number":21,"context_line":" This package contains FluxCD helm charts for the platform armada"},{"line_number":22,"context_line":" application."}],"source_content_type":"application/octet-stream","patch_set":8,"id":"f503e687_4caeb8c7","line":21,"in_reply_to":"d78621bd_96b16e66","updated":"2022-11-04 23:29:54.000000000","message":"Done","commit_id":"6c316bdffdd9896847ba06a3c38cde2cf98a9b71"}],"stx-platform-helm/debian/deb_folder/rules":[{"author":{"_account_id":28459,"name":"Bob Church","email":"robert.church@windriver.com","username":"rchurch"},"change_message_id":"f0bcdda4ae3f6cc146bd39a80f43662094874011","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":8,"id":"f85997c4_b9aa5daa","updated":"2022-11-03 09:11:25.000000000","message":"For this package, remove the CentOS build since we don\u0027t have a CentOS package for the new charts","commit_id":"6c316bdffdd9896847ba06a3c38cde2cf98a9b71"},{"author":{"_account_id":34926,"name":"Hediberto Cavalcante da Silva","display_name":"Hediberto C Silva","email":"hediberto.cavalcantedasilva@windriver.com","username":"hcavalca"},"change_message_id":"bb7f02154ff167ca8eb59b35cc9da113a9df5f6d","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":8,"id":"ef453d1e_7606affb","in_reply_to":"f85997c4_b9aa5daa","updated":"2022-11-04 23:29:54.000000000","message":"Done","commit_id":"6c316bdffdd9896847ba06a3c38cde2cf98a9b71"},{"author":{"_account_id":28459,"name":"Bob Church","email":"robert.church@windriver.com","username":"rchurch"},"change_message_id":"31104db08385fa46b0f3ab6622d7ad73e5b07bec","unresolved":true,"context_lines":[{"line_number":6,"context_line":"export EXTRA_CHARTS \u003d $(ROOT)/opt/extracharts"},{"line_number":7,"context_line":""},{"line_number":8,"context_line":"export DEB_VERSION \u003d $(shell dpkg-parsechangelog | egrep \u0027^Version:\u0027 | cut -f 2 -d \u0027 \u0027)"},{"line_number":9,"context_line":"export MAJOR \u003d $(shell echo $(DEB_VERSION) | cut -f 1 -d \u0027.\u0027)"},{"line_number":10,"context_line":"export MINOR_PATCH \u003d $(shell echo $(DEB_VERSION) | cut -f 2 -d \u0027.\u0027)"},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"export APP_NAME \u003d platform-integ-apps"},{"line_number":13,"context_line":"export APP_VERSION \u003d $(MAJOR).$(MINOR_PATCH)"}],"source_content_type":"application/octet-stream","patch_set":8,"id":"fd355d80_016dec86","line":10,"range":{"start_line":9,"start_character":0,"end_line":10,"end_character":67},"updated":"2022-11-04 18:54:03.000000000","message":"The package versioning isn\u0027t correct:\n\nWe need:\n\n modified   stx-platform-helm/debian/meta_data.yaml\n @@ -6,4 +6,6 @@ src_files:\n    - ${MY_REPO}/stx/helm-charts/node-feature-discovery/node-feature-discovery/helm-charts\n  revision:\n    dist: $STX_DIST\n -  PKG_GITREVCOUNT: true\n +  GITREVCOUNT:\n +    BASE_SRCREV: c608f2aaa92064b712e7076e4141a162b78fe995\n +    SRC_DIR: ${MY_REPO}/stx/platform-armada-app\n \n \n and\n \n  modified   stx-platform-helm/debian/deb_folder/rules\n @@ -6,16 +6,15 @@ export APP_FOLDER \u003d $(ROOT)/usr/local/share/applications/helm\n  export EXTRA_CHARTS \u003d $(ROOT)/opt/extracharts\n  \n  export DEB_VERSION \u003d $(shell dpkg-parsechangelog | egrep \u0027^Version:\u0027 | cut -f 2 -d \u0027 \u0027)\n -export MAJOR \u003d $(shell echo $(DEB_VERSION) | cut -f 1 -d \u0027.\u0027)\n -export MINOR_PATCH \u003d $(shell echo $(DEB_VERSION) | cut -f 2 -d \u0027.\u0027)\n +export MAJOR \u003d $(shell echo $(DEB_VERSION) | cut -f 1 -d \u0027-\u0027)\n +export MINOR_PATCH \u003d $(shell echo $(DEB_VERSION) | cut -f 4 -d \u0027.\u0027)\n  \n  export APP_NAME \u003d platform-integ-apps\n -export APP_VERSION \u003d $(MAJOR).$(MINOR_PATCH)\n +export APP_VERSION \u003d $(MAJOR)-$(MINOR_PATCH)\n \n \n Should produce:\n \n stx-platform-helm/stx-platform-helm_1.0-1.stx.94_amd64.deb:\n-rwxr-xr-x root/root     57639 2022-11-04 03:50 ./usr/local/share/applications/helm/platform-integ-apps-1.0-94.tgz","commit_id":"6c316bdffdd9896847ba06a3c38cde2cf98a9b71"},{"author":{"_account_id":34926,"name":"Hediberto Cavalcante da Silva","display_name":"Hediberto C Silva","email":"hediberto.cavalcantedasilva@windriver.com","username":"hcavalca"},"change_message_id":"bb7f02154ff167ca8eb59b35cc9da113a9df5f6d","unresolved":false,"context_lines":[{"line_number":6,"context_line":"export EXTRA_CHARTS \u003d $(ROOT)/opt/extracharts"},{"line_number":7,"context_line":""},{"line_number":8,"context_line":"export DEB_VERSION \u003d $(shell dpkg-parsechangelog | egrep \u0027^Version:\u0027 | cut -f 2 -d \u0027 \u0027)"},{"line_number":9,"context_line":"export MAJOR \u003d $(shell echo $(DEB_VERSION) | cut -f 1 -d \u0027.\u0027)"},{"line_number":10,"context_line":"export MINOR_PATCH \u003d $(shell echo $(DEB_VERSION) | cut -f 2 -d \u0027.\u0027)"},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"export APP_NAME \u003d platform-integ-apps"},{"line_number":13,"context_line":"export APP_VERSION \u003d $(MAJOR).$(MINOR_PATCH)"}],"source_content_type":"application/octet-stream","patch_set":8,"id":"7616878e_746aeec6","line":10,"range":{"start_line":9,"start_character":0,"end_line":10,"end_character":67},"in_reply_to":"fd355d80_016dec86","updated":"2022-11-04 23:29:54.000000000","message":"Done","commit_id":"6c316bdffdd9896847ba06a3c38cde2cf98a9b71"},{"author":{"_account_id":28459,"name":"Bob Church","email":"robert.church@windriver.com","username":"rchurch"},"change_message_id":"f0bcdda4ae3f6cc146bd39a80f43662094874011","unresolved":true,"context_lines":[{"line_number":15,"context_line":"export HELM_FOLDER \u003d /usr/lib/helm"},{"line_number":16,"context_line":"export HELM_REPO \u003d stx-platform"},{"line_number":17,"context_line":"export STAGING_FLUXCD \u003d staging-fluxcd"},{"line_number":18,"context_line":"export TOOLKIT_VERSION \u003d 0.2.19"},{"line_number":19,"context_line":""},{"line_number":20,"context_line":"%:"},{"line_number":21,"context_line":"\tdh $@"}],"source_content_type":"application/octet-stream","patch_set":8,"id":"38c0cb42_f232ca5a","line":18,"range":{"start_line":18,"start_character":0,"end_line":18,"end_character":31},"updated":"2022-11-03 09:11:25.000000000","message":"Remove with the helm-toolkit dependency","commit_id":"6c316bdffdd9896847ba06a3c38cde2cf98a9b71"},{"author":{"_account_id":34926,"name":"Hediberto Cavalcante da Silva","display_name":"Hediberto C Silva","email":"hediberto.cavalcantedasilva@windriver.com","username":"hcavalca"},"change_message_id":"bb7f02154ff167ca8eb59b35cc9da113a9df5f6d","unresolved":false,"context_lines":[{"line_number":15,"context_line":"export HELM_FOLDER \u003d /usr/lib/helm"},{"line_number":16,"context_line":"export HELM_REPO \u003d stx-platform"},{"line_number":17,"context_line":"export STAGING_FLUXCD \u003d staging-fluxcd"},{"line_number":18,"context_line":"export TOOLKIT_VERSION \u003d 0.2.19"},{"line_number":19,"context_line":""},{"line_number":20,"context_line":"%:"},{"line_number":21,"context_line":"\tdh $@"}],"source_content_type":"application/octet-stream","patch_set":8,"id":"34ce49d0_d2125d76","line":18,"range":{"start_line":18,"start_character":0,"end_line":18,"end_character":31},"in_reply_to":"38c0cb42_f232ca5a","updated":"2022-11-04 23:29:54.000000000","message":"Done","commit_id":"6c316bdffdd9896847ba06a3c38cde2cf98a9b71"},{"author":{"_account_id":28459,"name":"Bob Church","email":"robert.church@windriver.com","username":"rchurch"},"change_message_id":"f0bcdda4ae3f6cc146bd39a80f43662094874011","unresolved":true,"context_lines":[{"line_number":25,"context_line":"\t############"},{"line_number":26,"context_line":"\t#  COMMON  #"},{"line_number":27,"context_line":"\t############"},{"line_number":28,"context_line":"\t# Stage helm-toolkit in the local repo."},{"line_number":29,"context_line":"\tcp $(HELM_FOLDER)/helm-toolkit-$(TOOLKIT_VERSION).tgz helm-charts/"},{"line_number":30,"context_line":""},{"line_number":31,"context_line":"\t# Host a server for the helm charts."},{"line_number":32,"context_line":"\tchartmuseum --debug --port\u003d8879 --context-path\u003d\u0027/charts\u0027 --storage\u003d\"local\" \\"},{"line_number":33,"context_line":"\t\t--storage-local-rootdir\u003d\"./helm-charts\" \u0026"},{"line_number":34,"context_line":"\tsleep 2"},{"line_number":35,"context_line":"\thelm repo add local http://localhost:8879/charts"},{"line_number":36,"context_line":""},{"line_number":37,"context_line":"\t# Create the TGZ file."},{"line_number":38,"context_line":"\tcd helm-charts \u0026\u0026 make ceph-pools-audit"}],"source_content_type":"application/octet-stream","patch_set":8,"id":"b7d13683_6ab95954","line":35,"range":{"start_line":28,"start_character":0,"end_line":35,"end_character":49},"updated":"2022-11-03 09:11:25.000000000","message":"same comment as above","commit_id":"6c316bdffdd9896847ba06a3c38cde2cf98a9b71"},{"author":{"_account_id":34926,"name":"Hediberto Cavalcante da Silva","display_name":"Hediberto C Silva","email":"hediberto.cavalcantedasilva@windriver.com","username":"hcavalca"},"change_message_id":"bb7f02154ff167ca8eb59b35cc9da113a9df5f6d","unresolved":false,"context_lines":[{"line_number":25,"context_line":"\t############"},{"line_number":26,"context_line":"\t#  COMMON  #"},{"line_number":27,"context_line":"\t############"},{"line_number":28,"context_line":"\t# Stage helm-toolkit in the local repo."},{"line_number":29,"context_line":"\tcp $(HELM_FOLDER)/helm-toolkit-$(TOOLKIT_VERSION).tgz helm-charts/"},{"line_number":30,"context_line":""},{"line_number":31,"context_line":"\t# Host a server for the helm charts."},{"line_number":32,"context_line":"\tchartmuseum --debug --port\u003d8879 --context-path\u003d\u0027/charts\u0027 --storage\u003d\"local\" \\"},{"line_number":33,"context_line":"\t\t--storage-local-rootdir\u003d\"./helm-charts\" \u0026"},{"line_number":34,"context_line":"\tsleep 2"},{"line_number":35,"context_line":"\thelm repo add local http://localhost:8879/charts"},{"line_number":36,"context_line":""},{"line_number":37,"context_line":"\t# Create the TGZ file."},{"line_number":38,"context_line":"\tcd helm-charts \u0026\u0026 make ceph-pools-audit"}],"source_content_type":"application/octet-stream","patch_set":8,"id":"7568e119_29da64f7","line":35,"range":{"start_line":28,"start_character":0,"end_line":35,"end_character":49},"in_reply_to":"b7d13683_6ab95954","updated":"2022-11-04 23:29:54.000000000","message":"Done","commit_id":"6c316bdffdd9896847ba06a3c38cde2cf98a9b71"},{"author":{"_account_id":28459,"name":"Bob Church","email":"robert.church@windriver.com","username":"rchurch"},"change_message_id":"f0bcdda4ae3f6cc146bd39a80f43662094874011","unresolved":true,"context_lines":[{"line_number":38,"context_line":"\tcd helm-charts \u0026\u0026 make ceph-pools-audit"},{"line_number":39,"context_line":"\tcd helm-charts \u0026\u0026 make node-feature-discovery"},{"line_number":40,"context_line":""},{"line_number":41,"context_line":"\t# Terminate the helm chart server."},{"line_number":42,"context_line":"\tpkill chartmuseum"},{"line_number":43,"context_line":""},{"line_number":44,"context_line":"\t############"},{"line_number":45,"context_line":"\t#  FLUXCD  #"}],"source_content_type":"application/octet-stream","patch_set":8,"id":"66fc411a_0631a7f3","line":42,"range":{"start_line":41,"start_character":0,"end_line":42,"end_character":18},"updated":"2022-11-03 09:11:25.000000000","message":"same comment as above","commit_id":"6c316bdffdd9896847ba06a3c38cde2cf98a9b71"},{"author":{"_account_id":34926,"name":"Hediberto Cavalcante da Silva","display_name":"Hediberto C Silva","email":"hediberto.cavalcantedasilva@windriver.com","username":"hcavalca"},"change_message_id":"bb7f02154ff167ca8eb59b35cc9da113a9df5f6d","unresolved":false,"context_lines":[{"line_number":38,"context_line":"\tcd helm-charts \u0026\u0026 make ceph-pools-audit"},{"line_number":39,"context_line":"\tcd helm-charts \u0026\u0026 make node-feature-discovery"},{"line_number":40,"context_line":""},{"line_number":41,"context_line":"\t# Terminate the helm chart server."},{"line_number":42,"context_line":"\tpkill chartmuseum"},{"line_number":43,"context_line":""},{"line_number":44,"context_line":"\t############"},{"line_number":45,"context_line":"\t#  FLUXCD  #"}],"source_content_type":"application/octet-stream","patch_set":8,"id":"bd6476b1_87118bdb","line":42,"range":{"start_line":41,"start_character":0,"end_line":42,"end_character":18},"in_reply_to":"66fc411a_0631a7f3","updated":"2022-11-04 23:29:54.000000000","message":"Done","commit_id":"6c316bdffdd9896847ba06a3c38cde2cf98a9b71"},{"author":{"_account_id":28459,"name":"Bob Church","email":"robert.church@windriver.com","username":"rchurch"},"change_message_id":"261a9b5e339541e88808a07c20c6ef8e5e6aba71","unresolved":true,"context_lines":[{"line_number":19,"context_line":"%:"},{"line_number":20,"context_line":"\tdh $@"},{"line_number":21,"context_line":""},{"line_number":22,"context_line":"override_dh_auto_build:"},{"line_number":23,"context_line":"\t# Create the TGZ file."},{"line_number":24,"context_line":"\tcd helm-charts \u0026\u0026 make ceph-pools-audit"},{"line_number":25,"context_line":"\tcd helm-charts \u0026\u0026 make node-feature-discovery"}],"source_content_type":"application/octet-stream","patch_set":10,"id":"c2d72d44_69995da8","line":22,"updated":"2022-11-05 18:19:25.000000000","message":"Now that helm-toolkit/chartmuseum dependency is gone, we can remove:\n - stx-platform-helm/stx-platform-helm/files/index.yaml\n - stx-platform-helm/stx-platform-helm/files/repositories.yaml","commit_id":"ff78686ac8d87d875fa022a0721f4d09379a17c3"},{"author":{"_account_id":34926,"name":"Hediberto Cavalcante da Silva","display_name":"Hediberto C Silva","email":"hediberto.cavalcantedasilva@windriver.com","username":"hcavalca"},"change_message_id":"4eebd0560ab01cc6c62e93497ed9d024d5a7922a","unresolved":false,"context_lines":[{"line_number":19,"context_line":"%:"},{"line_number":20,"context_line":"\tdh $@"},{"line_number":21,"context_line":""},{"line_number":22,"context_line":"override_dh_auto_build:"},{"line_number":23,"context_line":"\t# Create the TGZ file."},{"line_number":24,"context_line":"\tcd helm-charts \u0026\u0026 make ceph-pools-audit"},{"line_number":25,"context_line":"\tcd helm-charts \u0026\u0026 make node-feature-discovery"}],"source_content_type":"application/octet-stream","patch_set":10,"id":"fef17bd1_37fde58d","line":22,"in_reply_to":"c2d72d44_69995da8","updated":"2022-11-05 19:41:17.000000000","message":"Done","commit_id":"ff78686ac8d87d875fa022a0721f4d09379a17c3"}],"stx-platform-helm/stx-platform-helm/fluxcd-manifests/cephfs-provisioner/cephfs-provisioner-static-overrides.yaml":[{"author":{"_account_id":28459,"name":"Bob Church","email":"robert.church@windriver.com","username":"rchurch"},"change_message_id":"a64981b094215db801a23546227fc331ec3c4a14","unresolved":true,"context_lines":[{"line_number":58,"context_line":"  controllerExpandSecretNamespace: kube-system"},{"line_number":59,"context_line":"  nodeStageSecretNamespace: kube-system"},{"line_number":60,"context_line":"  annotations: {"},{"line_number":61,"context_line":"    \"storageclass.kubernetes.io/is-default-class\": \"true\","},{"line_number":62,"context_line":"    \"helm.sh/hook\": \"pre-upgrade, pre-install\","},{"line_number":63,"context_line":"    \"helm.sh/hook-delete-policy\": \"before-hook-creation\""},{"line_number":64,"context_line":"  }"}],"source_content_type":"text/x-yaml","patch_set":12,"id":"7273e059_440fb021","line":61,"updated":"2022-11-07 20:14:57.000000000","message":"This should be \"false\". This is producing 2 default storage classes and we can only have one:\n\n [sysadmin@controller-0 ~(keystone_admin)]$ kubectl get sc --all-namespaces \n NAME                PROVISIONER           RECLAIMPOLICY   VOLUMEBINDINGMODE   ALLOWVOLUMEEXPANSION   AGE\n cephfs (default)    cephfs.csi.ceph.com   Delete          Immediate           true                   17m\n general (default)   rbd.csi.ceph.com      Delete          Immediate           true                   17m","commit_id":"00eeb799637c356b34b0e44829c067d960161eaa"},{"author":{"_account_id":34926,"name":"Hediberto Cavalcante da Silva","display_name":"Hediberto C Silva","email":"hediberto.cavalcantedasilva@windriver.com","username":"hcavalca"},"change_message_id":"d4becbfbf1964220bb67ac79db0e6e9f5cbf0f8a","unresolved":false,"context_lines":[{"line_number":58,"context_line":"  controllerExpandSecretNamespace: kube-system"},{"line_number":59,"context_line":"  nodeStageSecretNamespace: kube-system"},{"line_number":60,"context_line":"  annotations: {"},{"line_number":61,"context_line":"    \"storageclass.kubernetes.io/is-default-class\": \"true\","},{"line_number":62,"context_line":"    \"helm.sh/hook\": \"pre-upgrade, pre-install\","},{"line_number":63,"context_line":"    \"helm.sh/hook-delete-policy\": \"before-hook-creation\""},{"line_number":64,"context_line":"  }"}],"source_content_type":"text/x-yaml","patch_set":12,"id":"5740c894_479f323a","line":61,"in_reply_to":"7273e059_440fb021","updated":"2022-11-07 20:41:10.000000000","message":"Done","commit_id":"00eeb799637c356b34b0e44829c067d960161eaa"}],"stx-platform-helm/stx-platform-helm/fluxcd-manifests/cephfs-provisioner/helmrelease.yaml":[{"author":{"_account_id":28459,"name":"Bob Church","email":"robert.church@windriver.com","username":"rchurch"},"change_message_id":"f0bcdda4ae3f6cc146bd39a80f43662094874011","unresolved":true,"context_lines":[{"line_number":15,"context_line":"  chart:"},{"line_number":16,"context_line":"    spec:"},{"line_number":17,"context_line":"      chart: ceph-csi-cephfs"},{"line_number":18,"context_line":"      version: 3-canary"},{"line_number":19,"context_line":"      sourceRef:"},{"line_number":20,"context_line":"        kind: HelmRepository"},{"line_number":21,"context_line":"        name: stx-platform"}],"source_content_type":"text/x-yaml","patch_set":8,"id":"7fc08d2e_0954b6f2","line":18,"updated":"2022-11-03 09:11:25.000000000","message":"should be 3.6.2","commit_id":"6c316bdffdd9896847ba06a3c38cde2cf98a9b71"},{"author":{"_account_id":34926,"name":"Hediberto Cavalcante da Silva","display_name":"Hediberto C Silva","email":"hediberto.cavalcantedasilva@windriver.com","username":"hcavalca"},"change_message_id":"bb7f02154ff167ca8eb59b35cc9da113a9df5f6d","unresolved":false,"context_lines":[{"line_number":15,"context_line":"  chart:"},{"line_number":16,"context_line":"    spec:"},{"line_number":17,"context_line":"      chart: ceph-csi-cephfs"},{"line_number":18,"context_line":"      version: 3-canary"},{"line_number":19,"context_line":"      sourceRef:"},{"line_number":20,"context_line":"        kind: HelmRepository"},{"line_number":21,"context_line":"        name: stx-platform"}],"source_content_type":"text/x-yaml","patch_set":8,"id":"d81b0506_d3036d8d","line":18,"in_reply_to":"7fc08d2e_0954b6f2","updated":"2022-11-04 23:29:54.000000000","message":"Done","commit_id":"6c316bdffdd9896847ba06a3c38cde2cf98a9b71"}],"stx-platform-helm/stx-platform-helm/fluxcd-manifests/rbd-provisioner/helmrelease.yaml":[{"author":{"_account_id":28459,"name":"Bob Church","email":"robert.church@windriver.com","username":"rchurch"},"change_message_id":"f0bcdda4ae3f6cc146bd39a80f43662094874011","unresolved":true,"context_lines":[{"line_number":15,"context_line":"  chart:"},{"line_number":16,"context_line":"    spec:"},{"line_number":17,"context_line":"      chart: ceph-csi-rbd"},{"line_number":18,"context_line":"      version: 3-canary"},{"line_number":19,"context_line":"      sourceRef:"},{"line_number":20,"context_line":"        kind: HelmRepository"},{"line_number":21,"context_line":"        name: stx-platform"}],"source_content_type":"text/x-yaml","patch_set":8,"id":"50f93bde_7e5d0fe6","line":18,"updated":"2022-11-03 09:11:25.000000000","message":"should be 3.6.2","commit_id":"6c316bdffdd9896847ba06a3c38cde2cf98a9b71"},{"author":{"_account_id":34926,"name":"Hediberto Cavalcante da Silva","display_name":"Hediberto C Silva","email":"hediberto.cavalcantedasilva@windriver.com","username":"hcavalca"},"change_message_id":"bb7f02154ff167ca8eb59b35cc9da113a9df5f6d","unresolved":false,"context_lines":[{"line_number":15,"context_line":"  chart:"},{"line_number":16,"context_line":"    spec:"},{"line_number":17,"context_line":"      chart: ceph-csi-rbd"},{"line_number":18,"context_line":"      version: 3-canary"},{"line_number":19,"context_line":"      sourceRef:"},{"line_number":20,"context_line":"        kind: HelmRepository"},{"line_number":21,"context_line":"        name: stx-platform"}],"source_content_type":"text/x-yaml","patch_set":8,"id":"0d107626_59d2e470","line":18,"in_reply_to":"50f93bde_7e5d0fe6","updated":"2022-11-04 23:29:54.000000000","message":"Done","commit_id":"6c316bdffdd9896847ba06a3c38cde2cf98a9b71"}],"stx-platform-helm/stx-platform-helm/helm-charts/ceph-pools-audit/templates/job-ceph-pools-audit.yaml":[{"author":{"_account_id":28459,"name":"Bob Church","email":"robert.church@windriver.com","username":"rchurch"},"change_message_id":"261a9b5e339541e88808a07c20c6ef8e5e6aba71","unresolved":true,"context_lines":[{"line_number":18,"context_line":"metadata:"},{"line_number":19,"context_line":"  name: {{ $serviceAccountName }}"},{"line_number":20,"context_line":"  namespace: {{ $envAll.Release.namespace }}"},{"line_number":21,"context_line":""},{"line_number":22,"context_line":"---"},{"line_number":23,"context_line":"#"},{"line_number":24,"context_line":"# The CronJob makes sure all the Ceph pools have the right replication,"}],"source_content_type":"text/x-yaml","patch_set":10,"id":"24021be2_84919e29","line":21,"updated":"2022-11-05 18:19:25.000000000","message":"I think this is missing:\n\n imagePullSecrets:\n   - name: default-registry-key","commit_id":"ff78686ac8d87d875fa022a0721f4d09379a17c3"},{"author":{"_account_id":34926,"name":"Hediberto Cavalcante da Silva","display_name":"Hediberto C Silva","email":"hediberto.cavalcantedasilva@windriver.com","username":"hcavalca"},"change_message_id":"4eebd0560ab01cc6c62e93497ed9d024d5a7922a","unresolved":false,"context_lines":[{"line_number":18,"context_line":"metadata:"},{"line_number":19,"context_line":"  name: {{ $serviceAccountName }}"},{"line_number":20,"context_line":"  namespace: {{ $envAll.Release.namespace }}"},{"line_number":21,"context_line":""},{"line_number":22,"context_line":"---"},{"line_number":23,"context_line":"#"},{"line_number":24,"context_line":"# The CronJob makes sure all the Ceph pools have the right replication,"}],"source_content_type":"text/x-yaml","patch_set":10,"id":"3ab2584e_673a00dc","line":21,"in_reply_to":"24021be2_84919e29","updated":"2022-11-05 19:41:17.000000000","message":"Done","commit_id":"ff78686ac8d87d875fa022a0721f4d09379a17c3"}],"stx-platform-helm/stx-platform-helm/helm-charts/ceph-pools-audit/values.yaml":[{"author":{"_account_id":28459,"name":"Bob Church","email":"robert.church@windriver.com","username":"rchurch"},"change_message_id":"f0bcdda4ae3f6cc146bd39a80f43662094874011","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":8,"id":"041a7bb4_ac7e04d3","updated":"2022-11-03 09:11:25.000000000","message":"need to drop the helm-toolkit dependencies in this chart:\n- drop requirements.yaml\n- Embed templates/bin/_ceph-pools-audit.sh.tpl in templates/confimap-bin.yaml\n- replace the service account dependency in templates/job-ceph-pools-audit.yaml by declaring it directlly in the file","commit_id":"6c316bdffdd9896847ba06a3c38cde2cf98a9b71"},{"author":{"_account_id":34926,"name":"Hediberto Cavalcante da Silva","display_name":"Hediberto C Silva","email":"hediberto.cavalcantedasilva@windriver.com","username":"hcavalca"},"change_message_id":"bb7f02154ff167ca8eb59b35cc9da113a9df5f6d","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":8,"id":"f1888096_8792721a","in_reply_to":"041a7bb4_ac7e04d3","updated":"2022-11-04 23:29:54.000000000","message":"Done","commit_id":"6c316bdffdd9896847ba06a3c38cde2cf98a9b71"}]}