Add PS operator support#836

Draft

maxkondr wants to merge 6 commits intomainfrom

EVEREST-2058-PS-PoC

Contributor

maxkondr commented Jul 22, 2025

CHANGE DESCRIPTION

Problem:
EVEREST-0

Short explanation of the problem.

Related pull requests

[link]

Cause:
Short explanation of the root cause of the issue if applicable.

Solution:
Short explanation of the solution we are providing with this PR.

CHECKLIST

Helm chart

Is the helm chart updated with the new changes? (if applicable)

Jira

Is the Jira ticket created and referenced properly?

Tests

Is an Integration test/test case added for the new feature/change?
Are unit tests added where appropriate?


          Add PS operator support

d21dc95

maxkondr self-assigned this


          Merge branch 'main' into EVEREST-2058-PS-PoC

6f35f23

maxkondr added the auto-update-base label

maxkondr added 2 commits

July 30, 2025 13:26


          basic implementation

a9a6057


          Merge branch 'main' into EVEREST-2058-PS-PoC

ae32004

github-actions bot reviewed

View reviewed changes

Contributor

github-actions bot left a comment

Remaining comments which cannot be posted as a review comment to avoid GitHub Rate Limit

golangci-lint

[golangci-lint] _{reported by reviewdog 🐶}

https://github.com/percona/everest-operator/blob/ae32004497aac2822172c63999fcc78fed2e95b9/internal/controller/providers/ps/applier.go#L373-L373

[golangci-lint] _{reported by reviewdog 🐶}

https://github.com/percona/everest-operator/blob/ae32004497aac2822172c63999fcc78fed2e95b9/internal/controller/providers/ps/applier.go#L393-L393

[golangci-lint] _{reported by reviewdog 🐶}

https://github.com/percona/everest-operator/blob/ae32004497aac2822172c63999fcc78fed2e95b9/internal/controller/providers/ps/applier.go#L407-L407

[golangci-lint] _{reported by reviewdog 🐶}

https://github.com/percona/everest-operator/blob/ae32004497aac2822172c63999fcc78fed2e95b9/internal/controller/providers/ps/applier.go#L408-L408

[golangci-lint] _{reported by reviewdog 🐶}

https://github.com/percona/everest-operator/blob/ae32004497aac2822172c63999fcc78fed2e95b9/internal/controller/providers/ps/applier.go#L417-L417

[golangci-lint] _{reported by reviewdog 🐶}

https://github.com/percona/everest-operator/blob/ae32004497aac2822172c63999fcc78fed2e95b9/internal/controller/providers/ps/applier.go#L419-L419

[golangci-lint] _{reported by reviewdog 🐶}

https://github.com/percona/everest-operator/blob/ae32004497aac2822172c63999fcc78fed2e95b9/internal/controller/providers/ps/applier.go#L425-L425

[golangci-lint] _{reported by reviewdog 🐶}

https://github.com/percona/everest-operator/blob/ae32004497aac2822172c63999fcc78fed2e95b9/internal/controller/providers/ps/applier.go#L431-L431

[golangci-lint] _{reported by reviewdog 🐶}

https://github.com/percona/everest-operator/blob/ae32004497aac2822172c63999fcc78fed2e95b9/internal/controller/providers/ps/applier.go#L434-L434

[golangci-lint] _{reported by reviewdog 🐶}

https://github.com/percona/everest-operator/blob/ae32004497aac2822172c63999fcc78fed2e95b9/internal/controller/providers/ps/applier.go#L437-L437

[golangci-lint] _{reported by reviewdog 🐶}

https://github.com/percona/everest-operator/blob/ae32004497aac2822172c63999fcc78fed2e95b9/internal/controller/providers/ps/applier.go#L444-L444

[golangci-lint] _{reported by reviewdog 🐶}

https://github.com/percona/everest-operator/blob/ae32004497aac2822172c63999fcc78fed2e95b9/internal/controller/providers/ps/applier.go#L460-L460

[golangci-lint] _{reported by reviewdog 🐶}

https://github.com/percona/everest-operator/blob/ae32004497aac2822172c63999fcc78fed2e95b9/internal/controller/providers/ps/applier.go#L499-L499

[golangci-lint] _{reported by reviewdog 🐶}

https://github.com/percona/everest-operator/blob/ae32004497aac2822172c63999fcc78fed2e95b9/internal/controller/providers/ps/applier.go#L516-L516

[golangci-lint] _{reported by reviewdog 🐶}

https://github.com/percona/everest-operator/blob/ae32004497aac2822172c63999fcc78fed2e95b9/internal/controller/providers/ps/applier.go#L522-L522

[golangci-lint] _{reported by reviewdog 🐶}

https://github.com/percona/everest-operator/blob/ae32004497aac2822172c63999fcc78fed2e95b9/internal/controller/providers/ps/applier.go#L541-L541

[golangci-lint] _{reported by reviewdog 🐶}

https://github.com/percona/everest-operator/blob/ae32004497aac2822172c63999fcc78fed2e95b9/internal/controller/providers/ps/haproxy_configs.go#L27-L27

[golangci-lint] _{reported by reviewdog 🐶}

https://github.com/percona/everest-operator/blob/ae32004497aac2822172c63999fcc78fed2e95b9/internal/controller/providers/ps/provider.go#L61-L61

[golangci-lint] _{reported by reviewdog 🐶}

https://github.com/percona/everest-operator/blob/ae32004497aac2822172c63999fcc78fed2e95b9/internal/controller/providers/ps/provider.go#L73-L73

[golangci-lint] _{reported by reviewdog 🐶}

https://github.com/percona/everest-operator/blob/ae32004497aac2822172c63999fcc78fed2e95b9/internal/controller/providers/ps/provider.go#L102-L102

[golangci-lint] _{reported by reviewdog 🐶}

https://github.com/percona/everest-operator/blob/ae32004497aac2822172c63999fcc78fed2e95b9/internal/controller/providers/ps/provider.go#L107-L107

[golangci-lint] _{reported by reviewdog 🐶}

https://github.com/percona/everest-operator/blob/ae32004497aac2822172c63999fcc78fed2e95b9/internal/controller/providers/ps/provider.go#L125-L125

[golangci-lint] _{reported by reviewdog 🐶}

https://github.com/percona/everest-operator/blob/ae32004497aac2822172c63999fcc78fed2e95b9/internal/controller/providers/ps/provider.go#L131-L131

[golangci-lint] _{reported by reviewdog 🐶}

https://github.com/percona/everest-operator/blob/ae32004497aac2822172c63999fcc78fed2e95b9/internal/controller/providers/ps/provider.go#L152-L152

[golangci-lint] _{reported by reviewdog 🐶}

https://github.com/percona/everest-operator/blob/ae32004497aac2822172c63999fcc78fed2e95b9/internal/controller/providers/ps/provider.go#L186-L186

[golangci-lint] _{reported by reviewdog 🐶}

https://github.com/percona/everest-operator/blob/ae32004497aac2822172c63999fcc78fed2e95b9/internal/controller/providers/ps/provider.go#L222-L222

[golangci-lint] _{reported by reviewdog 🐶}

https://github.com/percona/everest-operator/blob/ae32004497aac2822172c63999fcc78fed2e95b9/internal/controller/providers/ps/provider.go#L227-L227

maxkondr added 2 commits

July 31, 2025 11:50


          Merge branch 'main' into EVEREST-2058-PS-PoC

2f440f5


          Merge branch 'main' into EVEREST-2058-PS-PoC

f04f369

github-actions bot reviewed

View reviewed changes

Contributor

github-actions bot left a comment

Remaining comments which cannot be posted as a review comment to avoid GitHub Rate Limit

golangci-lint

[golangci-lint] _{reported by reviewdog 🐶}

https://github.com/percona/everest-operator/blob/f04f369cd6b98229e04e1403338acd50f958ab39/internal/controller/providers/ps/applier.go#L373-L373

[golangci-lint] _{reported by reviewdog 🐶}

https://github.com/percona/everest-operator/blob/f04f369cd6b98229e04e1403338acd50f958ab39/internal/controller/providers/ps/applier.go#L393-L393

[golangci-lint] _{reported by reviewdog 🐶}

https://github.com/percona/everest-operator/blob/f04f369cd6b98229e04e1403338acd50f958ab39/internal/controller/providers/ps/applier.go#L407-L407

[golangci-lint] _{reported by reviewdog 🐶}

https://github.com/percona/everest-operator/blob/f04f369cd6b98229e04e1403338acd50f958ab39/internal/controller/providers/ps/applier.go#L408-L408

[golangci-lint] _{reported by reviewdog 🐶}

https://github.com/percona/everest-operator/blob/f04f369cd6b98229e04e1403338acd50f958ab39/internal/controller/providers/ps/applier.go#L417-L417

[golangci-lint] _{reported by reviewdog 🐶}

https://github.com/percona/everest-operator/blob/f04f369cd6b98229e04e1403338acd50f958ab39/internal/controller/providers/ps/applier.go#L419-L419

[golangci-lint] _{reported by reviewdog 🐶}

https://github.com/percona/everest-operator/blob/f04f369cd6b98229e04e1403338acd50f958ab39/internal/controller/providers/ps/applier.go#L425-L425

[golangci-lint] _{reported by reviewdog 🐶}

https://github.com/percona/everest-operator/blob/f04f369cd6b98229e04e1403338acd50f958ab39/internal/controller/providers/ps/applier.go#L431-L431

[golangci-lint] _{reported by reviewdog 🐶}

https://github.com/percona/everest-operator/blob/f04f369cd6b98229e04e1403338acd50f958ab39/internal/controller/providers/ps/applier.go#L434-L434

[golangci-lint] _{reported by reviewdog 🐶}

https://github.com/percona/everest-operator/blob/f04f369cd6b98229e04e1403338acd50f958ab39/internal/controller/providers/ps/applier.go#L437-L437

[golangci-lint] _{reported by reviewdog 🐶}

https://github.com/percona/everest-operator/blob/f04f369cd6b98229e04e1403338acd50f958ab39/internal/controller/providers/ps/applier.go#L444-L444

[golangci-lint] _{reported by reviewdog 🐶}

https://github.com/percona/everest-operator/blob/f04f369cd6b98229e04e1403338acd50f958ab39/internal/controller/providers/ps/applier.go#L460-L460

[golangci-lint] _{reported by reviewdog 🐶}

https://github.com/percona/everest-operator/blob/f04f369cd6b98229e04e1403338acd50f958ab39/internal/controller/providers/ps/applier.go#L499-L499

[golangci-lint] _{reported by reviewdog 🐶}

https://github.com/percona/everest-operator/blob/f04f369cd6b98229e04e1403338acd50f958ab39/internal/controller/providers/ps/applier.go#L516-L516

[golangci-lint] _{reported by reviewdog 🐶}

https://github.com/percona/everest-operator/blob/f04f369cd6b98229e04e1403338acd50f958ab39/internal/controller/providers/ps/applier.go#L522-L522

[golangci-lint] _{reported by reviewdog 🐶}

https://github.com/percona/everest-operator/blob/f04f369cd6b98229e04e1403338acd50f958ab39/internal/controller/providers/ps/applier.go#L541-L541

[golangci-lint] _{reported by reviewdog 🐶}

https://github.com/percona/everest-operator/blob/f04f369cd6b98229e04e1403338acd50f958ab39/internal/controller/providers/ps/haproxy_configs.go#L27-L27

[golangci-lint] _{reported by reviewdog 🐶}

https://github.com/percona/everest-operator/blob/f04f369cd6b98229e04e1403338acd50f958ab39/internal/controller/providers/ps/provider.go#L61-L61

[golangci-lint] _{reported by reviewdog 🐶}

https://github.com/percona/everest-operator/blob/f04f369cd6b98229e04e1403338acd50f958ab39/internal/controller/providers/ps/provider.go#L73-L73

[golangci-lint] _{reported by reviewdog 🐶}

https://github.com/percona/everest-operator/blob/f04f369cd6b98229e04e1403338acd50f958ab39/internal/controller/providers/ps/provider.go#L102-L102

[golangci-lint] _{reported by reviewdog 🐶}

https://github.com/percona/everest-operator/blob/f04f369cd6b98229e04e1403338acd50f958ab39/internal/controller/providers/ps/provider.go#L107-L107

[golangci-lint] _{reported by reviewdog 🐶}

https://github.com/percona/everest-operator/blob/f04f369cd6b98229e04e1403338acd50f958ab39/internal/controller/providers/ps/provider.go#L125-L125

[golangci-lint] _{reported by reviewdog 🐶}

https://github.com/percona/everest-operator/blob/f04f369cd6b98229e04e1403338acd50f958ab39/internal/controller/providers/ps/provider.go#L131-L131

[golangci-lint] _{reported by reviewdog 🐶}

https://github.com/percona/everest-operator/blob/f04f369cd6b98229e04e1403338acd50f958ab39/internal/controller/providers/ps/provider.go#L152-L152

[golangci-lint] _{reported by reviewdog 🐶}

https://github.com/percona/everest-operator/blob/f04f369cd6b98229e04e1403338acd50f958ab39/internal/controller/providers/ps/provider.go#L186-L186

[golangci-lint] _{reported by reviewdog 🐶}

https://github.com/percona/everest-operator/blob/f04f369cd6b98229e04e1403338acd50f958ab39/internal/controller/providers/ps/provider.go#L222-L222

[golangci-lint] _{reported by reviewdog 🐶}

https://github.com/percona/everest-operator/blob/f04f369cd6b98229e04e1403338acd50f958ab39/internal/controller/providers/ps/provider.go#L227-L227

internal/controller/common/helper.go

		@@ -703,7 +703,7 @@ func deleteBackupsForDatabase(
		return false, nil

Contributor

github-actions bot Aug 25, 2025

[golangci-lint] _{reported by reviewdog 🐶}

Suggested change

      
            	return false, nil
          
            	return false, nil

internal/controller/databaseengine_controller.go

    
              	installPlanRefs := getInstallPlanRefsForUpgrade(dbEngine, subscription, installPlans)

              	result := []everestv1alpha1.OperatorUpgrade{}

              	var result []everestv1alpha1.OperatorUpgrade

Contributor

github-actions bot Aug 25, 2025

[golangci-lint] _{reported by reviewdog 🐶}

Suggested change

      
            	var result []everestv1alpha1.OperatorUpgrade
          
            	var result []everestv1alpha1.OperatorUpgrade

internal/controller/databaseengine_controller.go

    
              func (r *DatabaseEngineReconciler) ensureDBEnginesInNamespaces(ctx context.Context, namespaces []string) ([]reconcile.Request, error) {

              	requests := []reconcile.Request{}

              	var requests []reconcile.Request

              	for _, ns := range namespaces {

Contributor

github-actions bot Aug 25, 2025

[golangci-lint] _{reported by reviewdog 🐶}

Suggested change

      
            	for _, ns := range namespaces {
          
            	for _, ns := range namespaces {

internal/controller/databaseengine_controller.go

    
              func getDatabaseEngineRequestsFromInstallPlan(_ context.Context, o client.Object) []reconcile.Request {

              	result := []reconcile.Request{}

              	var result []reconcile.Request

              	installPlan, ok := o.(*opfwv1alpha1.InstallPlan)

Contributor

github-actions bot Aug 25, 2025

[golangci-lint] _{reported by reviewdog 🐶}

Suggested change

      
            	installPlan, ok := o.(*opfwv1alpha1.InstallPlan)
          
            	installPlan, ok := o.(*opfwv1alpha1.InstallPlan)

internal/controller/providers/ps/applier.go

+              type applier struct {
+              	*Provider
+              	ctx context.Context //nolint:containedctx

Contributor

github-actions bot Aug 25, 2025

[golangci-lint] _{reported by reviewdog 🐶}

Suggested change

      
            	ctx context.Context //nolint:containedctx
          
            	ctx context.Context //nolint:containedctx

internal/controller/providers/ps/applier.go

+              	case everestv1alpha1.EngineSizeSmall:
+              		haProxy.PodSpec.Resources = haProxyResourceRequirementsSmall
+              	case everestv1alpha1.EngineSizeMedium:
+              		haProxy.PodSpec.Resources = haProxyResourceRequirementsMedium

Contributor

github-actions bot Aug 25, 2025

[golangci-lint] _{reported by reviewdog 🐶}

Suggested change

      
            		haProxy.PodSpec.Resources = haProxyResourceRequirementsMedium
          
            		haProxy.Resources = haProxyResourceRequirementsMedium

internal/controller/providers/ps/applier.go

+              	case everestv1alpha1.EngineSizeMedium:
+              		haProxy.PodSpec.Resources = haProxyResourceRequirementsMedium
+              	case everestv1alpha1.EngineSizeLarge:
+              		haProxy.PodSpec.Resources = haProxyResourceRequirementsLarge

Contributor

github-actions bot Aug 25, 2025

[golangci-lint] _{reported by reviewdog 🐶}

Suggested change

      
            		haProxy.PodSpec.Resources = haProxyResourceRequirementsLarge
          
            		haProxy.Resources = haProxyResourceRequirementsLarge

internal/controller/providers/ps/applier.go

+              	}
+              	if a.DB.Spec.Proxy.Replicas != nil {
+              		haProxy.PodSpec.Size = *a.DB.Spec.Proxy.Replicas

Contributor

github-actions bot Aug 25, 2025

[golangci-lint] _{reported by reviewdog 🐶}

Suggested change

      
            		haProxy.PodSpec.Size = *a.DB.Spec.Proxy.Replicas
          
            		haProxy.Size = *a.DB.Spec.Proxy.Replicas

internal/controller/providers/ps/applier.go

+              	if a.DB.Spec.Proxy.Replicas != nil {
+              		haProxy.PodSpec.Size = *a.DB.Spec.Proxy.Replicas
+              	} else {
+              		haProxy.PodSpec.Size = a.DB.Spec.Engine.Replicas

Contributor

github-actions bot Aug 25, 2025

[golangci-lint] _{reported by reviewdog 🐶}

Suggested change

      
            		haProxy.PodSpec.Size = a.DB.Spec.Engine.Replicas
          
            		haProxy.Size = a.DB.Spec.Engine.Replicas

internal/controller/providers/ps/applier.go

+              	}
+              	if a.DB.Spec.Proxy.Config != "" {
+              		haProxy.PodSpec.Configuration = a.DB.Spec.Proxy.Config

Contributor

github-actions bot Aug 25, 2025

[golangci-lint] _{reported by reviewdog 🐶}

Suggested change

      
            		haProxy.PodSpec.Configuration = a.DB.Spec.Proxy.Config
          
            		haProxy.Configuration = a.DB.Spec.Proxy.Config

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

auto-update-base