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

Add missing attribute 'DbtSourceGcpCloudRunJobOperator' in module 'cosmos.operators.gcp_cloud_run_job' #1290

Open
wants to merge 7 commits into
base: main
Choose a base branch
from

Conversation

anai-s
Copy link

@anai-s anai-s commented Oct 30, 2024

Description

this PR add a missing attribute DbtSourceGcpCloudRunJobOperator in cosmos.operators.gcp_cloud_run_job.py to be able to use the source_rendering_behavior params in cosmos.config.RenderConfig class with the new GCP_CLOUD_RUN_JOB execution mode (available in 1.7.0) .

Related Issue(s)

closes #1276

Breaking Change?

Checklist

  • I have made corresponding changes to the documentation (if required)
  • I have added tests that prove my fix is effective or that my feature works

@dosubot dosubot bot added the size:S This PR changes 10-29 lines, ignoring generated files. label Oct 30, 2024
@dosubot dosubot bot added the area:execution Related to the execution environment/mode, like Docker, Kubernetes, Local, VirtualEnv, etc label Oct 30, 2024
Copy link

netlify bot commented Oct 30, 2024

Deploy Preview for sunny-pastelito-5ecb04 canceled.

Name Link
🔨 Latest commit c94928e
🔍 Latest deploy log https://app.netlify.com/sites/sunny-pastelito-5ecb04/deploys/672487aa816f080008bf7445

@anai-s anai-s marked this pull request as draft October 30, 2024 16:28
@anai-s anai-s marked this pull request as ready for review October 30, 2024 22:58
@anai-s
Copy link
Author

anai-s commented Oct 31, 2024

Hello @tatiana My PR is ready for a review.

Copy link
Collaborator

@tatiana tatiana left a comment

Choose a reason for hiding this comment

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

@anai-s this looks great! Thanks for reporting the issue and creating the PR to fix it.

Could we also have a test related to this?

It may be possible to accomplish this by extending an existing test, such as test_dbt_gcp_cloud_run_job_build_command:
https://github.com/astronomer/astronomer-cosmos/blob/main/tests/operators/test_gcp_cloud_run_job.py

Tests are important so we make sure we don't have a regression on this in the future.

@anai-s
Copy link
Author

anai-s commented Oct 31, 2024

@tatiana sure I will add the test! Thank you for the review.

@anai-s
Copy link
Author

anai-s commented Nov 1, 2024

@tatiana I added the test for the module DbtSourceGcpCloudRunJobOperator. Can you review it please?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area:execution Related to the execution environment/mode, like Docker, Kubernetes, Local, VirtualEnv, etc size:S This PR changes 10-29 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Bug] missing attribute 'DbtSourceGcpCloudRunJobOperator' in new executionMode GCP_CLOUD_RUN_JOB
2 participants