New strict type validation in puppetlabs-apache breaks existing usage
Bug #1983300 reported by
Takashi Kajinami
This bug affects 2 people
Bug Description
The following change was recently merged in puppetlabs-apache which made a few type validations more strict.
https:/
The current usage of apache::vhost in our modules are not compliant with the new validations, and we should fix the following points.
1. The priority parameter no longer accepts a string. It accepts only an integer
2. The access_log_* parameters no longer accept 'false'. We should use 'undef' instead.
Changed in puppet-aodh: | |
importance: | Undecided → High |
Changed in puppet-barbican: | |
importance: | Undecided → High |
Changed in puppet-cinder: | |
importance: | Undecided → High |
Changed in puppet-cloudkitty: | |
importance: | Undecided → High |
Changed in puppet-designate: | |
importance: | Undecided → High |
Changed in puppet-glance: | |
importance: | Undecided → High |
Changed in puppet-gnocchi: | |
importance: | Undecided → High |
Changed in puppet-heat: | |
importance: | Undecided → High |
Changed in puppet-horizon: | |
importance: | Undecided → High |
Changed in puppet-keystone: | |
importance: | Undecided → High |
Changed in puppet-magnum: | |
importance: | Undecided → High |
Changed in puppet-manila: | |
importance: | Undecided → High |
Changed in puppet-neutron: | |
importance: | Undecided → High |
Changed in puppet-nova: | |
importance: | Undecided → High |
Changed in puppet-openstacklib: | |
importance: | Undecided → High |
Changed in puppet-placement: | |
importance: | Undecided → High |
Changed in puppet-sahara: | |
importance: | Undecided → High |
Changed in puppet-vitrage: | |
importance: | Undecided → High |
Changed in puppet-watcher: | |
importance: | Undecided → High |
Changed in puppet-zaqar: | |
importance: | Undecided → High |
Changed in puppet-horizon: | |
status: | New → In Progress |
Changed in puppet-horizon: | |
assignee: | nobody → Takashi Kajinami (kajinamit) |
status: | In Progress → New |
Changed in puppet-keystone: | |
assignee: | nobody → Takashi Kajinami (kajinamit) |
Changed in puppet-aodh: | |
assignee: | nobody → Takashi Kajinami (kajinamit) |
Changed in puppet-barbican: | |
assignee: | nobody → Takashi Kajinami (kajinamit) |
Changed in puppet-cinder: | |
assignee: | nobody → Takashi Kajinami (kajinamit) |
Changed in puppet-cloudkitty: | |
assignee: | nobody → Takashi Kajinami (kajinamit) |
Changed in puppet-designate: | |
assignee: | nobody → Takashi Kajinami (kajinamit) |
Changed in puppet-glance: | |
assignee: | nobody → Takashi Kajinami (kajinamit) |
Changed in puppet-gnocchi: | |
assignee: | nobody → Takashi Kajinami (kajinamit) |
Changed in puppet-heat: | |
assignee: | nobody → Takashi Kajinami (kajinamit) |
Changed in puppet-magnum: | |
assignee: | nobody → Takashi Kajinami (kajinamit) |
Changed in puppet-manila: | |
assignee: | nobody → Takashi Kajinami (kajinamit) |
Changed in puppet-neutron: | |
assignee: | nobody → Takashi Kajinami (kajinamit) |
Changed in puppet-nova: | |
assignee: | nobody → Takashi Kajinami (kajinamit) |
Changed in puppet-openstacklib: | |
assignee: | nobody → Takashi Kajinami (kajinamit) |
Changed in puppet-placement: | |
assignee: | nobody → Takashi Kajinami (kajinamit) |
Changed in puppet-sahara: | |
assignee: | nobody → Takashi Kajinami (kajinamit) |
Changed in puppet-vitrage: | |
assignee: | nobody → Takashi Kajinami (kajinamit) |
Changed in puppet-watcher: | |
assignee: | nobody → Takashi Kajinami (kajinamit) |
Changed in puppet-zaqar: | |
assignee: | nobody → Takashi Kajinami (kajinamit) |
Changed in puppet-aodh: | |
status: | New → In Progress |
Changed in puppet-cinder: | |
status: | New → In Progress |
Changed in puppet-cloudkitty: | |
status: | New → In Progress |
Changed in puppet-gnocchi: | |
status: | New → In Progress |
Changed in puppet-keystone: | |
status: | New → In Progress |
Changed in puppet-heat: | |
status: | New → In Progress |
Changed in puppet-barbican: | |
status: | New → In Progress |
Changed in puppet-designate: | |
status: | New → In Progress |
Changed in puppet-openstacklib: | |
status: | New → In Progress |
Changed in puppet-horizon: | |
status: | New → In Progress |
To post a comment you must log in.
Fix proposed to branch: master /review. opendev. org/c/openstack /puppet- magnum/ +/851842
Review: https:/