[node-patches] Change in ovirt-node[master]: Make sure the Puppet certificate name is a valid hostname or...

[rbarry at redhat.com]

Ryan Barry has uploaded a new change for review.

Change subject: Make sure the Puppet certificate name is a valid hostname or FQDN
......................................................................

Make sure the Puppet certificate name is a valid hostname or FQDN

The certname field is intended to submit a hostname other than
the actual hostname of the system in case it's desirable to
identify the node by a different name. Previously, we didn't
validate this. Do so.

Change-Id: I19ce15a30dad9487fff41fdc9aeb2c9f58c04e82
Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=987793
Signed-off-by: Ryan Barry <rbarry at redhat.com>
---
M src/ovirt/node/setup/puppet/puppet_page.py
1 file changed, 1 insertion(+), 0 deletions(-)


  git pull ssh://gerrit.ovirt.org:29418/ovirt-node refs/changes/90/17490/1

diff --git a/src/ovirt/node/setup/puppet/puppet_page.py b/src/ovirt/node/setup/puppet/puppet_page.py
index 57a5ada..4bed3af 100644
--- a/src/ovirt/node/setup/puppet/puppet_page.py
+++ b/src/ovirt/node/setup/puppet/puppet_page.py
@@ -51,6 +51,7 @@
 
     def validators(self):
         return {"puppet.server": valid.FQDNOrIPAddress() | valid.Empty(),
+                "puppet.certname": valid.FQDN() | valid.Empty()
                 }
 
     def ui_content(self):


-- 
To view, visit http://gerrit.ovirt.org/17490
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: I19ce15a30dad9487fff41fdc9aeb2c9f58c04e82
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-node
Gerrit-Branch: master
Gerrit-Owner: Ryan Barry <rbarry at redhat.com>