15:27:32 <hillpd> Going forward, passing the cinder.conf "host" value to os-brick seems sensible. Unfortunately, it wouldn't be safe to backport this due to how the host value is used in existing deployments.
15:27:59 <hillpd> To address this in previous releases, we are considering the addition of a "[cinder-oceanstor]" config option, "use_host_fqdn" to handle this in the Huawei driver. Would that be a reasonable approach?
<whoami-rajat> hillpd, hey, I'm unsure how this would pan out. The code that establishes the connection via os-brick isn't part of the driver, so how would the Huawei driver utilize this config option? Are you suggesting we disregard the connector['host'] value and use the config option instead?
@hillpd, please respond here so that the conversation doesn't get lost. It would be even better if you could propose a WIP patch. This way, we can see how it fully functions and continue the discussion based on the actual code.
This bug was discussed on May 31, 2023. You can refer to the discussion here: https:/ /meetings. opendev. org/meetings/ cinder_ bs/2023/ cinder_ bs.2023- 05-31-15. 03.log. html#l- 67 and https:/ /meetings. opendev. org/irclogs/ %23openstack- cinder/ %23openstack- cinder. 2023-06- 01.log. html
15:27:32 <hillpd> Going forward, passing the cinder.conf "host" value to os-brick seems sensible. Unfortunately, it wouldn't be safe to backport this due to how the host value is used in existing deployments. oceanstor] " config option, "use_host_fqdn" to handle this in the Huawei driver. Would that be a reasonable approach?
15:27:59 <hillpd> To address this in previous releases, we are considering the addition of a "[cinder-
<whoami-rajat> hillpd, hey, I'm unsure how this would pan out. The code that establishes the connection via os-brick isn't part of the driver, so how would the Huawei driver utilize this config option? Are you suggesting we disregard the connector['host'] value and use the config option instead?
@hillpd, please respond here so that the conversation doesn't get lost. It would be even better if you could propose a WIP patch. This way, we can see how it fully functions and continue the discussion based on the actual code.
Thank you!