Restarting PHP-FPM in Deploy Script

Best Answer

Level 65

Forge supplies a button to activate opcache now?

php-fpm is restarted with sudo service phpX-fpm reload. Just add that command to sudoers via sudo visudo

6 likes

Lars-Janssen

Level 33

@bashy if I activate OPCACHE on forge do I have to add sudo service phpX-fpm reload at the end of my deploy script aswel?

Thanks!

skylerkatz

Level 29

@bashy,

Yeah the functionality was released yesterday.

I think the issue is that I added it incorrectly to sudo visudo.

Here is what I have in the file now

forge ALL=NOPASSWD: /usr/sbin/service php7.1-fpm restart

Should that look like something else?

1 like

davielee

Level 11

@skylerkatz That looks right, but if you are worried you could disable opcache through the UI and re-enable it, but after that, what @bashy wrote should do the trick for you.

Level 65

@skylerkatz A reload should suffice else you'll get downtime from a restart I think.

1 like

skylerkatz

Level 29

Yeah, it turns out I was running the wrong command in the deployment script :)

willvincent

Level 54

You can also do it in php without having to restart the php process...

Just call:

opcache_reset();

Level 65

@willvincent And in a deployment system, how would you do that?

I also wouldn't "restart", just "reload"

1 like

willvincent

Level 54

@bashy I would probably wrap an artisan command around it that gets fired by the deploy script.

etelford

Level 5

There's also this: https://github.com/appstract/laravel-opcache

1 like

PhilippeTellier

Level 30

I don't think a package is really necessary for something that simple. Here is my Command class :

<?php

namespace App\Console\Commands\Opcache;

use Illuminate\Console\Command;
use App\Models\User;

class Clear extends Command
{
    /**
     * The name and signature of the console command.
     *
     * @var string
     */
    protected $signature = 'opcache:clear';

    /**
     * The console command description.
     *
     * @var string
     */
    protected $description = 'Clear the opcode cache';

    /**
     * Create a new command instance.
     *
     * @return void
     */
    public function __construct()
    {
        parent::__construct();
    }

    /**
     * Execute the console command.
     *
     * @return mixed
     */
    public function handle()
    {
        if (function_exists('opcache_reset') && opcache_reset()) {
            $this->info('Opcode cache cleared');
        } else {
            $this->line('Opcode cache: Nothing to clear');
        }
    }
}

3 likes

azimidev

Level 55

@PhilippeTellier Adding this in deploy script wont clear the opcache! This works for sure https://github.com/appstract/laravel-opcache if you however have Opcache running in forge you'll need to reset once.

shez1983

Level 23

@bashy sorry to dredge up this post - i can see forge has automatically? added a file

forge ALL=NOPASSWD: /usr/sbin/service php7.1-fpm reload
forge ALL=NOPASSWD: /usr/sbin/service php7.0-fpm reload
forge ALL=NOPASSWD: /usr/sbin/service php5.6-fpm reload
forge ALL=NOPASSWD: /usr/sbin/service php5-fpm reload

in /etc/sudoers.d but this stil gives me the error when i press deploy? i have used lots of variants:

sudo /usr/sbin/service php7.1-fpm reload sudo service php7.1-fpm reload service php-fpm reload

and every time i get interactive authentication required?

2 likes

Level 65

@shez1983 Sorry, I've never actually used Forge so I'm not sure what you're seeing/getting in response to your deployment.

coreproc

Level 20

@shez1983

sudo /usr/sbin/service php7.1-fpm reload should work.

I had the same problem a few minutes ago - turns out I was running "restart" instead of "reload"

2 likes

Level 65

@coreproc Already been marked as answered and your reply adds nothing. Restart is pretty much the same as reload but you drop client connections instantly...

aurawindsurfing