Skip to content

Standardize logging to use logrus consistently across the repository#429

Draft
Copilot wants to merge 2 commits intomainfrom
copilot/refactor-logging-style-again
Draft

Standardize logging to use logrus consistently across the repository#429
Copilot wants to merge 2 commits intomainfrom
copilot/refactor-logging-style-again

Conversation

Copy link

Copilot AI commented Feb 27, 2026

  • pkg/backend/attach.go — Concise wording, move config/struct dumps to Debug, remove "starting...operation" and "successfully" boilerplate
  • pkg/backend/build.go — Same pattern: concise verbs, struct dumps to Debug
  • pkg/backend/pull.go — Same pattern
  • pkg/backend/push.go — Same pattern
  • pkg/backend/fetch.go — Same pattern
  • pkg/backend/extract.go — Same pattern
  • pkg/backend/login.go — Concise wording
  • pkg/backend/logout.go — Concise wording
  • pkg/backend/rm.go — Concise wording
  • pkg/backend/list.go — Concise wording
  • pkg/backend/tag.go — Concise wording
  • pkg/backend/upload.go — Concise wording
  • pkg/backend/prune.go — Concise wording
  • pkg/backend/inspect.go — Concise wording, move config dump to Debug
  • pkg/backend/build/builder.go — Add "builder:" prefix to all logs, move digest calculation to Debug
  • pkg/backend/processor/base.go — Concise wording
  • pkg/backend/remote/client.go — Add "remote:" prefix
  • pkg/codec/raw.go — Add "codec:" prefix
  • internal/pb/pb.go — Add "progress:" prefix
  • pkg/modelprovider/mlflow/downloader.go — Optimize wording, adjust levels
  • pkg/modelprovider/mlflow/provider.go — Optimize warning messages
  • pkg/backend/fetch_by_d7y.go — Concise wording
  • pkg/backend/pull_by_d7y.go — Concise wording
  • Run make lint, make build, make test to verify

💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

- Replace log alias (log "github.com/sirupsen/logrus") in mlflow package
  with direct "github.com/sirupsen/logrus" import
- Convert all log.Printf/log.Println calls to proper logrus level methods
  (logrus.Infof, logrus.Debugf, logrus.Errorf, logrus.Warnf)
- Add consistent "mlflow:" prefix to all mlflow log messages matching the
  existing "component: action" pattern used across the backend package
- Replace stdlib log.Fatal with logrus.Fatal in cmd/root.go
- Remove unused stdlib "log" import from cmd/root.go

Co-authored-by: chlins <31262637+chlins@users.noreply.github.com>
Copilot AI changed the title [WIP] Refactor logging for consistent and standardized style Standardize logging to use logrus consistently across the repository Feb 27, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants