Bug #16913
closedValidate format of dhcp-range option
Description
Cloned from https://bugzilla.redhat.com/show_bug.cgi?id=1281743
Description of problem:
katello-installer with --capsule-dhcp-range is not able to set to default when using "false" or it doesn't check the syntax.
Version-Release number of selected component (if applicable):
6.1.3
How reproducible:
install satellite witch capsule and dhcp support
Steps to Reproduce:
1. After the satellite with capsule is installed, run:
2. katello-installer --capsule-dhcp-range "1 1" -> will fail, in the config
will be row like this: range "1 1"
3. katello-installer --capusle-dhcp-range false -> will fail, in the config
will be row like this: range false
Actual results:
There is wrong value for dhcp range
Expected results:
User receive an error message, when the syntax is not correct or in the case of "false", which supposed to be default, the range will be set to default
Updated by Dominic Cleal over 7 years ago
- Project changed from Foreman to Installer
- Subject changed from Validate format of dhcp-range option to Validate format of dhcp-range option
- Category set to Foreman modules
Updated by Anonymous almost 7 years ago
Stephen, what's the status here with recent versions?
Updated by Ewoud Kohl van Wijngaarden almost 7 years ago
Looks like theforeman-dhcp doesn't use Puppet 4 types and theforeman-foreman_proxy accepts String. We could narrow this down to a Pattern to validate this.
Updated by The Foreman Bot over 6 years ago
- Status changed from New to Ready For Testing
- Assignee set to Ewoud Kohl van Wijngaarden
- Pull request https://github.com/theforeman/puppet-dhcp/pull/112 added
Updated by Ewoud Kohl van Wijngaarden over 6 years ago
- Status changed from Ready For Testing to Closed
- % Done changed from 0 to 100
Applied in changeset puppet-dhcp|7d31525491d362f090a31fad3bfe1802adcd59f4.