Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

set_fact _monitor_address - 'ipaddr: unknown filter type: 172.120.3.0/24,172.117.3.0/24,172.118.3.0/24,172.119.3.0/24' #4333

Closed
hjensas opened this issue Aug 13, 2019 · 0 comments · Fixed by #4339

Comments

@hjensas
Copy link
Contributor

hjensas commented Aug 13, 2019

Bug Report

What happened:
The public_network is a L3 routed network with multiple subnet CIDR's. When configuring the monitor_address_block using the same list of CIDR's as used for the public_network the deployment fails with the following error:

TASK [ceph-facts : set_fact _monitor_address to monitor_address_block ipv4]
task path: /usr/share/ceph-ansible/roles/ceph-facts/tasks/set_monitor_address.yml:2
Friday 09 August 2019  23:43:45 +0000 (0:00:00.921)       0:01:17.061 ********* 
fatal: [hostname]: FAILED! => 
msg: 'ipaddr: unknown filter type: 172.120.3.0/24,172.117.3.0/24,172.118.3.0/24,172.119.3.0/24'

What you expected to happen:
set_fact _monitor_address to monitor_address_block ipv4 should support a comma separated list of address blocks in the same way public_network and cluster_network does. So that monitor address for monitors placed in different subnets can be installed without setting the monitor_address or monitor_interface variables.

How to reproduce it (minimal and precise):
Set monitor_address_block to include a comma separated list of address blocks. For example:
monitor_address_block: '172.120.3.0/24,172.117.3.0/24,172.118.3.0/24,172.119.3.0/24'

Share your group_vars files, inventory

Environment:

  • OS (e.g. from /etc/os-release): Red Hat Enterprise Linux 8.0 (Ootpa)
  • Kernel (e.g. uname -a): Linux overcloud-controller-0 4.18.0-80.7.2.el8_0.x86_64 #1 SMP Fri Jul 26 10:48:21 UTC 2019 x86_64 x86_64 x86_64 GNU/Linux
  • Ansible version (e.g. ansible-playbook --version): ansible-playbook 2.8.3
  • ceph-ansible version (e.g. git head or tag or stable branch): ceph-ansible-4.0.0-0.1.rc13.el8cp.noarch
  • Ceph version (e.g. ceph -v): Ceph 4.0
hjensas added a commit to hjensas/ceph-ansible that referenced this issue Aug 14, 2019
To enable deploying monitors accross different subnets in
an L3 routed network the monitor_address_block parameter
needs to support a list of subnet CIDRs.

This change implements a filter_plugin that is used in the
ceph-facts and ceph-validate roles, replacing the use of the
ipaddr filter. The new filter plugin will return a list of
all IP address that reside in any one of the given IP ranges.

ceph.conf already support a comma separated list of CIDRs
for the public and cluster network options. With this
change a comma separated list of subnet CIDRs can also be
used for monitor_address_block.

NOTE: This change also updates the RPM spec file so that
      the filter_plugins are copied to the buildroot.

Related-To: https://bugs.launchpad.net/tripleo/+bug/1840030
Related-To: https://bugzilla.redhat.com/show_bug.cgi?id=1740283

Closes: ceph#4333
Please backport to stable-4.0

Signed-off-by: Harald Jensås <hjensas@redhat.com>
hjensas added a commit to hjensas/ceph-ansible that referenced this issue Aug 14, 2019
To enable deploying monitors accross different subnets in
an L3 routed network the monitor_address_block parameter
needs to support a list of subnet CIDRs.

This change implements a filter_plugin that is used in the
ceph-facts and ceph-validate roles, replacing the use of the
ipaddr filter. The new filter plugin will return a list of
all IP address that reside in any one of the given IP ranges.

ceph.conf already support a comma separated list of CIDRs
for the public and cluster network options. With this
change a comma separated list of subnet CIDRs can also be
used for monitor_address_block.

NOTE: This change also updates the RPM spec file so that
      the filter_plugins are copied to the buildroot.

Related-To: https://bugs.launchpad.net/tripleo/+bug/1840030
Related-To: https://bugzilla.redhat.com/show_bug.cgi?id=1740283

Closes: ceph#4333
Please backport to stable-4.0

