-
Notifications
You must be signed in to change notification settings - Fork 312
feat: Add GCP_GKE and WATCHER_GCP_GKE execution modes
#2488
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
Merged
Merged
Changes from 7 commits
Commits
Show all changes
52 commits
Select commit
Hold shift + click to select a range
65a44a6
feat: add GCP_GKE and WATCHER_GCP_GKE execution modes for GKE support
vricciardulli c40c7db
fix: try different cmd invocation
vricciardulli c73aae5
Merge remote-tracking branch 'upstream/main' into gcp-modes
vricciardulli a947590
🎨 [pre-commit.ci] Auto format from pre-commit.com hooks
pre-commit-ci[bot] 8416d7e
fix: bots comments
vricciardulli d046ae2
fix: tests
vricciardulli 885e2e2
fix: bots comments
vricciardulli 2e64343
Merge branch 'main' into gcp-modes
vricciardulli ce703e9
Merge remote-tracking branch 'upstream/main' into gcp-modes
vricciardulli 5cccfe6
Merge branch 'gcp-modes' of https://github.com/vricciardulli/astronom…
vricciardulli 8b4cdd0
fix: bots comments
vricciardulli 6a250cf
Merge remote-tracking branch 'upstream/main' into gcp-modes
vricciardulli 7ac5f7a
Merge branch 'main' into gcp-modes
vricciardulli d01bd53
fix: remove retries = 0
vricciardulli 340a463
fix: tests
vricciardulli 8bc0e4d
Merge remote-tracking branch 'upstream/main' into gcp-modes
vricciardulli 11fca9e
Merge remote-tracking branch 'upstream/main' into gcp-modes
vricciardulli aedbd4e
fix: adapt to changes on main
vricciardulli 8e69307
add more tests
vricciardulli bf37489
add more tests
vricciardulli 150b780
feat(DEV): fix wrong docstrings
vricciardulli a24d1ae
refactor: deduplicate code across all k8s-based modes
vricciardulli d437c3e
Merge branch 'gcp-modes-deduplication' into gcp-modes
vricciardulli 2b972d7
🎨 [pre-commit.ci] Auto format from pre-commit.com hooks
pre-commit-ci[bot] 8b72654
fix: bot comments
vricciardulli 440de44
fix: bring back inits
vricciardulli 604ffcb
tests: add tests for execute_watcher_producer
vricciardulli 0e5da20
docs: document changes
vricciardulli 159347b
fix: attempt to improve type checking
vricciardulli 4e01e14
tests: group tests with parametrize
vricciardulli 8ec48cc
docs: fix pip install
vricciardulli 446dbf8
fix: remove integration tests
vricciardulli 2b458a5
Merge remote-tracking branch 'upstream/main' into gcp-modes
vricciardulli d41cea2
Merge remote-tracking branch 'upstream/main' into gcp-modes
vricciardulli eb9c56e
fix: bring back inits
vricciardulli 6139d46
Merge remote-tracking branch 'upstream/main' into gcp-modes
vricciardulli 7a4ac8c
Merge remote-tracking branch 'upstream/main' into gcp-modes
vricciardulli 0d3f2cf
🎨 [pre-commit.ci] Auto format from pre-commit.com hooks
pre-commit-ci[bot] 737c036
fix: enforce pre-commit checks
vricciardulli 0ba5e9b
fix: simplify
vricciardulli aab1cb1
Merge remote-tracking branch 'upstream/main' into gcp-modes
vricciardulli c1552ad
Use lazy logging in _k8s_common warning handler
vricciardulli 6534b9a
Document GCP GKE execution modes in AGENTS.md
vricciardulli ad54d57
Add versionadded note for WATCHER_GCP_GKE mode
vricciardulli f87ea74
Add versionadded note for GCP_GKE mode
vricciardulli e8437b4
fix(DEV): rework docs a little
vricciardulli 4491a5b
Merge remote-tracking branch 'upstream/main' into gcp-modes
vricciardulli dc76b16
Handle tests for WATCHER_GCP_GKE mode
vricciardulli 98bb7f6
Merge branch 'main' into gcp-modes
pankajkoti 57f6fd7
Merge remote-tracking branch 'upstream/main' into gcp-modes
vricciardulli 424753f
Preserve dbt prefix in cloud-docs K8s command; reuse name helper
vricciardulli b449c48
fix: correct test arg to empty list
vricciardulli File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Uh oh!
There was an error while loading. Please reload this page.