Reverts commits 79908b3c6, 083c5a042, e1ff75624, ff77fa248.

[winglian]

The non-root user approach had multiple issues with RunPod compatibility, sudo PATH handling, and tmux in exec sessions. Restoring root as the default user for now.

Description

verified this does have zero net changes

Motivation and Context

How has this been tested?

AI Usage Disclaimer

Screenshots (if appropriate)

Types of changes

Social Handles (Optional)

Summary by CodeRabbit

• Refactor
  • Simplified container startup and initialization process
  • Streamlined entrypoint script execution and user context handling
  • Removed intermediate setup steps for more direct service initialization
  • Consolidated Docker image configuration for cleaner startup flow

[coderabbitai]

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: 799eb8a3-b20c-435b-84c3-4c478c79bb33

📥 Commits

Reviewing files that changed from the base of the PR and between ff77fa2 and 85a556d.

📒 Files selected for processing (4)

• docker/Dockerfile-cloud-uv
• docker/Dockerfile-uv
• docker/Dockerfile-uv-base
• scripts/cloud-entrypoint.sh

---

📝 Walkthrough

Walkthrough

This PR removes the ubuntu user from Docker images across all Dockerfiles and the cloud entrypoint script, consolidating setup logic to run under the root user context. It updates script paths, adjusts home directory references, and removes sudo-dependent logic and user-switching branching.

Changes

Cohort / File(s)	Summary
Docker base configuration docker/Dockerfile-uv-base	Removes ubuntu user creation and sudo setup; adds root PATH environment variable for local binaries; simplifies UV Python installation and removes workspace ownership transfer.
Docker application images docker/Dockerfile-uv, docker/Dockerfile-cloud-uv	Removes ubuntu user references and home directory customization; updates entrypoint script paths from /etc/ and /home/ubuntu/ to /root/; consolidates SSH and tmux configuration to target root home directory.
Cloud entrypoint script scripts/cloud-entrypoint.sh	Removes sudo detection and RUN_AS_USER branching logic; simplifies add_keys_to_authorized() function signature (removes target_home parameter); updates SSH key setup, service startup, and Jupyter invocation to use direct execution without user context switching.

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~20 minutes

Possibly related PRs

• use ubuntu user instead of root for uv docker images #3491: Modifies the same Dockerfiles and scripts/cloud-entrypoint.sh with opposite user-context changes (introduces ubuntu user instead of removing it).
• check ubuntu user and set uv python dir #3492: Modifies docker/Dockerfile-uv and docker/Dockerfile-uv-base with related changes to user handling and UV Python installation configuration.
• add uv axolotl builds #3431: Modifies the same Dockerfiles and cloud entrypoint script, affecting entrypoint paths, user/home handling, and SSH key setup logic.

✨ Finishing Touches

• 📝 Generate docstrings (stacked PR)
• 📝 Generate docstrings (commit on current branch)

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment
• Commit unit tests in branch revert-uv-ubuntu

📝 Coding Plan

• Generate coding plan for human review comments

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}