cloudnativedaysjp / dreamkast-ui

MIT License
7 stars 2 forks source link

Update for CNDF2023 #445

Closed hrk091 closed 1 year ago

hrk091 commented 1 year ago

CNDF2023向けに、以下を実施しました。

環境変数が追加されています。staging/prodへのrelease前に、 NEXT_PUBLIC_WEAVER_URL を各環境のmanifestに追加する必要があります。

github-actions[bot] commented 1 year ago

Review app

hrk091 commented 1 year ago

IVS建てないと結合試験できないので、dk-weaverとの結合時には以下のような処理をdk-weaver側に突っ込んでstabとして動作させて検証してます(trackIDは、CICD2023のテストデータと合致する値です)

diff --git a/internal/dkui/dkui.go b/internal/dkui/dkui.go
index d867b32..bab8118 100644
--- a/internal/dkui/dkui.go
+++ b/internal/dkui/dkui.go
@@ -273,6 +273,34 @@ func (v *ServiceImpl) ListViewerCounts(ctx context.Context, confName value.ConfN
        if err != nil {
                return nil, err
        }
+       arn, _ := value.NewChannelArn("arnawsivs:us-west-2326937407773channel/0Y1lcs4U7jk5")
+
+       dvc.Items = append(dvc.Items, *domain.NewViewerCount(
+               newTrackID(38),
+               arn,
+               newTrackName("A"),
+               10,
+       ), *domain.NewViewerCount(
+               newTrackID(39),
+               arn,
+               newTrackName("B"),
+               20,
+       ), *domain.NewViewerCount(
+               newTrackID(40),
+               arn,
+               newTrackName("C"),
+               30,
+       ))

        return dvc, nil
 }
+
+func newTrackID(v int32) value.TrackID {
+       trackID, _ := value.NewTrackID(v)
+       return trackID
+}
+
+func newTrackName(v string) value.TrackName {
+       name, _ := value.NewTrackName(v)
+       return name
+}
ystkfujii commented 1 year ago

Review app

起動時にerrorが発生してそうですー

https://argocd.dev.cloudnativedays.jp/applications/argocd/dreamkast-ui-445?view=tree&node=%2FPod%2Fdreamkast-ui-445%2Fdreamkast-ui-875856c5c-2msw8%2F0&tab=events&resource=sync%3ASynced%2Csync%3AOutOfSync

ystkfujii commented 1 year ago

dk-weaverとの結合テストの観点では、LIVEの数字が取得できたので良さそうでした! コードの修正内容は見れてないので、どなたかレビューお願いします 🙇

image

hrk091 commented 1 year ago

起動時にerrorが発生してそうですー

環境変数が足りてないからですね。 追加しますー

hrk091 commented 1 year ago

reviewありがとうございます! mergeしますー