Skip to content

Commit 9001428

Browse files
redhatHameedredhatHameed
committed
feat : Feast Operator support log level configuration for services
Signed-off-by: Abdul Hameed <ahameed@redhat.com>
1 parent 6089905 commit 9001428

File tree

4 files changed

+168
-0
lines changed

4 files changed

+168
-0
lines changed

‎infra/feast-operator/api/v1alpha1/featurestore_types.go‎

Lines changed: 12 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -77,6 +77,10 @@ type OfflineStore struct {
7777
ServiceConfigs `json:",inline"`
7878
Persistence *OfflineStorePersistence `json:"persistence,omitempty"`
7979
TLS *OfflineTlsConfigs `json:"tls,omitempty"`
80+
// LogLevel sets the logging level for the offline store service
81+
// Allowed values: "debug", "info", "warning", "error", "critical".
82+
// +kubebuilder:validation:Enum=debug;info;warning;error;critical
83+
LogLevel string `json:"logLevel,omitempty"`
8084
}
8185

8286
// OfflineTlsConfigs configures server TLS for the offline feast service. in an openshift cluster, this is configured by default using service serving certificates.
@@ -128,6 +132,10 @@ type OnlineStore struct {
128132
ServiceConfigs `json:",inline"`
129133
Persistence *OnlineStorePersistence `json:"persistence,omitempty"`
130134
TLS *TlsConfigs `json:"tls,omitempty"`
135+
// LogLevel sets the logging level for the online store service
136+
// Allowed values: "debug", "info", "warning", "error", "critical".
137+
// +kubebuilder:validation:Enum=debug;info;warning;error;critical
138+
LogLevel string `json:"logLevel,omitempty"`
131139
}
132140

