[config reload] Removed replace irreversibly job-mode for sonic.target restart

[vivekrnv]

Signed-off-by: Vivek Reddy Karri [email protected]

What I did

As discussed in this PR #1814 (comment), only the stop.job should have job-mode set to replace irreversibly.

Otherwise, simultaneous config reloads in the quick succession, can lead to the behavior.

Although ,when the restart job (and all the other dependent jobs) are finished and is taken out of systemd's job queue, the next stop job will not be cancelled.

admin@sonic:~$ sudo systemctl restart sonic.target --job-mode replace-irreversibly
admin@sonic:~$ sudo systemctl stop sonic.target --job-mode replace-irreversibly (Ran Immediately)
Failed to stop sonic.target: Transaction for sonic.target/stop is destructive (ntp-config.service has 'start' job queued, but 'stop' is included in transaction).
See system logs and 'systemctl status sonic.target' for details.
(After all the dependent jobs of restart sonic.target  are done, it works)
admin@sonic:~$ sudo systemctl stop sonic.target --job-mode replace-irreversibly
admin@sonic:

How I did it

How to verify it

Previous command output (if the output of a command-line utility has changed)

New command output (if the output of a command-line utility has changed)

[vivekrnv]

@stepanblyschak , @nazariig, @rajendra-dendukuri, @qiluo-msft Please review

[vivekrnv]

Required for 202012

[nazariig]

sonic-net/sonic-buildimage#8657

[qiluo-msft]

Do you need it on 202106 branch?

[vivekrnv]

Do you need it on 20202106 branch?

Yes