Skip to content
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

Update rdp-server.md to add instructions to add EPEL repository #2647

Merged
merged 1 commit into from
Mar 12, 2025

Conversation

rjys365
Copy link
Contributor

@rjys365 rjys365 commented Mar 12, 2025

Add instructions to add EPEL repository before installing xrdp

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)

Add instructions to add EPEL repository before installing xrdp
@rjys365
Copy link
Contributor Author

rjys365 commented Mar 12, 2025

The guide involves installing xrdp, a package in EPEL, but didn't mention how to add it, which may lead to confusion to beginners. I added the info on this before the instruction to install xrdp.

Btw, is there some templates to instruct how to add EPEL repository so that it can be easily added before any instructions on EPEL software installation?

Copy link

Test results for 10518fa:

Number of broken URLs: 1

URL,RESULT,FILENAME
 http://node-0:$,failed,labs/kubernetes-the-hard-way/lab12-smoke-test.md

@wsoyinka
Copy link
Member

The guide involves installing xrdp, a package in EPEL, but didn't mention how to add it, which may lead to confusion to beginners. I added the info on this before the instruction to install xrdp.

Btw, is there some templates to instruct how to add EPEL repository so that it can be easily added before any instructions on EPEL software installation?

Thank you @rjys365 :-)

We don't have any such templates. Also it might be an overkill to include it or expect it to be included it in every doc/guide. We leave that decision up to the original author.

@rjys365
Copy link
Contributor Author

rjys365 commented Mar 12, 2025

Also it might be an overkill to include it or expect it to be included it in every doc/guide.

@wsoyinka I see, that sounds reasonable. I'm just kinda curious of whether that kinds of template exists so that I'm not doing redundant work. :)

However, the original tutorial didn't mention anything about EPEL, and the following statement suggests that the tutorial may be targeted to all Rocky Linux users including beginners:

For beginners, you will be using RDP. RDP stands for Remote Desktop Protocol, and it does exactly what that implies...

You need an RDP server to make your Rocky Linux desktop-accessible remotely. For our purposes, 'xrdp' will more than suffice. You'll need to use the terminal for it, though, as it's a CLI-only program.

So (suppose that) when some beginner on a default Rocky install types sudo dnf install xrdp into the shell and see the error message Error: Unable to find a match: xrdp , I guess it would be somehow confusing. So I wonder if it is appropriate to add my short piece about how to add EPEL into the tutorial.

Thanks for your time and patience.

Copy link
Contributor

@sspencerwire sspencerwire 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 @rjys365

@sspencerwire
Copy link
Contributor

@all-contributors add @rjys365 for content

Copy link
Contributor

@sspencerwire

I've put up a pull request to add @rjys365! 🎉

@wsoyinka
Copy link
Member

Also it might be an overkill to include it or expect it to be included it in every doc/guide.

@wsoyinka I see, that sounds reasonable. I'm just kinda curious of whether that kinds of template exists so that I'm not doing redundant work. :)

However, the original tutorial didn't mention anything about EPEL, and the following statement suggests that the tutorial may be targeted to all Rocky Linux users including beginners:

For beginners, you will be using RDP. RDP stands for Remote Desktop Protocol, and it does exactly what that implies...

You need an RDP server to make your Rocky Linux desktop-accessible remotely. For our purposes, 'xrdp' will more than suffice. You'll need to use the terminal for it, though, as it's a CLI-only program.

So (suppose that) when some beginner on a default Rocky install types sudo dnf install xrdp into the shell and see the error message Error: Unable to find a match: xrdp , I guess it would be somehow confusing. So I wonder if it is appropriate to add my short piece about how to add EPEL into the tutorial.

Thanks for your time and patience.

Yes, you are right. We'll merge this. Thank you for taking the time to contribute this improvement :-)

p.s. - I've always been curious to know if we could make use of templates (or macros) in our documentation. It'll save authors some time and help with standardization too. If you want to volunteer to help us research and look into this, the starting point will be https://squidfunk.github.io/mkdocs-material/ . We rely on mkdocs-material. (mkdocs-material is a documentation framework built on top of MkDocs).

Cheers.

@sspencerwire sspencerwire merged commit 443c859 into rocky-linux:main Mar 12, 2025
2 of 3 checks passed
@rjys365
Copy link
Contributor Author

rjys365 commented Mar 12, 2025

Thank you for your approval. I currently don't have much time to commit myself to the edition of the tutorial wiki, but I would check the documentation you mentioned if I have more spare time some day. The awesome guides of Rocky linux such as WordPress installation have helped me a lot and it is my pleasure to contribute to your project.

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.

3 participants