Skip to content

chore: add .git to .dockerignore#148

Merged
maze88 merged 1 commit intoMellanox:masterfrom
maze88:master
Feb 4, 2026
Merged

chore: add .git to .dockerignore#148
maze88 merged 1 commit intoMellanox:masterfrom
maze88:master

Conversation

@maze88
Copy link
Collaborator

@maze88 maze88 commented Feb 4, 2026

No description provided.

@github-actions
Copy link

github-actions bot commented Feb 4, 2026

Thanks for your PR,
To run vendors CIs, Maintainers can use one of:

  • /test-all: To run all tests for all vendors.
  • /test-e2e-all: To run all E2E tests for all vendors.
  • /test-e2e-nvidia-all: To run all E2E tests for NVIDIA vendor.

To skip the vendors CIs, Maintainers can use one of:

  • /skip-all: To skip all tests for all vendors.
  • /skip-e2e-all: To skip all E2E tests for all vendors.
  • /skip-e2e-nvidia-all: To skip all E2E tests for NVIDIA vendor.
    Best regards.

@greptile-apps
Copy link

greptile-apps bot commented Feb 4, 2026

Greptile Overview

Greptile Summary

This PR adds a .dockerignore file that excludes .git/ from the Docker build context. This reduces build-context size and prevents VCS metadata from being copied into images, aligning with typical Docker build best practices.

Confidence Score: 5/5

  • This PR is safe to merge with minimal risk.
  • Change is limited to adding .git/ to .dockerignore, which only affects Docker build context contents and does not alter runtime logic.
  • No files require special attention

Important Files Changed

Filename Overview
.dockerignore Adds .dockerignore entry to exclude the .git directory from Docker build context.

Sequence Diagram

sequenceDiagram
    participant Dev as Developer
    participant Git as Git repo
    participant Docker as Docker build

    Dev->>Git: Add `.dockerignore` with `.git/`
    Dev->>Docker: docker build (sends build context)
    Docker->>Git: Exclude `.git/` from build context per `.dockerignore`
    Docker-->>Dev: Build uses smaller context (no VCS metadata)
Loading

Copy link

@greptile-apps greptile-apps bot left a comment

Choose a reason for hiding this comment

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

1 file reviewed, no comments

Edit Code Review Agent Settings | Greptile

@maze88 maze88 merged commit 47c85e9 into Mellanox:master Feb 4, 2026
12 of 16 checks passed
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.

1 participant