Skip to content
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

Fix environment yaml paths #1551

Merged
merged 5 commits into from
Mar 15, 2024

Conversation

efajardo-nv
Copy link
Contributor

@efajardo-nv efajardo-nv commented Mar 7, 2024

Description

  • Update environment yaml paths to point to conda/environments instead of morpheus/conda/environments.
  • Update docker/Dockerfile to copy all yaml files from conda/environments to runtime image.
  • Fix broken environment yaml path in DFP Dockerfile to use above examples yaml.
  • Remove DFP conda_env.yml which is out of date and no longer used. DFP dependencies have already been moved to the examples yaml with updated versions.

Closes #1554

By Submitting this PR I confirm:

  • I am familiar with the Contributing Guidelines.
  • When the PR is ready for review, new or existing tests cover these changes.
  • When the PR is ready for review, the documentation is up to date with these changes.

@efajardo-nv efajardo-nv added non-breaking Non-breaking change improvement Improvement to existing functionality labels Mar 7, 2024
@efajardo-nv efajardo-nv self-assigned this Mar 7, 2024
@efajardo-nv efajardo-nv requested review from a team as code owners March 7, 2024 16:47
docker/Dockerfile Outdated Show resolved Hide resolved
@efajardo-nv
Copy link
Contributor Author

/merge

@rapids-bot rapids-bot bot merged commit 7bb4ec2 into nv-morpheus:branch-24.03 Mar 15, 2024
11 checks passed
@efajardo-nv efajardo-nv deleted the fix-yaml-paths branch March 15, 2024 15:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
improvement Improvement to existing functionality non-breaking Non-breaking change
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

[BUG]: Error when building DFP container
2 participants