-
Notifications
You must be signed in to change notification settings - Fork 2
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
FE-187 remove copy project pipeline and tests #332
Merged
Merged
Changes from all commits
Commits
Show all changes
43 commits
Select commit
Hold shift + click to select a range
6f392ac
remove copy_project test
bahill 50f0d3d
remove copy_project_to_new_dataset_job (unused and calls copy_project…
bahill 94a31ff
remove copy_project pipeline solids
bahill bec35b7
remove copy_project pipeline solids test
bahill eadc961
remove copy_project pipeline
bahill 8689195
remove copy_project test config
bahill f844c1c
remove copy_project import
bahill ef3675f
linting for style
bahill d47ccfb
Merge ef3675f88316f14b3dd066eea6956b207f7cd123 into 4af78838a70a2d350…
bahill 34b1f9c
Update requirements.txt
6cd5b28
removing unused copy_project related config
bahill 325773a
Merge remote-tracking branch 'origin/FE-187-remove-copy_project-pipel…
bahill 7716a63
Merge 325773a1c6d3cc8f7ea37bd508c7263bd9880e0e into 4af78838a70a2d350…
bahill 73421af
Update requirements.txt
7385f6b
removing unused reference. (clean up from previous commit)
bahill cda082e
removing unused references (clean up from previous commit)
bahill 08f4e51
Adding concurrency limiter - IE if a new push for the PR comes throug…
bahill 347db28
Merge remote-tracking branch 'origin/FE-187-remove-copy_project-pipel…
bahill 7c9bd98
Merge 347db289de27f3ae52ccab25209b521d597a9623 into 4af78838a70a2d350…
bahill 889da0c
Update requirements.txt
1772fa4
linting
bahill 2feac9f
Merge remote-tracking branch 'origin/FE-187-remove-copy_project-pipel…
bahill 2d0b777
linting
bahill a6dd141
first attempt to add sleep to load_hca to see if the e2e issue is tim…
bahill c854614
Merge a6dd14142ce3b9a6217d7a43d62e2448be120912 into 4af78838a70a2d350…
bahill 2cbf749
Update requirements.txt
d4bec13
linting
bahill 9dd2fa6
Merge remote-tracking branch 'origin/FE-187-remove-copy_project-pipel…
bahill 7a37497
Merge 9dd2fa663a715261e4f70d936bd2cc968efd7b36 into 4af78838a70a2d350…
bahill 03a3646
Update requirements.txt
c6cff66
linting
bahill fc11e60
Merge c6cff66414f0436d25888bebb397704417d3b526 into 4af78838a70a2d350…
bahill e7fa7d1
Update requirements.txt
dfca3fe
FE-203 disable test load hca e2e (#333)
bahill 122bcb4
Merge dfca3fed47118e5b8edbf7abddd0227d61254baf into 4af78838a70a2d350…
bahill 0ec574a
Update requirements.txt
5e67b83
add notes about skipping of test_load_hca.py
bahill cd19afb
Merge 5e67b83f722cc095b864aa47a0c38f467618bb23 into 4af78838a70a2d350…
bahill 8abe8cc
Update requirements.txt
0a8eeee
linting
bahill edc8291
linting
bahill 6765c9a
Merge edc82913adcc483bd3c875ffc28771b20d09459c into 4af78838a70a2d350…
bahill 67b8797
Update requirements.txt
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 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 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 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
18 changes: 0 additions & 18 deletions
18
.../hca_orchestration/config/dev_refresh/run_config/copy_project_new_dataset_run_config.yaml
This file was deleted.
Oops, something went wrong.
22 changes: 0 additions & 22 deletions
22
orchestration/hca_orchestration/config/dev_refresh/run_config/copy_project_run_config.yaml
This file was deleted.
Oops, something went wrong.
This file contains 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
29 changes: 0 additions & 29 deletions
29
...stration/hca_orchestration/config/prod_migration/run_config/dcp1_real_prod_migration.yaml
This file was deleted.
Oops, something went wrong.
29 changes: 0 additions & 29 deletions
29
...stration/hca_orchestration/config/prod_migration/run_config/dcp2_real_prod_migration.yaml
This file was deleted.
Oops, something went wrong.
11 changes: 0 additions & 11 deletions
11
...hca_orchestration/config/prod_migration/run_config/dcp2_real_prod_migration_snapshot.yaml
This file was deleted.
Oops, something went wrong.
This file contains 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 was deleted.
Oops, something went wrong.
This file contains 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 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
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -17,9 +17,6 @@ | |
dev_run_config_for_dcp_release_per_project_partition, | ||
run_config_per_project_public_snapshot_job, | ||
) | ||
from hca_orchestration.config.dev_refresh.dev_refresh import ( | ||
run_config_for_per_project_dataset_partition, | ||
) | ||
from hca_orchestration.config.prod_migration.prod_migration import ( | ||
run_config_per_project_snapshot_job_dev, | ||
) | ||
|
@@ -35,7 +32,6 @@ | |
staging_area_validator, | ||
validate_ingress_graph, | ||
) | ||
from hca_orchestration.repositories.common import copy_project_to_new_dataset_job | ||
from hca_orchestration.resources import bigquery_service, load_tag | ||
from hca_orchestration.resources.config.dagit import dagit_config | ||
from hca_orchestration.resources.config.datasets import ( | ||
|
@@ -127,15 +123,12 @@ def per_project_load_hca() -> PipelineDefinition: | |
@repository | ||
def all_jobs() -> list[PipelineDefinition]: | ||
jobs = [ | ||
copy_project_to_new_dataset_job("prod", "dev"), | ||
make_snapshot_public_job("dev", "dev"), | ||
cut_project_snapshot_job("dev", "dev", "[email protected]"), | ||
legacy_cut_snapshot_job("dev", "[email protected]"), | ||
per_project_load_hca(), | ||
validate_ingress_job() | ||
] | ||
jobs += configure_partitions_for_pipeline("copy_project_to_new_dataset", | ||
run_config_for_per_project_dataset_partition) | ||
jobs += configure_partitions_for_pipeline("make_snapshot_public_job_dev", | ||
run_config_per_project_public_snapshot_job) | ||
jobs += configure_partitions_for_pipeline("cut_project_snapshot_job_dev", | ||
|
Oops, something went wrong.
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.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Interesting, is the idea that multiple e2e tests in flight are putting too much load on dev TDR?