Skip to content

fix: job state name print#6640

Closed
racinmat wants to merge 4 commits into
googleapis:mainfrom
racinmat:racinsky/fix-job-state
Closed

fix: job state name print#6640
racinmat wants to merge 4 commits into
googleapis:mainfrom
racinmat:racinsky/fix-job-state

Conversation

@racinmat
Copy link
Copy Markdown
Contributor

@racinmat racinmat commented Apr 24, 2026 •
edited by BenjaminKazemi
Loading

Thank you for opening a Pull Request! Before submitting your PR, there are a few things you can do to make sure it goes smoothly:

  • Make sure to open an issue as a bug/issue before writing your code! That way we can discuss the change, evaluate designs, and agree on the general idea
  • Ensure the tests and linter pass
  • Code coverage does not decrease (if any source code was changed)
  • Appropriate docs were updated (if necessary)
  • Get the necessary approvals
  • Once the last commit on the PR has been approved, add the "ready to pull" label to the Pull Request

Note: do not merge your PR from GitHub. Adding the "ready to pull" label is the final step in the review process.
After approvals, the changes in your PR will be committed to the main branch and this PR will be closed.

Fixes #6526 🦕

@racinmat racinmat requested a review from a team as a code owner April 24, 2026 10:53
@product-auto-label product-auto-label Bot added size: m Pull request size is medium. api: vertex-ai Issues related to the googleapis/python-aiplatform API. labels Apr 24, 2026
@racinmat racinmat force-pushed the racinsky/fix-job-state branch from 9c00237 to 598fc8a Compare April 30, 2026 08:06
@racinmat
Copy link
Copy Markdown
Contributor Author

racinmat commented Apr 30, 2026

@matthew29tang thank you for merging my other PR, could you please take a look at this one too? And also PRs #6641 and #6642 ? I updated all of them and formatted them properly. Thank you.

@BenjaminKazemi BenjaminKazemi force-pushed the racinsky/fix-job-state branch from 598fc8a to 63436df Compare May 29, 2026 16:57
@BenjaminKazemi BenjaminKazemi added the kokoro:force-run Add this label to force Kokoro to re-run the tests. label May 29, 2026
@yoshi-kokoro yoshi-kokoro removed the kokoro:force-run Add this label to force Kokoro to re-run the tests. label May 29, 2026
@racinmat
Copy link
Copy Markdown
Contributor Author

racinmat commented May 29, 2026

@BenjaminKazemi thanks for running the tests, but as they don't run in github actions, I can't see the error message, when I clicked on details of the failed build https://btx.cloud.google.com/invocations/89d56829-67cb-4417-ab6e-17c35f2aa866 I got
image

how can I see what is wrong so I could fix it?

@BenjaminKazemi BenjaminKazemi added the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Jun 1, 2026
@yoshi-kokoro yoshi-kokoro removed the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Jun 1, 2026
@BenjaminKazemi BenjaminKazemi added the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Jun 1, 2026
@yoshi-kokoro yoshi-kokoro removed the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Jun 1, 2026
@BenjaminKazemi BenjaminKazemi added the ready to pull Ready to be merged into the codebase. label Jun 1, 2026
@BenjaminKazemi BenjaminKazemi force-pushed the racinsky/fix-job-state branch from f606941 to 73afc14 Compare June 1, 2026 19:06
BenjaminKazemi
BenjaminKazemi reviewed Jun 1, 2026
View reviewed changes
Comment thread tests/unit/aiplatform/test_pipeline_job_schedules.py
@BenjaminKazemi BenjaminKazemi added ready to pull Ready to be merged into the codebase. kokoro:force-run Add this label to force Kokoro to re-run the tests. and removed ready to pull Ready to be merged into the codebase. labels Jun 1, 2026
@yoshi-kokoro yoshi-kokoro removed kokoro:force-run Add this label to force Kokoro to re-run the tests. labels Jun 1, 2026
@BenjaminKazemi BenjaminKazemi added kokoro:force-run Add this label to force Kokoro to re-run the tests. and removed ready to pull Ready to be merged into the codebase. labels Jun 2, 2026
@yoshi-kokoro yoshi-kokoro removed the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Jun 2, 2026
@BenjaminKazemi BenjaminKazemi added the ready to pull Ready to be merged into the codebase. label Jun 2, 2026
copybara-service Bot pushed a commit that referenced this pull request Jun 2, 2026
@racinmat @copybara-github
fix: job state name print
be945df 
style: format

COPYBARA_INTEGRATE_REVIEW=#6640 from racinmat:racinsky/fix-job-state 98ca81e
PiperOrigin-RevId: 925416387
@BenjaminKazemi
Copy link
Copy Markdown
Contributor

BenjaminKazemi commented Jun 2, 2026

This change has been committed to the main branch by Copybara: be945df

Closing the PR.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@BenjaminKazemi BenjaminKazemi BenjaminKazemi approved these changes

@sararob sararob sararob approved these changes

Assignees

No one assigned

Labels

api: vertex-ai Issues related to the googleapis/python-aiplatform API. ready to pull Ready to be merged into the codebase. size: m Pull request size is medium.

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Job state printing is broken for python > 3.10

4 participants

@racinmat @BenjaminKazemi @sararob @yoshi-kokoro