Advertising

IMHO the original text was good.
Tomas, why is the huge blob thing in exception bad?
Issues with new code/text.
1. it is supported but not for domain level != 0:
self.log.error("Using replica files to set up IPA replicas is not "
+ "supported."
+ )
2. format() is not needed:
+ self.log.info(
+ "To create a replica, you must promote an existing "
+ "IPA client.".format(domain_level=domain_level)
+ )

3. I don't like that the exception text says 'requires', which might
imply something to be done - lower domain level - which is not possible.
"allowed only" might be better.

Just changing RuntimeError to InvalidDomainLevelError would be fine with
me since the MIN_DOMAIN_LEVEL was already changed to DOMAIN_LEVEL_0.