plugins/bundle: prevent ns-level polling by validating intervals

Fix polling interval validation in Reconfigure to avoid nanosecond ticker
loops

Fixes: #8081

Signed-off-by: jjhwan.h <[email protected]>

Author: jjhwan-h

v1/plugins/bundle/plugin.go

@@ -158,8 +158,15 @@ func (p *Plugin) Reconfigure(ctx context.Context, config any) {
 	// Look for any bundles that have had their config changed, are new, or have been removed
 	newConfig := config.(*Config)
 	newBundles, updatedBundles, deletedBundles := p.configDelta(newConfig)
-	p.config = *newConfig
 
+	err := newConfig.ValidateAndInjectDefaults()
+	if err != nil {
+		p.manager.Logger().Error("Failed to validate bundle configuration: %s", err)
+		p.cfgMtx.Unlock()
+		return
+	}
+
+	p.config = *newConfig
 	p.cfgMtx.Unlock()
 
 	if len(updatedBundles) == 0 && len(newBundles) == 0 && len(deletedBundles) == 0 {
@@ -194,7 +201,7 @@ func (p *Plugin) Reconfigure(ctx context.Context, config any) {
 	// Deactivate the bundles that were removed
 	params := storage.WriteParams
 	params.Context = storage.NewContext() // TODO(sr): metrics?
-	err := storage.Txn(ctx, p.manager.Store, params, func(txn storage.Transaction) error {
+	err = storage.Txn(ctx, p.manager.Store, params, func(txn storage.Transaction) error {
 		opts := &bundle.DeactivateOpts{
 			Ctx:           ctx,
 			Store:         p.manager.Store,