adoroszlai commented on code in PR #47:
URL: https://github.com/apache/ozone-helm-charts/pull/47#discussion_r3653093275


##########
charts/ozone/templates/_helpers.tpl:
##########
@@ -104,38 +84,6 @@ app.kubernetes.io/instance: {{ .Release.Name }}
     {{- $nodes | join ","}}
 {{- end }}
 
-{{/* List of decommission scm nodes */}}
-{{- define "ozone.scm.decommissioned.nodes" -}}

Review Comment:
   Yes, I see that it is currently unused.  My question is whether it is useful 
for any planned future improvement.  (We can remove and restore later in any 
case, but if it is going to be used in a near future task, then not worth 
removing.)



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to