Movatterモバイル変換

[0]ホーム
URL:

Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

 Sign up
Appearance settings

chore: update template examples and docs to install code-server under /tmp#6138

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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Merged
johnstcn merged 5 commits intomainfromcj/docs-install-code-server-tmp
Feb 10, 2023

Conversation

johnstcn
Copy link
Member

This PR updates the example templates to install code-server under /tmp using the standalone method, which should require no root privileges.

We found that the default Kubernetes template in OpenShift would fail to install code-server due to lack of sudo access.

Copy link
Member

@mafredrimafredri left a comment•
edited
Loading

Choose a reason for hiding this comment

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

Two (potential) issues with storing in/tmp:

  1. It might be mountedtmpfs in which case it'll use up RAM
  2. Could be mountednoexec which is a somewhat common security measure

Edit: Only meant this as a "to keep in mind", don't see it as a real blocker.

Copy link
Member

@mafredrimafredri left a comment

Choose a reason for hiding this comment

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

👌🏻

Co-authored-by: Mathias Fredriksson <mafredri@gmail.com>
@johnstcnjohnstcn merged commitab9cba9 intomainFeb 10, 2023
@johnstcnjohnstcn deleted the cj/docs-install-code-server-tmp branchFebruary 10, 2023 13:36
@github-actionsgithub-actionsbot locked and limited conversation to collaboratorsFeb 10, 2023
Sign up for freeto subscribe to this conversation on GitHub. Already have an account?Sign in.
Reviewers

@mafredrimafredrimafredri approved these changes

@sharkymarksharkymarkAwaiting requested review from sharkymark

@ericpaulsenericpaulsenAwaiting requested review from ericpaulsen

@bpmctbpmctAwaiting requested review from bpmct

Assignees

@johnstcnjohnstcn

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

2 participants
@johnstcn@mafredri
[8]ページ先頭
©2009-2025 Movatter.jp