Skip to content

wip: envoy gatewayの章の追加 - #555

Open
csenet wants to merge 4 commits into
mainfrom
add-envoy-gateway
Open

wip: envoy gatewayの章の追加#555
csenet wants to merge 4 commits into
mainfrom
add-envoy-gateway

Conversation

@csenet

@csenet csenet commented Apr 7, 2026

Copy link
Copy Markdown
Member

No description provided.

@gemini-code-assist gemini-code-assist Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Code Review

This pull request introduces a new hands-on chapter for Envoy Gateway, covering installation, various HTTP routing strategies (basic, path-based, header-based, and weighted), external service routing using the Backend resource, and health check policies. It also includes a detailed section on debugging with the egctl CLI tool. Feedback focuses on improving the tutorial's robustness: specifically, ensuring port 8888 is accessible in cloud environments, correctly handling the installation path of the egctl binary, and providing a command to clean up background port-forward processes during the final step.

Comment thread chapter_envoy-gateway/README.md Outdated

```sh
export ENVOY_SERVICE=$(kubectl get svc -n envoy-gateway-system --selector=gateway.envoyproxy.io/owning-gateway-namespace=envoy-gateway-handson,gateway.envoyproxy.io/owning-gateway-name=eg -o jsonpath='{.items[0].metadata.name}')
kubectl -n envoy-gateway-system port-forward service/${ENVOY_SERVICE} 8888:8080 &

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

medium

ここで使用しているポート 8888 は、chapter_setup/README.md のセキュリティグループ作成手順(25行目)で許可されているポートリストに含まれていません。AWS 等のクラウド環境で VM を作成して実施している場合、ローカルマシンのブラウザや curl からこのポートにアクセスできず、接続エラーになります。セキュリティグループの設定に 8888 を追加で許可するか、既に許可されているポート(例: 18080)を使用するように手順を調整してください。

### egctlのインストール

```sh
curl -fsSL https://gateway.envoyproxy.io/get-egctl.sh | VERSION=v1.7.1 bash

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

medium

get-egctl.sh スクリプトは、デフォルトでカレントディレクトリの ./bin/egctl にバイナリをインストールします。そのため、その後の手順(625行目など)で egctl を直接実行すると「command not found」エラーになります。バイナリをパスの通ったディレクトリ(/usr/local/bin など)に移動する手順を追加するか、実行時のパスを ./bin/egctl に修正してください。

Suggested change
curl -fsSL https://gateway.envoyproxy.io/get-egctl.sh | VERSION=v1.7.1 bash
curl -fsSL https://gateway.envoyproxy.io/get-egctl.sh | VERSION=v1.7.1 bash
sudo mv ./bin/egctl /usr/local/bin/


ハンズオンで作成したリソースを削除します。

```sh

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

medium

セットアップセクション(131行目)でバックグラウンド実行した port-forward プロセスを停止する手順が含まれていません。ハンズオン終了後もプロセスが残り続けるのを防ぐため、リソースの削除と合わせてプロセスを停止するコマンドを追加することを推奨します。

Suggested change
```sh
# port-forward プロセスの停止
pkill -f "port-forward.*8888:8080"
kubectl delete -f manifests/backend-traffic-policy.yaml --ignore-not-found=true

csenet and others added 3 commits April 7, 2026 15:01
* feat: 新しいコンテナポート32081を追加し、ホストポート8081を設定

* fix: Gatewayのポート番号を8080から8081に変更
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