Skip to content

Make Docker build successful and update base image#305

Merged
kou merged 9 commits into
red-data-tools:mainfrom
tikkss:fix-build-docker
Apr 28, 2025
Merged

Make Docker build successful and update base image#305
kou merged 9 commits into
red-data-tools:mainfrom
tikkss:fix-build-docker

Conversation

@tikkss
Copy link
Copy Markdown
Contributor

@tikkss tikkss commented Apr 27, 2025

This PR includes the following changes:

  • Replace some deprecated for Docker
  • Use Quay.io instead of Docker Hub
    • Because Jupyter images hosted on Docker Hub are no longer updated
  • Fix some missing file error
  • Update latest jupyter/minimal-notebook base image

tikkss added 9 commits April 27, 2025 06:32
Because the `version` field is no longer required in Docker Compose v2.
This patch will suppress the following warning:

```console
$ docker compose build
WARN[0000] the attribute `version` is obsolete, it will be ignored,
please remove it to avoid potential confusion
```
Because Jupyter images hosted on Docker Hub are no longer updated.
This patch will resolve the following error:

```console
$ docker compose build
(snip)
failed to solve: jupyter/minimal-notebook:3692e3a84cd6: failed to
resolve source metadata for
docker.io/jupyter/minimal-notebook:3692e3a84cd6:
docker.io/jupyter/minimal-notebook:3692e3a84cd6: not found
```
Since e04839b, Gemfile.lock file is deleted.
This patch will resolve the following error:

```console
$ docker compose build
(snip)
 => ERROR [minimal-redamber 7/9] COPY --chown=1000:100 Gemfile.lock
 /home/jovyan
```
Since e04839b, example script is moved to bin directory. bin directory
is not accessible in the container. Because compose.yml's context is `.`
(`.` means docker directory).
This patch will resolve an occurred error while installing psych.
@kou
Copy link
Copy Markdown
Member

kou commented Apr 27, 2025

Is the build container / build failure unrelated?

@tikkss
Copy link
Copy Markdown
Contributor Author

tikkss commented Apr 27, 2025

Yes, unrelated. The build container / build failure that root cause is may unable to iruby register --force in

$HOME/.rbenv/bin/rbenv exec iruby register --force
. Because jupyter kernel for ruby is not found as follows:

https://github.com/red-data-tools/red_amber/actions/runs/14686571719/job/41215968467?pr=305#step:4:34464

[2025-04-27T00:43:57.085Z] ERROR: Jupyter kernel 'ruby' not found. Known kernels: python3. Run 'quarto check jupyter' with your python environment activated to check python version used.

I would like to open a separate PR.

@kou kou merged commit 7f48138 into red-data-tools:main Apr 28, 2025
8 of 10 checks passed
@kou
Copy link
Copy Markdown
Member

kou commented Apr 28, 2025

OK. Let's work on it a separated task.

@tikkss
Copy link
Copy Markdown
Contributor Author

tikkss commented Apr 28, 2025

Thanks for your review!

@tikkss tikkss deleted the fix-build-docker branch April 28, 2025 03:06
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.

2 participants