Skip to content

client: use 'Podman' instead of 'Docker' in messages to user#7172

Merged
AenBleidd merged 6 commits into
masterfrom
dpa_docker35
Jul 12, 2026
Merged

client: use 'Podman' instead of 'Docker' in messages to user#7172
AenBleidd merged 6 commits into
masterfrom
dpa_docker35

Conversation

@davidpanderson

@davidpanderson davidpanderson commented Jul 11, 2026

Copy link
Copy Markdown
Contributor

Summary by cubic

Default to “Podman” in user-facing messages and help links, but show “Docker” when Docker is the detected runtime. Also fixes an uninitialized field in project list entries. Improves logs, host info, cleanup output, and WSL notices.

  • Refactors

    • Messages use the detected runtime name (Docker/Podman) in host info and compose lines (e.g., “%s version %s”).
    • Install/help links now point to Podman; show a Podman install notice when missing; Windows WSL notices mention “Docker or Podman”.
    • Cleanup logs include the runtime (e.g., “Removed unused %s container/image”); CPU scheduler alerts refer to Podman when appropriate.
    • Inlined docker_type_str() in hostinfo.h with proper capitalization; removed old implementation and fixed a Windows build issue.
    • Initialized PROJECT_LIST_ENTRY::is_account_manager to false to avoid uninitialized state.
  • New Features

    • Added dont_use_podman in cc_config.xml (alias of dont_use_docker); logs reflect “don’t use Podman/Docker” and skip Podman apps when disabled.

Written for commit 3ea21a7. Summary will update on new commits.

Review in cubic

@cubic-dev-ai cubic-dev-ai Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

All reported issues were addressed

Reply with feedback, questions, or to request a fix.

Re-trigger cubic

Comment thread client/hostinfo_wsl.cpp Outdated
Comment thread client/client_state.cpp Outdated
Comment thread client/cs_apps.cpp Outdated
@github-project-automation github-project-automation Bot moved this to In progress in Client/Manager Jul 12, 2026
@AenBleidd AenBleidd added this to the Client/Manager 8.2.16 milestone Jul 12, 2026
@AenBleidd AenBleidd merged commit 693ed47 into master Jul 12, 2026
272 checks passed
@AenBleidd AenBleidd deleted the dpa_docker35 branch July 12, 2026 16:26
@github-project-automation github-project-automation Bot moved this from In progress to Merged in Client/Manager Jul 12, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

Status: Merged

Development

Successfully merging this pull request may close these issues.

2 participants