Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add omitempty tag everywhere in main spec #319

Closed
wants to merge 1 commit into from
Closed

Add omitempty tag everywhere in main spec #319

Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
24 changes: 12 additions & 12 deletions api/v1/ytsaurus_types.go
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -174,7 +174,7 @@ type BaseLoggerSpec struct {
Compression LogCompression `json:"compression,omitempty"`
//+kubebuilder:default:=false
//+optional
UseTimestampSuffix bool `json:"useTimestampSuffix"`
UseTimestampSuffix bool `json:"useTimestampSuffix,omitempty"`
RotationPolicy *LogRotationPolicy `json:"rotationPolicy,omitempty"`
}

Expand Down Expand Up @@ -440,7 +440,7 @@ type ExecNodesSpec struct {
Sidecars []string `json:"sidecars,omitempty"`
//+kubebuilder:default:=true
//+optional
Privileged bool `json:"privileged"`
Privileged bool `json:"privileged,omitempty"`
JobProxyLoggers []TextLoggerSpec `json:"jobProxyLoggers,omitempty"`
// Resources dedicated for running jobs.
//+optional
Expand Down Expand Up @@ -482,11 +482,11 @@ type UISpec struct {
// If defined allows insecure (over http) authentication.
//+kubebuilder:default:=true
//+optional
UseInsecureCookies bool `json:"useInsecureCookies"`
UseInsecureCookies bool `json:"useInsecureCookies,omitempty"`
// Use secure connection to the cluster's http-proxies.
//+kubebuilder:default:=false
//+optional
Secure bool `json:"secure"`
Secure bool `json:"secure,omitempty"`
Resources corev1.ResourceRequirements `json:"resources,omitempty"`
InstanceCount int32 `json:"instanceCount,omitempty"`

Expand Down Expand Up @@ -568,21 +568,21 @@ type CommonSpec struct {

//+kubebuilder:default:=false
//+optional
UseIPv6 bool `json:"useIpv6"`
UseIPv6 bool `json:"useIpv6,omitempty"`
//+kubebuilder:default:=false
//+optional
UseIPv4 bool `json:"useIpv4"`
UseIPv4 bool `json:"useIpv4,omitempty"`
//+kubebuilder:default:=true
//+optional
UseShortNames bool `json:"useShortNames"`
UseShortNames bool `json:"useShortNames,omitempty"`
// Use the host's network namespace for all components.
//+kubebuilder:default:=false
//+optional
HostNetwork bool `json:"hostNetwork"`
HostNetwork bool `json:"hostNetwork,omitempty"`

//+kubebuilder:default:=false
//+optional
UsePorto bool `json:"usePorto"`
UsePorto bool `json:"usePorto,omitempty"`

ExtraPodAnnotations map[string]string `json:"extraPodAnnotations,omitempty"`

Expand All @@ -601,15 +601,15 @@ type YtsaurusSpec struct {

//+kubebuilder:default:=true
//+optional
IsManaged bool `json:"isManaged"`
IsManaged bool `json:"isManaged,omitempty"`
//+kubebuilder:default:=true
//+optional
EnableFullUpdate bool `json:"enableFullUpdate"`
EnableFullUpdate bool `json:"enableFullUpdate,omitempty"`
//+optional
//+kubebuilder:validation:Enum={"","Nothing","StatelessOnly","MasterOnly","TabletNodesOnly","ExecNodesOnly","Everything"}
// UpdateSelector is an experimental field. Behaviour may change.
// If UpdateSelector is not empty EnableFullUpdate is ignored.
UpdateSelector UpdateSelector `json:"updateSelector"`
UpdateSelector UpdateSelector `json:"updateSelector,omitempty"`

Bootstrap *BootstrapSpec `json:"bootstrap,omitempty"`

Expand Down
Loading