This is an automatically generated e-mail. To reply, visit: http://reviewboard-fedoraserver.rhcloud.com/r/2/

On July 14th, 2014, 1:24 p.m. UTC, Thomas Woerner wrote:

config/roles/databaseserver/role.py (Diff revision 1)
38
        "backup_paths": [ ]

backup_path was in the testrole by accident before. The methods to handle backup_path and also the property is disabled at the moment. Should it just be enabled?

No, we're not likely to have it implemented properly in time for F21. I'll remove this.


- Stephen


On July 14th, 2014, 11:42 a.m. UTC, Stephen Gallagher wrote:

Review request for RoleKit Mailing List, Stephen Gallagher and Thomas Woerner.
By Stephen Gallagher.

Updated July 14, 2014, 11:42 a.m.

Repository: rolekit

Description

Add placeholders for Domain Controller and Database roles

Testing

Started up rolekit and examined with d-feet that the new roles were available and functioned equivalently to the testrole.

Diffs

  • config/Makefile.am (88f68818c877eeadf05e3c6bccea77e6693d428c)
  • config/roles/databaseserver/role.py (PRE-CREATION)
  • config/roles/domaincontroller/role.py (PRE-CREATION)

View Diff