-
Notifications
You must be signed in to change notification settings - Fork 28
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Consider validation of '/system/archetype/os/name' #178
Comments
Another potential challenge: t's used in different ways at different sites. So e.g. we use it for Linux/Solaris but in @jouvin's setup docs he uses it to differentiate between Linux distros. |
Perhaps the validation should be nothing more than "is a complete lower-case word with no funny characters" |
One of the types in #180 perhaps? |
I think that if we want to be able to test against it in other standard templates, it would be better to have an exhaustive list of possible values. Is it really much more than linux, centos, scientific-linux, rhel, debian, ubuntu? It looks to me as a rather finite list that we can update when needed... |
I don't think we can meaningfully validate the content in any other way than enforcing a lowercase |
As mentioned in quattor/configuration-modules-core#1282.
For:
Against:
The text was updated successfully, but these errors were encountered: