[node-patches] Change in ovirt-node[master]: Make FQDN validator abide by RFC1035

fabiand at fedoraproject.org fabiand at fedoraproject.org
Thu Jun 13 16:44:42 UTC 2013


Fabian Deutsch has posted comments on this change.

Change subject: Make FQDN validator abide by RFC1035
......................................................................


Patch Set 1:

I agree with you on the point that it's nice to give some differentiated feedback to the user.
I'd still suggest to use a regex to validate the length of the sections. The solution looks a bit cumbersome ...
If it's valuable for other places too we could even create a validator which takes several validators and validates one after another and spits out the msg assigned to the validator.

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I747a29c1a15b601bf1eea364b76e56818078e80b
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-node
Gerrit-Branch: master
Gerrit-Owner: Ryan Barry <rbarry at redhat.com>
Gerrit-Reviewer: Fabian Deutsch <fabiand at fedoraproject.org>
Gerrit-Reviewer: Ryan Barry <rbarry at redhat.com>
Gerrit-Reviewer: oVirt Jenkins CI Server



More information about the node-patches mailing list