m-lab / etl-gardener

Gardener provides services for maintaining and reprocessing mlab data.
Apache License 2.0
13 stars 5 forks source link

Remove dependency on datastore #393

Closed stephen-soltesz closed 1 year ago

stephen-soltesz commented 1 year ago

This change removes all dependencies on datastore, relying now exclusively on the "local savers" for storing persistent information relevant to gardener.

We previously migrated our production deployments to use the local saver in https://github.com/m-lab/etl-gardener/pull/367, so the datastore logic can be safely removed without consequence.

Tested with unit tests and overnight in sandbox.

Clean up part of https://github.com/m-lab/etl-gardener/issues/349


This change is Reviewable

coveralls commented 1 year ago

Pull Request Test Coverage Report for Build 3416


Changes Missing Coverage Covered Lines Changed/Added Lines %
tracker/tracker.go 14 18 77.78%
<!-- Total: 17 21 80.95% -->
Totals Coverage Status
Change from base Build 3407: -0.2%
Covered Lines: 1521
Relevant Lines: 2089

💛 - Coveralls