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

Usage of analysis docker file #3259

Closed
ukinimod opened this issue Mar 29, 2023 · 2 comments
Closed

Usage of analysis docker file #3259

ukinimod opened this issue Mar 29, 2023 · 2 comments
Labels
bug Only issues that describe bugs. good first issue pr-analysis Issues that touch the analysis pr(oject). priority:high Set by PO tech For technical stories without user impact (=refactoring stories).

Comments

@ukinimod
Copy link
Contributor

Bug

Expected Behavior

WHEN

I ran the analysis docker image via

docker run -v $(pwd):/src --workdir /src codecharta/codecharta-analysis:1.114.0 csvimport some.csv

THEN

It should run ccsh csvimport in the docker container

Actual Behavior

It starts a docker container with entrypoint "tail -f /dev/null" and hangs

Additional Info

  • There is a workaround for this: fixing the entrypoint to use "ccsh" instead of the config of the docker image.
  • Using docker run -v $(pwd):/src --workdir /src codecharta/codecharta-analysis:1.114.0 ccsh csvimport some.csv amounts to the same result.

Steps to Reproduce the Problem

  1. docker run -v $(pwd):/src --workdir /src codecharta/codecharta-analysis:1.114.0 csvimport some.csv

Specifications

  • CodeCharta Version: 1.114.0
  • OS: Windows 11 WSL
  • Browser: not relevant
@ukinimod ukinimod added the bug Only issues that describe bugs. label Mar 29, 2023
@ce-bo ce-bo added priority:high Set by PO pr-analysis Issues that touch the analysis pr(oject). tech For technical stories without user impact (=refactoring stories). good first issue labels Mar 30, 2023
@ukinimod
Copy link
Contributor Author

@ce-bo If there is nobody on this, I could implement this. It should be a quick fix.

@ce-bo
Copy link
Collaborator

ce-bo commented May 15, 2023

@ukinimod The image provides more commands than just ccsh. Thus, the entrypoint should not prevent the usage of the other commands. I guess the way to go then is docker run -v $(pwd):/src --workdir /src codecharta/codecharta-analysis:1.114.0 ccsh csvimport some.csv.

ukinimod added a commit to ukinimod/codecharta that referenced this issue May 15, 2023
ce-bo pushed a commit that referenced this issue May 16, 2023
* Fix wrong entrypoint of analysis docker #3259

* Fix wrong entrypoint of analysis docker #3259

* Delete entrypoint of analysis docker #3259
@ce-bo ce-bo closed this as completed May 16, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Only issues that describe bugs. good first issue pr-analysis Issues that touch the analysis pr(oject). priority:high Set by PO tech For technical stories without user impact (=refactoring stories).
Projects
None yet
Development

No branches or pull requests

2 participants