[xena][cephfs] Using hostname as mount URI cause warning log every minute

[Romain]

Description
===========

From manila 13.0.3 it's now possible to set an FQDN in `cephfs_ganesha_server_ip` or `cephfs_ganesha_export_ips`. The process accepts to start but it will log a WARNING message because a function tries to determine the IP version from those settings then will trigger an exception because you cannot determine IP version from FQDN.

The problem is each 'updating share status' request will trigger this warning (so every minutes)

Steps to reproduce
==================

set one of these options with FQDN:

cephfs_ganesha_server_ip = host.domain.tld
cephfs_ganesha_export_ips = host.domain.tld

Expected result
===============

Everything is ok

Actual result
=============

A warning log every minutes

Environment
===========

dpkg -l | grep manila
ii manila-common 1:13.0.3-0ubuntu1~cloud0 all OpenStack shared file system as a service - common files
ii manila-share 1:13.0.3-0ubuntu1~cloud0 all OpenStack shared file system as a service - Share server
ii python3-manila 1:13.0.3-0ubuntu1~cloud0 all OpenStack shared file system as a service - Python 3 libs
ii python3-manilaclient 3.0.0-0ubuntu1~cloud0 all OpenStack shared file system as a service - Python 3.x client

Backend: CephFS

Networking: no network configured in manila

Logs & Configs
==============

Logs:

2022-10-05 09:37:53.110 396 WARNING manila.share.drivers.cephfs.driver [req-7f9c523b-ca2f-4da5-946d-1f887aa0fa2f - - - - -] Setting configured IP versions to [4, 6] since a hostname (rather than IP address) was supplied in 'cephfs_ganesha_server_ip' or in 'cephfs_ganesha_export_ips'

Config:
[cephfs_nfs_xxx]
share_backend_name = cephfs_nfs_xxx
ganesha_rados_store_enable = True
ganesha_rados_store_pool_name = cephfs_data_xxx
#ganesha_rados_store_pool_name = nfs-ganesha
driver_handles_share_servers = False
share_driver = manila.share.drivers.cephfs.driver.CephFSDriver
cephfs_protocol_helper_type = NFS
cephfs_conf_path = /etc/ceph/ceph.conf
cephfs_auth_id = manila
cephfs_cluster_name = ceph
cephfs_ganesha_server_is_remote = False
cephfs_ganesha_server_ip = o1.o2.o3.o4
cephfs_ganesha_export_ips = FQDN
cephfs_filesystem_name = cephfs_xxx
backend_availability_zone = xxx

[Goutham Pacha Ravi (gouthamr)]

Hi Romain,

Thank you for reporting this bug. Yes; that warning can be annoying and must be emitted just once. Just curious, in your environment, do you support IPv6 clients, or just IPv4 clients or both?

[OpenStack Infra (hudson-openstack)]

Fix proposed to branch: master
Review: https://review.opendev.org/c/openstack/manila/+/860982

[Romain (romain-chanu)]

Hi,

Our network support only IPv4.

[Goutham Pacha Ravi (gouthamr)]

> Our network support only IPv4.

Thanks for the feedback!

[OpenStack Infra (hudson-openstack)]

Fix proposed to branch: stable/zed
Review: https://review.opendev.org/c/openstack/manila/+/861066

[OpenStack Infra (hudson-openstack)]

Reviewed: https://review.opendev.org/c/openstack/manila/+/860982
Committed: https://opendev.org/openstack/manila/commit/bdfafcdada6c4912c7bc93c59c9690a18296dc8a
Submitter: "Zuul (22348)"
Branch: master

commit bdfafcdada6c4912c7bc93c59c9690a18296dc8a
Author: Goutham Pacha Ravi <email address hidden>
Date: Tue Oct 11 13:49:55 2022 -0700

    Set configured_ip_versions fact in cephfs driver

    Without it, we lose the memoization optimization
    in the "get_configured_ip_versions" method, and
    cause a warning log to be emitted one too many
    times than desired.

    Change-Id: Ibd05406b4af7c99795f1bca2d4989790430b616b
    Closes-Bug: #1991776
    Signed-off-by: Goutham Pacha Ravi <email address hidden>

[OpenStack Infra (hudson-openstack)]

Reviewed: https://review.opendev.org/c/openstack/manila/+/861066
Committed: https://opendev.org/openstack/manila/commit/5f9c037a3e929a90ced92f34eaaef32a9ebf2aac
Submitter: "Zuul (22348)"
Branch: stable/zed

commit 5f9c037a3e929a90ced92f34eaaef32a9ebf2aac
Author: Goutham Pacha Ravi <email address hidden>
Date: Tue Oct 11 13:49:55 2022 -0700

    Set configured_ip_versions fact in cephfs driver

    Without it, we lose the memoization optimization
    in the "get_configured_ip_versions" method, and
    cause a warning log to be emitted one too many
    times than desired.

    Change-Id: Ibd05406b4af7c99795f1bca2d4989790430b616b
    Closes-Bug: #1991776
    Signed-off-by: Goutham Pacha Ravi <email address hidden>
    (cherry picked from commit bdfafcdada6c4912c7bc93c59c9690a18296dc8a)

[OpenStack Infra (hudson-openstack)]

Fix proposed to branch: stable/yoga
Review: https://review.opendev.org/c/openstack/manila/+/862966

[OpenStack Infra (hudson-openstack)]

This issue was fixed in the openstack/manila 15.1.0 release.

[OpenStack Infra (hudson-openstack)]

Reviewed: https://review.opendev.org/c/openstack/manila/+/862966
Committed: https://opendev.org/openstack/manila/commit/f2111d5414ce45dc1189c34d10718147bcd783b6
Submitter: "Zuul (22348)"
Branch: stable/yoga

commit f2111d5414ce45dc1189c34d10718147bcd783b6
Author: Goutham Pacha Ravi <email address hidden>
Date: Tue Oct 11 13:49:55 2022 -0700

    Set configured_ip_versions fact in cephfs driver

    Without it, we lose the memoization optimization
    in the "get_configured_ip_versions" method, and
    cause a warning log to be emitted one too many
    times than desired.

    Change-Id: Ibd05406b4af7c99795f1bca2d4989790430b616b
    Closes-Bug: #1991776
    Signed-off-by: Goutham Pacha Ravi <email address hidden>
    (cherry picked from commit bdfafcdada6c4912c7bc93c59c9690a18296dc8a)
    (cherry picked from commit 5f9c037a3e929a90ced92f34eaaef32a9ebf2aac)

[OpenStack Infra (hudson-openstack)]

This issue was fixed in the openstack/manila 14.0.1 release.

[OpenStack Infra (hudson-openstack)]

This issue was fixed in the openstack/manila 16.0.0.0rc1 release candidate.