Skip to content
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

FIX: GCS Bucket subdirectory not being creating while using ReasoningEngine create method #3651

Closed
wants to merge 3 commits into from

Conversation

ItsMeAbby
Copy link
Contributor

Fixes #3650 🦕

Earlier blobs subfolder were created using

blob = gcs_bucket.blob(os.path.join(gcs_dir_name, _EXTRA_PACKAGES_FILE))

which resulted in files with name like gcs_dir_name_EXTRA_PACKAGES_FILE instead of creating folder.

image

now fixed it with blob = gcs_bucket.blob(f"{gcs_dir_name}/{_EXTRA_PACKAGES_FILE}")

this creates a sub folder

image

This fixes the issue
404 Can not copy from "gs://aretec-vertex-lc-stagging/reasoning_engine/reasoning_engine.pkl" to "gs://1006586376595-996210311323189248-1713810220200/code.pkl", check if the source object and target bucket exist.

Earlier blobs subfolder were created using

blob = gcs_bucket.blob(os.path.join(gcs_dir_name, _EXTRA_PACKAGES_FILE))

which resulted in files with name like gcs_dir_name\_EXTRA_PACKAGES_FILE instead of creating folder.

now fixed it with blob = gcs_bucket.blob(f"{gcs_dir_name}/{_EXTRA_PACKAGES_FILE}")

this creates a sub folder
Copy link

conventional-commit-lint-gcf bot commented Apr 22, 2024

🤖 I detect that the PR title and the commit message differ and there's only one commit. To use the PR title for the commit history, you can use Github's automerge feature with squashing, or use automerge label. Good luck human!

-- conventional-commit-lint bot
https://conventionalcommits.org/

@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 Apr 22, 2024
@yeesian yeesian added the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Apr 24, 2024
@yoshi-kokoro yoshi-kokoro removed the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Apr 24, 2024
@yeesian yeesian added the ready to pull Ready to be merged into the codebase. label Apr 24, 2024
@yeesian yeesian self-requested a review April 24, 2024 19:31
Copy link
Contributor

@yeesian yeesian left a comment

Choose a reason for hiding this comment

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

Thank you!

@yeesian yeesian added ready to pull Ready to be merged into the codebase. and removed ready to pull Ready to be merged into the codebase. labels Apr 24, 2024
@sasha-gitg sasha-gitg added the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Apr 25, 2024
@yoshi-kokoro yoshi-kokoro removed the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Apr 25, 2024
copybara-service bot pushed a commit that referenced this pull request Apr 25, 2024
…Engine create method

COPYBARA_INTEGRATE_REVIEW=#3651 from ItsMeAbby:patch-1 dba25ee
PiperOrigin-RevId: 628104988
@yeesian
Copy link
Contributor

yeesian commented May 1, 2024

This has been reflected in the main branch (commit) and released on v1.49.0.

Thank you again for the report and fix!

@yeesian yeesian closed this May 1, 2024
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. ready to pull Ready to be merged into the codebase. size: xs Pull request size is extra small.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

GCS Bucket subdirectory not being creating while using ReasoningEngine create method
4 participants