Signed-off-by: Harald Jensås <hjensas@redhat.com>
hjensas added a commit to hjensas/ceph-ansible that referenced this issue Aug 15, 2019
To enable deploying monitors accross different subnets in
an L3 routed network the monitor_address_block parameter
needs to support a list of subnet CIDRs.

This change implements a filter_plugin that is used in the
ceph-facts and ceph-validate roles, replacing the use of the
ipaddr filter. The new filter plugin will return a list of
all IP address that reside in any one of the given IP ranges.

ceph.conf already support a comma separated list of CIDRs
for the public and cluster network options. With this
change a comma separated list of subnet CIDRs can also be
used for monitor_address_block.

NOTE:
  * This change also updates the RPM spec file so that
    the filter_plugins are copied to the buildroot.
  * Add a comment include 'ansible' in tests/requirements.txt
    so that netaddr is installed as requiement for the
    ceph-ansible-pr-syntax-check.

Related-To: https://bugs.launchpad.net/tripleo/+bug/1840030
Related-To: https://bugzilla.redhat.com/show_bug.cgi?id=1740283

Closes: ceph#4333
Please backport to stable-4.0

Signed-off-by: Harald Jensås <hjensas@redhat.com>
hjensas added a commit to hjensas/ceph-ansible that referenced this issue Aug 19, 2019
To enable deploying monitors accross different subnets in
an L3 routed network the monitor_address_block parameter
needs to support a list of subnet CIDRs.

This change implements a filter_plugin that is used in the
ceph-facts and ceph-validate roles, replacing the use of the
ipaddr filter. The new filter plugin will return a list of
all IP address that reside in any one of the given IP ranges.

ceph.conf already support a comma separated list of CIDRs
for the public and cluster network options. With this
change a comma separated list of subnet CIDRs can also be
used for monitor_address_block.

NOTE:
  * This change also updates the RPM spec file so that
    the filter_plugins are copied to the buildroot.
  * Add a comment include 'ansible' in tests/requirements.txt
    so that netaddr is installed as requiement for the
    ceph-ansible-pr-syntax-check.

Related-To: https://bugs.launchpad.net/tripleo/+bug/1840030
Related-To: https://bugzilla.redhat.com/show_bug.cgi?id=1740283

Closes: ceph#4333
Please backport to stable-4.0

Signed-off-by: Harald Jensås <hjensas@redhat.com>
hjensas added a commit to hjensas/ceph-ansible that referenced this issue Aug 19, 2019
To enable deploying monitors accross different subnets in
an L3 routed network the monitor_address_block parameter
needs to support a list of subnet CIDRs.

This change implements a filter_plugin that is used in the
ceph-facts and ceph-validate roles, replacing the use of the
ipaddr filter. The new filter plugin will return a list of
all IP address that reside in any one of the given IP ranges.

ceph.conf already support a comma separated list of CIDRs
for the public and cluster network options. With this
change a comma separated list of subnet CIDRs can also be
used for monitor_address_block.

NOTE:
  * This change also updates the RPM spec file so that
    the filter_plugins are copied to the buildroot.
  * Add a comment include 'ansible' in tests/requirements.txt
    so that netaddr is installed as requiement for the
    ceph-ansible-pr-syntax-check.

Related-To: https://bugs.launchpad.net/tripleo/+bug/1840030
Related-To: https://bugzilla.redhat.com/show_bug.cgi?id=1740283

Closes: ceph#4333
Please backport to stable-4.0

Signed-off-by: Harald Jensås <hjensas@redhat.com>
hjensas added a commit to hjensas/ceph-ansible that referenced this issue Aug 21, 2019
This change implements a filter_plugin that is used in the
ceph-facts and ceph-validate roles, replacing the use of the
ipaddr filter. The new filter plugin will return a list of
all IP address that reside in any one of the given IP ranges.

ceph.conf already support a comma separated list of CIDRs
for the public and cluster network options. With this
change a comma separated list of subnet CIDRs can also be
used for monitor_address_block and radosgw_address_block.

