Remove deprecated cleanup hooks (#1972)

Signed-off-by: Dave Henderson <[email protected]>

data/datasource.go

@@ -39,14 +39,6 @@ type fileContent struct {
 	b           []byte
 }
 
-// Cleanup - clean up datasources before shutting the process down - things
-// like Logging out happen here
-func (d *Data) Cleanup() {
-	for _, s := range d.Sources {
-		s.cleanup()
-	}
-}
-
 // NewData - constructor for Data
 //
 // Deprecated: will be replaced in future
@@ -98,13 +90,6 @@ type Source struct {
 	mediaType string
 }
 
-// Deprecated: no-op
-func (s *Source) cleanup() {
-	// if s.kv != nil {
-	// 	s.kv.Logout()
-	// }
-}
-
 // String is the method to format the flag's value, part of the flag.Value interface.
 // The String method's output will be used in diagnostics.
 func (s *Source) String() string {

gomplate.go

@@ -29,7 +29,6 @@ func RunTemplates(o *Config) error {
 // Run all gomplate templates specified by the given configuration
 func Run(ctx context.Context, cfg *config.Config) error {
 	Metrics = newMetrics()
-	defer runCleanupHooks()
 
 	// apply defaults before validation
 	cfg.ApplyDefaults()

render.go

@@ -168,9 +168,6 @@ func NewRenderer(opts Options) *Renderer {
 		Sources:      sources,
 	}
 
-	// make sure data cleanups are run on exit
-	addCleanupHook(d.Cleanup)
-
 	if opts.Funcs == nil {
 		opts.Funcs = template.FuncMap{}
 	}