-
Notifications
You must be signed in to change notification settings - Fork 108
enhancement : Moved Docker images repo to docker folder in main branch #2932
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
Conversation
|
Hey @Helion55 thanks for opening this PR! we will have a look during the following days, in nebari we use pre-commit to lint the files and commit changes; its just a few typos, but if you want then to be self-fixed, you can |
|
Hey, @ Helion55, I am giving you a heads-up on the review status. We are finishing the latest release of Nebari. Once that's out,, we will include this PR on our discussion board next week. Once again, thanks for opening the PR! |
|
Thanks @viniciusdc, I was curious about it. I have fixed the thing that you had mentioned. Waiting to catch up on next week. |
docker/Dockerfile.dask-worker
Outdated
| @@ -0,0 +1,41 @@ | |||
| # Copyright (c) Nebari Development Team. | |||
Check failure
Code scanning / Trivy
Image user should not be 'root' High
Type: dockerfile
Vulnerability DS002
Severity: HIGH
Message: Specify at least 1 USER command in Dockerfile with non-root user as argument
Link: DS002
docker/Dockerfile.dask-worker
Outdated
| @@ -0,0 +1,41 @@ | |||
| # Copyright (c) Nebari Development Team. | |||
Check notice
Code scanning / Trivy
No HEALTHCHECK defined Low
Type: dockerfile
Vulnerability DS026
Severity: LOW
Message: Add HEALTHCHECK instruction in your Dockerfile
Link: DS026
docker/Dockerfile.jupyterhub
Outdated
| @@ -0,0 +1,41 @@ | |||
| # Copyright (c) Nebari Development Team. | |||
Check failure
Code scanning / Trivy
Image user should not be 'root' High
Type: dockerfile
Vulnerability DS002
Severity: HIGH
Message: Specify at least 1 USER command in Dockerfile with non-root user as argument
Link: DS002
docker/Dockerfile.jupyterhub
Outdated
| @@ -0,0 +1,41 @@ | |||
| # Copyright (c) Nebari Development Team. | |||
Check notice
Code scanning / Trivy
No HEALTHCHECK defined Low
Type: dockerfile
Vulnerability DS026
Severity: LOW
Message: Add HEALTHCHECK instruction in your Dockerfile
Link: DS026
docker/Dockerfile.jupyterlab
Outdated
| @@ -0,0 +1,70 @@ | |||
| # Copyright (c) Nebari Development Team. | |||
Check failure
Code scanning / Trivy
Image user should not be 'root' High
Type: dockerfile
Vulnerability DS002
Severity: HIGH
Message: Specify at least 1 USER command in Dockerfile with non-root user as argument
Link: DS002
docker/Dockerfile.jupyterlab
Outdated
| @@ -0,0 +1,70 @@ | |||
| # Copyright (c) Nebari Development Team. | |||
Check notice
Code scanning / Trivy
No HEALTHCHECK defined Low
Type: dockerfile
Vulnerability DS026
Severity: LOW
Message: Add HEALTHCHECK instruction in your Dockerfile
Link: DS026
| @@ -0,0 +1,61 @@ | |||
| # Copyright (c) Nebari Development Team. | |||
Check failure
Code scanning / Trivy
Image user should not be 'root' High
Type: dockerfile
Vulnerability DS002
Severity: HIGH
Message: Specify at least 1 USER command in Dockerfile with non-root user as argument
Link: DS002
| @@ -0,0 +1,61 @@ | |||
| # Copyright (c) Nebari Development Team. | |||
Check notice
Code scanning / Trivy
No HEALTHCHECK defined Low
Type: dockerfile
Vulnerability DS026
Severity: LOW
Message: Add HEALTHCHECK instruction in your Dockerfile
Link: DS026
|
Will start looking into it today |
|
Hey @Helion55, I did a quick look at your PR. Since this will be an initial part of a big file move, I would like to disable or not copy the action files initially. Also, we will be making some changes this week in that repo to make things faster to build and more stable overall, so you will probably need to copy/modify the files again. In the meantime, to not leave you empty-handed, feel free to check any of our issues with |
|
Thanks @viniciusdc, Sure, I will do that. |
|
Hello, I have removed those two workflow files; kindly check the PR now. And I apologize for my delayed activity. |
|
Hey @Helion55 sorry I missed your message, thanks. I will need to have a look at this later on...though you might need to copy the files again since a lot has changed in that repo since then |
|
Sure @viniciusdc . I'll check and copy them. |
…rfile misconfigurations (#214) commit
|
Hi team, I have updated the branch with the latest commits from nebari-docker-images repo. Kindly check once. |
|
Thanks @Helion55, I will have time next week to look into this |
.github/ISSUE_TEMPLATE/RFD.md
Outdated
| @@ -0,0 +1,52 @@ | |||
| --- | |||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
These template files are not needed
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hi @viniciusdc, is it fine now?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes, thanks! I need to take a closer look at what we need to do internally to make this all work from here. I haven't had the time to do so yet, but I plan to check this later today or tomorrow, or next Wednesday.
|
Hey @Helion55 sorry this got quite shifted, I hope to have some time to look into it this Friday |
|
Also, I suggest you create feature branches in your Fork and create the PRs from those instead of from your main. This gets your git history clean and entirely in sync with ours. |
|
Hi @viniciusdc, Now I have created a PR for this from a separate feature branch. Kindly check once #3177. |
Reference Issues or PRs
Fixes #2871
What does this implement/fix?
Moved docker-images repo back to main nebari's repo. Changes made,
dockerand moved the necessary files from docker repository..github/workflowsfolder by adding thebuild-push-docker.yamlandtest-images.yamlfiles from docker repo's workflow folder.documentation.ymlandRFD.mdfiles to.github/ISSUE_TEMPLATEfolder..pre-commit-config.yamlfile.Testing
no
no
Any other comments?
no