Skip to content

chore: add warn log when fix_venv failed #338

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Mar 21, 2025

Conversation

zhaohaidao
Copy link
Contributor

Overview:

Error result from super::fix_venv(venv, py) is not properly handled

Details:

Add a warn log to reflect the error result.

Where should the reviewer start?

Related Issues: (use one of the action keywords Closes / Fixes / Resolves / Relates to)

  • closes GitHub issue: #xxx

Copy link

copy-pr-bot bot commented Mar 21, 2025

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.

Copy link

👋 Hi zhaohaidao! Thank you for contributing to ai-dynamo/dynamo.

Just a reminder: The NVIDIA Test Github Validation CI runs an essential subset of the testing framework to quickly catch errors.Your PR reviewers may elect to test the changes comprehensively before approving your changes.

🚀

@github-actions github-actions bot added the external-contribution Pull request is from an external contributor label Mar 21, 2025
@zhaohaidao zhaohaidao changed the title [chore] add warn log when fix_venv failed chore: add warn log when fix_venv failed Mar 21, 2025
@grahamking grahamking merged commit cb24669 into ai-dynamo:main Mar 21, 2025
9 of 10 checks passed
rainj-me pushed a commit to bytedance-iaas/dynamo that referenced this pull request Mar 21, 2025
kylehh pushed a commit to kylehh/dynamo that referenced this pull request Apr 11, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
external-contribution Pull request is from an external contributor
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants