Bug #17595
closedforeman-installer does not check for correct values in DNS/DHCP providers
Description
Cloned from https://bugzilla.redhat.com/show_bug.cgi?id=1402062
Description of problem:
You can run satellite-installer with any value you wish in the DNS/DHCP provider!
Version-Release number of selected component (if applicable):
6.2.4 and probably earlier versions as well.
How reproducible:
Always.
Steps to Reproduce:
1. Run the following command:
satellite-installer -S satellite \
--foreman-proxy-dhcp-provider virsh \
--foreman-proxy-dhcp-server 192.168.122.1 \
--foreman-proxy-dns-server 192.168.122.1 \
--foreman-proxy-dns-provider blabla \
--foreman-proxy-dns true \
--foreman-proxy-dhcp true \
--foreman-proxy-dhcp-interface eth0
Installing Done [100%] [..........................................................]
Success!
* Satellite is running at https://rhss62.testenv
* To install additional capsule on separate machine continue by running:
capsule-certs-generate --capsule-fqdn "$CAPSULE" --certs-tar "~/$CAPSULE-certs.tar"
The full log is at /var/log/foreman-installer/satellite.log
Actual results:
satellite-installer starts and complete the installation without any warning/error message!
Expected results:
It should accept only the foreman-proxy providers.
Additional info:
cat ./dns.yml---
- DNS management
:enabled: https - valid providers:
- dns_dnscmd (Microsoft Windows native implementation)
- dns_nsupdate
- dns_nsupdate_gss (for GSS-TSIG support)
- dns_virsh (simple implementation for libvirt)
:use_provider: dns_blabla - use this setting if you want to override default TTL setting (86400)
:dns_ttl: 86400
Updated by Dominic Cleal over 8 years ago
- Subject changed from satellite-installer does not check for correct values in DNS/DHCP providers to foreman-installer does not check for correct values in DNS/DHCP providers
- Status changed from New to Feedback
Updated by The Foreman Bot over 8 years ago
- Status changed from Feedback to Ready For Testing
- Assignee set to Daniel Lobato Garcia
- Pull request https://github.com/theforeman/puppet-foreman_proxy/pull/314 added
Updated by Daniel Lobato Garcia over 8 years ago
- Related to Bug #17631: Validate realms/puppetrun providers added
Updated by Dominic Cleal over 8 years ago
- Status changed from Ready For Testing to Feedback
- Pull request deleted (
https://github.com/theforeman/puppet-foreman_proxy/pull/314)
Updated by Ewoud Kohl van Wijngaarden over 7 years ago
- Status changed from Needs design to Resolved