search

RCOSDP / RDM-osf.io

Facilitating Open Science
https://rcos.nii.ac.jp
Apache License 2.0
13 stars 24 forks source link

Feature/orthros [39386]認証保証レベル高度化(IAL2, AAL2)の実装 #479

Closed Yusaku-Kitabatake closed 2 months ago

Yusaku-Kitabatake commented 5 months ago

Purpose

https://redmine.devops.rcos.nii.ac.jp/issues/39386 上記チケットで示す機能のデプロイ

Changes

QA Notes

Documentation

Side Effects

Ticket

Yusaku-Kitabatake commented 5 months ago

取り急ぎコンクリフトは解消しました。ですがコンクリフトを解消することで、ビルドチェックにおいて下記問題が発生しております。 rcos-releaseブランチではOSFuserテーブルに新たなカラムが追加されているようで、それに基づいたapi_tests/institutions/views/test_institution_auth.pyのテストが通らずビルドチェックに失敗します。このテストを無理やりスルーしようとapi_tests/institutions/views/test_institution_auth.pyのコードを変更すると、コンクリフトが発生します。 あちらを立てればこちらが立たずといった状況なのですが、この場合feature/orthros_R-2023-55ブランチにrcos-releaseブランチの変更を取り込む方法が正着となりますでしょうか？

hide24 commented 5 months ago

この場合feature/orthros_R-2023-55ブランチにrcos-releaseブランチの変更を取り込む方法が正着となりますでしょうか？

将来的にdevelopブランチへの合流を目指すなら 現状のfeature/orthros_R-2023-55ブランチから、develop向けのブランチ(現状ママ)と rcos-release向けのブランチを分岐させる(そして双方をメンテナンスする) 必要があるかと思います。

お手数をおかけします。

Yusaku-Kitabatake commented 5 months ago

承知しました。 rcos-release向けのブランチですが、とりあえずビルドチェックをスルーするための最低限の変更の取込のみ行う、といった方法はNGでしょうか。

hide24 commented 5 months ago

rcos-release向けのブランチですが、とりあえずビルドチェックをスルーするための最低限の変更の取込のみ行う、といった方法はNGでしょうか。

適切なコミットを抽出できるのであれば問題ないかと思います。

Yusaku-Kitabatake commented 2 months ago

@hide24 大変お待たせ致しました。 本ブランチに対し、ビルドチェックをスルーするためのrcos-releaseブランチの最低限の変更のみ取込を行い、ビルドチェックをスルー致しました。 統合処理をお願いしたいのですが、本ブランチはRDM-cas-overlayの更新が必須となっております。 https://github.com/RCOSDP/RDM-cas-overlay/tree/feature/orthros 上記のブランチのPRを送りたいのですが、統合先はどのブランチを指定すべきでしょうか。

hide24 commented 2 months ago

上記のブランチのPRを送りたいのですが

とりあえず、developあてに出しておいてください。 RCOS環境では上記のブランチのままの状態で利用します。