nik/OMDSCloud
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity
950 Commits 1 Branch 50 Tags
Commit Graph

12 Commits

Author SHA1 Message Date
saito.k
a0ab885526 Azureリソースの構成変更のためテンプレートを更新 2023-11-29 20:04:19 +09:00
saito.k
cbb412218e Azureリソース作成したためテンプレートを更新 2023-11-29 19:28:46 +09:00
masaaki
40162ef3af Merged PR 568: Azure Function App上でのDBアクセス・メール送信疎通実施 
## 概要
[Task2867: Azure Function App上でのDBアクセス・メール送信疎通実施](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2867)

- 元PBI or タスクへのリンク（内容・目的などはそちらにあるはず）
- 何をどう変更したか、追加したライブラリなど
  - ライセンスアラート処理がAzureFunction上で動作するよう構成変更を行いました
    - DBアクセス、SendGrid、AADB2Cアクセスに必要な環境変数を追加
    - kv-odms-secret-devのアクセスポリシにfunc-odms-dictation-devを追加
    - ネットワーク設定にvnet-odms-network-dev/snet-odms-integration-devを追加
  - プロジェクトの内容もいくつか修正しています
    - 最新のpackage.jsonに合わせてpackage-lock.jsonを生成
    - licenseAlertのスケジュールを毎日1:00（UTC）に変更
    - ビルド用にDockerfileを復活
- このPull Requestでの対象/対象外
- 影響範囲（他の機能にも影響があるか）

## レビューポイント
- 特段注視する部分はありません。ソース上の修正はスケジュールの変更のみです。

## UIの変更
- 無し

## 動作確認状況
- unitテスト実施、ローカルで確認

## 補足
- 相談、参考資料などがあれば
 2023-11-14 12:08:30 +00:00
oura.a
42dab2a45d Merged PR 532: function構築の動作確認用ブランチ 
## 概要
[Task2954: function構築の動作確認用ブランチ](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2954)

- 元PBI or タスクへのリンク（内容・目的などはそちらにあるはず）
  - dev環境にfunctionAppが動作する環境を構築しました。
      - 新規にDICTATION_FUNCTIONプロジェクトを追加
      - 関数アプリリソース「funcapp-odms-dictation-dev」を追加
      - 「funcapp-odms-dictation-dev」に付随してストレージアカウント「saomdsdevelopfuncapp」を追加
      - pipeline「ODMS Function Deploy」を追加
  - 構築の流れをwikiに更新しています。
    https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/OMDSDictation/_wiki/wikis/OMDSDictation_wiki/280/FunctionApp%E3%81%AE%E4%BD%9C%E6%88%90
- このPull Requestでの対象/対象外
  - configurationsについて、結構差分がたまっている状態でした。本タスクで更新したのは以下になり、それ以外は対象外です。
    - dev-application-rg.json：funcapp-odms-dictation-dev
    - dev-application-rg.json：saomdsdevelopfuncapp
  - pipelineについては精査が必要です。「タスク 2992: pipelineの最適化」にて対応予定
- 影響範囲（他の機能にも影響があるか）

## レビューポイント
- 特にレビューしてほしい箇所
  - .devcontainerの内容について、VSCodeからコンテナで動かすためDICTATION_SERVERから取得しています。入れておくと問題のある設定があれば指摘いただければと思います。
- 軽微なものや自明なものは記載不要
- 修正範囲が大きい場合などに記載
- 全体的にや仕様を満たしているか等は本当に必要な時のみ記載

## UIの変更
- 無し

## 動作確認状況
- ローカルで確認＋develop環境で確認

## 補足
- 相談、参考資料などがあれば
 2023-11-06 00:29:47 +00:00
masaaki
d3aeaea777 Merged PR 306: auth/tokenのAPIでWAFのルールに引っかかることがある問題を解決する 
## 概要
[Task2272: auth/tokenのAPIでWAFのルールに引っかかることがある問題を解決する](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2272)

- WAFのルールについて、idTokenに対するREQUEST-942-APPLICATION-ATTACK-SQLIグループのルールを一律除外するよう設定しました

## レビューポイント
- 特になし

## UIの変更
- 特になし

## 動作確認状況
- 無し

## 補足
- 相談、参考資料などがあれば
 2023-08-31 08:35:27 +00:00
湯本 開
781162f2d2 Merged PR 287: Azureリソースの現状を更新 | エラーの挙動について確認する 
## 概要
[Task2302: エラーの挙動について確認する](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2302)

- ApplicationGatewayの正常性プローブの監視先を `/` から `/health` に変更
- httpの対象を監視となっていたものをhttpsにしてバックエンド設定を指定

## レビューポイント
- 情報共有

## 動作確認状況
- develop環境で確認
 2023-07-31 07:58:41 +00:00
saito.k
d9c0ee4abd Merged PR 271: 不要になったAzureリソース・設定を削除する 
## 概要
[Task2279: 不要になったAzureリソース・設定を削除する](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2279)

- 不要になったAzureリソース削除
  - vm-odms-pipeline-maintenance(今までマイグレーションを行っていたVM)
    - vm_odms_pipeline_maintenance433(NIC)
    - nsg_odms_pipeline_maintenance(NSG)
    - vm_odms_pipeline_maintenance_key(SSHキー)
    - snet-odms-pipeline-maintenance(サブネット)

