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

Added question how to add ssh key in faq.md #285

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

amolvkharche
Copy link

No description provided.

@netlify
Copy link

netlify bot commented Feb 16, 2023

Deploy Preview for harvester-preview ready!

Name Link
🔨 Latest commit c6c73cd
🔍 Latest deploy log https://app.netlify.com/sites/harvester-preview/deploys/63f4594c6a421d0009107970
😎 Deploy Preview https://deploy-preview-285--harvester-preview.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site settings.

@bk201
Copy link
Member

bk201 commented Feb 20, 2023

@amolvkharche Thanks for the doc. It would be nice to also mention that updating /home/rancher/.ssh/authorized_keys can change the runtime setting (but it's not persistent between reboots)

@LucasSaintarbor LucasSaintarbor self-requested a review March 2, 2023 19:49
Copy link
Contributor

@LucasSaintarbor LucasSaintarbor left a comment

Choose a reason for hiding this comment

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

docs/faq.md Outdated Show resolved Hide resolved
docs/faq.md Outdated Show resolved Hide resolved
docs/faq.md Outdated Show resolved Hide resolved
docs/faq.md Outdated Show resolved Hide resolved
docs/faq.md Outdated Show resolved Hide resolved
docs/faq.md Outdated Show resolved Hide resolved
@LucasSaintarbor LucasSaintarbor removed the request for review from guangbochen March 13, 2023 21:27
@LucasSaintarbor
Copy link
Contributor

@guangbochen Is this PR still valid?

Co-authored-by: Lucas Saintarbor <lucas.saintarbor@suse.com>
@vickyhella vickyhella added this to the v1.2.0 milestone Aug 16, 2023
Copy link
Contributor

@Vicente-Cheng Vicente-Cheng left a comment

Choose a reason for hiding this comment

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

Just a nit, others LGTM.

To dynamically add users in the runtime without rebooting, add ssh authorized keys to /home/rancher/.ssh/authorized_keys. These changes are not persistent between reboots.

Option 2. Persistent
After you ssh login to Harvester, add ssh authorized keys to your /oem/99_custom.yaml or 90_custom.yaml. These changes are persistent between reboots.
Copy link
Contributor

Choose a reason for hiding this comment

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

I thought in the harvester cluster, we only have /oem/90_custom.yaml.
Maybe we could remove the /oem/99_custom.yaml?

@LucasSaintarbor LucasSaintarbor removed this from the v1.2.0 milestone Sep 7, 2023
@vickyhella vickyhella added this to the v1.2.0 milestone Sep 8, 2023
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.

6 participants