Skip to content

Commit

Permalink
fix: PR suggestions
Browse files Browse the repository at this point in the history
  • Loading branch information
Jorres committed Dec 28, 2023
1 parent 2cdd3cb commit 79c6dd2
Show file tree
Hide file tree
Showing 2 changed files with 7 additions and 4 deletions.
4 changes: 2 additions & 2 deletions api/v1alpha1/storage_types.go
Original file line number Diff line number Diff line change
Expand Up @@ -44,15 +44,15 @@ type StorageSpec struct {
// `false` means the default state of the system, all Pods running.
// +kubebuilder:default:=false
// +optional
Pause bool `json:"pause,omitempty"`
Pause bool `json:"pause"`

// Enables or disables operator's reconcile loop.
// `false` means all the Pods are running, but the reconcile is effectively turned off.
// `true` means the default state of the system, all Pods running, operator reacts
// to specification change of this Storage resource.
// +kubebuilder:default:=true
// +optional
OperatorSync bool `json:"operatorSync,omitempty"`
OperatorSync bool `json:"operatorSync"`

// (Optional) Name of the root storage domain
// Default: root
Expand Down
7 changes: 5 additions & 2 deletions e2e/tests/test-objects/objects.go
Original file line number Diff line number Diff line change
Expand Up @@ -55,6 +55,7 @@ func DefaultStorage(storageYamlConfigPath string) *v1alpha1.Storage {
},
Spec: v1alpha1.StorageSpec{
Nodes: 8,
OperatorSync: true,
Configuration: string(storageConfig),
Erasure: "block-4-2",
DataStore: []corev1.PersistentVolumeClaimSpec{},
Expand Down Expand Up @@ -103,7 +104,8 @@ func StorageWithStaticCredentials(storageYamlConfigPath string) *v1alpha1.Storag
Namespace: YdbNamespace,
},
Spec: v1alpha1.StorageSpec{
Nodes: 8,
Nodes: 8,
OperatorSync: true,
OperatorConnection: &v1alpha1.ConnectionOptions{
StaticCredentials: &v1alpha1.StaticCredentialsAuth{
Username: "root",
Expand Down Expand Up @@ -154,7 +156,8 @@ func DefaultDatabase() *v1alpha1.Database {
Namespace: YdbNamespace,
},
Spec: v1alpha1.DatabaseSpec{
Nodes: 8,
Nodes: 8,
OperatorSync: true,
Resources: &v1alpha1.DatabaseResources{
StorageUnits: []v1alpha1.StorageUnit{
{
Expand Down

0 comments on commit 79c6dd2

Please sign in to comment.