)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":22636,"name":"Cliff Parsons","email":"cliffhparsons@aol.com","username":"cliffparsons"},"change_message_id":"7ca70d2d7dc216380da8560b7541e647335d9e72","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"182c86ad_2a7eda6c","updated":"2022-01-03 03:01:32.000000000","message":"Looks good so far! As discussed before, we still need to eliminate the use of the namespace-client-ceph-config-manager job, and instead have the ceph-rbd-pool job dynamically populate the mon host IP address in its own /etc/ceph/ceph.conf file, (from inside the pool init code), as we discussed before. Also, as discussed, we need to update the rgw ceph.conf whenever a mon host IP address changes. Please let me know if you run into any roadblocks or issues, I am happy to help out.","commit_id":"71e00e89bb21584dbb935ac02850e63cf747f605"},{"author":{"_account_id":22636,"name":"Cliff Parsons","email":"cliffhparsons@aol.com","username":"cliffparsons"},"change_message_id":"299a8996067c1d8315eec28d95511a4f3816fc4d","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":14,"id":"0dc5808c_925a136a","updated":"2022-01-05 16:44:56.000000000","message":"I like the approach, it just needs to be refined a bit to handle multiple RGWs, if possible.","commit_id":"596c47104f373741d60d45d3716e24890d4ca109"},{"author":{"_account_id":22636,"name":"Cliff Parsons","email":"cliffhparsons@aol.com","username":"cliffparsons"},"change_message_id":"a74291aa74b3df5c29570709c7e45d96b6c1c128","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":26,"id":"5b4ac774_10b6be85","updated":"2022-01-13 16:14:34.000000000","message":"Thanks for addressing the comments!  Just a couple more minor adjustments.","commit_id":"2dd5c3dd31c2488b5f3364465988338579cc3d9d"},{"author":{"_account_id":22636,"name":"Cliff Parsons","email":"cliffhparsons@aol.com","username":"cliffparsons"},"change_message_id":"3fb20a91fecad44678ef348d77e33ca4331d1dc4","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":36,"id":"afdb81ec_403759be","updated":"2022-02-10 16:09:51.000000000","message":"Need to address one more thing.  See comments in deployment-checkdns.yaml.","commit_id":"f9b538add5d79181f1d0d4ca4a6a4ee2f19581f4"},{"author":{"_account_id":29974,"name":"Stephen Taylor","email":"stephen.taylor.1@att.com","username":"st053q"},"change_message_id":"b7af29cd3ccc5d9419527db843014b987f6f8480","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":40,"id":"c579e83e_281f03ce","updated":"2022-02-14 12:17:37.000000000","message":"recheck","commit_id":"728c340dc0c6a8aa57f43eeee04a524ac2683647"}],"ceph-client/templates/bin/_namespace-client-ceph-config-manager.sh.tpl":[{"author":{"_account_id":22636,"name":"Cliff Parsons","email":"cliffhparsons@aol.com","username":"cliffparsons"},"change_message_id":"bfdb048df9f75e55762e61f8f52356b3f80dce0f","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":23,"id":"839467ef_cca749fd","updated":"2022-01-12 19:34:41.000000000","message":"This file and the associated job definition can be removed. I\u0027ve looked through the ceph-client extensively, and this job isn\u0027t needed because all the jobs and pods that use the ceph-mon endpoint are using the helm-toolkit function to update the mon endpoint IPs on their own.","commit_id":"c0d53ea526370bd76b267077b72930a81bd74f1e"},{"author":{"_account_id":32433,"name":"Vladimir Sigunov","email":"vladimir.sigunov@att.com","username":"vs422h"},"change_message_id":"3d22552a555cd7ef013e2115cda1d98b005f29a8","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":23,"id":"c461823d_25702427","in_reply_to":"839467ef_cca749fd","updated":"2022-01-12 23:09:40.000000000","message":"Implemented with PS #25","commit_id":"c0d53ea526370bd76b267077b72930a81bd74f1e"}],"ceph-client/templates/bin/utils/_checkDNS.sh.tpl":[{"author":{"_account_id":22636,"name":"Cliff Parsons","email":"cliffhparsons@aol.com","username":"cliffparsons"},"change_message_id":"bfdb048df9f75e55762e61f8f52356b3f80dce0f","unresolved":true,"context_lines":[{"line_number":13,"context_line":"See the License for the specific language governing permissions and"},{"line_number":14,"context_line":"limitations under the License."},{"line_number":15,"context_line":"*/}}"},{"line_number":16,"context_line":"# TBD: Add fixes to RGWs located in the different namespace"},{"line_number":17,"context_line":": \"${CEPH_CONF:\u003d\"/etc/ceph/${CLUSTER}.conf\"}\""},{"line_number":18,"context_line":"ENDPOINT\u003d\"{$1}\""},{"line_number":19,"context_line":""}],"source_content_type":"text/x-smarty","patch_set":23,"id":"61939fb0_76ba5dbc","line":16,"range":{"start_line":16,"start_character":0,"end_line":16,"end_character":1},"updated":"2022-01-12 19:34:41.000000000","message":"This comment can be removed, right?","commit_id":"c0d53ea526370bd76b267077b72930a81bd74f1e"},{"author":{"_account_id":32433,"name":"Vladimir Sigunov","email":"vladimir.sigunov@att.com","username":"vs422h"},"change_message_id":"3d22552a555cd7ef013e2115cda1d98b005f29a8","unresolved":false,"context_lines":[{"line_number":13,"context_line":"See the License for the specific language governing permissions and"},{"line_number":14,"context_line":"limitations under the License."},{"line_number":15,"context_line":"*/}}"},{"line_number":16,"context_line":"# TBD: Add fixes to RGWs located in the different namespace"},{"line_number":17,"context_line":": \"${CEPH_CONF:\u003d\"/etc/ceph/${CLUSTER}.conf\"}\""},{"line_number":18,"context_line":"ENDPOINT\u003d\"{$1}\""},{"line_number":19,"context_line":""}],"source_content_type":"text/x-smarty","patch_set":23,"id":"849f836a_d80e582d","line":16,"range":{"start_line":16,"start_character":0,"end_line":16,"end_character":1},"in_reply_to":"61939fb0_76ba5dbc","updated":"2022-01-12 23:09:40.000000000","message":"Implemented with PS #24\nThank you for suggestions!","commit_id":"c0d53ea526370bd76b267077b72930a81bd74f1e"}],"ceph-client/templates/bin/utils/_checkDNS_start.sh.tpl":[{"author":{"_account_id":22636,"name":"Cliff Parsons","email":"cliffhparsons@aol.com","username":"cliffparsons"},"change_message_id":"299a8996067c1d8315eec28d95511a4f3816fc4d","unresolved":true,"context_lines":[{"line_number":19,"context_line":"{{ include \"helm-toolkit.snippets.mon_host_from_k8s_ep\" . }}"},{"line_number":20,"context_line":"# TBD: Add fixes to RGWs located in the different namespace"},{"line_number":21,"context_line":""},{"line_number":22,"context_line":"rgwNS\u003d{{ .Values.endpoints.ceph_object_store.namespace }}"},{"line_number":23,"context_line":""},{"line_number":24,"context_line":"function check_mon_dns {"},{"line_number":25,"context_line":"  NS\u003d${1}"}],"source_content_type":"text/x-smarty","patch_set":14,"id":"e0dfa694_29a9c373","line":22,"range":{"start_line":22,"start_character":0,"end_line":22,"end_character":1},"updated":"2022-01-05 16:44:56.000000000","message":"might need to use range command or a for loop here as there could be multple RGWs in a cluster..","commit_id":"596c47104f373741d60d45d3716e24890d4ca109"},{"author":{"_account_id":22636,"name":"Cliff Parsons","email":"cliffhparsons@aol.com","username":"cliffparsons"},"change_message_id":"03e3a71dfcf1ac57588e485d3131689ddfb51a17","unresolved":false,"context_lines":[{"line_number":19,"context_line":"{{ include \"helm-toolkit.snippets.mon_host_from_k8s_ep\" . }}"},{"line_number":20,"context_line":"# TBD: Add fixes to RGWs located in the different namespace"},{"line_number":21,"context_line":""},{"line_number":22,"context_line":"rgwNS\u003d{{ .Values.endpoints.ceph_object_store.namespace }}"},{"line_number":23,"context_line":""},{"line_number":24,"context_line":"function check_mon_dns {"},{"line_number":25,"context_line":"  NS\u003d${1}"}],"source_content_type":"text/x-smarty","patch_set":14,"id":"87d4b714_1c6be3fd","line":22,"range":{"start_line":22,"start_character":0,"end_line":22,"end_character":1},"in_reply_to":"e0dfa694_29a9c373","updated":"2022-02-14 15:03:02.000000000","message":"Done","commit_id":"596c47104f373741d60d45d3716e24890d4ca109"}],"ceph-client/templates/deployment-checkdns.yaml":[{"author":{"_account_id":22636,"name":"Cliff Parsons","email":"cliffhparsons@aol.com","username":"cliffparsons"},"change_message_id":"80cef30ee84e4c91367785c4d17563f9d0acfefb","unresolved":true,"context_lines":[{"line_number":42,"context_line":"kind: RoleBinding"},{"line_number":43,"context_line":"metadata:"},{"line_number":44,"context_line":"  name: {{ $cephRoleBindingName }}"},{"line_number":45,"context_line":"  namespace: {{ $envAll.Release.Namespace }}"},{"line_number":46,"context_line":"roleRef:"},{"line_number":47,"context_line":"  apiGroup: rbac.authorization.k8s.io"},{"line_number":48,"context_line":"  kind: ClusterRole"}],"source_content_type":"text/x-yaml","patch_set":23,"id":"4fd5d3a1_5dd12180","line":45,"range":{"start_line":45,"start_character":0,"end_line":45,"end_character":3},"updated":"2022-01-12 15:07:05.000000000","message":"Will this allow the checkDNS pod to see the RGW pods of the other namespace(s)? We might need one of these for each RGW namespace.","commit_id":"c0d53ea526370bd76b267077b72930a81bd74f1e"},{"author":{"_account_id":22636,"name":"Cliff Parsons","email":"cliffhparsons@aol.com","username":"cliffparsons"},"change_message_id":"5ab3dd6b38195724776f73148a5327751e452a83","unresolved":false,"context_lines":[{"line_number":42,"context_line":"kind: RoleBinding"},{"line_number":43,"context_line":"metadata:"},{"line_number":44,"context_line":"  name: {{ $cephRoleBindingName }}"},{"line_number":45,"context_line":"  namespace: {{ $envAll.Release.Namespace }}"},{"line_number":46,"context_line":"roleRef:"},{"line_number":47,"context_line":"  apiGroup: rbac.authorization.k8s.io"},{"line_number":48,"context_line":"  kind: ClusterRole"}],"source_content_type":"text/x-yaml","patch_set":23,"id":"2c179a8a_bf634cf2","line":45,"range":{"start_line":45,"start_character":0,"end_line":45,"end_character":3},"in_reply_to":"4fd5d3a1_5dd12180","updated":"2022-01-12 15:15:30.000000000","message":"Nevermind. This is a cluster role, so it should work.","commit_id":"c0d53ea526370bd76b267077b72930a81bd74f1e"},{"author":{"_account_id":22636,"name":"Cliff Parsons","email":"cliffhparsons@aol.com","username":"cliffparsons"},"change_message_id":"3fb20a91fecad44678ef348d77e33ca4331d1dc4","unresolved":true,"context_lines":[{"line_number":52,"context_line":"roleRef:"},{"line_number":53,"context_line":"  apiGroup: rbac.authorization.k8s.io"},{"line_number":54,"context_line":"  kind: ClusterRole"},{"line_number":55,"context_line":"  name: clusterrole-checkdns"},{"line_number":56,"context_line":"subjects:"},{"line_number":57,"context_line":"  - kind: ServiceAccount"},{"line_number":58,"context_line":"    name: {{ $serviceAccountName }}"}],"source_content_type":"text/x-yaml","patch_set":36,"id":"4844fc7e_b857b38f","line":55,"range":{"start_line":55,"start_character":8,"end_line":55,"end_character":28},"updated":"2022-02-10 16:09:51.000000000","message":"I think we need a way to distinguish the undercloud Ceph clusterrole-checkdns from the tenant Ceph clusterrole-checkdns.","commit_id":"f9b538add5d79181f1d0d4ca4a6a4ee2f19581f4"},{"author":{"_account_id":22636,"name":"Cliff Parsons","email":"cliffhparsons@aol.com","username":"cliffparsons"},"change_message_id":"03e3a71dfcf1ac57588e485d3131689ddfb51a17","unresolved":false,"context_lines":[{"line_number":52,"context_line":"roleRef:"},{"line_number":53,"context_line":"  apiGroup: rbac.authorization.k8s.io"},{"line_number":54,"context_line":"  kind: ClusterRole"},{"line_number":55,"context_line":"  name: clusterrole-checkdns"},{"line_number":56,"context_line":"subjects:"},{"line_number":57,"context_line":"  - kind: ServiceAccount"},{"line_number":58,"context_line":"    name: {{ $serviceAccountName }}"}],"source_content_type":"text/x-yaml","patch_set":36,"id":"747e8f57_6021a605","line":55,"range":{"start_line":55,"start_character":8,"end_line":55,"end_character":28},"in_reply_to":"4844fc7e_b857b38f","updated":"2022-02-14 15:03:02.000000000","message":"Done","commit_id":"f9b538add5d79181f1d0d4ca4a6a4ee2f19581f4"}],"ceph-client/values.yaml":[{"author":{"_account_id":22636,"name":"Cliff Parsons","email":"cliffhparsons@aol.com","username":"cliffparsons"},"change_message_id":"299a8996067c1d8315eec28d95511a4f3816fc4d","unresolved":true,"context_lines":[{"line_number":613,"context_line":"        default: 9283"},{"line_number":614,"context_line":"    scheme:"},{"line_number":615,"context_line":"      default: http"},{"line_number":616,"context_line":"  ceph_object_store:"},{"line_number":617,"context_line":"    namespace: openstack"},{"line_number":618,"context_line":"    hosts:"},{"line_number":619,"context_line":"      default: ceph-rgw"},{"line_number":620,"context_line":"    host_fqdn_override:"},{"line_number":621,"context_line":"      default: null"},{"line_number":622,"context_line":""},{"line_number":623,"context_line":"monitoring:"},{"line_number":624,"context_line":"  prometheus:"}],"source_content_type":"text/x-yaml","patch_set":14,"id":"4f3ebfa8_0ae02cbf","line":621,"range":{"start_line":616,"start_character":0,"end_line":621,"end_character":4},"updated":"2022-01-05 16:44:56.000000000","message":"This is a good thought, but keep in mind that there are multiple RGWs in a cluster (multiple namespaces). Using this approach, can we have multiple ceph_object_store entries, one for each namespace RGW is deployed?21","commit_id":"596c47104f373741d60d45d3716e24890d4ca109"},{"author":{"_account_id":32433,"name":"Vladimir Sigunov","email":"vladimir.sigunov@att.com","username":"vs422h"},"change_message_id":"72664aa26ff72e46145c69bea411a5cd086b5586","unresolved":true,"context_lines":[{"line_number":613,"context_line":"        default: 9283"},{"line_number":614,"context_line":"    scheme:"},{"line_number":615,"context_line":"      default: http"},{"line_number":616,"context_line":"  ceph_object_store:"},{"line_number":617,"context_line":"    namespace: openstack"},{"line_number":618,"context_line":"    hosts:"},{"line_number":619,"context_line":"      default: ceph-rgw"},{"line_number":620,"context_line":"    host_fqdn_override:"},{"line_number":621,"context_line":"      default: null"},{"line_number":622,"context_line":""},{"line_number":623,"context_line":"monitoring:"},{"line_number":624,"context_line":"  prometheus:"}],"source_content_type":"text/x-yaml","patch_set":14,"id":"d73815a4_174c5640","line":621,"range":{"start_line":616,"start_character":0,"end_line":621,"end_character":4},"in_reply_to":"4f3ebfa8_0ae02cbf","updated":"2022-01-06 16:51:04.000000000","message":"Thanks Cliff. This is just a hard-coded patch to prove the approach. The main idea is here:\nhttps://review.opendev.org/c/openstack/openstack-helm-infra/+/821474/14/helm-toolkit/templates/endpoints/_hostname_namespaced_endpoint_namespace_lookup.tpl","commit_id":"596c47104f373741d60d45d3716e24890d4ca109"},{"author":{"_account_id":22636,"name":"Cliff Parsons","email":"cliffhparsons@aol.com","username":"cliffparsons"},"change_message_id":"03e3a71dfcf1ac57588e485d3131689ddfb51a17","unresolved":false,"context_lines":[{"line_number":613,"context_line":"        default: 9283"},{"line_number":614,"context_line":"    scheme:"},{"line_number":615,"context_line":"      default: http"},{"line_number":616,"context_line":"  ceph_object_store:"},{"line_number":617,"context_line":"    namespace: openstack"},{"line_number":618,"context_line":"    hosts:"},{"line_number":619,"context_line":"      default: ceph-rgw"},{"line_number":620,"context_line":"    host_fqdn_override:"},{"line_number":621,"context_line":"      default: null"},{"line_number":622,"context_line":""},{"line_number":623,"context_line":"monitoring:"},{"line_number":624,"context_line":"  prometheus:"}],"source_content_type":"text/x-yaml","patch_set":14,"id":"90e79fac_7e9909b5","line":621,"range":{"start_line":616,"start_character":0,"end_line":621,"end_character":4},"in_reply_to":"d73815a4_174c5640","updated":"2022-02-14 15:03:02.000000000","message":"Done","commit_id":"596c47104f373741d60d45d3716e24890d4ca109"},{"author":{"_account_id":22636,"name":"Cliff Parsons","email":"cliffhparsons@aol.com","username":"cliffparsons"},"change_message_id":"a74291aa74b3df5c29570709c7e45d96b6c1c128","unresolved":true,"context_lines":[{"line_number":253,"context_line":"    startingDeadlineSecs: 60"},{"line_number":254,"context_line":"  rbd_pool:"},{"line_number":255,"context_line":"    restartPolicy: OnFailure"},{"line_number":256,"context_line":"  client_ceph_config_update:"},{"line_number":257,"context_line":"    restartPolicy: OnFailure"},{"line_number":258,"context_line":""},{"line_number":259,"context_line":"conf:"},{"line_number":260,"context_line":"  features:"}],"source_content_type":"text/x-yaml","patch_set":26,"id":"5215ef9b_37a76a67","line":257,"range":{"start_line":256,"start_character":0,"end_line":257,"end_character":4},"updated":"2022-01-13 16:14:34.000000000","message":"I believe this is related to the namespace-ceph-client-config job that was deleted. So I think this can be deleted as well.","commit_id":"2dd5c3dd31c2488b5f3364465988338579cc3d9d"},{"author":{"_account_id":22636,"name":"Cliff Parsons","email":"cliffhparsons@aol.com","username":"cliffparsons"},"change_message_id":"03e3a71dfcf1ac57588e485d3131689ddfb51a17","unresolved":false,"context_lines":[{"line_number":253,"context_line":"    startingDeadlineSecs: 60"},{"line_number":254,"context_line":"  rbd_pool:"},{"line_number":255,"context_line":"    restartPolicy: OnFailure"},{"line_number":256,"context_line":"  client_ceph_config_update:"},{"line_number":257,"context_line":"    restartPolicy: OnFailure"},{"line_number":258,"context_line":""},{"line_number":259,"context_line":"conf:"},{"line_number":260,"context_line":"  features:"}],"source_content_type":"text/x-yaml","patch_set":26,"id":"73f68e40_49f6d983","line":257,"range":{"start_line":256,"start_character":0,"end_line":257,"end_character":4},"in_reply_to":"5215ef9b_37a76a67","updated":"2022-02-14 15:03:02.000000000","message":"Done","commit_id":"2dd5c3dd31c2488b5f3364465988338579cc3d9d"},{"author":{"_account_id":22636,"name":"Cliff Parsons","email":"cliffhparsons@aol.com","username":"cliffparsons"},"change_message_id":"a74291aa74b3df5c29570709c7e45d96b6c1c128","unresolved":true,"context_lines":[{"line_number":637,"context_line":"  deployment_mgr: true"},{"line_number":638,"context_line":"  deployment_checkdns: true"},{"line_number":639,"context_line":"  job_bootstrap: false"},{"line_number":640,"context_line":"  job_ns_client_ceph_config: true"},{"line_number":641,"context_line":"  job_cephfs_client_key: true"},{"line_number":642,"context_line":"  job_image_repo_sync: true"},{"line_number":643,"context_line":"  job_rbd_pool: true"}],"source_content_type":"text/x-yaml","patch_set":26,"id":"7a060a1b_615265af","line":640,"range":{"start_line":640,"start_character":0,"end_line":640,"end_character":4},"updated":"2022-01-13 16:14:34.000000000","message":"I think this can be deleted as well.","commit_id":"2dd5c3dd31c2488b5f3364465988338579cc3d9d"},{"author":{"_account_id":32433,"name":"Vladimir Sigunov","email":"vladimir.sigunov@att.com","username":"vs422h"},"change_message_id":"d5003cca2b4d43761c028fe0262c5bf197e181bc","unresolved":true,"context_lines":[{"line_number":637,"context_line":"  deployment_mgr: true"},{"line_number":638,"context_line":"  deployment_checkdns: true"},{"line_number":639,"context_line":"  job_bootstrap: false"},{"line_number":640,"context_line":"  job_ns_client_ceph_config: true"},{"line_number":641,"context_line":"  job_cephfs_client_key: true"},{"line_number":642,"context_line":"  job_image_repo_sync: true"},{"line_number":643,"context_line":"  job_rbd_pool: true"}],"source_content_type":"text/x-yaml","patch_set":26,"id":"e862210f_017ebe4e","line":640,"range":{"start_line":640,"start_character":0,"end_line":640,"end_character":4},"in_reply_to":"7a060a1b_615265af","updated":"2022-01-18 14:42:15.000000000","message":"Looks like this line should be removed, as well\nhttps://review.opendev.org/c/openstack/openstack-helm-infra/+/821474/27/ceph-client/templates/configmap-bin.yaml#34","commit_id":"2dd5c3dd31c2488b5f3364465988338579cc3d9d"},{"author":{"_account_id":22636,"name":"Cliff Parsons","email":"cliffhparsons@aol.com","username":"cliffparsons"},"change_message_id":"03e3a71dfcf1ac57588e485d3131689ddfb51a17","unresolved":false,"context_lines":[{"line_number":637,"context_line":"  deployment_mgr: true"},{"line_number":638,"context_line":"  deployment_checkdns: true"},{"line_number":639,"context_line":"  job_bootstrap: false"},{"line_number":640,"context_line":"  job_ns_client_ceph_config: true"},{"line_number":641,"context_line":"  job_cephfs_client_key: true"},{"line_number":642,"context_line":"  job_image_repo_sync: true"},{"line_number":643,"context_line":"  job_rbd_pool: true"}],"source_content_type":"text/x-yaml","patch_set":26,"id":"acbc356b_8035f674","line":640,"range":{"start_line":640,"start_character":0,"end_line":640,"end_character":4},"in_reply_to":"e862210f_017ebe4e","updated":"2022-02-14 15:03:02.000000000","message":"Done","commit_id":"2dd5c3dd31c2488b5f3364465988338579cc3d9d"}],"ceph-rgw/templates/deployment-rgw.yaml":[{"author":{"_account_id":22636,"name":"Cliff Parsons","email":"cliffhparsons@aol.com","username":"cliffparsons"},"change_message_id":"4b2a2464d6de6b65e7173c8845aafbe88975aa11","unresolved":true,"context_lines":[{"line_number":215,"context_line":"              mountPath: /run"},{"line_number":216,"context_line":"            - name: pod-etc-ceph"},{"line_number":217,"context_line":"              mountPath: /etc/ceph"},{"line_number":218,"context_line":"            - name: ceph-rgw-bin"},{"line_number":219,"context_line":"              mountPath: /tmp/rgw-start.sh"},{"line_number":220,"context_line":"              subPath: rgw-start.sh"},{"line_number":221,"context_line":"              readOnly: true"},{"line_number":222,"context_line":"            - name: ceph-rgw-etc"},{"line_number":223,"context_line":"              mountPath: /etc/ceph/ceph.conf.template"},{"line_number":224,"context_line":"              subPath: ceph.conf"}],"source_content_type":"text/x-yaml","patch_set":37,"id":"aea8e915_53ee2a9c","line":221,"range":{"start_line":218,"start_character":0,"end_line":221,"end_character":4},"updated":"2022-02-10 17:11:43.000000000","message":"Need to provide a mount point for the checkDNS.sh script, like this rgw-start.sh is being mounted.","commit_id":"5c67c7d16528a2459b56ad04fb8e38965a437b55"},{"author":{"_account_id":22636,"name":"Cliff Parsons","email":"cliffhparsons@aol.com","username":"cliffparsons"},"change_message_id":"03e3a71dfcf1ac57588e485d3131689ddfb51a17","unresolved":false,"context_lines":[{"line_number":215,"context_line":"              mountPath: /run"},{"line_number":216,"context_line":"            - name: pod-etc-ceph"},{"line_number":217,"context_line":"              mountPath: /etc/ceph"},{"line_number":218,"context_line":"            - name: ceph-rgw-bin"},{"line_number":219,"context_line":"              mountPath: /tmp/rgw-start.sh"},{"line_number":220,"context_line":"              subPath: rgw-start.sh"},{"line_number":221,"context_line":"              readOnly: true"},{"line_number":222,"context_line":"            - name: ceph-rgw-etc"},{"line_number":223,"context_line":"              mountPath: /etc/ceph/ceph.conf.template"},{"line_number":224,"context_line":"              subPath: ceph.conf"}],"source_content_type":"text/x-yaml","patch_set":37,"id":"44571970_00b9f784","line":221,"range":{"start_line":218,"start_character":0,"end_line":221,"end_character":4},"in_reply_to":"aea8e915_53ee2a9c","updated":"2022-02-14 15:03:02.000000000","message":"Done","commit_id":"5c67c7d16528a2459b56ad04fb8e38965a437b55"},{"author":{"_account_id":22636,"name":"Cliff Parsons","email":"cliffhparsons@aol.com","username":"cliffparsons"},"change_message_id":"49f8a79d2413afc985685319754689954d9dc2b8","unresolved":true,"context_lines":[{"line_number":137,"context_line":"              mountPath: /tmp/init-dirs.sh"},{"line_number":138,"context_line":"              subPath: init-dirs.sh"},{"line_number":139,"context_line":"              readOnly: true"},{"line_number":140,"context_line":"            - name: ceph-rgw-bin"},{"line_number":141,"context_line":"              mountPath: /tmp/utils-checkDNS.sh"},{"line_number":142,"context_line":"              subPath: utils-checkDNS.sh"},{"line_number":143,"context_line":"              readOnly: true"},{"line_number":144,"context_line":"            - name: pod-var-lib-ceph"},{"line_number":145,"context_line":"              mountPath: /var/lib/ceph"},{"line_number":146,"context_line":"              readOnly: false"}],"source_content_type":"text/x-yaml","patch_set":38,"id":"835e1e19_b0adfefa","line":143,"range":{"start_line":140,"start_character":0,"end_line":143,"end_character":4},"updated":"2022-02-10 19:49:03.000000000","message":"Don\u0027t you need this in the main ceph-rgw container?  By the time you need to call this script, this init container will be completed.","commit_id":"6f5630d2627c74103cb14cd4a4adb71017e50fc2"},{"author":{"_account_id":22636,"name":"Cliff Parsons","email":"cliffhparsons@aol.com","username":"cliffparsons"},"change_message_id":"03e3a71dfcf1ac57588e485d3131689ddfb51a17","unresolved":false,"context_lines":[{"line_number":137,"context_line":"              mountPath: /tmp/init-dirs.sh"},{"line_number":138,"context_line":"              subPath: init-dirs.sh"},{"line_number":139,"context_line":"              readOnly: true"},{"line_number":140,"context_line":"            - name: ceph-rgw-bin"},{"line_number":141,"context_line":"              mountPath: /tmp/utils-checkDNS.sh"},{"line_number":142,"context_line":"              subPath: utils-checkDNS.sh"},{"line_number":143,"context_line":"              readOnly: true"},{"line_number":144,"context_line":"            - name: pod-var-lib-ceph"},{"line_number":145,"context_line":"              mountPath: /var/lib/ceph"},{"line_number":146,"context_line":"              readOnly: false"}],"source_content_type":"text/x-yaml","patch_set":38,"id":"72e4f54a_f1010ae3","line":143,"range":{"start_line":140,"start_character":0,"end_line":143,"end_character":4},"in_reply_to":"835e1e19_b0adfefa","updated":"2022-02-14 15:03:02.000000000","message":"Done","commit_id":"6f5630d2627c74103cb14cd4a4adb71017e50fc2"}]}