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

[v2] Remove minReplicaCount and cooldownPeriod from ScaledJob #955

Closed
TsuyoshiUshio opened this issue Jul 30, 2020 · 7 comments
Closed

[v2] Remove minReplicaCount and cooldownPeriod from ScaledJob #955

TsuyoshiUshio opened this issue Jul 30, 2020 · 7 comments
Assignees
Labels
feature-request All issues for new features that have not been committed to needs-discussion
Milestone

Comments

@TsuyoshiUshio
Copy link
Contributor

A clear and concise description of what you want to happen.

Use-Case

For the ScaledJob on v2, minReplicaCount and coolddownPeriod is not make sense for ScaledJob.
I want to remove before release v2 beta. Do you agree with it?

Specification

apiVersion: v1
kind: Secret
metadata:
  name: rabbitmq-consumer
data:
  RabbitMqHost: YW1xcDovL3VzZXI6UEFTU1dPUkRAcmFiYml0bXEuZGVmYXVsdC5zdmMuY2x1c3Rlci5sb2NhbDo1Njcy
  LocalHost: YW1xcDovL3VzZXI6UEFTU1dPUkRAMTI3LjAuMC4xOjU2NzI=
---
apiVersion: keda.sh/v1alpha1
kind: ScaledJob
metadata:
  name: rabbitmq-consumer
  namespace: default
spec:
  jobTargetRef:
    template:
      spec:
        containers:
        - name: rabbitmq-client
          image: tsuyoshiushio/rabbitmq-client:dev3
          imagePullPolicy: Always
          command: ["receive",  "amqp://user:[email protected]:5672", "job"]
          envFrom:
            - secretRef:
                name: rabbitmq-consumer
        restartPolicy: Never
    backoffLimit: 4  
  pollingInterval: 10   # Optional. Default: 30 seconds
  cooldownPeriod: 30   # Optional. Default: 300 seconds   **<- removed from ScaledJob**
  minReplicaCount: 0    # Optional.  **<- removed from ScaledJob**
  maxReplicaCount: 30  # Optional. Default: 100
@TsuyoshiUshio TsuyoshiUshio added needs-discussion feature-request All issues for new features that have not been committed to labels Jul 30, 2020
@tomkerkhove
Copy link
Member

Makes sense.

@tomkerkhove tomkerkhove added this to the v2.0 milestone Jul 30, 2020
@TsuyoshiUshio
Copy link
Contributor Author

Thank you, @tomkerkhove
I'll create a new pr for this issue for v2!

@tomkerkhove
Copy link
Member

@tomkerkhove tomkerkhove reopened this Aug 3, 2020
@TsuyoshiUshio
Copy link
Contributor Author

Good catch! Thanks!

@zroubalik
Copy link
Member

Fixing in #971

@TsuyoshiUshio
Copy link
Contributor Author

Thanks. 👍

SpiritZhou pushed a commit to SpiritZhou/keda that referenced this issue Jul 18, 2023
@NamanChandra1
Copy link

I need this parameter for my scaledjob. What if i want the scaler to activate 1 hour after the previous run? In that use case I can set cooldownperiod as 1 hour.

Correct me if I am missing something

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature-request All issues for new features that have not been committed to needs-discussion
Projects
None yet
Development

No branches or pull requests

5 participants