Skip to content

[infra] Replace nnas docker-run with direct docker commands#16230

Merged
hseok-oh merged 1 commit into
Samsung:masterfrom
hseok-oh:fix_docker_run
Oct 24, 2025
Merged

[infra] Replace nnas docker-run with direct docker commands#16230
hseok-oh merged 1 commit into
Samsung:masterfrom
hseok-oh:fix_docker_run

Conversation

@hseok-oh
Copy link
Copy Markdown
Contributor

This commit replaces the nnas docker-run command with direct docker run commands in the CI workflow.
nnas docker-run command set user as directory owner, but directory owner ID in github action is 1001, not root. So it causes permission error when running nnas docker-run command.

ONE-DCO-1.0-Signed-off-by: Hyeongseok Oh hseok82.oh@samsung.com

This commit replaces the `nnas docker-run` command with direct docker run commands in the CI workflow.
`nnas docker-run` command set user as directory owner, but directory owner ID in github action is 1001, not root. So it causes permission error when running `nnas docker-run` command.

ONE-DCO-1.0-Signed-off-by: Hyeongseok Oh <hseok82.oh@samsung.com>
@hseok-oh hseok-oh added the PR/ready for review It is ready to review. Please review it. label Oct 23, 2025
Copy link
Copy Markdown
Contributor

@ragmani ragmani left a comment

Choose a reason for hiding this comment

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

LGTM

@hseok-oh hseok-oh merged commit 9381924 into Samsung:master Oct 24, 2025
8 checks passed
@hseok-oh hseok-oh deleted the fix_docker_run branch October 24, 2025 00:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

PR/ready for review It is ready to review. Please review it.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants