Remove spurious log that adds no value and make ITs

sometimes fail because of an additional line.

Signed-off-by: David Gageot <david@gageot.net>
This commit is contained in:
David Gageot 2016-01-26 17:02:57 +01:00
parent b6462a1969
commit b4209ee84b
1 changed files with 3 additions and 7 deletions

View File

@ -109,7 +109,8 @@ func (f *DefaultRPCClientDriverFactory) Close() error {
for _, openedDriver := range f.openedDrivers { for _, openedDriver := range f.openedDrivers {
if err := openedDriver.close(); err != nil { if err := openedDriver.close(); err != nil {
log.Warnf("Error closing a plugin driver: %s", err) // No need to display an error.
// There's nothing we can do and it doesn't add value to the user.
} }
} }
f.openedDrivers = []*RPCClientDriver{} f.openedDrivers = []*RPCClientDriver{}
@ -213,14 +214,9 @@ func (c *RPCClientDriver) close() error {
} }
log.Debug("Successfully made call to close driver server") log.Debug("Successfully made call to close driver server")
log.Debug("Making call to close connection to plugin binary") log.Debug("Making call to close connection to plugin binary")
if err := c.plugin.Close(); err != nil { return c.plugin.Close()
return err
}
return nil
} }
// Helper method to make requests which take no arguments and return simply a // Helper method to make requests which take no arguments and return simply a