[2.1] Print logs from flaky test shouldUpgradeDatabase().

@jakewins The initial calls are actually using log.error() but ConsoleLogger is relying on its realLogger (TestLogger which extends StringLogger) which it calls without purveying the info|warn|error information. The logMessage() interface has not been deprecated anywhere that I have seen and since implementations of different classes still use it, just throwing an exception doesn't seem appropriate. I think removing it properly is orthogonal to what I am trying to accomplish so I am trying to not dig deeper into this right now. When the test case fails it tries to print using log.error() but it will throw the now removed exception, which is the reason for putting it back.