Skip to content

Conversation

alemorvan
Copy link
Contributor

@alemorvan alemorvan commented Jul 9, 2023

Documentation about nfsserver

Author checklist (Completed by original Author)

  • Good fit for the Rocky Linux project? Title and Author Metatags inserted ?
  • If applicable, steps and instructions have been tested to work
  • Initial self-review to fix basic typos and grammar completed

Rocky Documentation checklist (Completed by Rocky team)

  • 1st Pass (Document is good fit for project and author checklist completed)
  • 2nd Pass (Technical Review - check for technical correctness)
  • 3rd Pass (Detailed Editorial Review and Peer Review)
  • Final approval (Final Review)

Documentation about nfsserver

## Client configuration

Access to shared resources on an NFS server is done by mount point on the client.
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I don't know how to explain that in english.
It means you have to create a mount point on the client side to access to the data on server.

Copy link
Contributor

@SergeCroise SergeCroise Jul 9, 2023

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Shared resources on an NFS server are accessible through the mount point on the client.
(serait plus facile à traduire?)

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

through a mount point
or
through the mount point

Copy link
Contributor

@SergeCroise SergeCroise left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hello @alemorvan and @sspencerwire ,
Please review my proposals, feel free to overwrite my changes!


## Client configuration

Access to shared resources on an NFS server is done by mount point on the client.
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I don't know how to explain that in english.
It means you have to create a mount point on the client side to access to the data on server.

@alemorvan
Copy link
Contributor Author

Thx @SergeCroise I have validated your proposals.

Copy link
Contributor

@SergeCroise SergeCroise left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@alemorvan , @sspencerwire
please review

sspencerwire and others added 3 commits July 9, 2023 08:06
Those darn spaces!

Co-authored-by: Serge Croisé <[email protected]>
wording change

Co-authored-by: Serge Croisé <[email protected]>
@sspencerwire
Copy link
Contributor

@alemorvan this looks good! I've committed @SergeCroise suggestions, as they were mostly cosmetic and fixed space issues, etc. (some wording changes also were approved). I'm going to convert this to a draft and will plan on doing a little more editing tomorrow (Monday). By converting it to a draft it just means it won't be accidentally merged before the editing has been done.

@sspencerwire sspencerwire marked this pull request as draft July 9, 2023 13:14
wsoyinka
wsoyinka previously approved these changes Jul 9, 2023
Copy link
Member

@wsoyinka wsoyinka left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you @alemorvan :-) !! Beautiful contribution

@github-actions
Copy link

github-actions bot commented Jul 9, 2023

Test results for 88c2e64:

Number of broken URLs: 4

URL,RESULT,FILENAME
 http://www.flaviosiqueira.com.br/,failed,guides/contribute/README.md
 https://ersei.saggis.com/,failed,guides/contribute/README.md
 https://github.com/andrewthiesen,failed,guides/contribute/README.md
 https://github.com/bdacus01,failed,guides/contribute/README.md

@sspencerwire sspencerwire changed the base branch from main to nfserver_edit July 10, 2023 11:20
@sspencerwire sspencerwire marked this pull request as ready for review July 10, 2023 11:21
@sspencerwire sspencerwire merged commit ff519d3 into rocky-linux:nfserver_edit Jul 10, 2023
sspencerwire added a commit that referenced this pull request Jul 12, 2023
* Create nfsserver.md (#1335)

* Create nfsserver.md

Documentation about nfsserver

* Apply suggestions from code review

Thx @serge

Co-authored-by: Serge Croisé <[email protected]>

* Update docs/guides/file_sharing/nfsserver.md

Those darn spaces!

Co-authored-by: Serge Croisé <[email protected]>

* Update docs/guides/file_sharing/nfsserver.md

Co-authored-by: Serge Croisé <[email protected]>

* Update docs/guides/file_sharing/nfsserver.md

wording change

Co-authored-by: Serge Croisé <[email protected]>

* Update nfsserver.md

---------

Co-authored-by: Serge Croisé <[email protected]>
Co-authored-by: sspencerwire <[email protected]>
Co-authored-by: wale soyinka <[email protected]>

* NFS server edits:
* remove most (not all) passive voice
* remove punctuation on bullet points except those under "Case studies" which are a command parameters with a qualifying sentence.
* some minor wording changes for better understanding (for instance "conserved" becomes "preserves")
* added Serge to contributors

* Update nfsserver.md

Some changes not saved... edited to put them back in.

* Update docs/guides/file_sharing/nfsserver.md

Co-authored-by: Serge Croisé <[email protected]>

---------

Co-authored-by: Antoine Le Morvan <[email protected]>
Co-authored-by: Serge Croisé <[email protected]>
Co-authored-by: wale soyinka <[email protected]>
@alemorvan alemorvan deleted the feature/nfsserver branch December 28, 2023 13:05
@alemorvan alemorvan restored the feature/nfsserver branch December 28, 2023 13:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants