Merge pull request #2931 from dgageot/remove-annoying-log

Remove spurious log that adds no value and make ITs fail sometimes because of an additional line
This commit is contained in:
Jean-Laurent de Morlhon 2016-01-26 17:33:50 +01:00
commit 7f708edcec
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