Created attachment 1144819 [details]
Mark puppet environment as a required field
This screenshot shows that "Puppet Environment" is a required field for provisioning new hosts. However, the form does not flag this field as being required. Other required fields have an "*" next to them to denote them as being required.
The puppet environment is required field only when a puppet master is selected on the host. http://projects.theforeman.org/issues/7450. Given the current complexity of the host form UI, I don't think the gain for making sure we flag the environment as required only when puppet master is selected is worth the effort (even not sure it would get accepted upstream).
Since it's not just a missed flag on obviously required field and there are reasons why it's not flagged by default, I don't agree on the blocker flag here.
I think this won't be implemented until we have a way of specifying required fields validations based on other fields selections. That sounds more like a new way of rendering forms. Also it should not be addressed just for this field but rather every field, e.g. subnet has some fields required for ipv4 only.
Converting to a more generic RFE
Created redmine issue http://projects.theforeman.org/issues/21678 from this bug
Thank you for your interest in Satellite 6. We have evaluated this request, and we do not expect this to be implemented in product in the forseeable future. We are therefore closing this out as WONTFIX. If you have any concerns about this, please feel free to contact Rich Jerrido or Bryan Kearney. Thank you.