NOTE:
  * This change also updates the RPM spec file so that
    the filter_plugins are copied to the buildroot.
  * Add a comment include 'ansible' in tests/requirements.txt
    so that netaddr is installed as requiement for the
    ceph-ansible-pr-syntax-check.

Related-To: https://bugs.launchpad.net/tripleo/+bug/1840030
Related-To: https://bugzilla.redhat.com/show_bug.cgi?id=1740283

Closes: ceph#4333
Please backport to stable-4.0

Signed-off-by: Harald Jensås <hjensas@redhat.com>
hjensas added a commit to hjensas/ceph-ansible that referenced this issue Sep 4, 2019
This change implements a filter_plugin that is used in the
ceph-facts, ceph-validate roles and infrastucture-playbooks.
The new filter plugin will return a list of all IP address
that reside in any one of the given IP ranges. The new filter
replaces the use of the ipaddr filter.

ceph.conf already support a comma separated list of CIDRs
for the public_network and cluster_network options.

Changes: [1] and [2] introduced a regression in ceph-ansible
where public_network can no longer be a comma separated list
of cidrs.

With this change a comma separated list of subnet CIDRs can
also be used for monitor_address_block and radosgw_address_block.

NOTE:
  * This change also updates the RPM spec file so that
    the filter_plugins are copied to the buildroot.
  * Add a comment include 'ansible' in tests/requirements.txt
    so that netaddr is installed as requiement for the
    ceph-ansible-pr-syntax-check.

[1] commit: d67230b
[2] commit: 20e4852

Related-To: https://bugs.launchpad.net/tripleo/+bug/1840030
Related-To: https://bugzilla.redhat.com/show_bug.cgi?id=1740283

Closes: ceph#4333
Please backport to stable-4.0

Signed-off-by: Harald Jensås <hjensas@redhat.com>
hjensas added a commit to hjensas/ceph-ansible that referenced this issue Sep 6, 2019
This change implements a filter_plugin that is used in the
ceph-facts, ceph-validate roles and infrastucture-playbooks.
The new filter plugin will return a list of all IP address
that reside in any one of the given IP ranges. The new filter
replaces the use of the ipaddr filter.

ceph.conf already support a comma separated list of CIDRs
for the public_network and cluster_network options.

Changes: [1] and [2] introduced a regression in ceph-ansible
where public_network can no longer be a comma separated list
of cidrs.

With this change a comma separated list of subnet CIDRs can
also be used for monitor_address_block and radosgw_address_block.

[1] commit: d67230b
[2] commit: 20e4852

Related-To: https://bugs.launchpad.net/tripleo/+bug/1840030
Related-To: https://bugzilla.redhat.com/show_bug.cgi?id=1740283

Closes: ceph#4333
Please backport to stable-4.0

Signed-off-by: Harald Jensås <hjensas@redhat.com>
hjensas added a commit to hjensas/ceph-ansible that referenced this issue Sep 6, 2019
ceph.conf supports a comma separated list of
subnet CIDR's for the public_network and the
cluster network. ceph-ansible should support
setting up the firewall for this configuration.

Closes: ceph#4425
Related: ceph#4333
https://docs.ceph.com/docs/nautilus/rados/configuration/network-config-ref/#network-config-settings

Signed-off-by: Harald Jensås <hjensas@redhat.com>
hjensas added a commit to hjensas/ceph-ansible that referenced this issue Sep 6, 2019
ceph.conf supports a comma separated list of
subnet CIDR's for the public_network and the
cluster network. ceph-ansible should support
setting up the firewall for this configuration.

Closes: ceph#4425
Related: ceph#4333
https://docs.ceph.com/docs/nautilus/rados/configuration/network-config-ref/#network-config-settings

Signed-off-by: Harald Jensås <hjensas@redhat.com>
hjensas added a commit to hjensas/ceph-ansible that referenced this issue Sep 6, 2019
This change implements a filter_plugin that is used in the
ceph-facts, ceph-validate roles and infrastucture-playbooks.
The new filter plugin will return a list of all IP address
that reside in any one of the given IP ranges. The new filter
replaces the use of the ipaddr filter.

ceph.conf already support a comma separated list of CIDRs
for the public_network and cluster_network options.

