Skip to content

Add validation for primaryScalerReplicas field in the CRD#1702

Merged
stefanprodan merged 1 commit intofluxcd:mainfrom
aufarg:add-autoscaler-ref-validation
Nov 23, 2024
Merged

Add validation for primaryScalerReplicas field in the CRD#1702
stefanprodan merged 1 commit intofluxcd:mainfrom
aufarg:add-autoscaler-ref-validation

Conversation

@aufarg
Copy link
Contributor

@aufarg aufarg commented Sep 11, 2024

Canary object right now accepts values lower than one (1) for primaryScalerReplicas counts. This results in Flagger keep trying to update primary HPA replica count, but failing. The validation is added to prevent the invalid object definition to be applied in the first place.

@aufarg aufarg force-pushed the add-autoscaler-ref-validation branch 2 times, most recently from 82e9c4c to 35ec90a Compare September 11, 2024 06:52
Signed-off-by: Aufar Gilbran <aufargilbran@gmail.com>
@aufarg aufarg force-pushed the add-autoscaler-ref-validation branch from 35ec90a to b719427 Compare September 12, 2024 08:50
@stefanprodan stefanprodan changed the title Add validation for primaryScalerReplicas counts Add validation for primaryScalerReplicas field in the CRD Nov 23, 2024
Copy link
Member

@stefanprodan stefanprodan left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Thanks @aufarg

@stefanprodan stefanprodan merged commit 92daf51 into fluxcd:main Nov 23, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants