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.

Sign up for GitHub

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

Jump to bottom

Test that callback functionality works for ExecutionMode.VIRTUALENV #1399

Closed
pankajkoti opened this issue Dec 17, 2024 · 0 comments · Fixed by #1401
Closed

Test that callback functionality works for ExecutionMode.VIRTUALENV #1399

pankajkoti opened this issue Dec 17, 2024 · 0 comments · Fixed by #1401
Assignees
@pankajkoti
Labels
area:execution Related to the execution environment/mode, like Docker, Kubernetes, Local, VirtualEnv, etc execution:virtualenv Related to Virtualenv execution environment

Comments

@pankajkoti
Copy link
Contributor

Since the ability to specify a callback argument is already supported in ExecutionMode.LOCAL (as demonstrated in this code reference), and PR #1389 showcases how to leverage this functionality, we need to verify that this callback support is inherited and functions correctly with ExecutionMode.VIRTUALENV, given that DbtVirtualenvBaseOperator extends DbtLocalBaseOperator.

Related: #1349
@dosubot dosubot bot added area:execution Related to the execution environment/mode, like Docker, Kubernetes, Local, VirtualEnv, etc execution:virtualenv Related to Virtualenv execution environment labels Dec 17, 2024
@pankajkoti pankajkoti mentioned this issue Dec 18, 2024
Merged
@pankajkoti pankajkoti self-assigned this Dec 18, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@pankajkoti pankajkoti

Labels
area:execution Related to the execution environment/mode, like Docker, Kubernetes, Local, VirtualEnv, etc execution:virtualenv Related to Virtualenv execution environment
Projects
None yet
Milestone
No milestone
1 participant
@pankajkoti