Skip to content

feat: cli funcionality to deploy an Agent to a running GKE cluster #1607

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
wants to merge 2 commits into from

Conversation

vicentefb
Copy link
Contributor

@vicentefb vicentefb commented Jun 23, 2025

@vicentefb vicentefb marked this pull request as draft June 23, 2025 21:21
@vicentefb
Copy link
Contributor Author

@tomergee

@vicentefb vicentefb marked this pull request as ready for review June 24, 2025 19:24
@vicentefb vicentefb marked this pull request as draft June 24, 2025 19:25
@polong-lin polong-lin requested a review from hangfei June 24, 2025 19:34
@hangfei hangfei marked this pull request as ready for review June 26, 2025 00:11
@hangfei
Copy link
Collaborator

hangfei commented Jun 26, 2025

Sry i miss clicked "Ready to review". I can't unclick it. If it's not ready, please unclick from your side.

@vicentefb
Copy link
Contributor Author

Yes, It's ready to be reviewed @hangfei , thanks!

Copy link
Collaborator

@hangfei hangfei left a comment

Choose a reason for hiding this comment

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

Could you ensure cloud run still works as part of your testing? Thanks.

@vicentefb vicentefb requested a review from hangfei June 26, 2025 16:38
@vicentefb
Copy link
Contributor Author

@hangfei ping

@moficodes
Copy link

@hangfei is there any blocker to merging this?

@hangfei
Copy link
Collaborator

hangfei commented Jul 10, 2025

LGTM. @vicentefb Could you update the manual e2e testing(GKE, CloudRun) you have done?

Please squash into single commit.

@vicentefb vicentefb force-pushed the GkeDeployAgent branch 3 times, most recently from bb13f0b to 7b1571b Compare July 16, 2025 21:04
@hangfei hangfei added the ready to pull [Status] This PR is ready to be importing back to Google label Jul 16, 2025
@vicentefb vicentefb force-pushed the GkeDeployAgent branch 2 times, most recently from e8f3aca to 4f4e1b9 Compare July 18, 2025 05:06
@hangfei hangfei removed the ready to pull [Status] This PR is ready to be importing back to Google label Jul 18, 2025
@vicentefb vicentefb force-pushed the GkeDeployAgent branch 2 times, most recently from f892762 to 86911b4 Compare July 18, 2025 18:04
@hangfei hangfei added the ready to pull [Status] This PR is ready to be importing back to Google label Jul 18, 2025
@vicentefb vicentefb force-pushed the GkeDeployAgent branch 2 times, most recently from ab735df to 3fa5cc2 Compare July 24, 2025 02:10
@moficodes
Copy link

@hangfei @polong-lin

Anything stopping us from merging this?

@hangfei hangfei removed the ready to pull [Status] This PR is ready to be importing back to Google label Jul 24, 2025
@vicentefb vicentefb force-pushed the GkeDeployAgent branch 2 times, most recently from 73329dc to c829547 Compare July 24, 2025 19:54
@hangfei hangfei added the ready to pull [Status] This PR is ready to be importing back to Google label Jul 24, 2025
feat: cli funcionality to deploy an Agent to a running GKE cluster

feat: cli funcionality to deploy an Agent to a running GKE cluster

added tests for tools click adn cli deploy

revert some  changes on cli tools click

added test for tools click

fixed lint issues

removed eval storage uri scope for now

addressed comments on allow origins and nits

formatting

improved logging and updated tests

added label to deployment so that we can keep track in keywords pipeline

added labels to pod spec

fixing conflicts when rebasing

solving conflicts and log_level

updated test

pylint

feat: cli funcionality to deploy an Agent to a running GKE cluster

added tests for tools click adn cli deploy

revert some  changes on cli tools click

added test for tools click

fixed lint issues

removed eval storage uri scope for now

addressed comments on allow origins and nits

formatting

improved logging and updated tests

added label to deployment so that we can keep track in keywords pipeline

added labels to pod spec

fixing conflicts when rebasing

solving conflicts and log_level

updated test

pylint

