Skip to content

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
Contributor

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.

@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 <[email protected]>
@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