Skip to content

Rename repo name and add image tag vesrion#4

Merged
denisPust merged 1 commit into1.9.xfrom
update-ci-v2
Jul 29, 2025
Merged

Rename repo name and add image tag vesrion#4
denisPust merged 1 commit into1.9.xfrom
update-ci-v2

Conversation

@denisPust
Copy link

@denisPust denisPust commented Jul 29, 2025

PR Type

enhancement, configuration changes


Description

  • Update Docker image repository name in CI workflow

  • Add version tag (1.9.2) to Docker image build and push

  • Ensure CI uses new image naming convention


Changes walkthrough 📝

Relevant files
Configuration changes
ci.yaml
Update Docker image repo and tag in CI pipeline                   

.github/workflows/ci.yaml

  • Changed Docker image repository from mlrun-ui-pt to mlrun-ui
  • Added version tag 1.9.2 to Docker image name in build and push steps
  • Updated all relevant CI steps to use new naming convention
  • +2/-2     

    Need help?
  • Type /help how to ... in the comments thread for any questions about PR-Agent usage.
  • Check out the documentation for more information.
  • @pull-request-agent
    Copy link

    PR Reviewer Guide 🔍

    Here are some key observations to aid the review process:

    ⏱️ Estimated effort to review: 2 🔵🔵⚪⚪⚪
    🧪 No relevant tests
    🔒 No security concerns identified
    ⚡ Recommended focus areas for review

    Tagging Consistency

    Ensure that the new Docker image tag format (proteantecs/mlrun-ui:1.9.2-${{ github.sha }}) is consistently used throughout all relevant deployment scripts and documentation to avoid confusion or deployment issues.

        docker build -t proteantecs/mlrun-ui:1.9.2-${{ github.sha }} . > build.log 2>&1 || (cat build.log; exit 1)
    
    - name: Push Docker image
      if: success()
      run: |
        docker push proteantecs/mlrun-ui:1.9.2-${{ github.sha }}

    @pull-request-agent
    Copy link

    PR Code Suggestions ✨

    No code suggestions found for the PR.

    @denisPust denisPust merged commit 5639c75 into 1.9.x Jul 29, 2025
    5 checks passed
    Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

    Projects

    None yet

    Development

    Successfully merging this pull request may close these issues.

    1 participant