test not working locally

feat: cli funcionality to deploy an Agent to a running GKE cluster

feat: cli funcionality to deploy an Agent to a running GKE cluster

feat: cli funcionality to deploy an Agent to a running GKE cluster

added tests for tools click adn cli deploy

revert some  changes on cli tools click

added test for tools click

fixed lint issues

removed eval storage uri scope for now

addressed comments on allow origins and nits

formatting

improved logging and updated tests

added label to deployment so that we can keep track in keywords pipeline

added labels to pod spec

fixing conflicts when rebasing

solving conflicts and log_level

updated test

pylint

feat: cli funcionality to deploy an Agent to a running GKE cluster

added tests for tools click adn cli deploy

revert some  changes on cli tools click

added test for tools click

fixed lint issues

removed eval storage uri scope for now

addressed comments on allow origins and nits

formatting

improved logging and updated tests

added label to deployment so that we can keep track in keywords pipeline

added labels to pod spec

fixing conflicts when rebasing

solving conflicts and log_level

updated test

pylint

test not working locally

updated test based on merge conflict

feat: cli funcionality to deploy an Agent to a running GKE cluster

feat: cli funcionality to deploy an Agent to a running GKE cluster

feat: cli funcionality to deploy an Agent to a running GKE cluster

added tests for tools click adn cli deploy

revert some  changes on cli tools click

added test for tools click

fixed lint issues

removed eval storage uri scope for now

addressed comments on allow origins and nits

formatting

improved logging and updated tests

added label to deployment so that we can keep track in keywords pipeline

added labels to pod spec

fixing conflicts when rebasing

solving conflicts and log_level

updated test

pylint

feat: cli funcionality to deploy an Agent to a running GKE cluster

added tests for tools click adn cli deploy

revert some  changes on cli tools click

added test for tools click

fixed lint issues

removed eval storage uri scope for now

addressed comments on allow origins and nits

formatting

improved logging and updated tests

added label to deployment so that we can keep track in keywords pipeline

added labels to pod spec

fixing conflicts when rebasing

solving conflicts and log_level

updated test

pylint

test not working locally

feat: cli funcionality to deploy an Agent to a running GKE cluster

feat: cli funcionality to deploy an Agent to a running GKE cluster

feat: cli funcionality to deploy an Agent to a running GKE cluster

added tests for tools click adn cli deploy

revert some  changes on cli tools click

added test for tools click

fixed lint issues

removed eval storage uri scope for now

addressed comments on allow origins and nits

formatting

improved logging and updated tests

added label to deployment so that we can keep track in keywords pipeline

added labels to pod spec

fixing conflicts when rebasing

solving conflicts and log_level

updated test

pylint

feat: cli funcionality to deploy an Agent to a running GKE cluster

added tests for tools click adn cli deploy

revert some  changes on cli tools click

added test for tools click

fixed lint issues

removed eval storage uri scope for now

addressed comments on allow origins and nits

formatting

improved logging and updated tests

added label to deployment so that we can keep track in keywords pipeline

added labels to pod spec

fixing conflicts when rebasing

solving conflicts and log_level

updated test

pylint

test not working locally

updated test based on merge conflict

fixed cli deploy test
@hangfei hangfei removed the ready to pull [Status] This PR is ready to be importing back to Google label Jul 24, 2025
@hangfei hangfei added the ready to pull [Status] This PR is ready to be importing back to Google label Jul 24, 2025
copybara-service bot pushed a commit that referenced this pull request Jul 24, 2025
Merge #1607

- Added CLI functionality so that we can deploy and Agent onto a GKE cluster
- Related documentation google/adk-docs#445

COPYBARA_INTEGRATE_REVIEW=#1607 from vicentefb:GkeDeployAgent 42f35d9
PiperOrigin-RevId: 786857789
@hangfei
Copy link
Collaborator

hangfei commented Jul 24, 2025

merged by copybara. thanks for the contribution!

@hangfei hangfei closed this Jul 24, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ready to pull [Status] This PR is ready to be importing back to Google
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants