m-lab / etl

M-Lab ingestion pipeline
Apache License 2.0
22 stars 7 forks source link

Remove obsolete v1 flags, cloudbuild steps, and related configurations #1090

Closed stephen-soltesz closed 2 years ago

stephen-soltesz commented 2 years ago

This change continues the decommission of the v1 logic by removing obsolete flags from etl_worker.go, removing commented steps from cloudbuild.yaml, and removing obsolete Dockerfile for the worker.

This change also updates the -output_dir flag to -output_location so that it can be used for both output types, "local" or "gcs". This makes the output bucket configurable, where before it was unconditionally etl-<PROJECT>.

Part of:


This change is Reviewable

coveralls commented 2 years ago

Pull Request Test Coverage Report for Build 7356


Changes Missing Coverage Covered Lines Changed/Added Lines %
cmd/etl_worker/etl_worker.go 2 3 66.67%
<!-- Total: 2 3 66.67% -->
Totals Coverage Status
Change from base Build 7352: 0.05%
Covered Lines: 3335
Relevant Lines: 4965

💛 - Coveralls