sts-sponsors team mailing list archive

[Alberto Donato <mp+443084@xxxxxxxxxxxxxxxxxx>]

Alberto Donato has proposed merging ~ack/maas-kpi:pass-dailystats-days into maas-kpi:master.

Commit message:
allow passing number of days to go back for daily stats



Requested reviews:
  MAAS Committers (maas-committers)

For more details, see:
https://code.launchpad.net/~ack/maas-kpi/+git/maas-kpi/+merge/443084
-- 
Your team MAAS Committers is requested to review the proposed merge of ~ack/maas-kpi:pass-dailystats-days into maas-kpi:master.

diff --git a/Makefile b/Makefile
index 1ec3de4..52e3ce1 100644
--- a/Makefile
+++ b/Makefile
@@ -2,6 +2,7 @@ LP_CREDENTIALS ?= launchpad.creds
 SNAP_STORE_MACAROON ?= snapcraft.macaroon
 INFLUXDB_CREDENTIALS ?= influxdb.creds
 INFLUXDB_DBNAME ?= maas
+DAILYSTATS_DAYS ?= 1
 SWIFT_KEY ?= swift.key
 export DATASOURCE ?= maas-influxdb.cloud.kpi.internal
 
@@ -51,7 +52,10 @@ $(GENERATED)/snap.metrics: $(SNAP_STORE_MACAROON) | $(GENERATED) $(VIRTUALENV)
 	$(VIRTUALENV)/bin/maaskpi-snap -o $@ --macaroon $(SNAP_STORE_MACAROON)
 
 $(GENERATED)/dailystats.metrics: | $(GENERATED) $(VIRTUALENV)
-	$(VIRTUALENV)/bin/maaskpi-dailystats -o $@ --swift-key $(SWIFT_KEY)
+	$(VIRTUALENV)/bin/maaskpi-dailystats \
+		--days $(DAILYSTATS_DAYS) \
+		-o $@ \
+		--swift-key $(SWIFT_KEY)
 
 $(GENERATED)/bugs.metrics: | $(GENERATED) $(VIRTUALENV)
 	$(VIRTUALENV)/bin/maaskpi-bugs -o $@ --lp-credentials $(LP_CREDENTIALS)