Feat: add option to skip owner transfer of home-dir
まだ誰も着手していません。
評価
調査の方向性
まず、/home/coder の所有権が移譲される envbuilder.go の829-842行付近を読みます。既存のオプションがどのように定義され、このステップに渡されているかを追跡し、現在のデフォルト動作を維持したまま移譲をスキップするオプションを追加します。そのオプションを使用でき、有効にしたときに所有権の移譲ステップが省略されれば完了です。
索引モデルが issue の本文から書いたものです。
説明
Hey!
We see that some of our workspaces with large home-dirs uses quite a bit of time on the owner-transfer-step:
#3: 🔄 Updating ownership of /home/coder...
#3: 🏡 Updated ownership of /home/coder! [2m1.175064722s]
This might be a somewhat extreme case, where the home-dir has a couple of pipx-installs, and pyenv-venvs. Where the home-dir has been prebuild, and comes with the image.
In our case this is already owned by the coder-user, and the ownership does not seem to be shifted to root on envbuild, so in this case, the ownership-transfer seems to be redundant.
So tested with removing the ownership-transfer (https://github.com/coder/envbuilder/blob/7eabaa4d876c3ce54f7271b6e68fea37b6719c66/envbuilder.go#L829-L842), and could not see any issues with ownership in our case. So if envbuilder had an option to disable that, that would be great
- 主要言語
- Go
- スター
- 300
- フォーク
- 64
- 平均マージ
- 20分
- マージ済み PR(30日)
- 1
コントリビューションガイド
このリポジトリのコントリビューションガイドは索引されていません
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
coder/envbuilder のほかの issue
-
bug
難易度 1/5 1時間未満 初心者へのやさしさ 90/100
coder/envbuilder#506 ·
-
bug envbuilder needs-investigation
難易度 4/5 3〜5日 初心者へのやさしさ 35/100
coder/envbuilder#495 ·
-
envbuilder Feature
難易度 3/5 1〜2日 初心者へのやさしさ 56/100
coder/envbuilder#492 ·
-
難易度 4/5 3〜5日 初心者へのやさしさ 25/100
coder/envbuilder#484 ·
-
難易度 3/5 1〜2日 初心者へのやさしさ 38/100
coder/envbuilder#483 ·
coder/envbuilder の issue をすべて見る
似ている issue
-
難易度 1/5 1時間未満 初心者へのやさしさ 60/100
github/gh-aw-mcpg#13748 ·
-
agentic-workflows
難易度 2/5 1〜3時間 初心者へのやさしさ 65/100
-
needs-triage
難易度 2/5 1〜3時間 初心者へのやさしさ 75/100
-
難易度 2/5 1〜3時間 初心者へのやさしさ 65/100
-
難易度 2/5 1〜3時間 初心者へのやさしさ 75/100
googleapis/librarian#7670 · コメント 2 件 ·