Skip to content

fix: keeping the non-default version of the model resource#2786

Closed
racinmat wants to merge 1 commit intogoogleapis:mainfrom
racinmat:main
Closed

fix: keeping the non-default version of the model resource#2786
racinmat wants to merge 1 commit intogoogleapis:mainfrom
racinmat:main

Conversation

@racinmat
Copy link
Copy Markdown

@racinmat racinmat commented Oct 7, 2023

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 #2619 🦕

The whole problem is described in the issue.
I do not have rights to add labels, could you please label this PR as hacktoberfest-accepted?
Thanks.

@product-auto-label product-auto-label Bot added size: xs Pull request size is extra small. api: vertex-ai Issues related to the googleapis/python-aiplatform API. labels Oct 7, 2023
@racinmat racinmat changed the title keeping the non-default version of the model resource fix: keeping the non-default version of the model resource Oct 8, 2023
"""Sync GAPIC service representation of client class resource.
Uses versioned resource name so the non-default version is not lost.
"""
self._gca_resource = self._get_gca_resource(
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Please update the unit tests to test the behavior change.

@sasha-gitg sasha-gitg added the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Oct 9, 2023
@yoshi-kokoro yoshi-kokoro removed the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Oct 9, 2023
@matthew29tang
Copy link
Copy Markdown
Contributor

Closing due to inactivity.

@racinmat
Copy link
Copy Markdown
Author

racinmat commented Jul 1, 2025

I am using windows and I was not able to run the tests locally. Is there a simple way how to run them?

@racinmat racinmat mentioned this pull request Apr 24, 2026
6 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

api: vertex-ai Issues related to the googleapis/python-aiplatform API. size: xs Pull request size is extra small.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Syncing gca resource for model with non-default version mutates the version id

4 participants