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.

Sign up for GitHub

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

Jump to bottom

Add runtime environment output to fix building the release container #1496

Merged
merged 72 commits into from
Feb 7, 2024
Merged

Add runtime environment output to fix building the release container #1496

merged 72 commits into from
Feb 7, 2024

Conversation

cwharris
Copy link
Contributor

@cwharris cwharris commented Feb 6, 2024

Description

Fixes the release container build.

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.

@cwharris
upgrade to cudf-23.10
1485b88
@cwharris
Merge branch 'branch-24.03' of github.com:nv-morpheus/Morpheus into c…
082ec58 
…udf-upgrade-23.10
@cwharris
rapids 23.12
ddf2c4f
@cwharris
unpin dep versions
e03953b
@cwharris
fix sql password hiding breaking change
d1d7789
@cwharris
fix graph_sage_stage bug
38f9e50
@cwharris
pin dependencies
fa54c64
@cwharris
.
2122371
@cwharris
add all example deps to dependencies.yaml
795645d
@cwharris
remove unnecessary requirements files
045a1bc
@cwharris
expand *click in dependencies.yaml
978991e
@cwharris
cmake update and file formatting
25d0e32
@cwharris
update conda channels
a0cad2c
@cwharris
Merge branch 'branch-24.03' of github.com:nv-morpheus/Morpheus into c…
4365488 
…udf-upgrade-24.02
@cwharris
update to cccl and fix matx
45fc8fe
@cwharris
get libcudacxx from cccl, fix matx usages
b0eaf09
@cwharris
fix cudf api misuse, use correct column owners
60d9c72
@cwharris
try fix ci
e86de1f
@cwharris
require cxx standard 20
d9a8a42
@cwharris
try fix ci
9ecb19b
@cwharris
try fix ci
ffec30b
@cwharris
try fix ci
0345ef2
@cwharris
try fix ci
050fe69
@cwharris
get grpcio-status from conda
d1811f5
@cwharris
try fix ci
71415a8
@cwharris
pin pytest
d5d6fbe
@cwharris
always remove and re-add index when adding slices to multi_message
8414ad8
@cwharris
fix matx util bug
41ea18b
@cwharris
cleanup
06d6913
@cwharris
address review feedback
59ba379
@cwharris cwharris added bug Something isn't working non-breaking Non-breaking change labels Feb 6, 2024
@cwharris cwharris marked this pull request as ready for review February 6, 2024 14:40
@cwharris cwharris requested review from a team as code owners February 6, 2024 14:40
mdemoret-nv
mdemoret-nv requested changes Feb 6, 2024
View reviewed changes
Copy link
Contributor

@mdemoret-nv mdemoret-nv left a comment

Choose a reason for hiding this comment

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

Just want to confirm that we are using the new environments in conda/environments and all references to docker/conda has been removed.

docker/Dockerfile Outdated Show resolved Hide resolved
docker/build_container.sh Show resolved Hide resolved
@cwharris
Copy link
Contributor Author

cwharris commented Feb 6, 2024

Addressed the feedback and opened #1497. We'll need to work with the data science team to make sure we get this updated properly, so suggest we go ahead and merge this PR.

@mdemoret-nv mdemoret-nv changed the title Fix release container Add runtime environment output to fix building the release container Feb 7, 2024
@mdemoret-nv
Copy link
Contributor

/merge

@rapids-bot rapids-bot bot merged commit 6388da1 into nv-morpheus:branch-24.03 Feb 7, 2024
11 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@mdemoret-nv mdemoret-nv mdemoret-nv approved these changes

Assignees
No one assigned
Labels
bug Something isn't working non-breaking Non-breaking change
Projects
Morpheus Boards
Archived in project
Milestone
24.03 - Dependencies, Build System, a...
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@cwharris @mdemoret-nv