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

[BUG] php-fpm is not required #19

Closed
1 task done
killbus opened this issue Aug 25, 2023 · 5 comments · Fixed by #20
Closed
1 task done

[BUG] php-fpm is not required #19

killbus opened this issue Aug 25, 2023 · 5 comments · Fixed by #20

Comments

@killbus
Copy link

killbus commented Aug 25, 2023

Is there an existing issue for this?

  • I have searched the existing issues

Current Behavior

the php-fpm is up with the container.

Expected Behavior

drop the php-fpm service.

Steps To Reproduce

  1. docker-compose.yml
    ---
    version: "2.1"
    services:
      snapdrop:
        image: lscr.io/linuxserver/snapdrop:latest
        container_name: snapdrop
        restart: always
        environment:
          - PUID=1000
          - PGID=1000
          - TZ=Etc/UTC
  1. docker compose up -d
  2. docker compose exec snapdrop ps -aux

Environment

No response

CPU architecture

arm64

Docker creation

---
      version: "2.1"
      services:
        snapdrop:
          image: lscr.io/linuxserver/snapdrop:latest
          container_name: snapdrop
          restart: always
          environment:
            - PUID=1000
            - PGID=1000
            - TZ=Etc/UTC

Container logs

USER         PID %CPU %MEM    VSZ   RSS TTY      STAT START   TIME COMMAND
root           1  0.1  0.0    200    64 ?        Ss   06:24   0:00 /package/admi
root          15  0.0  0.0    204    68 ?        S    06:24   0:00 s6-supervise 
root          17  0.0  0.0    192     4 ?        Ss   06:24   0:00 /package/admi
root          38  0.0  0.0    204    68 ?        S    06:24   0:00 s6-supervise 
root          39  0.0  0.0    204    68 ?        S    06:24   0:00 s6-supervise 
root          40  0.0  0.0    204    64 ?        S    06:24   0:00 s6-supervise 
root          41  0.0  0.0    204    64 ?        S    06:24   0:00 s6-supervise 
root          42  0.0  0.0    204    68 ?        S    06:24   0:00 s6-supervise 
root          43  0.0  0.0    204    68 ?        S    06:24   0:00 s6-supervise 
root          51  0.0  0.0    180    48 ?        Ss   06:24   0:00 /package/admi
abc          235  2.0  0.2 243780 49760 ?        Ssl  06:24   0:00 /usr/bin/node
root         236  0.2  0.0  33732 14796 ?        Ss   06:24   0:00 php-fpm: mast
root         239  0.0  0.0   1692   984 ?        Ss   06:24   0:00 /usr/sbin/cro
root         241  0.1  0.0  18100  6328 ?        Ss   06:24   0:00 nginx: master
abc          251  0.0  0.0  18548  2608 ?        S    06:24   0:00 nginx: worker
abc          252  0.0  0.0  18548  2608 ?        S    06:24   0:00 nginx: worker
abc          253  0.0  0.0  18548  2608 ?        S    06:24   0:00 nginx: worker
abc          254  0.0  0.0  18548  2236 ?        S    06:24   0:00 nginx: worker
abc          265  0.0  0.0  33740  3504 ?        S    06:24   0:00 php-fpm: pool
abc          266  0.0  0.0  33740  3504 ?        S    06:24   0:00 php-fpm: pool
root         277 33.3  0.0   3200  2280 pts/0    Rs+  06:24   0:00 ps -aux
@github-actions
Copy link

Thanks for opening your first issue here! Be sure to follow the relevant issue templates, or risk having this issue marked as invalid.

@Roxedus
Copy link
Member

Roxedus commented Aug 25, 2023

This is a artifact of us using our nginx base image, where php-fpm is running by default

@killbus
Copy link
Author

killbus commented Aug 25, 2023

This is a artifact of us using our nginx base image, where php-fpm is running by default

I did check the base image. It does. Should we get rid of it?

@Roxedus Roxedus mentioned this issue Aug 25, 2023
1 task
@Roxedus
Copy link
Member

Roxedus commented Aug 25, 2023

Please test the PR artifact from #20.

lspipepr/snapdrop:eac78009-pkg-41fa80c4-dev-98c0a6e2a1b21f6b401a77354672513b4b7deb13-pr-20

@killbus
Copy link
Author

killbus commented Aug 25, 2023

Please test the PR artifact from #20.

lspipepr/snapdrop:eac78009-pkg-41fa80c4-dev-98c0a6e2a1b21f6b401a77354672513b4b7deb13-pr-20

Tested. It works as expected. 👍

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Archived in project
Development

Successfully merging a pull request may close this issue.

2 participants