Changes: [1] and [2] introduced a regression in ceph-ansible
where public_network can no longer be a comma separated list
of cidrs.

With this change a comma separated list of subnet CIDRs can
also be used for monitor_address_block and radosgw_address_block.

[1] commit: d67230b
[2] commit: 20e4852

Related-To: https://bugs.launchpad.net/tripleo/+bug/1840030
Related-To: https://bugzilla.redhat.com/show_bug.cgi?id=1740283

Closes: ceph#4333
Please backport to stable-4.0

Signed-off-by: Harald Jensås <hjensas@redhat.com>
hjensas added a commit to hjensas/ceph-ansible that referenced this issue Sep 6, 2019
ceph.conf supports a comma separated list of
subnet CIDR's for the public_network and the
cluster network. ceph-ansible should support
setting up the firewall for this configuration.

Closes: ceph#4425
Related: ceph#4333
https://docs.ceph.com/docs/nautilus/rados/configuration/network-config-ref/#network-config-settings

Signed-off-by: Harald Jensås <hjensas@redhat.com>
dsavineau pushed a commit that referenced this issue Sep 9, 2019
ceph.conf supports a comma separated list of
subnet CIDR's for the public_network and the
cluster network. ceph-ansible should support
setting up the firewall for this configuration.

Closes: #4425
Related: #4333
https://docs.ceph.com/docs/nautilus/rados/configuration/network-config-ref/#network-config-settings

Signed-off-by: Harald Jensås <hjensas@redhat.com>
mergify bot pushed a commit that referenced this issue Sep 9, 2019
ceph.conf supports a comma separated list of
subnet CIDR's for the public_network and the
cluster network. ceph-ansible should support
setting up the firewall for this configuration.

Closes: #4425
Related: #4333
https://docs.ceph.com/docs/nautilus/rados/configuration/network-config-ref/#network-config-settings

Signed-off-by: Harald Jensås <hjensas@redhat.com>
(cherry picked from commit d942292)
hjensas added a commit to hjensas/ceph-ansible that referenced this issue Sep 9, 2019
This change implements a filter_plugin that is used in the
ceph-facts, ceph-validate roles and infrastucture-playbooks.
The new filter plugin will return a list of all IP address
that reside in any one of the given IP ranges. The new filter
replaces the use of the ipaddr filter.

ceph.conf already support a comma separated list of CIDRs
for the public_network and cluster_network options.

Changes: [1] and [2] introduced a regression in ceph-ansible
where public_network can no longer be a comma separated list
of cidrs.

With this change a comma separated list of subnet CIDRs can
also be used for monitor_address_block and radosgw_address_block.

[1] commit: d67230b
[2] commit: 20e4852

Related-To: https://bugs.launchpad.net/tripleo/+bug/1840030
Related-To: https://bugzilla.redhat.com/show_bug.cgi?id=1740283

Closes: ceph#4333
Please backport to stable-4.0

Signed-off-by: Harald Jensås <hjensas@redhat.com>
hjensas added a commit to hjensas/ceph-ansible that referenced this issue Sep 9, 2019
This change implements a filter_plugin that is used in the
ceph-facts, ceph-validate roles and infrastucture-playbooks.
The new filter plugin will return a list of all IP address
that reside in any one of the given IP ranges. The new filter
replaces the use of the ipaddr filter.

ceph.conf already support a comma separated list of CIDRs
for the public_network and cluster_network options.

Changes: [1] and [2] introduced a regression in ceph-ansible
where public_network can no longer be a comma separated list
of cidrs.

With this change a comma separated list of subnet CIDRs can
also be used for monitor_address_block and radosgw_address_block.

[1] commit: d67230b
[2] commit: 20e4852

Related-To: https://bugs.launchpad.net/tripleo/+bug/1840030
Related-To: https://bugzilla.redhat.com/show_bug.cgi?id=1740283

Closes: ceph#4333
Please backport to stable-4.0

Signed-off-by: Harald Jensås <hjensas@redhat.com>
dsavineau pushed a commit that referenced this issue Sep 10, 2019
ceph.conf supports a comma separated list of
subnet CIDR's for the public_network and the
cluster network. ceph-ansible should support
setting up the firewall for this configuration.

