Skip to content

ci: add CI evaluation framework for AlloyDB with model configs and automated Cloud Build pipeline - #154

Merged
omkargaikwad23 merged 5 commits into
mainfrom
release-please--evalbench-ci
Apr 30, 2026
Merged

ci: add CI evaluation framework for AlloyDB with model configs and automated Cloud Build pipeline#154
omkargaikwad23 merged 5 commits into
mainfrom
release-please--evalbench-ci

Conversation

@omkargaikwad23

@omkargaikwad23 omkargaikwad23 commented Apr 29, 2026

Copy link
Copy Markdown
Contributor

This pull request introduces a comprehensive CI evaluation framework for AlloyDB using evalbench. It includes a Cloud Build configuration, evaluation datasets, model and run configurations, and a utility script for environment variable substitution.

@gemini-code-assist gemini-code-assist Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code Review

This pull request introduces a comprehensive CI evaluation framework for AlloyDB using evalbench. It includes a Cloud Build configuration, evaluation datasets, model and run configurations, and a utility script for environment variable substitution. Key feedback points include correcting the GitHub repository variable in the build script, fixing an invalid Vertex AI model name (gemini-2.5-pro), and resolving a recursive symbolic link issue in the workspace. Additionally, it is recommended to replace hardcoded project IDs with environment variables in the dataset and reporting configurations to improve portability.

Comment thread cloudbuild.yaml
Comment thread evals/gemini_2.5_pro_model.yaml
Comment thread cloudbuild.yaml
Comment thread evals/model_config.yaml
Comment thread evals/dataset.json Outdated
Comment thread evals/substitute_env.py Outdated
Comment thread evals/run_config.yaml Outdated
@omkargaikwad23 omkargaikwad23 added do not merge Indicates a pull request not ready for merge, due to either quality or timing. autorelease: pending Release please needs to do its work on this. and removed do not merge Indicates a pull request not ready for merge, due to either quality or timing. labels Apr 29, 2026
@omkargaikwad23 omkargaikwad23 changed the title ci: add CI evaluation framework for AlloyDB with model configs and automated Cloud Build pipeline ci: add CI evaluation framework for AlloyDB with model configs and automated Cloud Build pipeline release 0.0.0 Apr 30, 2026
@omkargaikwad23 omkargaikwad23 changed the title ci: add CI evaluation framework for AlloyDB with model configs and automated Cloud Build pipeline release 0.0.0 ci: add CI evaluation framework for AlloyDB with model configs and automated Cloud Build pipeline Apr 30, 2026
@omkargaikwad23 omkargaikwad23 removed the autorelease: pending Release please needs to do its work on this. label Apr 30, 2026
Comment thread evals/dataset.json Outdated
Comment thread evals/model_config.yaml Outdated
Comment thread evals/run_config.yaml
@omkargaikwad23
omkargaikwad23 merged commit 0723d3a into main Apr 30, 2026
9 checks passed
@omkargaikwad23
omkargaikwad23 deleted the release-please--evalbench-ci branch April 30, 2026 12:47
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.

3 participants