From 95b00283a25a005e76931d61ffe064d9a1dce7f0 Mon Sep 17 00:00:00 2001 From: Nick White Date: Tue, 24 Sep 2019 16:18:30 +0100 Subject: Add list of books done and in progress to lspipeline --- bookpipeline/cmd/lspipeline/main.go | 90 ++++++++++++++++++++++++++++++++++++- 1 file changed, 88 insertions(+), 2 deletions(-) (limited to 'bookpipeline/cmd/lspipeline') diff --git a/bookpipeline/cmd/lspipeline/main.go b/bookpipeline/cmd/lspipeline/main.go index 3cbc893..a491cbc 100644 --- a/bookpipeline/cmd/lspipeline/main.go +++ b/bookpipeline/cmd/lspipeline/main.go @@ -5,6 +5,7 @@ import ( "fmt" "log" "os" + "strings" "rescribe.xyz/go.git/bookpipeline" ) @@ -15,8 +16,8 @@ Lists useful things related to the pipeline. - Instances running - Messages in each queue -- Books not completed (from S3 without a best file) -- Books completed (from S3 with a best file) +- Books not completed +- Books done - Last 5 lines of bookpipeline logs from each running instance (with -v) ` @@ -27,6 +28,8 @@ type LsPipeliner interface { AnalyseQueueId() string GetQueueDetails(url string) (string, string, error) GetInstanceDetails() ([]bookpipeline.InstanceDetails, error) + ListObjects(bucket string, prefix string) ([]string, error) + WIPStorageId() string } // NullWriter is used so non-verbose logging may be discarded @@ -71,6 +74,76 @@ func getQueueDetails(conn LsPipeliner, qdetails chan queueDetails) { close(qdetails) } +// getBookStatus returns a list of in progress and done books. +// It determines this by listing all objects, and splitting the +// prefixes into two lists, those which have a 'graph.png' file, +// which are classed as done, and those which are not. +func getBookStatus(conn LsPipeliner) (inprogress []string, done []string, err error) { + allfiles, err := conn.ListObjects(conn.WIPStorageId(), "") + if err != nil { + log.Println("Error getting list of objects:", err) + return inprogress, done, err + } + for _, f := range allfiles { + parts := strings.Split(f, "/") + if parts[1] != "graph.png" { + continue + } + prefix := parts[0] + found := false + for _, i := range done { + if i == prefix { + found = true + continue + } + } + if !found { + done = append(done, prefix) + } + } + + for _, f := range allfiles { + parts := strings.Split(f, "/") + prefix := parts[0] + found := false + for _, i := range done { + if i == prefix { + found = true + continue + } + } + for _, i := range inprogress { + if i == prefix { + found = true + continue + } + } + if !found { + inprogress = append(inprogress, prefix) + } + } + + return inprogress, done, err +} + +func getBookStatusChan(conn LsPipeliner, inprogressc chan string, donec chan string) { + inprogress, done, err := getBookStatus(conn) + if err != nil { + log.Println("Error getting book status:", err) + close(inprogressc) + close(donec) + return + } + for _, i := range inprogress { + inprogressc <- i + } + close(inprogressc) + for _, i := range done { + donec <- i + } + close(donec) +} + func main() { verbose := flag.Bool("v", false, "verbose") flag.Usage = func() { @@ -96,9 +169,12 @@ func main() { instances := make(chan bookpipeline.InstanceDetails, 100) queues := make(chan queueDetails) + inprogress := make(chan string) + done := make(chan string) go getInstances(conn, instances) go getQueueDetails(conn, queues) + go getBookStatusChan(conn, inprogress, done) fmt.Println("# Instances") for i := range instances { @@ -120,5 +196,15 @@ func main() { fmt.Printf("%s: %s available, %s in progress\n", i.name, i.numAvailable, i.numInProgress) } + fmt.Println("\n# Books not completed") + for i := range inprogress { + fmt.Println(i) + } + + fmt.Println("\n# Books done") + for i := range done { + fmt.Println(i) + } + // TODO: See remaining items in the usage statement } -- cgit v1.2.1-24-ge1ad