-
Notifications
You must be signed in to change notification settings - Fork 291
Update 07_using_efm_utility.mdx #6700
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
@EFM-Bobby, please review and let us know what you think? |
We use "superuser privileges" in other parts of the doc (including this page for the 'efm upgrade-conf'), so if we make the change it should be consistent. Can't other users also have the privileges to run these commands with sudo besides root? Looking through the file history, we used to say "root privileges" but changed it to the current "superuser privileges." I didn't put any information in there about why the change was made, but I don't think it was my idea. It'd be good to know why we switched to this before switching away from it. |
Hi,
efm ALL=(enterprisedb) NOPASSWD: /usr/edb/as17/bin/* Kind Regards, |
But that's not related to whether the term "superuser privileges" or "as root user" is used in the text, is it? |
In the current documentation workspace, I don't see any instances of "invoked as root" but I see nine instances of "invoked with superuser" (in the efm and epas products). I would prefer the documentation stay consistent unless there is some strong reason to change this. I also don't understand what this sentence means: 'Since "Superuser privilege" usually indicates the "Superuser" Attribute for roles...' We're talking about running a linux command. |
Here is the PR that added the current text. Note that this change came from a customer request and was approved by the docs team: #5256 So far I can't find any other information about this or who the customer was. |
Hi. Nah, I just answered your question. So it is currently not available to run these commands with sudo besides root, at least, not without extra setups which is not described in the document.
Kind Regards, |
Hi, This is just a precaution for those are not used to reading technical documents or Database newbies. Please think of change all 9 parts to "as root user". This is most explicit because it is the general term and familiar for Linux users.
Kind Regards, |
@EFM-Bobby, do we have any action item here? |
I don't really know. This isn't an efm question; it's about whether the documentation should use "as root user" instead of "superuser privileges." I just want things to be consistent, not just across the efm docs but across all of the product documentation. FWIW I don't think "as root user" is technically correct, as you can do things with superuser privileges (e.g. install something) without being root. It's unfortunate that the database and operating system both have the term "superuser" but we can't change that. As written, I do not approve of this PR that changes one instance of the wording. My suggestion is to close this PR, and @yuki-tei can start a different conversation with the docs team about this terminology. A quick search shows that "superuser privileges" is used ~100 times across multiple products. |
Thanks, @EFM-Bobby. I am closing this PR. @yuki-tei we can have separate discussion on this if needed. |
Hi team,
Greetings. Hope you are doing well and healthy.
What Changed?
Since "Superuser privilege" usually indicates the "Superuser" Attribute for roles, I think it is better to explicitly say "as a root" user, so this would avoid any potential misunderstanding.