Mattermost-server: Refactor "store/sqlstore/upgrade.go" to use structured logging

[mattermod]

Mattermost is refactoring the logs to be fully structured and we're looking for contributors to help with that effort. This Help Wanted issue is to change the usages of mlog. methods with fmt.Sprintf to proper mlog.Field() calls in file store/sqlstore/upgrade.go.

The expected way to implement it is to open the file store/sqlstore/upgrade.go, look for all calls to mlog.Info, mlog.Debug, etc and in cases where fmt.Sprintf is used to write the log - modify the call to use mlog.Bool, mlog.String, etc to pass the parameters, instead of passing them as part of the format string to Sprintf.

Example: https://github.com/mattermost/mattermost-server/pull/12038

---

If you're interested please comment here and come join our "Contributors" community channel on our daily build server, where you can discuss questions with community members and the Mattermost core team. For technical advice or questions, please join our "Developers" community channel.

New contributors please see our Developer's Guide.

JIRA: https://mattermost.atlassian.net/browse/MM-18301

[niklabh]

Here is a PR:

https://github.com/mattermost/mattermost-server/pull/12085

[niklabh]

Sorry previous PR got closed. Here is a new one with all the review comments fixes.

https://github.com/mattermost/mattermost-server/pull/12628