Skip to content

Commit

Permalink
fix: make qos field in the RoceOptimized sections optional
Browse files Browse the repository at this point in the history
If not specified, trust will be set to dscp and pfc to 0,0,0,1,0,0,0,0

Signed-off-by: Alexander Maslennikov <[email protected]>
  • Loading branch information
almaslennikov committed Nov 1, 2024
1 parent de1a58e commit 670917d
Show file tree
Hide file tree
Showing 5 changed files with 1 addition and 5 deletions.
2 changes: 1 addition & 1 deletion api/v1alpha1/nicconfigurationtemplate_types.go
Original file line number Diff line number Diff line change
Expand Up @@ -59,7 +59,7 @@ type RoceOptimizedSpec struct {
// Optimize RoCE
Enabled bool `json:"enabled"`
// Quality of Service settings
Qos *QosSpec `json:"qos"`
Qos *QosSpec `json:"qos,omitempty"`
}

// GpuDirectOptimizedSpec specifies GPU Direct optimization settings
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -164,7 +164,6 @@ spec:
type: object
required:
- enabled
- qos
type: object
required:
- linkType
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -143,7 +143,6 @@ spec:
type: object
required:
- enabled
- qos
type: object
required:
- linkType
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -164,7 +164,6 @@ spec:
type: object
required:
- enabled
- qos
type: object
required:
- linkType
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -143,7 +143,6 @@ spec:
type: object
required:
- enabled
- qos
type: object
required:
- linkType
Expand Down

0 comments on commit 670917d

Please sign in to comment.