kaikaila
kaikaila
Hi @HumairAK I wanna bring this to your attention. In backend/src/apiserver/model/task.go line 26, the field name RunId ≠ column name RunUUID. This may cause confusion when using foreignKey: in GORM...
Closed in favor of #12013 with a more accurate branch name and scope. Please continue reviewing there!
Hi @muzzlol @mprahl , I fully respect @muzzlol's initial implementation and reasoning. However, since @mprahl has highlighted concrete user benefits for this feature, I'd like to propose a collaborative path...
Hey everyone, just a quick update! I’ve implemented the fix and am currently resolving some final Git branch/sign-off issues. Expecting to submit the PR later today or early tomorrow. Let...
Hi everyone, I'm Yunkai, a master's student at UC Berkeley. Since submitting my GSoC proposal for this project, I've started prototyping some of the core components — including a basic...
Currently, both MySQL and PGX setups use the DB superuser for all KFP operations, which is why client_manager.go contains a “create database if not exist” step [here](https://github.com/kubeflow/pipelines/blob/82f6b7d08e9bf7000974bed8d527a6a05bb24031/backend/src/apiserver/client_manager/client_manager.go#L416). From a security...
Thanks, @HumairAK — totally agree on the security point. Since this PR is already getting quite heavy, would you be okay if I leave the user permission changes for a...
## Question about the PostgreSQL test workflow organization ### Current situation The V2 integration tests for PostgreSQL logically belong in a "PostgreSQL counterpart" to [legacy-v2-api-integration-tests.yml](https://github.com/kubeflow/pipelines/blob/master/.github/workflows/legacy-v2-api-integration-tests.yml) However, I didn't want to...
Hi @nickalexander-disco, Thanks for your comment. There are two main reasons why I decided not to use GORM for the store package: - GORM excels at simple CRUD operations but...