Closes: #4425
Related: #4333
https://docs.ceph.com/docs/nautilus/rados/configuration/network-config-ref/#network-config-settings

Signed-off-by: Harald Jensås <hjensas@redhat.com>
(cherry picked from commit d942292)
hjensas added a commit to hjensas/ceph-ansible that referenced this issue Sep 10, 2019
This change implements a filter_plugin that is used in the
ceph-facts, ceph-validate roles and infrastucture-playbooks.
The new filter plugin will return a list of all IP address
that reside in any one of the given IP ranges. The new filter
replaces the use of the ipaddr filter.

ceph.conf already support a comma separated list of CIDRs
for the public_network and cluster_network options.

Changes: [1] and [2] introduced a regression in ceph-ansible
where public_network can no longer be a comma separated list
of cidrs.

With this change a comma separated list of subnet CIDRs can
also be used for monitor_address_block and radosgw_address_block.

[1] commit: d67230b
[2] commit: 20e4852

Related-To: https://bugs.launchpad.net/tripleo/+bug/1840030
Related-To: https://bugzilla.redhat.com/show_bug.cgi?id=1740283

Closes: ceph#4333
Please backport to stable-4.0

Signed-off-by: Harald Jensås <hjensas@redhat.com>
hjensas added a commit to hjensas/ceph-ansible that referenced this issue Sep 11, 2019
This change implements a filter_plugin that is used in the
ceph-facts, ceph-validate roles and infrastucture-playbooks.
The new filter plugin will return a list of all IP address
that reside in any one of the given IP ranges. The new filter
replaces the use of the ipaddr filter.

ceph.conf already support a comma separated list of CIDRs
for the public_network and cluster_network options.

Changes: [1] and [2] introduced a regression in ceph-ansible
where public_network can no longer be a comma separated list
of cidrs.

With this change a comma separated list of subnet CIDRs can
also be used for monitor_address_block and radosgw_address_block.

[1] commit: d67230b
[2] commit: 20e4852

Related-To: https://bugs.launchpad.net/tripleo/+bug/1840030
Related-To: https://bugzilla.redhat.com/show_bug.cgi?id=1740283

Closes: ceph#4333
Please backport to stable-4.0

Signed-off-by: Harald Jensås <hjensas@redhat.com>
hjensas added a commit to hjensas/ceph-ansible that referenced this issue Sep 12, 2019
This change implements a filter_plugin that is used in the
ceph-facts, ceph-validate roles and infrastucture-playbooks.
The new filter plugin will return a list of all IP address
that reside in any one of the given IP ranges. The new filter
replaces the use of the ipaddr filter.

ceph.conf already support a comma separated list of CIDRs
for the public_network and cluster_network options.

Changes: [1] and [2] introduced a regression in ceph-ansible
where public_network can no longer be a comma separated list
of cidrs.

With this change a comma separated list of subnet CIDRs can
also be used for monitor_address_block and radosgw_address_block.

[1] commit: d67230b
[2] commit: 20e4852

Related-To: https://bugs.launchpad.net/tripleo/+bug/1840030
Related-To: https://bugzilla.redhat.com/show_bug.cgi?id=1740283

Closes: ceph#4333
Please backport to stable-4.0

Signed-off-by: Harald Jensås <hjensas@redhat.com>
hjensas added a commit to hjensas/ceph-ansible that referenced this issue Sep 12, 2019
This change implements a filter_plugin that is used in the
ceph-facts, ceph-validate roles and infrastucture-playbooks.
The new filter plugin will return a list of all IP address
that reside in any one of the given IP ranges. The new filter
replaces the use of the ipaddr filter.

ceph.conf already support a comma separated list of CIDRs
for the public_network and cluster_network options.

Changes: [1] and [2] introduced a regression in ceph-ansible
where public_network can no longer be a comma separated list
of cidrs.

With this change a comma separated list of subnet CIDRs can
also be used for monitor_address_block and radosgw_address_block.

[1] commit: d67230b
[2] commit: 20e4852

Related-To: https://bugs.launchpad.net/tripleo/+bug/1840030
Related-To: https://bugzilla.redhat.com/show_bug.cgi?id=1740283

