pkg/chargeback: Don't exit after first promsum collection

Chance Zibolski · Chance Zibolski · commit e2fd596b8441 · 2018-05-15T11:04:28.000-07:00
Extra return caused collection to only happen once, and was added when
refactoring the logic to fix operator shutdown hanging.
diff --git a/pkg/chargeback/promsum.go b/pkg/chargeback/promsum.go
@@ -61,7 +61,6 @@ func (c *Chargeback) runPromsumWorker(stopCh <-chan struct{}) {
 				return
 			case <-tickerCh:
 				c.collectPromsumDataWithDefaultTimeBounds(ctx, logger)
-				return
 			}
 		}
 	}()

Original file line number	Diff line number	Diff line change
`@@ -61,7 +61,6 @@ func (c *Chargeback) runPromsumWorker(stopCh <-chan struct{}) {`
`61`	`61`	`return`
`62`	`62`	`case <-tickerCh:`
`63`	`63`	`c.collectPromsumDataWithDefaultTimeBounds(ctx, logger)`
`64`		`- return`
`65`	`64`	`}`
`66`	`65`	`}`
`67`	`66`	`}()`