Skip to content
This repository was archived by the owner on May 20, 2026. It is now read-only.

CPU & GPU: init + sample tests#24

Merged
pablo-garay merged 9 commits into
mainfrom
pablo-garay/add-cicd-testing
Nov 4, 2025
Merged

CPU & GPU: init + sample tests#24
pablo-garay merged 9 commits into
mainfrom
pablo-garay/add-cicd-testing

Conversation

@pablo-garay

@pablo-garay pablo-garay commented Nov 3, 2025

Copy link
Copy Markdown
Contributor
  • run when commit to main or release branch
  • CPU & GPU: init + sample tests

@copy-pr-bot

copy-pr-bot Bot commented Nov 3, 2025

Copy link
Copy Markdown

This pull request requires additional validation before any workflows can run on NVIDIA's runners.

Pull request vetters can view their responsibilities here.

Contributors can view more details about this message here.

Signed-off-by: Pablo Garay <pagaray@nvidia.com>
Signed-off-by: Pablo Garay <pagaray@nvidia.com>
@pablo-garay
pablo-garay force-pushed the pablo-garay/add-cicd-testing branch from 7a903b0 to df59692 Compare November 3, 2025 20:58
Signed-off-by: Pablo Garay <pagaray@nvidia.com>
Signed-off-by: Pablo Garay <pagaray@nvidia.com>
Signed-off-by: Pablo Garay <pagaray@nvidia.com>
Signed-off-by: Pablo Garay <pagaray@nvidia.com>
Signed-off-by: Pablo Garay <pagaray@nvidia.com>
@pablo-garay

Copy link
Copy Markdown
Contributor Author

/ok to test b17ecc4

Signed-off-by: Pablo Garay <pagaray@nvidia.com>
@pablo-garay

Copy link
Copy Markdown
Contributor Author

/ok to test b8d0ed5

Signed-off-by: Pablo Garay <pagaray@nvidia.com>
@pablo-garay

Copy link
Copy Markdown
Contributor Author

/ok to test 5888465

@abhinavg4 abhinavg4 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

looks good right now, but we will need to make sure that dockerfile has Mcore path also installed

Comment thread docker/Dockerfile.ci
FROM nvcr.io/nvidia/pytorch:25.09-py3

WORKDIR /workspace

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Can you please take a look at this dockerfile. We would want to copy their changes for the Mcore path:

https://github.com/NVIDIA-NeMo/Megatron-Bridge/blob/main/docker/Dockerfile.ci

I don't think we need third_party/mcore relates installations but can you please take a look if the overall dockerfile looks similar ?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

yes i can do :)

will address in follow up PR ; SG thankss👍

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Looks similar. MegatronBridge is using that thirdparty/mcore install though

Comment thread docker/Dockerfile.ci
COPY pyproject.toml uv.lock ./

# Install test dependencies using uv sync
RUN uv sync --link-mode copy --locked --group test --no-install-project

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

I think we need to think this through. This is only installing automodel path right now. We will also need Mcore path

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

yes i can do :)

will address in follow up PR ; SG thankss👍

@pablo-garay
pablo-garay merged commit 9d74ab1 into main Nov 4, 2025
13 checks passed
huvunvidia pushed a commit that referenced this pull request Feb 12, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants