Skip to content

Fix: Use moveit-pro Docker image name - #798

Merged
JWhitleyWork merged 1 commit into
mainfrom
fix/rename-moveit-pro-images
Jul 27, 2026
Merged

Fix: Use moveit-pro Docker image name#798
JWhitleyWork merged 1 commit into
mainfrom
fix/rename-moveit-pro-images

Conversation

@marty-mcfly-bot

Copy link
Copy Markdown

[written by AI]

Summary

  • Rename MoveIt Pro customer image references from moveit-studio to moveit-pro.
  • Update the default workspace base image and CI image resolution for paired PRs and fallback tags.

Validation

  • pre-commit run -a
  • Verified no tracked moveit-studio references remain.
  • git diff --check origin/main...HEAD

@github-actions

Copy link
Copy Markdown

⚠️ This PR modifies 2 file(s) that also exist in PickNikRobotics/moveit_pro_empty_ws.

Consider whether the change should land upstream in moveit_pro_empty_ws first so downstream forks pick it up on the next sync.

Overlapping files
  • .github/workflows/ci.yaml
  • Dockerfile

@marty-mcfly-bot marty-mcfly-bot Bot added this to the 10.0.0 milestone Jul 21, 2026
@coderabbitai

coderabbitai Bot commented Jul 21, 2026

Copy link
Copy Markdown

Review Change Stack

📝 Walkthrough

Walkthrough

The CI workflow and Dockerfile now consistently select customer images from moveit-pro instead of moveit-studio, while preserving existing tag and architecture handling.

Changes

MoveIt Pro image source alignment

Layer / File(s) Summary
CI image resolution
.github/workflows/ci.yaml
Paired-PR and fallback image_ref templates, along with related comments, now reference the moveit-pro Docker repository.
Docker base image default
Dockerfile
The default MOVEIT_PRO_BASE_IMAGE now uses picknikciuser/moveit-pro with the existing tag composition.

Possibly related PRs

Suggested reviewers: shaur-k


🔕 Pre-merge checks override applied

The pre-merge checks have been overridden successfully. You can now proceed with the merge.

Overridden by @JWhitleyWork via checkbox on 2026-07-27T13:39:44.485Z.

❌ Failed checks (1 error)

Check name Status Explanation Resolution
Human Review Check ❌ Error [IGNORED] PR changes CI workflow logic and Dockerfile base-image defaults, which are CI/CD and deployment/infra config under the check's fail list. This PR requires review by a requested human reviewer. After review, a non-author requested reviewer should override this pre-merge check.
✅ Passed checks (3 passed)
Check name Status Explanation
Description check ✅ Passed The description matches the changes: it renames image references to moveit-pro and updates CI and Dockerfile defaults.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.

Comment @coderabbitai help to get the list of available commands.

@github-actions

Copy link
Copy Markdown

MoveIt Pro Example WS - Objectives Integration Test Report

  • lab_sim
    • jazzy: no report produced — see run logs
  • hangar_sim
    • jazzy: no report produced — see run logs

@github-actions

Copy link
Copy Markdown

MoveIt Pro Example WS - Objectives Integration Test Report

@fdavulcu fdavulcu left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

LGTM!

@JWhitleyWork
JWhitleyWork merged commit 8ba187f into main Jul 27, 2026
25 of 28 checks passed
@JWhitleyWork
JWhitleyWork deleted the fix/rename-moveit-pro-images branch July 27, 2026 13:34
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