-
-
Notifications
You must be signed in to change notification settings - Fork 5.2k
Add caution in system+semaphore #13035
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
Conversation
* 5.0: Update error_handler.rst
This PR was merged into the 4.3 branch. Discussion ---------- Update argument_value_resolver.rst Commits ------- 0e7a1ec Update argument_value_resolver.rst
* 4.3: Update argument_value_resolver.rst
* 4.4: Update argument_value_resolver.rst
* 5.0: Update argument_value_resolver.rst
…erface (atailouloute) This PR was merged into the 5.0 branch. Discussion ---------- Make custom routers compatible with the LoaderInterface Commits ------- a49af0d Make custom routers compatible with the LoaderInterface
* 5.0: Make custom routers compatible with the LoaderInterface
This PR was merged into the 4.3 branch. Discussion ---------- Update routing.rst - Fix wrong XML Commits ------- 3d0fed0 Update routing.rst - Fix wrong XML
This PR was merged into the 4.3 branch. Discussion ---------- Update routing.rst Commits ------- 085f5fb Update routing.rst
This PR was merged into the 4.3 branch. Discussion ---------- Update routing.rst Remove missing return Commits ------- 2052995 Update routing.rst
This PR was submitted for the 5.0 branch but it was merged into the 4.3 branch instead (closes symfony#12839). Discussion ---------- Update event_dispatcher.rst Commits ------- 20520f2 Update event_dispatcher.rst
* 4.3: Update event_dispatcher.rst Update routing.rst
* 4.4: Update event_dispatcher.rst Update routing.rst
* 5.0: Update event_dispatcher.rst Update routing.rst
Grammar.
This PR was submitted for the 5.0 branch but it was merged into the 4.3 branch instead (closes symfony#12878). Discussion ---------- Update doctrine.rst Grammar. Commits ------- c23db39 Update doctrine.rst
* 4.3: Update doctrine.rst
* 4.4: Update doctrine.rst
* 5.0: Update doctrine.rst
* 3.4: Add a note about voters with access_control
* 4.3: Add a note about voters with access_control
* 4.4: Add a note about voters with access_control
* 5.0: Add a note about voters with access_control
* 3.4: Use HttpKernel Component
* 3.4: Use HttpKernel Component
* 4.4: Use HttpKernel Component
* 5.0: Use HttpKernel Component
Following the documentation, we check the Calendar application and get the string "Yep, this is a leap year!" instead of "Hello Fabien".
* 4.3: Removed leftover 3.4 versionadded directive
* 4.4: Removed leftover 3.4 versionadded directive
* 5.0: Removed leftover 3.4 versionadded directive
* 3.4: Added the other two removed component articles to the redirection map
* 4.3: Added the other two removed component articles to the redirection map
* 4.4: Added the other two removed component articles to the redirection map
* 5.0: Added the other two removed component articles to the redirection map
Hello,
Also, RemoveIPC=yes is the default value on some Linux distributions, I don't know what you mean by
I think it is a system wide parameter? (but I am not 100% sure of this). Finally, I do not think it is related to cron, I just tested it with cron, and people in the original thread were all using a cron. So I wrote explanations for their cases. You just need to have a user that logs in and logs out for systemd to trigger the removal of IPC resources. I hope it is clearer ? I can try to reproduce it without a cron if you want. |
removed link to issue |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Minor comments
@@ -631,6 +631,14 @@ can be two running containers in parallel. | |||
concurrent process on a new machine, check that other process are stopped | |||
on the old one. | |||
|
|||
.. caution:: | |||
|
|||
When runing on systemd with non-system user with option ``RemoveIPC=yes`` |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
with non-system user and option...
?
|
||
When runing on systemd with non-system user with option ``RemoveIPC=yes`` | ||
(default value), locks are deleted by systemd when that user logs out. | ||
Check that process is ran with a system user (UID <= SYS_UID_MAX) with |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
-ran
+run
* 3.4: minor. refs #13035 Add caution in system+semaphore
* 4.4: minor. refs #13035 Add caution in system+semaphore
* 5.0: minor. refs #13035 Add caution in system+semaphore
Add a warning in documentation about using semaphore + systemd + non-system user
see symfony/symfony#29952 (comment)