[Pki-devel] TRAC Ticket #869 - f19 ipa-server-install fails at step 6/22 of cert sys install - systemctl start pki-tomcatd.target fails

[Ade Lee]

Also, I think we should be trying to clean up pylint warnings as we go
through and not add any new ones.  That means for instance, that the new
method you added should contain a docstring class.

In fact, I would suggest making some changes to fix most of the pylint
warnings in say the Systemd class.  There aren't that many - and if
everyone does it, we can slowly improve the code as we go.

If you want to do this only in master, thats fine with me too.

Ade

On Thu, 2014-03-13 at 13:22 -0400, Ade Lee wrote:
> In general, the patch is fine.  Note that in the 10.1+, we have modified
> the code to pass an array of commands to the system call.  Make sure to
> do that on those branches.
> 
> ACK.
> 
> On Wed, 2014-03-12 at 19:25 -0700, Matthew Harmsen wrote:
> > Please review the following patch which addresses:
> >       * PKI TRAC Ticket #869 - f19 ipa-server-install fails at step
> >         6/22 of cert sys install - systemctl start pki-tomcatd.target
> >         fails
> > 
> > This patch has been tested on Dogtag 10.0 on Fedora 19 using packages
> > built from the DOGTAG_10_0_BRANCH.
> > 
> > 
> > _______________________________________________
> > Pki-devel mailing list
> > Pki-devel at redhat.com
> > https://www.redhat.com/mailman/listinfo/pki-devel
> 
> 
> _______________________________________________
> Pki-devel mailing list
> Pki-devel at redhat.com
> https://www.redhat.com/mailman/listinfo/pki-devel