Closes: ceph#4333
Please backport to stable-4.0

Signed-off-by: Harald Jensås <hjensas@redhat.com>
hjensas added a commit to hjensas/ceph-ansible that referenced this issue Sep 18, 2019
This change implements a filter_plugin that is used in the
ceph-facts, ceph-validate roles and infrastucture-playbooks.
The new filter plugin will return a list of all IP address
that reside in any one of the given IP ranges. The new filter
replaces the use of the ipaddr filter.

ceph.conf already support a comma separated list of CIDRs
for the public_network and cluster_network options.

Changes: [1] and [2] introduced a regression in ceph-ansible
where public_network can no longer be a comma separated list
of cidrs.

With this change a comma separated list of subnet CIDRs can
also be used for monitor_address_block and radosgw_address_block.

[1] commit: d67230b
[2] commit: 20e4852

Related-To: https://bugs.launchpad.net/tripleo/+bug/1840030
Related-To: https://bugzilla.redhat.com/show_bug.cgi?id=1740283

Closes: ceph#4333
Please backport to stable-4.0

Signed-off-by: Harald Jensås <hjensas@redhat.com>
hjensas added a commit to hjensas/ceph-ansible that referenced this issue Sep 18, 2019
This change implements a filter_plugin that is used in the
ceph-facts, ceph-validate roles and infrastucture-playbooks.
The new filter plugin will return a list of all IP address
that reside in any one of the given IP ranges. The new filter
replaces the use of the ipaddr filter.

ceph.conf already support a comma separated list of CIDRs
for the public_network and cluster_network options.

Changes: [1] and [2] introduced a regression in ceph-ansible
where public_network can no longer be a comma separated list
of cidrs.

With this change a comma separated list of subnet CIDRs can
also be used for monitor_address_block and radosgw_address_block.

[1] commit: d67230b
[2] commit: 20e4852

Related-To: https://bugs.launchpad.net/tripleo/+bug/1840030
Related-To: https://bugzilla.redhat.com/show_bug.cgi?id=1740283

Closes: ceph#4333
Please backport to stable-4.0

Signed-off-by: Harald Jensås <hjensas@redhat.com>
hjensas added a commit to hjensas/ceph-ansible that referenced this issue Sep 26, 2019
This change implements a filter_plugin that is used in the
ceph-facts, ceph-validate roles and infrastucture-playbooks.
The new filter plugin will return a list of all IP address
that reside in any one of the given IP ranges. The new filter
replaces the use of the ipaddr filter.

ceph.conf already support a comma separated list of CIDRs
for the public_network and cluster_network options.

Changes: [1] and [2] introduced a regression in ceph-ansible
where public_network can no longer be a comma separated list
of cidrs.

With this change a comma separated list of subnet CIDRs can
also be used for monitor_address_block and radosgw_address_block.

[1] commit: d67230b
[2] commit: 20e4852

Related-To: https://bugs.launchpad.net/tripleo/+bug/1840030
Related-To: https://bugzilla.redhat.com/show_bug.cgi?id=1740283

Closes: ceph#4333
Please backport to stable-4.0

Signed-off-by: Harald Jensås <hjensas@redhat.com>
hjensas added a commit to hjensas/ceph-ansible that referenced this issue Sep 26, 2019
This change implements a filter_plugin that is used in the
ceph-facts, ceph-validate roles and infrastucture-playbooks.
The new filter plugin will return a list of all IP address
that reside in any one of the given IP ranges. The new filter
replaces the use of the ipaddr filter.

ceph.conf already support a comma separated list of CIDRs
for the public_network and cluster_network options.

Changes: [1] and [2] introduced a regression in ceph-ansible
where public_network can no longer be a comma separated list
of cidrs.

With this change a comma separated list of subnet CIDRs can
also be used for monitor_address_block and radosgw_address_block.

[1] commit: d67230b
[2] commit: 20e4852

Related-To: https://bugs.launchpad.net/tripleo/+bug/1840030
Related-To: https://bugzilla.redhat.com/show_bug.cgi?id=1740283

Closes: ceph#4333
Please backport to stable-4.0

