Added default funcs for configgrpc

.chloggen/configgrpc-add-newdefault-funcs.yaml

@@ -0,0 +1,25 @@
+# Use this changelog template to create an entry for release notes.
+
+# One of 'breaking', 'deprecation', 'new_component', 'enhancement', 'bug_fix'
+change_type: enhancement
+
+# The name of the component, or a single word describing the area of concern, (e.g. otlpreceiver)
+component: configgrpc
+
+# A brief description of the change.  Surround your text with quotes ("") if it needs to start with a backtick (`).
+note: Adds `NewDefault*` functions for all the config structs.
+
+# One or more tracking issues or pull requests related to the change
+issues: [9654]
+
+# (Optional) One or more lines of additional information to render under the primary note.
+# These lines will be padded with 2 spaces and then inserted directly into the document.
+# Use pipe (|) for multiline entries.
+subtext:
+
+# Optional: The change log or logs in which this entry should be included.
+# e.g. '[user]' or '[user, api]'
+# Include 'user' if the change is relevant to end users.
+# Include 'api' if there is a change to a library API.
+# Default: '[user]'
+change_logs: [api]

config/configgrpc/configgrpc.go

@@ -46,6 +46,15 @@ type KeepaliveClientConfig struct {
 	PermitWithoutStream bool          `mapstructure:"permit_without_stream"`
 }
 
+// NewDefaultKeepaliveClientConfig returns a new instance of KeepaliveClientConfig with default values.
+
+func NewDefaultKeepaliveClientConfig() *KeepaliveClientConfig {
+	return &KeepaliveClientConfig{
+		Time:    time.Second * 10,
+		Timeout: time.Second * 10,
+	}
+}
+
 // ClientConfig defines common settings for a gRPC client configuration.
 type ClientConfig struct {
 	// The target to which the exporter is going to send traces or metrics,
@@ -90,12 +99,26 @@ type ClientConfig struct {
 	Auth *configauth.Authentication `mapstructure:"auth"`
 }
 
+// NewDefaultClientConfig returns a new instance of ClientConfig with default values.
+func NewDefaultClientConfig() *ClientConfig {
+	return &ClientConfig{
+		TLSSetting: configtls.NewDefaultClientConfig(),
+		Keepalive:  NewDefaultKeepaliveClientConfig(),
+	}
+}
+
 // KeepaliveServerConfig is the configuration for keepalive.
 type KeepaliveServerConfig struct {
 	ServerParameters  *KeepaliveServerParameters  `mapstructure:"server_parameters"`
 	EnforcementPolicy *KeepaliveEnforcementPolicy `mapstructure:"enforcement_policy"`
 }
 
+// NewDefaultKeepaliveServerConfig returns a new instance of KeepaliveServerConfig with default values.
+
+func NewDefaultKeepaliveServerConfig() *KeepaliveServerConfig {
+	return &KeepaliveServerConfig{}
+}
+
 // KeepaliveServerParameters allow configuration of the keepalive.ServerParameters.
 // The same default values as keepalive.ServerParameters are applicable and get applied by the server.
 // See https://godoc.org/google.golang.org/grpc/keepalive#ServerParameters for details.
@@ -150,6 +173,13 @@ type ServerConfig struct {
 	IncludeMetadata bool `mapstructure:"include_metadata"`
 }
 
+// NewDefaultServerConfig returns a new instance of ServerConfig with default values.
+func NewDefaultServerConfig() *ServerConfig {
+	return &ServerConfig{
+		Keepalive: NewDefaultKeepaliveServerConfig(),
+	}
+}
+
 // sanitizedEndpoint strips the prefix of either http:// or https:// from configgrpc.ClientConfig.Endpoint.
 func (gcs *ClientConfig) sanitizedEndpoint() string {
 	switch {

config/configgrpc/configgrpc_test.go

@@ -35,6 +35,47 @@ import (
 	"go.opentelemetry.io/collector/pdata/ptrace/ptraceotlp"
 )
 
+func TestNewDefaultKeepaliveClientConfig(t *testing.T) {
+	// Call the function to get a new instance of KeepaliveClientConfig
+	config := NewDefaultKeepaliveClientConfig()
+
+	// Check if the Time field has the default value of 10 seconds
+	expectedTime := time.Second * 10
+	if config.Time != expectedTime {
+		t.Errorf("Expected Time to be %v, but got %v", expectedTime, config.Time)
+	}
+
+	// Check if the Timeout field has the default value of 10 seconds
+	expectedTimeout := time.Second * 10
+	if config.Timeout != expectedTimeout {
+		t.Errorf("Expected Timeout to be %v, but got %v", expectedTimeout, config.Timeout)
+	}
+}
+
+func TestNewDefaultClientConfig(t *testing.T) {
+	// Call the function to get a new instance of ClientConfig
+	config := NewDefaultClientConfig()
+	// Check if Keepalive field is initialized
+	if config.Keepalive == nil {
+		t.Error("Expected Keepalive to be initialized")
+	}
+}
+func TestNewDefaultKeepaliveServerConfig(t *testing.T) {
+	config := NewDefaultKeepaliveServerConfig()
+
+	if config == nil {
+		t.Error("Expected non-nil KeepaliveServerConfig, got nil")
+	}
+
+}
+func TestNewDefaultServerConfig(t *testing.T) {
+	config := NewDefaultServerConfig()
+
+	if config == nil {
+		t.Error("Expected non-nil ServerConfig, got nil")
+	}
+}
+
 // testBalancerBuilder facilitates testing validateBalancerName().
 type testBalancerBuilder struct{}