133141
// OnlineStorePersistence configures the persistence settings for the online store service
@@ -240,6 +248,10 @@ type PvcCreate struct {
240248
type Registry struct {
241249
Local *LocalRegistryConfig `json:"local,omitempty"`
242250
Remote *RemoteRegistryConfig `json:"remote,omitempty"`
251+
// LogLevel sets the logging level for the registry service
252+
// Allowed values: "debug", "info", "warning", "error", "critical".
253+
// +kubebuilder:validation:Enum=debug;info;warning;error;critical
254+
LogLevel string `json:"logLevel,omitempty"`
243255
}
244256

245257
// RemoteRegistryConfig points to a remote feast registry server. When set, the operator will not deploy a registry for this FeatureStore CR.

‎infra/feast-operator/config/crd/bases/feast.dev_featurestores.yaml‎

Lines changed: 66 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -203,6 +203,17 @@ spec:
203203
description: PullPolicy describes a policy for if/when to
204204
pull a container image
205205
type: string
206+
logLevel:
207+
description: |-
208+
LogLevel sets the logging level for the offline store service
209+
Allowed values: "debug", "info", "warning", "error", "critical".
210+
enum:
211+
- debug
212+
- info
213+
- warning
214+
- error
215+
- critical
216+
type: string
206217
persistence:
207218
description: OfflineStorePersistence configures the persistence
208219
settings for the offline store service
@@ -550,6 +561,17 @@ spec:
550561
description: PullPolicy describes a policy for if/when to
551562
pull a container image
552563
type: string
564+
logLevel:
565+
description: |-
566+
LogLevel sets the logging level for the online store service
567+
Allowed values: "debug", "info", "warning", "error", "critical".
568+
enum:
569+
- debug
570+
- info
571+
- warning
572+
- error
573+
- critical
574+
type: string
553575
persistence:
554576
description: OnlineStorePersistence configures the persistence
555577
settings for the online store service
@@ -1149,6 +1171,17 @@ spec:
11491171
rule: '(!has(self.disable) || !self.disable) ? has(self.secretRef)
11501172
: true'
11511173
type: object
1174+
logLevel:
1175+
description: |-
1176+
LogLevel sets the logging level for the registry service
1177+
Allowed values: "debug", "info", "warning", "error", "critical".
1178+
enum:
1179+
- debug
1180+
- info
1181+
- warning
1182+
- error
1183+
- critical
1184+
type: string
11521185
remote:
11531186
description: |-
11541187
RemoteRegistryConfig points to a remote feast registry server. When set, the operator will not deploy a registry for this FeatureStore CR.
@@ -1373,6 +1406,17 @@ spec:
13731406
description: PullPolicy describes a policy for if/when
13741407
to pull a container image
13751408
type: string
1409+
logLevel:
1410+
description: |-
1411+
LogLevel sets the logging level for the offline store service
1412+
Allowed values: "debug", "info", "warning", "error", "critical".
1413+
enum:
1414+
- debug
1415+
- info
1416+
- warning
1417+
- error
1418+
- critical
1419+
type: string
13761420
persistence:
13771421
description: OfflineStorePersistence configures the persistence
13781422
settings for the offline store service
@@ -1725,6 +1769,17 @@ spec:
17251769
description: PullPolicy describes a policy for if/when
17261770
to pull a container image
17271771
type: string
1772+
logLevel:
1773+
description: |-
1774+
LogLevel sets the logging level for the online store service
1775+
Allowed values: "debug", "info", "warning", "error", "critical".
1776+
enum:
1777+
- debug
1778+
- info
1779+
- warning
1780+
- error
1781+
- critical
1782+
type: string
17281783
persistence:
17291784
description: OnlineStorePersistence configures the persistence
17301785
settings for the online store service
@@ -2335,6 +2390,17 @@ spec:
23352390
rule: '(!has(self.disable) || !self.disable) ? has(self.secretRef)
23362391
: true'
23372392
type: object
2393+
logLevel:
2394+
description: |-
2395+
LogLevel sets the logging level for the registry service
2396+
Allowed values: "debug", "info", "warning", "error", "critical".
2397+
enum:
2398+
- debug
2399+
- info
2400+
- warning
2401+
- error
2402+
- critical
2403+
type: string
23382404
remote:
23392405
description: |-
23402406
RemoteRegistryConfig points to a remote feast registry server. When set, the operator will not deploy a registry for this FeatureStore CR.

‎infra/feast-operator/dist/install.yaml‎

Lines changed: 66 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -211,6 +211,17 @@ spec:
211211
description: PullPolicy describes a policy for if/when to
212212
pull a container image
213213
type: string
214+
logLevel:
215+
description: |-
216+
LogLevel sets the logging level for the offline store service
217+
Allowed values: "debug", "info", "warning", "error", "critical".
218+
enum:
219+
- debug
220+
- info
221+
- warning
222+
- error
223+
- critical
224+
type: string
214225
persistence:
215226
description: OfflineStorePersistence configures the persistence
216227
settings for the offline store service
@@ -558,6 +569,17 @@ spec:
558569
description: PullPolicy describes a policy for if/when to
559570
pull a container image
560571
type: string
572+
logLevel:
573+
description: |-
574+
LogLevel sets the logging level for the online store service
575+
Allowed values: "debug", "info", "warning", "error", "critical".
576+
enum:
577+
- debug
578+
- info
579+
- warning
580+
- error
581+
- critical
582+
type: string
561583
persistence:
562584
description: OnlineStorePersistence configures the persistence
563585
settings for the online store service
@@ -1157,6 +1179,17 @@ spec:
11571179
rule: '(!has(self.disable) || !self.disable) ? has(self.secretRef)
11581180
: true'
11591181
type: object
1182+
logLevel:
1183+
description: |-
1184+
LogLevel sets the logging level for the registry service
1185+
Allowed values: "debug", "info", "warning", "error", "critical".
1186+
enum:
1187+
- debug
1188+
- info
1189+
- warning
1190+
- error
1191+
- critical
1192+
type: string
11601193
remote:
11611194
description: |-
11621195
RemoteRegistryConfig points to a remote feast registry server. When set, the operator will not deploy a registry for this FeatureStore CR.
@@ -1381,6 +1414,17 @@ spec:
13811414
description: PullPolicy describes a policy for if/when
13821415
to pull a container image
13831416
type: string
1417+
logLevel:
1418+
description: |-
1419+
LogLevel sets the logging level for the offline store service
1420+
Allowed values: "debug", "info", "warning", "error", "critical".
1421+
enum:
1422+
- debug
1423+
- info
1424+
- warning
1425+
- error
1426+
- critical
1427+
type: string
13841428
persistence:
13851429
description: OfflineStorePersistence configures the persistence
13861430
settings for the offline store service
@@ -1733,6 +1777,17 @@ spec:
17331777
description: PullPolicy describes a policy for if/when
17341778
to pull a container image
17351779
type: string
1780+
logLevel:
1781+
description: |-
1782+
LogLevel sets the logging level for the online store service
1783+
Allowed values: "debug", "info", "warning", "error", "critical".
1784+
enum:
1785+
- debug
1786+
- info
1787+
- warning
1788+
- error
1789+
- critical
1790+
type: string
17361791
persistence:
17371792
description: OnlineStorePersistence configures the persistence
17381793
settings for the online store service
@@ -2343,6 +2398,17 @@ spec:
23432398
rule: '(!has(self.disable) || !self.disable) ? has(self.secretRef)
23442399
: true'
23452400
type: object
2401+
logLevel:
2402+
description: |-
2403+
LogLevel sets the logging level for the registry service
2404+
Allowed values: "debug", "info", "warning", "error", "critical".
2405+
enum:
2406+
- debug
2407+
- info
2408+
- warning
2409+
- error
2410+
- critical
2411+
type: string
23462412
remote:
23472413
description: |-
23482414
RemoteRegistryConfig points to a remote feast registry server. When set, the operator will not deploy a registry for this FeatureStore CR.

‎infra/feast-operator/internal/controller/services/services.go‎

Lines changed: 24 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -358,6 +358,11 @@ func (feast *FeastServices) getContainerCommand(feastType FeastServiceType) []st
358358
deploySettings := FeastServiceConstants[feastType]
359359
targetPort := deploySettings.TargetHttpPort
360360
tls := feast.getTlsConfigs(feastType)
361+
logLevel := feast.getLogLevelForType(feastType)
362+
if logLevel != nil {
363+
deploySettings.Command = append(deploySettings.Command, "--log-level="+strings.ToUpper(*logLevel))
364+
}
365+
361366
if tls.IsTLS() {
362367
targetPort = deploySettings.TargetHttpsPort
363368
feastTlsPath := GetTlsPath(feastType)
@@ -474,6 +479,25 @@ func (feast *FeastServices) getServiceConfigs(feastType FeastServiceType) feastd
474479
return feastdevv1alpha1.ServiceConfigs{}
475480
}
476481

482+
func (feast *FeastServices) getLogLevelForType(feastType FeastServiceType) *string {
483+
services := feast.Handler.FeatureStore.Status.Applied.Services
484+
switch feastType {
485+
case OfflineFeastType:
486+
if services.OfflineStore != nil && services.OfflineStore.LogLevel != "" {
487+
return &services.OfflineStore.LogLevel
488+
}
489+
case OnlineFeastType:
490+
if services.OnlineStore != nil && services.OnlineStore.LogLevel != "" {
491+
return &services.OnlineStore.LogLevel
492+
}
493+
case RegistryFeastType:
494+
if services.Registry != nil && services.Registry.LogLevel != "" {
495+
return &services.Registry.LogLevel
496+
}
497+
}
498+
return nil
499+
}
500+
477501
// GetObjectMeta returns the feast k8s object metadata
478502
func (feast *FeastServices) GetObjectMeta(feastType FeastServiceType) metav1.ObjectMeta {
479503
return metav1.ObjectMeta{Name: feast.GetFeastServiceName(feastType), Namespace: feast.Handler.FeatureStore.Namespace}

0 commit comments

Comments
 (0)