On 2/9/2016 10:53 AM, Ade Lee wrote:
This check is unnecessary and is breaking continuous integration in
OpenStack. Removing from pkispawn/pkidestroy.
Ade
If I'm reading the code correctly the pki_dns_domainname is used as a
label to populate other properties (e.g. security domain names, admin
email, nickname). While the label itself is not crucial to server
operations, the patch removes the code that's supposed obtain the proper
label. So under normal usage (i.e. outside CI) the pki_dns_domainname
will be None, and those other properties will have incomplete values.
I think there are several options:
1. Instead of removing the check, the code should assign a better
default value (e.g. example.com) to pki_dns_domainname if the domain
name is not available due to CI.
2. Remove the check, but also remove all references to
pki_dns_domainname from the entire code.
--
Endi S. Dewata
_______________________________________________
Pki-devel mailing list
[email protected]
https://www.redhat.com/mailman/listinfo/pki-devel