- アクセス許可IPを削除
  - crOdmsRegistryMaintenance(コンテナリポジトリ)
  - kv_odms_secret_dev(Key Vault)
  - kv_odms_secret_stg(Key Vault)

## レビューポイント

## UIの変更
- Before/Afterのスクショなど
- スクショ置き場

## 動作確認状況
- ローカルで確認、develop環境で確認など

## 補足
- 相談、参考資料などがあれば
 2023-07-26 09:12:06 +00:00
saito.k
e9ab7cc10b Merged PR 266: Azure Notification Hubsのリソース作成 
## 概要
[Task2253: Azure Notification Hubsのリソース作成](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2253)

- dev-notification-rgとstg-notification-rgのテンプレートを出力するように修正

## レビューポイント
- 特にレビューしてほしい箇所
- 軽微なものや自明なものは記載不要
- 修正範囲が大きい場合などに記載
- 全体的にや仕様を満たしているか等は本当に必要な時のみ記載

## UIの変更
- Before/Afterのスクショなど
- スクショ置き場

## 動作確認状況
- ローカルで確認、develop環境で確認など

## 補足
- 相談、参考資料などがあれば
 2023-07-25 08:32:57 +00:00
saito.k
4f0e4f4a74 Merged PR 262: [Sprint13-1着手]Pipelineのbuild/pushを固定IPで行えるようにする 
## 概要
[Task2109: [Sprint13-1着手]Pipelineのbuild/pushを固定IPで行えるようにする](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2109)

- 新たに作成したリソース
  - maintenance-rg
    - vm-odms-deploy-maintenance(Pipeline実行環境のVM)
    - vm-odms-deploy-maintenance_disk1_1cbaf8f9bfdf4548902316f7be8189ed(VMのディスク)
    - nsg-odms-deploy-maintenance(VMがあるサブネットに付与したNSG)
    - vm-odms-deploy-maintenance759（VMに付与されたNIC）
    - vm-odms-deploy-maintenance_key_0720（SSH接続するために割り当てられたアクセスキー）
- 変更したリソース
  - Key Vault
    - PipelineVMがあるサブネットからのアクセスを許可する
  - nsg-odms-private-dev（DBのあるサブネットのNSG）
    - PipelineVMからのアクセスを許可する
  - Pieplineの処理

## レビューポイント
- 特にレビューしてほしい箇所
- 軽微なものや自明なものは記載不要
- 修正範囲が大きい場合などに記載
- 全体的にや仕様を満たしているか等は本当に必要な時のみ記載

## UIの変更
- Before/Afterのスクショなど
- スクショ置き場

## 動作確認状況
- deployされていることを確認

## 補足
- 相談、参考資料などがあれば
 2023-07-25 08:08:38 +00:00
masaaki
70cb66e517 Merged PR 254: API実装（パートナーライセンス情報取得API） 
## 概要
[Task2213: API実装（パートナーライセンス情報取得API）](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2213)

- パートナーライセンス情報取得APIについて実装を行いました
- このPull Request機能としては対象外。気づいたので直した部分
　・accounts.service.mock.ts
　　→整形されていなかったのを修正、不要な参照を削除
　・accounts.service.spec.ts
　　→整形されていなかったのを修正、不要な参照（userInfo）を削除
　・license.entity.ts
　　→テスト実施のため必須項目の定義を追加
　・dev-database-rg.json
　　→developからのマージだが、差分ファイルとして出てしまっている。
　　　（誤ってコミットのstagingを解除してしまい、後からpull操作したことが影響していると思われる）
- 影響範囲（他の機能にも影響があるか）
　→特になし

## レビューポイント
- MISOチームメンバについては、「このPull Request機能としては対象外。気づいたので直した部分」の確認をお願いします。整形や不要な参照はMISOで作りこんだものです。
- accounts.repository.service.tsについて、DRYの観点で一部処理をprivateなサブルーチンとしています（getAccountLicenseOrderStatus）。そもそもそういうことをやってよいか、分ける際のやりかた（引数など）について妥当か見ていただきたいです。
- repositoryからの戻り値型の名称についてわかりやすいかどうか。「PartnerLicenseInfoのうちリポジトリから取得した情報」として「PartnerLicenseInfoFromRepository」としていますが、Fromとかつけない方がよいのでは、とか呼び元を意識しすぎた名称になってる、とか気にしています。

## UIの変更
- 無し

## 動作確認状況
- ローカルで確認済

## 補足
- 相談、参考資料などがあれば
 2023-07-25 02:08:20 +00:00
湯本 開
5ee4c1e52f Merged PR 245: azureの構成をexportしてgit管理する 
## 概要
[Task2206: azureの構成をexportしてgit管理する](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2206)

- 主要なresource-groupをexportしてgit管理に追加
- 主要なresource-groupをexportするbatファイルを追加

## レビューポイント
- 自動生成されたと思われるresource-group等はgit管理に追加していないが、問題ないか？
- 配置ディレクトリは問題ないか
 2023-07-21 03:06:38 +00:00
湯本 開
f658201d5a Merged PR 240: pipelineの構造をexportしてgit管理する 
## 概要
[Task2205: pipelineの構造をexportしてgit管理する](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2205)

- PipelineをYAML形式でExportし、git管理に追加

## レビューポイント
- 配置するディレクトリ名などは適切であるか
- JSON形式だと未設定の項目の情報なども出て10倍弱のサイズになってしまうため、可読性の観点でJSONでなくYAMLを選択したが問題ないか
 2023-07-18 03:35:58 +00:00