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

Okteto with helm #8335

Merged
merged 1 commit into from
May 15, 2022
Merged

Okteto with helm #8335

merged 1 commit into from
May 15, 2022

Conversation

ikuradon
Copy link
Contributor

What

Oktetoでデプロイテストできるようにした

Why

Resolve #8331

Additional info (optional)

まだ標準の設定ファイルコピーしているだけなのでドメインの処理とかおかしいはず

PR時に動くようにするには次のファイルが必要(ユーザー名設定とかあるのでコミットには含めていない)
また、OKTETO_TOKENをActionsのSecretに設定する必要あり
.github/workflows/preview.yaml

on:
  pull_request:
    branches:
      - master

jobs:
  preview:
    runs-on: ubuntu-latest
    steps:
      - name: Context
        uses: okteto/context@latest
        with:
          token: ${{ secrets.OKTETO_TOKEN }}
          url: ${{ secrets.OKTETO_URL }}

      - name: Deploy preview environment
        uses: okteto/deploy-preview@latest
        env:
          GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
        with:
          name: pr-${{ github.event.number }}-ikuradon
          timeout: 15m

.github/workflows/preview-closed.yaml

on:
  pull_request:
    types:
      - closed

jobs:
  closed:
    runs-on: ubuntu-latest
    steps:
    - name: Context
      uses: okteto/context@latest
      with:
        token: ${{ secrets.OKTETO_TOKEN }}
        url: ${{ secrets.OKTETO_URL }}

    - name: Delete namespace
      uses: okteto/destroy-preview@latest
      with:
        name: pr-${{ github.event.number }}-ikuradon

@ikuradon
Copy link
Contributor Author

雑すぎるPRなので手元環境では動いたけど、もうすこし最適化した方がいい気がする

@rinsuki
Copy link
Contributor

rinsuki commented Feb 20, 2022

secrets って外部からの pull-request には乗らないはずだからちょっと微妙かも (仕方ないけど)

@ikuradon
Copy link
Contributor Author

on:
  pull_request:

ではなく

on:
  pull_request_target:

にするとSecretが使えるらしい

@ikuradon
Copy link
Contributor Author

ikuradon commented Mar 6, 2022

Ok To Testを使うとセキュリティ的にも良さそう

@ikuradon
Copy link
Contributor Author

Okteto自身がPR監視できるようになったため、GitHub Actionsが要らない≒Secrets問題解決したっぽい

@ikuradon
Copy link
Contributor Author

手元だと動いたっぽいのでDraft外します…

@ikuradon ikuradon marked this pull request as ready for review March 24, 2022 11:16
@ikuradon
Copy link
Contributor Author

image
image
image

@futchitwo
Copy link
Contributor

futchitwo commented Apr 17, 2022

これ試してみたんですけど、自動的にインスタンスが立ってくれて便利ですね
ただ、プルリクが来るとプレビューのURLが自動的に張られて誰でもadminになれてしまうので、連合を切ったりadmin対策(demo.misskey.ioの1時間ごとのリセットみたいな)が必要な感じです

あと自分の場合、無料枠だからかプルリク3つ分までしか展開されなかったのと、まだ新しい機能だからかプルリク後に新規コミットが来ても更新されないときがあるようなので注意が必要かもしれません(後者はここで言っても仕方ないですが)

@ikuradon
Copy link
Contributor Author

中身的には個別にDBを建てているので他の環境へは汚染が無いはず(アップデートテストができないともいう)
更新されないのはOktetoがどうにかしてくれないとだめかなー?

@syuilo
Copy link
Member

syuilo commented May 15, 2022

連合を切ったりadmin対策(demo.misskey.ioの1時間ごとのリセットみたいな)が必要な感じ

これは後々でもアプリケーションサイドでいくらでもコントロールできる部分だわね

無料枠だからかプルリク3つ分までしか展開されなかった

料金にもよるけど使用状況によっては有料契約しようと思う

@syuilo syuilo merged commit e1a467b into misskey-dev:okteto May 15, 2022
@syuilo
Copy link
Member

syuilo commented May 15, 2022

🙏🏻🙏🏻🙏🏻

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.

None yet

4 participants