Nelesh Singla
Nelesh Singla
**Out of scope for this PR:** we should try to remove the hard coded values in assertions to use the created objects in the setup/initialize phase
> @nsingla could you please address the CI failure and rebase? The CI failure is a genuine failure, I think its a performance issue with deleting pipeline versions with cacheEnabled=true...
> @nsingla > > > Rather than augmenting the junit xml, is there no way, we can just fetch the workflow logs directly from s3/minio and append the logs in...
> ## 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,...
> /retest seems like geniune test failures, I am fixing it
/lgtm
@CarterFendley FYI
@wassimbensalem I just asked my AI Code assistant to do some analysis on the PR, and as expected it provided some security risks: ## Risks 1. **CEL Expression Injection Risks...
> Hi! @nsingla I made intentional changes to the compiler, and manually updating all specs in test/compiled-workflow would be very time-consuming. I’ve already used the [following code](https://github.com/kubeflow/pipelines/pull/12269/files#diff-0e968b838baa6b52a64936e66c5f0c50332ebb260d19327ee342a3480f4b1fd0R133) on my side...
@sduvvuri1603 can you please add what this config is suppose to do, to the PR description? and a section about how you;ve validated the functionality.