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

tegra-tests: add nvidia-docker-tests #42

Merged
merged 1 commit into from
Sep 2, 2022

Conversation

atharvanan1
Copy link
Contributor

@atharvanan1 atharvanan1 commented Sep 1, 2022

  • This will test container compatibility on the device image
  • Tested on jetson-xavier-nx-devkit-emmc

@dwalkes
Copy link
Member

dwalkes commented Sep 1, 2022

Tested on jetson-xavier-nx-devkit-emmc

I think you need some tricks to get a /var partition greater than 16GB on this platform, correct?

@atharvanan1
Copy link
Contributor Author

atharvanan1 commented Sep 1, 2022

Ah yes. I needed to expand storage for pulling docker images.

@dwalkes
Copy link
Member

dwalkes commented Sep 1, 2022

Trying to figure out where we should document this. There are instructions in https://docs.google.com/spreadsheets/d/1EWG5QZvtWdk74gz48D4xCbegK-bsqMFJ8XOsyGdUUeI/edit#gid=593862884 but those suggest using the devkit build.

@atharvanan1
Copy link
Contributor Author

those suggest using the devkit build.

Do you mean the sdcard machine?

@dwalkes
Copy link
Member

dwalkes commented Sep 1, 2022

Do you mean the sdcard machine?

Yes, devkit and not devkit-emmc

@atharvanan1
Copy link
Contributor Author

Yes, devkit and not devkit-emmc

For this, I tried to use the extra space available on the sdcard. Ex. Using 32 GB SDcard, provided me with 16GB of extra space.

- This will test container compatibility on the device image

Signed-off-by: Atharva Nandanwar <[email protected]>
@atharvanan1
Copy link
Contributor Author

Should be updated with a small patch now.

@madisongh madisongh merged commit 90cb081 into OE4T:wip-jetpack-5-ga Sep 2, 2022
@atharvanan1 atharvanan1 deleted the add-docker-tests branch September 2, 2022 15:53
@atharvanan1
Copy link
Contributor Author

Thanks guys for your help!

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.

3 participants