Bug if using hostnames for memcache in manifest/cache

[Luca Cervigni]

Hello,

This commit https://github.com/openstack/puppet-oslo/commit/908503e3e972aa17220d2814762d77f91c689052

creates issues if using hostnames. In our environment we use hostnames for memcacheservers of the kind "memcache1:port" After that change has been implemented puppet gives an error. Replacing that line back to previous version restores a working environment.

I am not sure though if the problem is in the puppet-oslo module on that commit that I linked, or more relevant on the lib/puppet/functions/inet6_prefix.rb on openstacklib.

The function should:
# This is useful for services relying on python-memcached which require
# the inet6:[<ip_address]:<port> format.
#
# Returns the argument untouched otherwise.

For what I am understanding from this code, the input argument is supposed to be either an ipv4 or ipv6. Input it an hostnames creates this issue.

Thanks for investigating

[Luca Cervigni (cervigni)]

Also,

https://forge.puppet.com/puppetlabs/stdlib

Why not using standard stdlib functions to check if hostname / ipv4 / ipv6?

[Takashi Kajinami (kajinamit)]

Hello,

Sorry for the late response but could you please share the actual error you found ?

There was a patch merged to solve compatibility with old Ruby. I guess the error was fixed by this but we need actual error to confirm that.
 https://review.opendev.org/c/openstack/puppet-openstacklib/+/716258

[Takashi Kajinami (kajinamit)]

I'm closing this bug assuming [1] fixed the problem but please feel free to reopen this if you still face the same problem (and are still around the community).

[1] https://review.opendev.org/c/openstack/puppet-openstacklib/+/716258