Skip to content

Update versions in Dockerfile - #47

Merged
gitricko merged 6 commits into
mainfrom
gitricko-ver-update
Sep 5, 2026
Merged

Update versions in Dockerfile#47
gitricko merged 6 commits into
mainfrom
gitricko-ver-update

Conversation

@gitricko

@gitricko gitricko commented Sep 5, 2026

Copy link
Copy Markdown
Owner

This pull request updates the Docker build and startup scripts to improve reliability, update dependencies, and streamline the CI workflow. The most significant changes include updating several dependency versions, increasing timeouts and health checks for better startup robustness, and adjusting the GitHub Actions workflow for better concurrency and test accuracy.

Dependency Updates:

  • Updated OMNIROUTE_VERSION to 3.8.50, OLLAMA_VERSION to 0.33.3, PI_VERSION to 0.85.1, and MNEMON_VERSION to 0.2.8 in docker/Dockerfile for the latest features and fixes.

CI/CD Workflow Improvements:

  • Moved the permissions block, added a concurrency group to cancel in-progress runs for the same branch/PR, and improved test output and logic in .github/workflows/docker-publish.yml. [1] [2] [3] [4]

Startup Robustness:

  • Increased the dashboard readiness wait loop from 20 to 200 iterations (up to 10 minutes) in docker/start-hermes.sh to handle slower startups.
  • Increased the maximum attempts for OmniRoute readiness from 120 to 180 and switched the health check endpoint to /healthz in docker/start-omniroute.sh.
  • Updated the OmniRoute combo creation command to match the new CLI requirements and commented out legacy MCP enabling and model configuration code, reflecting changes in OmniRoute 3.8.50.

These changes collectively improve system reliability, keep dependencies up to date, and streamline the CI pipeline.

Copilot AI lite review requested due to automatic review settings September 5, 2026 17:45

Copilot AI 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.

🟢 Approval recommended

The changes are limited to straightforward Docker ARG version bumps with no functional logic modifications.

Pull request overview

Updates pinned component versions in the Docker build to pull newer upstream releases.

Changes:

  • Bump OMNIROUTE_VERSION from 3.8.49 to 3.8.50
  • Bump OLLAMA_VERSION from 0.33.2 to 0.33.3
  • Bump PI_VERSION from 0.84.4 to 0.85.1 and MNEMON_VERSION from 0.2.7 to 0.2.8
File summaries
File Description
docker/Dockerfile Updates build ARG version pins for several bundled components.
Review details
  • Files reviewed: 1/1 changed files
  • Comments generated: 0
  • Review effort level: Lite

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

@greptile-apps

greptile-apps Bot commented Sep 5, 2026

Copy link
Copy Markdown

Greptile Summary

The PR updates several bundled tool versions and adjusts container startup and CI integration behavior.

  • Updates OmniRoute, Ollama, Pi, and Mnemon versions.
  • Adapts OmniRoute readiness and combo creation for the newer CLI.
  • Extends Hermes and OmniRoute startup readiness windows.
  • Restores Claude-based Mnemon retrieval in the Docker workflow.

Confidence Score: 5/5

The PR appears safe to merge because no blocking failure remains within the eligible follow-up-review scope.

No blocking failure remains.

Important Files Changed

Filename Overview
.github/workflows/docker-publish.yml Moves workflow permissions, adds per-ref concurrency cancellation, and restores Claude-based Mnemon retrieval.
docker/Dockerfile Updates the pinned versions of OmniRoute, Ollama, Pi, and Mnemon.
docker/start-hermes.sh Extends the Hermes dashboard readiness polling window from one minute to ten minutes.
docker/start-omniroute.sh Updates readiness probing and combo creation for OmniRoute 3.8.50 while disabling automatic MCP setup.

Reviews (4): Last reviewed commit: "Update Docker publish workflow with conc..." | Re-trigger Greptile

Increased the maximum attempts for waiting for OmniRoute to be ready from 120 to 180. Updated the health check URL and modified the combo creation command to include model specifications.
Added concurrency settings to cancel in-progress runs.
@gitricko
gitricko merged commit 8736d01 into main Sep 5, 2026
5 checks passed
@gitricko
gitricko deleted the gitricko-ver-update branch September 5, 2026 21:12
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