Remove section for executing resetDell periodically

[chez-shanpu]

No description provided.

[masa213f]

Please refactor the caller of this facntion.
There is no longer a need to run this function as a goroutine.
https://github.com/cybozu-go/setup-hw/blob/v1.15.1/pkg/monitor-hw/cmd/root.go#L107-L109

[chez-shanpu]

I refactored it by deleting monitor functions. Also, I changed startExporter() to keep running without well.Wait().

[masa213f]

Please check whether monitor-hw exits normally.
As far as I have confirmed, the following test programs did not terminate when receiving signals.
I think monitor-hw will be the same.

package main

import (
	"context"
	"io"
	"net/http"
	"time"

	"github.com/cybozu-go/log"
	"github.com/cybozu-go/well"
	"github.com/spf13/cobra"
)

func startExporter() error {
	well.Go(func(ctx context.Context) error {
		for {
			log.Info("Do collector.Update(ctx)", nil)
			select {
			case <-time.After(time.Second):
			case <-ctx.Done():
				return nil
			}
		}
	})

	mux := http.NewServeMux()
	mux.HandleFunc("/metrics", func(w http.ResponseWriter, r *http.Request) {
		io.WriteString(w, "Hello")
	})
	return http.ListenAndServe(":9105", mux)
}

var rootCmd = &cobra.Command{
	PersistentPreRun: func(cmd *cobra.Command, args []string) {
		cmd.SilenceUsage = true
		err := well.LogConfig{}.Apply()
		if err != nil {
			log.ErrorExit(err)
		}
	},

	RunE: func(cmd *cobra.Command, args []string) error {
		return startExporter()
	},
}

func main() {
	err := rootCmd.Execute()
	if err != nil {
		log.ErrorExit(err)
	}
}

[chez-shanpu]

Thank you.
I rollbacked to use well.HTTPServer.ListenAndServe() and call well.Stop() and well.Wait at the parent function.

[masa213f]

LGTM