func main() {
flag.Parse()
if minutesBetweenRuns == 0 {
- err := singlerun(makeArvadosClient())
+ arv, err := makeArvadosClient()
+ if err != nil {
+ log.Fatalf("makeArvadosClient: %v", err)
+ }
+ err = singlerun(arv)
if err != nil {
log.Fatalf("singlerun: %v", err)
}
waitTime := time.Minute * time.Duration(minutesBetweenRuns)
for {
log.Println("Beginning Run")
- err := singlerun(makeArvadosClient())
+ arv, err := makeArvadosClient()
+ if err != nil {
+ log.Fatalf("makeArvadosClient: %v", err)
+ }
+ err = singlerun(arv)
if err != nil {
log.Printf("singlerun: %v", err)
}
}
}
-func makeArvadosClient() arvadosclient.ArvadosClient {
- arv, err := arvadosclient.MakeArvadosClient()
- if err != nil {
- log.Fatalf("Error setting up arvados client: %s", err)
- }
- return arv
+func makeArvadosClient() (arvadosclient.ArvadosClient, error) {
+ return arvadosclient.MakeArvadosClient()
}
func singlerun(arv arvadosclient.ArvadosClient) error {
kc, err := keepclient.MakeKeepClient(&arv)
if err != nil {
- loggerutil.FatalWithMessage(arvLogger,
- fmt.Sprintf("Error setting up keep client %s", err.Error()))
+ return fmt.Errorf("Error setting up keep client %v", err.Error())
}
// Log that we're finished. We force the recording, since go will
&keepServerInfo,
replicationSummary.KeepBlocksNotInCollections)
- summary.WritePullLists(arvLogger, pullLists)
+ err = summary.WritePullLists(arvLogger, pullLists)
+ if err != nil {
+ return err
+ }
if trashErr != nil {
return err
- } else {
- keep.SendTrashLists(kc, trashLists)
}
+ keep.SendTrashLists(kc, trashLists)
return nil
}
-// Returns a data fetcher that fetches data from remote servers.
+// BuildDataFetcher returns a data fetcher that fetches data from remote servers.
func BuildDataFetcher(arv arvadosclient.ArvadosClient) summary.DataFetcher {
return func(arvLogger *logger.Logger,
readCollections *collection.ReadCollections,
go func() {
collectionChannel <- collection.GetCollectionsAndSummarize(
+ arvLogger,
collection.GetCollectionsParams{
Client: arv,
Logger: arvLogger,
BatchSize: 50})
}()
- *keepServerInfo = keep.GetKeepServersAndSummarize(
+ var err error
+ *keepServerInfo, err = keep.GetKeepServersAndSummarize(
keep.GetKeepServersParams{
Client: arv,
Logger: arvLogger,
Limit: 1000})
+ if err != nil {
+ return
+ }
+
*readCollections = <-collectionChannel
}
}