diff options
| author | Rob Crittenden <rcritten@redhat.com> | 2015-03-26 14:55:27 -0400 |
|---|---|---|
| committer | Patrick Uiterwijk <puiterwijk@redhat.com> | 2015-03-27 19:46:52 +0100 |
| commit | 101022e3bf4dfe3f0c56ffb61abbf358a3b1ab26 (patch) | |
| tree | b0a78aab69144734c0d38c69188beffd61603aa6 /ipsilon/install/ipsilon-server-install | |
| parent | 83ac397cd5904cbbaa5a21adcac73815dda9fa63 (diff) | |
| download | ipsilon-101022e3bf4dfe3f0c56ffb61abbf358a3b1ab26.tar.gz ipsilon-101022e3bf4dfe3f0c56ffb61abbf358a3b1ab26.tar.xz ipsilon-101022e3bf4dfe3f0c56ffb61abbf358a3b1ab26.zip | |
Add a method to Installer classes to validate argument input
There was no way to validate argument input from plugins and
cause the installer to bail out. If a plugin needs to validate
some input it can use the validate_args() method and raise
ConfigurationError() if an issue is found.
https://fedorahosted.org/ipsilon/ticket/78
Signed-off-by: Rob Crittenden <rcritten@redhat.com>
Reviewed-by: Patrick Uiterwijk <puiterwijk@redhat.com>
Diffstat (limited to 'ipsilon/install/ipsilon-server-install')
| -rwxr-xr-x | ipsilon/install/ipsilon-server-install | 5 |
1 files changed, 5 insertions, 0 deletions
diff --git a/ipsilon/install/ipsilon-server-install b/ipsilon/install/ipsilon-server-install index edcccb6..8f915f3 100755 --- a/ipsilon/install/ipsilon-server-install +++ b/ipsilon/install/ipsilon-server-install @@ -348,6 +348,11 @@ def parse_args(plugins): if len(args['hostname'].split('.')) < 2: raise ConfigurationError('Hostname: %s is not a FQDN') + for plugin_group in plugins: + for plugin_name in plugins[plugin_group]: + plugin = plugins[plugin_group][plugin_name] + plugin.validate_args(args) + try: pwd.getpwnam(args['system_user']) except KeyError: |