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 for Ubuntu Server 22.04 LTS #460

Merged
merged 3 commits into from
May 2, 2024

Conversation

yasumichi
Copy link
Contributor

最新の Ubuntu Server の LTS バージョンと GROWI v7.0.2 に合わせて、インストール方法を修正してみました。

  • Toolchain のアップデート
    • node.js 18.x or 20.x
    • MongoDB 6.0
    • OpenJDK 17
    • Elasticsearch 7.x or 8.x
  • Turborepo のインストールについて追加
  • git-lfs のインストールについて追加
  • apt-key の deprecated に伴う修正(apt-key → gpg)
  • apt-get → apt
  • dpkg-query の使用方法の一部変更
  • Elasticsearch 関連
    • インストール時に表示されるパスワード情報について追加
    • TLS の設定無効化について追加
    • Elasticsearch の起動タイミングを設定変更後に修正
    • Elasticsearch 8.x の設定ファイルに既存のヒープメモリの設定がないため、表記を変更

参考リンク

* yarn
* MongoDB 3.x
* \(Option\) Elasticsearch 5.x
* MongoDB 4.4 以上
Copy link
Contributor

Choose a reason for hiding this comment

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

6.0 以上を推奨という旨も併記してください

Copy link
Contributor Author

Choose a reason for hiding this comment

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

  • MongoDB 4.4 以上 (6.0 以上を推奨)

に修正しました。

```

## GROWI

### Git LFS の導入

GROWI では、フォントファイルを [Git Large File Storage (LFS)](https://git-lfs.com/)で管理しています。そのため、`git-lfs` がインストールされていない状態で `git clone` を行うと正しくビルドが行えません。
Copy link
Contributor

Choose a reason for hiding this comment

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

今後フォントファイルに限らない可能性があるので、単に git-lfs 管理のファイルの取得に必要、という主旨に変えてください。「フォントファイルなど」のような書き方であれば入れても OK です。

Copy link
Contributor Author

Choose a reason for hiding this comment

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

「フォントファイルを」→「フォントファイルなどを」に修正しました。

@@ -280,8 +343,7 @@ npm start

...
# 以下のメッセージが表示されるまでしばらく待つ
> growi@3.1.9 server:prod /opt/growi
> env-cmd config/env.prod.js node app.js
{"name":"growi:crowi","hostname":"growi-server","pid":29259,"level":30,"msg":"[production] Express server is listening on port 3000","time":"2024-04-30T21:50:05.549Z","v":0}
Copy link
Contributor

@yuki-takei yuki-takei May 2, 2024

Choose a reason for hiding this comment

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

ここは好みですが、json 形式でのログを好まない場合は apps/app/.env.production.local を新規作成し以下を記述することでフォーマットされたログを出力できます。

FORMAT_NODE_LOG=true

https://docs.growi.org/ja/admin-guide/admin-cookbook/env-vars.html

Copy link
Contributor Author

Choose a reason for hiding this comment

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

ご教示ありがとうございます。なるほど、そういう手もあるのですね。
個人的には、環境変数か、ロギングに入れ込みたい気がします。

@yuki-takei
Copy link
Contributor

@yasumichi PR ありがとうございます。こういった活動は大変ありがたいです。

フィードバック内容をコメントしたので確認いただけますでしょうか。
また textlint job が失敗しているので、そちらも確認の上で表現の修正をお願いします。

@yasumichi
Copy link
Contributor Author

yasumichi commented May 2, 2024

フィードバックありがとうございます。以下の修正を行いました。

  • 「MongoDB 4.4 以上」に「(6.0 以上を推奨)」を補足しました。
  • Git LFS の説明で「フォントファイルを」→「フォントファイルなどを」に修正しました。
  • Textlint で指摘のあった「通信を行う」を「通信する」に修正しました。

ログの件は、原状のままとしています。

@yuki-takei yuki-takei merged commit 8e88342 into growilabs:master May 2, 2024
2 checks passed
@yuki-takei
Copy link
Contributor

@yasumichi yasumichi deleted the update-ubuntu branch May 2, 2024 11:33
@yasumichi
Copy link
Contributor Author

Merge ありがとうございました。公開されたことを確認できました。

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.

2 participants