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

Arch Linux: Fix package names and set correct php-fpm user (root -> http) #663

Merged
merged 3 commits into from
Jul 3, 2022

Conversation

jkroepke
Copy link
Contributor

@jkroepke jkroepke commented Jul 2, 2022

Pull Request (PR) description

This Pull Request (PR) fixes the following issues

@jkroepke jkroepke force-pushed the fix-arch branch 7 times, most recently from 0485da0 to 32979a9 Compare July 2, 2022 23:23
@jkroepke
Copy link
Contributor Author

jkroepke commented Jul 3, 2022

From logs, I can't see any error here. The error seems related to the extensions gd/intl, the daemon is running without them.

@bastelfreak
Copy link
Member

Very odd, the php-fpm service is working fine after the restart 🤔
2022-07-03-094547_1715x760_scrot

@bastelfreak
Copy link
Member

it's trying to rerestart php-fpm with service, which is wrong:

  Debug: /Stage[main]/Php/Php::Extension[gd]/Php::Extension::Config[gd]/Php::Config[gd]/Php::Config::Setting[/etc/php/conf.d/gd.ini: extension]/Ini_setting[/etc/php/conf.d/gd.ini: extension]: The container Php::Config::Setting[/etc/php/conf.d/gd.ini: extension] will propagate my refresh event
  Debug: Php::Config::Setting[/etc/php/conf.d/gd.ini: extension]: The container Php::Config[gd] will propagate my refresh event
  Debug: Php::Config[gd]: The container Php::Extension::Config[gd] will propagate my refresh event
  Debug: Php::Extension::Config[gd]: The container Php::Extension[gd] will propagate my refresh event
  Debug: Php::Extension[gd]: The container Class[Php] will propagate my refresh event
  Info: Php::Extension[gd]: Scheduling refresh of Service[php-fpm]
  Debug: Executing: '/usr/bin/systemctl is-active -- php-fpm'
  Debug: Executing: '/usr/bin/systemctl is-enabled -- php-fpm'
  Debug: Executing: '/usr/bin/systemctl is-active -- php-fpm'
  Debug: Executing: '/usr/bin/systemctl show --property=NeedDaemonReload -- php-fpm'
  Debug: Executing: 'service php-fpm reload'
  Debug: Running journalctl command to get logs for systemd restart failure: journalctl -n 50 --since '5 minutes ago' -u php-fpm --no-pager
  Debug: Executing: 'journalctl -n 50 --since '5 minutes ago' -u php-fpm --no-pager'
  Error: /Service[php-fpm]: Failed to call refresh: Systemd restart for php-fpm failed!
  journalctl log for php-fpm:
  Jul 03 07:51:11 archlinuxrolling-64-1 systemd[1]: Starting The PHP FastCGI Process Manager...
  Jul 03 07:51:11 archlinuxrolling-64-1 php-fpm[5105]: [NOTICE] fpm is running, pid 5105
  Jul 03 07:51:11 archlinuxrolling-64-1 php-fpm[5105]: [NOTICE] ready to handle connections
  Jul 03 07:51:11 archlinuxrolling-64-1 systemd[1]: Started The PHP FastCGI Process Manager.
  Jul 03 07:51:11 archlinuxrolling-64-1 php-fpm[5105]: [NOTICE] systemd monitor interval set to 10000ms
  Error: /Service[php-fpm]: Systemd restart for php-fpm failed!
  journalctl log for php-fpm:

@jkroepke
Copy link
Contributor Author

jkroepke commented Jul 3, 2022

If extensions are installed here, why is restarting wrong?

Edit: nvm, I re-read your comment and see the commit. Got it.

@bastelfreak
Copy link
Member

@jkroepke I pushed the systemd change in another PR. could you rebase your allow-unsupported against my fpm branch, and then rebase your fix-arch branch against your allow-unsupported? That will give us three seperate PRs where #663 has a clean CI.

@jkroepke jkroepke force-pushed the fix-arch branch 2 times, most recently from e5ac8ad to febe083 Compare July 3, 2022 08:11
@jkroepke
Copy link
Contributor Author

jkroepke commented Jul 3, 2022

Thanks god, the gh cli supports pr checkouts from upstream repositories flawless...

@bastelfreak bastelfreak changed the title fix arch tests Arch Linux: Fix package names and set correct php-fpm user (root -> http) Jul 3, 2022
@bastelfreak bastelfreak added the bug Something isn't working label Jul 3, 2022
@jkroepke
Copy link
Contributor Author

jkroepke commented Jul 3, 2022

Mention: I also disable pear on Arch, since it's not available no main repositories (missing in the title)

@bastelfreak bastelfreak merged commit 81165b9 into voxpupuli:master Jul 3, 2022
@jkroepke jkroepke deleted the fix-arch branch July 3, 2022 08:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants