-
Notifications
You must be signed in to change notification settings - Fork 9
Transient Clusters Triggered by teams.py #789
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 18 commits
Commits
Show all changes
69 commits
Select commit
Hold shift + click to select a range
1367dae
Add transient clusters to dataproc submitter
chewy-zlai 412852b
Remove wrong import
chewy-zlai 68b6510
fix import
chewy-zlai 59e848c
Add GCP to dataproc env variables
chewy-zlai 78ea196
Use transient clusters for canary tests
chewy-zlai 7722fc3
Make tags and initialization actions optional
chewy-zlai dd0273f
Remove opsagent initialization action
chewy-zlai 3cb5080
Test with 2 workers
chewy-zlai 0e867ed
Add artifact_prefix to metadata and switch to n2 as default machine type
chewy-zlai e243b9b
Fix metadata value
chewy-zlai a21daf3
Only create cluster if GCP_CREATE_CLUSTER is true
chewy-zlai 9e26a67
Only create cluster for backfill jobs
chewy-zlai c234f00
Switch gating flag to be GCP_CREATE_DATAPROC=true
chewy-zlai 68aadd3
Add GCP_DATAPROC_CLUSTER as default for canary tests other than backfill
chewy-zlai 4bac270
Create cluster for upload as well
chewy-zlai d315bc1
Reorder env variables for clarity
chewy-zlai 3e76796
Throw error if num_workers is not an int. Also scalafmt
chewy-zlai 2b2b9c9
Add better error handling for dataproc creation
chewy-zlai 4e03b27
Make master host type configurable
chewy-zlai 9ec8107
Undo change to cluster name order
chewy-zlai 4188b03
Merge branch 'main' of https://github.com/zipline-ai/chronon into che…
chewy-zlai 96ff2ee
Make subnetwork configurable
chewy-zlai 6b7bb66
Update timeout for cluster creation to 10 minutes, and use smaller ma…
chewy-zlai 135d62e
Use transient clusters for uploads instead of backfills
chewy-zlai 6936697
Set timeout for cluster creation to 15 minutes
chewy-zlai 9287f58
Merge branch 'main' of https://github.com/zipline-ai/chronon into che…
chewy-zlai 43d6ad0
scalafmt
chewy-zlai 61cb324
Add option of setting cluster config by file
chewy-zlai 6e15c9e
Parse json from config file
chewy-zlai cb82b54
Merge branch 'main' of https://github.com/zipline-ai/chronon into che…
chewy-zlai 81e3d38
Merge branch 'main' of https://github.com/zipline-ai/chronon into che…
chewy-zlai 2c1cfa6
Add error handling for failure to parse cluster config
chewy-zlai f4c5417
Create cluster config section of teams.py
chewy-zlai b1a1950
Merge branch 'main' of https://github.com/zipline-ai/chronon into che…
chewy-zlai b06692b
Plumb cluster config into dataproc submitter
chewy-zlai 7d978b1
Merge branch 'main' of https://github.com/zipline-ai/chronon into che…
chewy-zlai 08fdef4
Use types.py to import ClusterConfigProperties
chewy-zlai 82018fe
fix cluster conf extraction in job submitter
chewy-zlai b07d668
fix JobSubmitter errors
chewy-zlai c8fbf2f
Make ClusterConfigProperties a map of string to string for consistency
chewy-zlai 337181a
Null out cluster name for upload jobs. Use dataproc.config as map key…
chewy-zlai 20d8241
Use seconds instead of hours for idle ttl
chewy-zlai 5973e19
drop network from config and only include subnetwork
chewy-zlai 3e8a28f
Shorten transient cluster name
chewy-zlai 97e9492
Handle artifact_prefix being given with trailing /
chewy-zlai 1fba8ed
python lint
chewy-zlai 71d37fe
sort imports
chewy-zlai 5b3c55a
sort imports
chewy-zlai 1d9c592
ruff --fix
chewy-zlai dfd14a7
Update DataprocSubmitterTest
chewy-zlai e81aa66
Remove references to environment variable version of transient clusters
chewy-zlai 24583fe
Revert back to println from logger as it seems to have removed the lo…
chewy-zlai 902bb84
Make idle timeout configurable via generate_dataproc_cluster_config p…
chewy-zlai c97357a
flush the log message so it prints before the cluster is created
chewy-zlai 7b46949
Use System.out.flush
chewy-zlai 16c9ac7
Use scala.Console.println
chewy-zlai 19b4712
Clean up imports
chewy-zlai e3bd354
Handle errors for getCluster call
chewy-zlai b780d50
readd transient clustername as zipline-{uuid}
chewy-zlai c6af5b5
Add a small sleep so the buffer gets flushed
chewy-zlai d0c820b
Test having both clustername and a cluster config set
chewy-zlai afbc897
Try using slf4j logger again with added import
chewy-zlai 739d77f
Remove clustername from test for uploads
chewy-zlai f858596
import logback
chewy-zlai c763043
Use System.err.println to see if that gets printed quickly
chewy-zlai 49e5875
Add test of name and config given
chewy-zlai 927a621
Have canary include a name of the transient cluster so it only gets m…
chewy-zlai a3c68a6
scalafmt
chewy-zlai bdbd94c
Merge branch 'main' of https://github.com/zipline-ai/chronon into che…
chewy-zlai 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
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
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.