Skip to content

Show min/max replica count when using kubectl get scaledobject#1534

Merged
zroubalik merged 5 commits into
kedacore:mainfrom
Ritikaa96:add-min-max-replicaCount
Jan 26, 2021
Merged

Show min/max replica count when using kubectl get scaledobject#1534
zroubalik merged 5 commits into
kedacore:mainfrom
Ritikaa96:add-min-max-replicaCount

Conversation

@Ritikaa96

@Ritikaa96 Ritikaa96 commented Jan 25, 2021

Copy link
Copy Markdown
Contributor

Signed-off-by: Ritikaa96 ritika@india.nec.com

Checklist

Fixes #1410

Signed-off-by: Ritikaa96 <ritika@india.nec.com>
Comment thread api/v1alpha1/scaledobject_types.go Outdated
- jsonPath: .spec.triggers[*].type
name: Triggers
type: string
- jsonPath: .spec.minReplicaCount

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

We'll have to update our Helm chart as well; taking mental note

…ectl get so

Signed-off-by: Ritikaa96 <ritika@india.nec.com>

@zroubalik zroubalik left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Looking good, @Ritikaa96 could you please add the same for ScaledJob?

Comment thread api/v1alpha1/scaledobject_types.go
…sing kubectl get scaledjob

Signed-off-by: Ritikaa96 <ritika@india.nec.com>
@Ritikaa96

Ritikaa96 commented Jan 26, 2021

Copy link
Copy Markdown
Contributor Author

@tomkerkhove @zroubalik @coderanger I've shuffled the min/max in scaledObject and added min/max for scaledJob. PTAL.

@zroubalik zroubalik left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

@Ritikaa96 could you please update Changelog (improvements section) :)

@Ritikaa96

Copy link
Copy Markdown
Contributor Author

@Ritikaa96 could you please update Changelog (improvements section) :)

Sure.

…bject

Signed-off-by: Ritikaa96 <ritika@india.nec.com>
Comment thread CHANGELOG.md Outdated
…get scaledobject/scaledjob

Signed-off-by: Ritikaa96 <ritika@india.nec.com>

@zroubalik zroubalik left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM

@zroubalik zroubalik merged commit bc725e6 into kedacore:main Jan 26, 2021
@tomkerkhove

Copy link
Copy Markdown
Member

Thanks @Ritikaa96!

ycabrer pushed a commit to ycabrer/keda that referenced this pull request Mar 1, 2021
…job (kedacore#1534)

Signed-off-by: Ritikaa96 <ritika@india.nec.com>
Rodolfodc pushed a commit to sidilabs/keda that referenced this pull request Mar 11, 2021
…job (kedacore#1534)

Signed-off-by: Ritikaa96 <ritika@india.nec.com>
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.

Show min/max replica count when using kubectl get scaledobject

4 participants