refactor: Remove the product-config dependency

Drop the now-dead `impl Configuration for OpaConfigFragment` and
`impl KeyValueOverridesProvider for OpaConfigOverrides` (both no-ops kept only to
satisfy the product-config pipeline, which is no longer used), and remove the
`product-config` crate from the workspace and operator-binary manifests.

OPA never derived real config from product-config, so nothing is lost. The empty
`deploy/config-spec/properties.yaml` is kept (it is still mounted by the Docker
image and Helm chart) and normalized to match the other operators.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>

Author: maltesander

Cargo.lock

@@ -10,7 +10,6 @@ edition = "2021"
 repository = "https://github.com/stackabletech/opa-operator"
 
 [workspace.dependencies]
-product-config = { git = "https://github.com/stackabletech/product-config.git", tag = "0.8.0" }
 stackable-operator = { git = "https://github.com/stackabletech/operator-rs.git", rev = "57d8d1270016fdbe159fdf39b60cd02b465935af", features = ["webhook"] }
 krb5 = { git = "https://github.com/stackabletech/krb5-rs.git", tag = "v0.1.0" }
 

Cargo.toml

@@ -1,5 +1,5 @@
+---
 version: 0.1.0
 spec:
   units: []
-
 properties: []

deploy/config-spec/properties.yaml

@@ -9,7 +9,6 @@ repository.workspace = true
 publish = false
 
 [dependencies]
-product-config.workspace = true
 stackable-operator.workspace = true
 
 anyhow.workspace = true

rust/operator-binary/Cargo.toml

@@ -1,4 +1,4 @@
-use std::{collections::BTreeMap, str::FromStr};
+use std::str::FromStr;
 
 use serde::{Deserialize, Serialize};
 use snafu::{OptionExt, ResultExt, Snafu};
@@ -16,11 +16,9 @@ use stackable_operator::{
         fragment::{self, Fragment, ValidationError},
         merge::Merge,
     },
-    config_overrides::KeyValueOverridesProvider,
     deep_merger::ObjectOverrides,
     k8s_openapi::apimachinery::pkg::api::resource::Quantity,
     kube::{CustomResource, ResourceExt},
-    product_config_utils::Configuration,
     product_logging::{self, spec::Logging},
     role_utils::{EmptyRoleConfig, GenericCommonConfig, Role, RoleGroup, RoleGroupRef},
     schemars::{self, JsonSchema},
@@ -181,12 +179,6 @@ pub struct OpaConfigOverrides {
     pub config_json: JsonOrKeyValueConfigOverrides,
 }
 
-// OPA has no key-value config files, all overrides go through JsonConfigOverrides.
-// This impl is still required because the shared product config pipeline
-// (`transform_all_roles_to_config`) requires the `KeyValueOverridesProvider` bound
-// at compile time. The default implementation returns an empty map.
-impl KeyValueOverridesProvider for OpaConfigOverrides {}
-
 #[allow(clippy::derive_partial_eq_without_eq)]
 #[derive(Clone, Debug, Default, Fragment, JsonSchema, PartialEq)]
 #[fragment_attrs(
@@ -330,38 +322,6 @@ impl OpaConfig {
     }
 }
 
-impl Configuration for OpaConfigFragment {
-    type Configurable = v1alpha2::OpaCluster;
-
-    fn compute_env(
-        &self,
-        _resource: &Self::Configurable,
-        _role_name: &str,
-    ) -> Result<BTreeMap<String, Option<String>>, stackable_operator::product_config_utils::Error>
-    {
-        Ok(BTreeMap::new())
-    }
-
-    fn compute_cli(
-        &self,
-        _resource: &Self::Configurable,
-        _role_name: &str,
-    ) -> Result<BTreeMap<String, Option<String>>, stackable_operator::product_config_utils::Error>
-    {
-        Ok(BTreeMap::new())
-    }
-
-    fn compute_files(
-        &self,
-        _resource: &Self::Configurable,
-        _role_name: &str,
-        _file: &str,
-    ) -> Result<BTreeMap<String, Option<String>>, stackable_operator::product_config_utils::Error>
-    {
-        Ok(BTreeMap::new())
-    }
-}
-
 impl v1alpha2::OpaCluster {
     /// Returns a reference to the role.
     pub fn role(&self, role_variant: &OpaRole) -> &OpaRoleType {