Fix validate_udev_discovery_details to return Ok for empty discovery details

bindsi · bindsi · commit 49f4337f0341 · 2025-04-16T17:17:48.000+02:00
Signed-off-by: Marcel Bindseil &lt;marcelbindseil@gmail.com&gt;
diff --git a/webhooks/validating/configuration/src/main.rs b/webhooks/validating/configuration/src/main.rs
@@ -204,7 +204,7 @@ fn validate_udev_discovery_details(config: &Configuration) -> Result<(), ()> {
     match &config.spec.discovery_handler.discovery_details {
         details if details.is_empty() => {
             println!("Discovery details are empty");
-            Err(())
+            Ok(())
         }
         details => {
             // Try to parse the discovery details as UdevDiscoveryDetails

Original file line number	Diff line number	Diff line change
`@@ -204,7 +204,7 @@ fn validate_udev_discovery_details(config: &Configuration) -> Result<(), ()> {`
`204`	`204`	`match &config.spec.discovery_handler.discovery_details {`
`205`	`205`	`details if details.is_empty() => {`
`206`	`206`	`println!("Discovery details are empty");`
`207`		`- Err(())`
	`207`	`+ Ok(())`
`208`	`208`	`}`
`209`	`209`	`details => {`
`210`	`210`	`// Try to parse the discovery details as UdevDiscoveryDetails`