Rename gnmiext.Configurable to DataElement

The Configurable interface is used for both configuration and state
data, making the old name misleading. Rename it to DataElement, the
term used by the gNMI specification for any item addressable by a
path — regardless of whether it represents a leaf, container, or
entire subtree.

Update variable names in the nxos provider package to reflect the
intent of each call site: `updates` for Update, `deletes` for
Delete, `patches` for Patch, and `el` for generic slices, replacing
the ambiguous `conf` that implied configuration-only usage.

Author: felix-kaestner

internal/provider/cisco/iosxr/provider_test.go

@@ -21,12 +21,12 @@ import (
 
 type TestCase struct {
 	name string
-	val  gnmiext.Configurable
+	val  gnmiext.DataElement
 }
 
 var tests []TestCase
 
-func Register(name string, val gnmiext.Configurable) {
+func Register(name string, val gnmiext.DataElement) {
 	tests = append(tests, TestCase{
 		name: name,
 		val:  val,
@@ -92,11 +92,11 @@ func Test_Payload(t *testing.T) {
 type MockClient struct {
 	// Function fields for mocking different methods
 	CapabilitiesFunc func() *gnmiext.Capabilities
-	GetConfigFunc    func(ctx context.Context, conf ...gnmiext.Configurable) error
-	PatchFunc        func(ctx context.Context, conf ...gnmiext.Configurable) error
-	UpdateFunc       func(ctx context.Context, conf ...gnmiext.Configurable) error
-	DeleteFunc       func(ctx context.Context, conf ...gnmiext.Configurable) error
-	GetStateFunc     func(ctx context.Context, conf ...gnmiext.Configurable) error
+	GetConfigFunc    func(ctx context.Context, configs ...gnmiext.DataElement) error
+	PatchFunc        func(ctx context.Context, patches ...gnmiext.DataElement) error
+	UpdateFunc       func(ctx context.Context, updates ...gnmiext.DataElement) error
+	DeleteFunc       func(ctx context.Context, deletes ...gnmiext.DataElement) error
+	GetStateFunc     func(ctx context.Context, states ...gnmiext.DataElement) error
 }
 
 var _ gnmiext.Client = (*MockClient)(nil)
@@ -109,37 +109,37 @@ func (m *MockClient) Capabilities() *gnmiext.Capabilities {
 	return nil
 }
 
-func (m *MockClient) GetConfig(ctx context.Context, conf ...gnmiext.Configurable) error {
+func (m *MockClient) GetConfig(ctx context.Context, configs ...gnmiext.DataElement) error {
 	if m.GetConfigFunc != nil {
-		return m.GetConfigFunc(ctx, conf...)
+		return m.GetConfigFunc(ctx, configs...)
 	}
 	return nil
 }
 
-func (m *MockClient) GetState(ctx context.Context, conf ...gnmiext.Configurable) error {
+func (m *MockClient) GetState(ctx context.Context, states ...gnmiext.DataElement) error {
 	if m.GetStateFunc != nil {
-		return m.GetStateFunc(ctx, conf...)
+		return m.GetStateFunc(ctx, states...)
 	}
 	return nil
 }
 
-func (m *MockClient) Patch(ctx context.Context, conf ...gnmiext.Configurable) error {
+func (m *MockClient) Patch(ctx context.Context, patches ...gnmiext.DataElement) error {
 	if m.PatchFunc != nil {
-		return m.PatchFunc(ctx, conf...)
+		return m.PatchFunc(ctx, patches...)
 	}
 	return nil
 }
 
-func (m *MockClient) Update(ctx context.Context, conf ...gnmiext.Configurable) error {
+func (m *MockClient) Update(ctx context.Context, updates ...gnmiext.DataElement) error {
 	if m.UpdateFunc != nil {
-		return m.UpdateFunc(ctx, conf...)
+		return m.UpdateFunc(ctx, updates...)
 	}
 	return nil
 }
 
-func (m *MockClient) Delete(ctx context.Context, conf ...gnmiext.Configurable) error {
+func (m *MockClient) Delete(ctx context.Context, deletes ...gnmiext.DataElement) error {
 	if m.DeleteFunc != nil {
-		return m.DeleteFunc(ctx, conf...)
+		return m.DeleteFunc(ctx, deletes...)
 	}
 	return nil
 }
@@ -187,8 +187,8 @@ func Test_EnsureInterface(t *testing.T) {
 
 func Test_GetState(t *testing.T) {
 	m := &MockClient{
-		GetStateFunc: func(ctx context.Context, conf ...gnmiext.Configurable) error {
-			conf[0].(*PhysIfState).State = "im-state-up"
+		GetStateFunc: func(ctx context.Context, states ...gnmiext.DataElement) error {
+			states[0].(*PhysIfState).State = "im-state-up"
 			return nil
 		},
 	}

internal/provider/cisco/nxos/acl.go

@@ -10,7 +10,7 @@ import (
 	"github.com/ironcore-dev/network-operator/internal/transport/gnmiext"
 )
 
-var _ gnmiext.Configurable = (*ACL)(nil)
+var _ gnmiext.DataElement = (*ACL)(nil)
 
 // ACL represents an IPv4 or IPv6 access control list, depending on the rules it contains.
 // It can only contain either IPv4 or IPv6 rules, never both. It's name must be unique

internal/provider/cisco/nxos/banner.go

@@ -11,8 +11,8 @@ import (
 )
 
 var (
-	_ gnmiext.Configurable = (*Banner)(nil)
-	_ gnmiext.Defaultable  = (*Banner)(nil)
+	_ gnmiext.DataElement = (*Banner)(nil)
+	_ gnmiext.Defaultable = (*Banner)(nil)
 )
 
 // Banner represents the pre-login banner configuration of the device.

internal/provider/cisco/nxos/bgp.go

@@ -14,8 +14,8 @@ import (
 )
 
 var (
-	_ gnmiext.Configurable = (*BGP)(nil)
-	_ gnmiext.Configurable = (*BGPDom)(nil)
+	_ gnmiext.DataElement = (*BGP)(nil)
+	_ gnmiext.DataElement = (*BGPDom)(nil)
 )
 
 type BGP struct {

internal/provider/cisco/nxos/bgw.go

@@ -7,7 +7,7 @@ import (
 	"github.com/ironcore-dev/network-operator/internal/transport/gnmiext"
 )
 
-var _ gnmiext.Configurable = (*MultisiteItems)(nil)
+var _ gnmiext.DataElement = (*MultisiteItems)(nil)
 
 type MultisiteItems struct {
 	SiteID              string  `json:"siteId"`

internal/provider/cisco/nxos/cert.go

@@ -91,8 +91,8 @@ func (c *Certificate) EncodeKeyPair() (private, public []byte, err error) {
 }
 
 var (
-	_ gnmiext.Configurable = (*Trustpoint)(nil)
-	_ gnmiext.Configurable = (*KeyPair)(nil)
+	_ gnmiext.DataElement = (*Trustpoint)(nil)
+	_ gnmiext.DataElement = (*KeyPair)(nil)
 )
 
 // Trustpoint represents a PKI trustpoint configuration on a NX-OS device.

internal/provider/cisco/nxos/dhcprelay.go

@@ -9,7 +9,7 @@ import (
 	"github.com/ironcore-dev/network-operator/internal/transport/gnmiext"
 )
 
-var _ gnmiext.Configurable = (*DHCPRelayConfig)(nil)
+var _ gnmiext.DataElement = (*DHCPRelayConfig)(nil)
 
 // DHCPRelayConfig represents the complete DHCP relay configuration tree.
 type DHCPRelayConfig struct {

internal/provider/cisco/nxos/dns.go

@@ -5,7 +5,7 @@ package nxos
 
 import "github.com/ironcore-dev/network-operator/internal/transport/gnmiext"
 
-var _ gnmiext.Configurable = (*DNS)(nil)
+var _ gnmiext.DataElement = (*DNS)(nil)
 
 // DNS represents the DNS configuration on a NX-OS device.
 type DNS struct {

internal/provider/cisco/nxos/evi.go

@@ -14,7 +14,7 @@ import (
 	"github.com/ironcore-dev/network-operator/internal/transport/gnmiext"
 )
 
-var _ gnmiext.Configurable = (*BDEVI)(nil)
+var _ gnmiext.DataElement = (*BDEVI)(nil)
 
 // BDEVI represents a Bridge Domain Ethernet VPN Instance (MAC-VRF).
 type BDEVI struct {

internal/provider/cisco/nxos/feat.go

@@ -6,8 +6,8 @@ package nxos
 import "github.com/ironcore-dev/network-operator/internal/transport/gnmiext"
 
 var (
-	_ gnmiext.Configurable = (*Feature)(nil)
-	_ gnmiext.Defaultable  = (*Feature)(nil)
+	_ gnmiext.DataElement = (*Feature)(nil)
+	_ gnmiext.Defaultable = (*Feature)(nil)
 )
 
 // Feature represents a dynamic feature configuration on a NX-OS device.