Signed-off-by: Harald Jensås <hjensas@redhat.com>
guits pushed a commit that referenced this issue Sep 27, 2019
This change implements a filter_plugin that is used in the
ceph-facts, ceph-validate roles and infrastucture-playbooks.
The new filter plugin will return a list of all IP address
that reside in any one of the given IP ranges. The new filter
replaces the use of the ipaddr filter.

ceph.conf already support a comma separated list of CIDRs
for the public_network and cluster_network options.

Changes: [1] and [2] introduced a regression in ceph-ansible
where public_network can no longer be a comma separated list
of cidrs.

With this change a comma separated list of subnet CIDRs can
also be used for monitor_address_block and radosgw_address_block.

[1] commit: d67230b
[2] commit: 20e4852

Related-To: https://bugs.launchpad.net/tripleo/+bug/1840030
Related-To: https://bugzilla.redhat.com/show_bug.cgi?id=1740283

Closes: #4333
Please backport to stable-4.0

Signed-off-by: Harald Jensås <hjensas@redhat.com>
mergify bot pushed a commit that referenced this issue Sep 27, 2019
This change implements a filter_plugin that is used in the
ceph-facts, ceph-validate roles and infrastucture-playbooks.
The new filter plugin will return a list of all IP address
that reside in any one of the given IP ranges. The new filter
replaces the use of the ipaddr filter.

ceph.conf already support a comma separated list of CIDRs
for the public_network and cluster_network options.

Changes: [1] and [2] introduced a regression in ceph-ansible
where public_network can no longer be a comma separated list
of cidrs.

With this change a comma separated list of subnet CIDRs can
also be used for monitor_address_block and radosgw_address_block.

[1] commit: d67230b
[2] commit: 20e4852

Related-To: https://bugs.launchpad.net/tripleo/+bug/1840030
Related-To: https://bugzilla.redhat.com/show_bug.cgi?id=1740283

Closes: #4333
Please backport to stable-4.0

Signed-off-by: Harald Jensås <hjensas@redhat.com>
(cherry picked from commit e695efc)
guits pushed a commit that referenced this issue Sep 27, 2019
This change implements a filter_plugin that is used in the
ceph-facts, ceph-validate roles and infrastucture-playbooks.
The new filter plugin will return a list of all IP address
that reside in any one of the given IP ranges. The new filter
replaces the use of the ipaddr filter.

ceph.conf already support a comma separated list of CIDRs
for the public_network and cluster_network options.

Changes: [1] and [2] introduced a regression in ceph-ansible
where public_network can no longer be a comma separated list
of cidrs.

With this change a comma separated list of subnet CIDRs can
also be used for monitor_address_block and radosgw_address_block.

[1] commit: d67230b
[2] commit: 20e4852

Related-To: https://bugs.launchpad.net/tripleo/+bug/1840030
Related-To: https://bugzilla.redhat.com/show_bug.cgi?id=1740283

Closes: #4333
Please backport to stable-4.0

Signed-off-by: Harald Jensås <hjensas@redhat.com>
(cherry picked from commit e695efc)
dsavineau pushed a commit to dsavineau/ceph-ansible that referenced this issue Oct 31, 2019
ceph.conf supports a comma separated list of
subnet CIDR's for the public_network and the
cluster network. ceph-ansible should support
setting up the firewall for this configuration.

Closes: https://bugzilla.redhat.com/show_bug.cgi?id=1767392
Closes: ceph#4425
Related: ceph#4333
https://docs.ceph.com/docs/nautilus/rados/configuration/network-config-ref/#network-config-settings

Signed-off-by: Harald Jensås <hjensas@redhat.com>
(cherry picked from commit d942292)
dsavineau pushed a commit that referenced this issue Nov 1, 2019
ceph.conf supports a comma separated list of
subnet CIDR's for the public_network and the
cluster network. ceph-ansible should support
setting up the firewall for this configuration.

Closes: https://bugzilla.redhat.com/show_bug.cgi?id=1767392
Closes: #4425
Related: #4333
https://docs.ceph.com/docs/nautilus/rados/configuration/network-config-ref/#network-config-settings

Signed-off-by: Harald Jensås <hjensas@redhat.com>
(cherry picked from commit d942292)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant