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.

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

Use Replica Override to manually scale a component #665

Merged
merged 34 commits into from
Sep 11, 2024
Merged
Show file tree
Hide file tree
Changes from 12 commits
Commits
Show all changes
34 commits
Select commit Hold shift + click to select a range
9144898
Use Replica Override to manually scale a component
Richard87 Aug 27, 2024
3de4fc8
parse component status
Richard87 Aug 27, 2024
c62637e
make readable
Richard87 Aug 27, 2024
c8911ab
handle missing k8s deployment
Richard87 Aug 27, 2024
feabafc
simplify code
Richard87 Aug 27, 2024
dac015b
use component pods instead of all pods
Richard87 Aug 28, 2024
14b1360
Deprecate and add reset-scale endpoints
Richard87 Aug 28, 2024
715228d
refactor component status
Richard87 Aug 29, 2024
b64134d
reafctor environment, aux resources, component spec
Richard87 Aug 30, 2024
3e3048e
bugfixes
Richard87 Sep 2, 2024
18e416b
bugfixesuse status from component, not auxiliar service
Richard87 Sep 2, 2024
f6d0991
upadte tests
Richard87 Sep 3, 2024
615af37
Upgrade golang ci lint, generate swagger, fix lint error
Richard87 Sep 3, 2024
0ffbe31
Update api/deployments/models/component_deployment.go
Richard87 Sep 3, 2024
c4306a7
Update api/environments/component_handler.go
Richard87 Sep 3, 2024
3a5f555
Update api/deployments/models/component_status.go
Richard87 Sep 3, 2024
302f3f7
Update api/deployments/models/component_status.go
Richard87 Sep 3, 2024
f2dd858
Update api/deployments/models/component_status.go
Richard87 Sep 3, 2024
6da2ded
Update api/deployments/models/component_status.go
Richard87 Sep 3, 2024
e146ad1
fix panic in tests
Richard87 Sep 3, 2024
70b68a3
fix lint bugs
Richard87 Sep 3, 2024
00be7a9
Update api/environments/environment_handler.go
Richard87 Sep 4, 2024
b985ceb
Update api/environments/environment_handler.go
Richard87 Sep 4, 2024
b8ecd65
remove unused function
Richard87 Sep 4, 2024
0fae6c2
Add documentation
Richard87 Sep 4, 2024
23a15e4
return err if deployment not found
Richard87 Sep 4, 2024
032646e
Update api/utils/predicate/kubernetes.go
Richard87 Sep 4, 2024
7e387b2
update swagger
Richard87 Sep 4, 2024
57165b7
remove outdated null check
Richard87 Sep 5, 2024
3b2ea00
remove outdated test, introduce WithComponentStatuserFunc
Richard87 Sep 5, 2024
83b320d
Test component actions with status
Richard87 Sep 5, 2024
274fc64
revert go 1.23
Richard87 Sep 5, 2024
9e4e9c4
cleanup unused code
Richard87 Sep 6, 2024
3aafadb
update radix-operator
Richard87 Sep 11, 2024
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
17 changes: 8 additions & 9 deletions api/deployments/deployment_handler.go
Original file line number Diff line number Diff line change
Expand Up @@ -13,7 +13,7 @@ import (
"github.com/equinor/radix-api/models"
"github.com/equinor/radix-common/utils/slice"
"github.com/equinor/radix-operator/pkg/apis/kube"
v1 "github.com/equinor/radix-operator/pkg/apis/radix/v1"
radixv1 "github.com/equinor/radix-operator/pkg/apis/radix/v1"
operatorUtils "github.com/equinor/radix-operator/pkg/apis/utils"
radixlabels "github.com/equinor/radix-operator/pkg/apis/utils/labels"
"k8s.io/apimachinery/pkg/api/errors"
Expand All @@ -28,7 +28,6 @@ type DeployHandler interface {
GetDeploymentsForApplicationEnvironment(ctx context.Context, appName, environment string, latest bool) ([]*deploymentModels.DeploymentSummary, error)
GetComponentsForDeploymentName(ctx context.Context, appName, deploymentID string) ([]*deploymentModels.Component, error)
GetComponentsForDeployment(ctx context.Context, appName, deploymentName, envName string) ([]*deploymentModels.Component, error)
GetLatestDeploymentForApplicationEnvironment(ctx context.Context, appName, environment string) (*deploymentModels.DeploymentSummary, error)
GetDeploymentsForPipelineJob(context.Context, string, string) ([]*deploymentModels.DeploymentSummary, error)
GetJobComponentDeployments(context.Context, string, string, string) ([]*deploymentModels.DeploymentItem, error)
}
Expand Down Expand Up @@ -64,6 +63,7 @@ func (deploy *deployHandler) GetLogs(ctx context.Context, appName, podName strin

return log, nil
}

return nil, deploymentModels.NonExistingPod(appName, podName)
}

Expand Down Expand Up @@ -198,7 +198,7 @@ func (deploy *deployHandler) GetDeploymentWithName(ctx context.Context, appName,
return dep, nil
}

func (deploy *deployHandler) getRadixDeploymentRadixJob(ctx context.Context, appName string, rd *v1.RadixDeployment) (*v1.RadixJob, error) {
func (deploy *deployHandler) getRadixDeploymentRadixJob(ctx context.Context, appName string, rd *radixv1.RadixDeployment) (*radixv1.RadixJob, error) {
jobName := rd.GetLabels()[kube.RadixJobNameLabel]
radixJob, err := kubequery.GetRadixJob(ctx, deploy.accounts.UserAccount.RadixClient, appName, jobName)
if err != nil {
Expand All @@ -211,15 +211,14 @@ func (deploy *deployHandler) getRadixDeploymentRadixJob(ctx context.Context, app
}

func (deploy *deployHandler) getEnvironmentNames(ctx context.Context, appName string) ([]string, error) {
radixlabels.ForApplicationName(appName).AsSelector()
labelSelector := radixlabels.ForApplicationName(appName).AsSelector()

reList, err := deploy.accounts.ServiceAccount.RadixClient.RadixV1().RadixEnvironments().List(ctx, metav1.ListOptions{LabelSelector: labelSelector.String()})
if err != nil {
return nil, err
}

return slice.Map(reList.Items, func(re v1.RadixEnvironment) string {
return slice.Map(reList.Items, func(re radixv1.RadixEnvironment) string {
return re.Spec.EnvName
}), nil
}
Expand All @@ -239,7 +238,7 @@ func (deploy *deployHandler) getDeployments(ctx context.Context, appName string,
rdLabelSelector = rdLabelSelector.Add(*jobNameLabel)
}

var radixDeploymentList []v1.RadixDeployment
var radixDeploymentList []radixv1.RadixDeployment
namespaces := slice.Map(environments, func(env string) string { return operatorUtils.GetEnvironmentNamespace(appName, env) })
for _, ns := range namespaces {
rdList, err := deploy.accounts.UserAccount.RadixClient.RadixV1().RadixDeployments(ns).List(ctx, metav1.ListOptions{LabelSelector: rdLabelSelector.String()})
Expand All @@ -250,7 +249,7 @@ func (deploy *deployHandler) getDeployments(ctx context.Context, appName string,
}

appNamespace := operatorUtils.GetAppNamespace(appName)
radixJobMap := make(map[string]*v1.RadixJob)
radixJobMap := make(map[string]*radixv1.RadixJob)

if jobName != "" {
radixJob, err := deploy.accounts.UserAccount.RadixClient.RadixV1().RadixJobs(appNamespace).Get(ctx, jobName, metav1.GetOptions{})
Expand Down Expand Up @@ -278,7 +277,7 @@ func (deploy *deployHandler) getDeployments(ctx context.Context, appName string,
rds := sortRdsByActiveFromDesc(radixDeploymentList)
var deploymentSummaries []*deploymentModels.DeploymentSummary
for _, rd := range rds {
if latest && rd.Status.Condition == v1.DeploymentInactive {
if latest && rd.Status.Condition == radixv1.DeploymentInactive {
continue
}

Expand All @@ -298,7 +297,7 @@ func (deploy *deployHandler) getDeployments(ctx context.Context, appName string,
return deploymentSummaries, nil
}

func sortRdsByActiveFromDesc(rds []v1.RadixDeployment) []v1.RadixDeployment {
func sortRdsByActiveFromDesc(rds []radixv1.RadixDeployment) []radixv1.RadixDeployment {
sort.Slice(rds, func(i, j int) bool {
if rds[j].Status.ActiveFrom.IsZero() {
return true
Expand Down
15 changes: 0 additions & 15 deletions api/deployments/mock/deployment_handler_mock.go

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

3 changes: 3 additions & 0 deletions api/deployments/models/component_builder.go
Original file line number Diff line number Diff line change
Expand Up @@ -54,6 +54,7 @@ type componentBuilder struct {
gitTags string
resources *radixv1.ResourceRequirements
runtime *radixv1.Runtime
replicasOverride *int
}

func (b *componentBuilder) WithStatus(status ComponentStatus) ComponentBuilder {
Expand Down Expand Up @@ -99,6 +100,7 @@ func (b *componentBuilder) WithComponent(component radixv1.RadixCommonDeployComp
b.commitID = component.GetEnvironmentVariables()[defaults.RadixCommitHashEnvironmentVariable]
b.gitTags = component.GetEnvironmentVariables()[defaults.RadixGitTagsEnvironmentVariable]
b.runtime = component.GetRuntime()
b.replicasOverride = component.GetReplicasOverride()

ports := []Port{}
if component.GetPorts() != nil {
Expand Down Expand Up @@ -252,6 +254,7 @@ func (b *componentBuilder) BuildComponent() (*Component, error) {
Variables: variables,
Replicas: b.podNames,
ReplicaList: b.replicaSummaryList,
ReplicasOverride: b.replicasOverride,
SchedulerPort: b.schedulerPort,
ScheduledJobPayloadPath: b.scheduledJobPayloadPath,
AuxiliaryResource: b.auxResource,
Expand Down
8 changes: 8 additions & 0 deletions api/deployments/models/component_deployment.go
Original file line number Diff line number Diff line change
Expand Up @@ -85,6 +85,14 @@ type Component struct {
// required: false
ReplicaList []ReplicaSummary `json:"replicaList"`

// Set if manuall control of replicas is in place. null means automatic controll, 0 means stopped and >= 1 is manually scaled.
Richard87 marked this conversation as resolved.
Show resolved Hide resolved
//
// required: false
// example: 5
// Extensions:
// x-nullable: true
ReplicasOverride *int `json:"replicasOverride"`

// HorizontalScaling defines horizontal scaling summary for this component
//
// required: false
Expand Down
61 changes: 52 additions & 9 deletions api/deployments/models/component_status.go
Original file line number Diff line number Diff line change
@@ -1,6 +1,17 @@
package models

import appsv1 "k8s.io/api/apps/v1"
import (
"strings"

"github.com/equinor/radix-api/api/utils/owner"
commonutils "github.com/equinor/radix-common/utils"
"github.com/equinor/radix-common/utils/pointers"
operatordefaults "github.com/equinor/radix-operator/pkg/apis/defaults"
"github.com/equinor/radix-operator/pkg/apis/kube"
radixv1 "github.com/equinor/radix-operator/pkg/apis/radix/v1"
"github.com/rs/zerolog/log"
appsv1 "k8s.io/api/apps/v1"
)

// ComponentStatus Enumeration of the statuses of component
type ComponentStatus int
Expand Down Expand Up @@ -31,15 +42,47 @@ func (p ComponentStatus) String() string {
return [...]string{"Stopped", "Consistent", "Reconciling", "Restarting", "Outdated"}[p]
}

func ComponentStatusFromDeployment(deployment *appsv1.Deployment) ComponentStatus {
status := ConsistentComponent
func ComponentStatusFromDeployment(component radixv1.RadixCommonDeployComponent, kd *appsv1.Deployment, rd *radixv1.RadixDeployment) ComponentStatus {
if kd == nil || kd.GetName() == "" {
return ComponentReconciling
}
replicasUnavailable := kd.Status.UnavailableReplicas
replicasReady := kd.Status.ReadyReplicas
replicas := pointers.Val(kd.Spec.Replicas)

if isCopmonentRestarting(component, rd) {
Richard87 marked this conversation as resolved.
Show resolved Hide resolved
return ComponentRestarting
}

if !owner.VerifyCorrectObjectGeneration(rd, kd, kube.RadixDeploymentObservedGeneration) {
return ComponentOutdated
}

if replicas == 0 {
return StoppedComponent
}

switch {
case deployment.Status.Replicas == 0:
status = StoppedComponent
case deployment.Status.UnavailableReplicas > 0:
status = ComponentReconciling
// Check if component is scaling up or down
if replicasUnavailable > 0 || replicas < replicasReady {
return ComponentReconciling
}

return status
return ConsistentComponent
}

func isCopmonentRestarting(component radixv1.RadixCommonDeployComponent, rd *radixv1.RadixDeployment) bool {
Richard87 marked this conversation as resolved.
Show resolved Hide resolved
restarted := component.GetEnvironmentVariables()[operatordefaults.RadixRestartEnvironmentVariable]
if strings.EqualFold(restarted, "") {
Richard87 marked this conversation as resolved.
Show resolved Hide resolved
return false
}
restartedTime, err := commonutils.ParseTimestamp(restarted)
if err != nil {
log.Logger.Warn().Err(err).Msgf("unable to parse restarted time %v, component: %s", restarted, component.GetName())
return false
}
reconciledTime := rd.Status.Reconciled
if reconciledTime.IsZero() || restartedTime.After(reconciledTime.Time) {
return true
}
return false
Richard87 marked this conversation as resolved.
Show resolved Hide resolved
}
73 changes: 73 additions & 0 deletions api/deployments/models/component_status_test.go
Original file line number Diff line number Diff line change
@@ -0,0 +1,73 @@
package models_test

import (
"testing"
"time"

"github.com/equinor/radix-api/api/deployments/models"
radixutils "github.com/equinor/radix-common/utils"
"github.com/equinor/radix-common/utils/pointers"
operatordefaults "github.com/equinor/radix-operator/pkg/apis/defaults"
"github.com/equinor/radix-operator/pkg/apis/kube"
radixv1 "github.com/equinor/radix-operator/pkg/apis/radix/v1"
"github.com/stretchr/testify/assert"
appsv1 "k8s.io/api/apps/v1"
metav1 "k8s.io/apimachinery/pkg/apis/meta/v1"
)

func TestNoKubeDeployments_IsReconciling(t *testing.T) {
status := models.ComponentStatusFromDeployment(&radixv1.RadixDeployComponent{}, nil, nil)
assert.Equal(t, models.ComponentReconciling, status)
}

func TestKubeDeploymentsWithRestartLabel_IsRestarting(t *testing.T) {
status := models.ComponentStatusFromDeployment(
&radixv1.RadixDeployComponent{EnvironmentVariables: map[string]string{operatordefaults.RadixRestartEnvironmentVariable: radixutils.FormatTimestamp(time.Now())}},
createKubeDeployment(0),
&radixv1.RadixDeployment{
ObjectMeta: metav1.ObjectMeta{Generation: 2},
Status: radixv1.RadixDeployStatus{Reconciled: metav1.NewTime(time.Now().Add(-10 * time.Minute))},
})

assert.Equal(t, models.ComponentRestarting, status)
}

func TestKubeDeploymentsWithoutReplicas_IsStopped(t *testing.T) {
status := models.ComponentStatusFromDeployment(
&radixv1.RadixDeployComponent{},
createKubeDeployment(0),
&radixv1.RadixDeployment{
ObjectMeta: metav1.ObjectMeta{Generation: 1},
})
assert.Equal(t, models.StoppedComponent, status)
}

func TestKubeDeployment_IsConsistent(t *testing.T) {
status := models.ComponentStatusFromDeployment(
&radixv1.RadixDeployComponent{},
createKubeDeployment(1),
&radixv1.RadixDeployment{
ObjectMeta: metav1.ObjectMeta{Generation: 1},
})
assert.Equal(t, models.ConsistentComponent, status)
}

func TestKubeDeployment_IsOutdated(t *testing.T) {
status := models.ComponentStatusFromDeployment(
&radixv1.RadixDeployComponent{},
createKubeDeployment(1),
&radixv1.RadixDeployment{
ObjectMeta: metav1.ObjectMeta{Generation: 2},
})
assert.Equal(t, models.ComponentOutdated, status)
}

func createKubeDeployment(replicas int32) *appsv1.Deployment {
return &appsv1.Deployment{
ObjectMeta: metav1.ObjectMeta{
Name: "helloworld",
Annotations: map[string]string{kube.RadixDeploymentObservedGeneration: "1"},
OwnerReferences: []metav1.OwnerReference{{Controller: pointers.Ptr(true)}}},
Spec: appsv1.DeploymentSpec{Replicas: pointers.Ptr[int32](replicas)},
}
}
Loading