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

Provide configuration option for counting invisible/visible messages in Azure Storage Queue scaler #4478

Open
tomkerkhove opened this issue Apr 26, 2023 · 3 comments
Labels
feature All issues for new features that have been committed to

Comments

@tomkerkhove
Copy link
Member

Proposal

I believe we should introduce a new configuration option that will influence this:

  metadata:
    queueName: orders
    queueLength: '5'
+   queueLengthStrategy: Default|VisibleOnly|InvisibleOnly
    activationQueueLength: '50'
    connectionFromEnv: STORAGE_CONNECTIONSTRING_ENV_NAME
    accountName: storage-account-name
    cloud: AzureUSGovernmentCloud

Use-Case

We've been going back and forward if Azure Storage Queue scaler should consider invisible during evaluation or not, but this is not ideal for everyone. Example #4432.

Is this a feature you are interested in implementing yourself?

No

Anything else?

No response

@tomkerkhove tomkerkhove added needs-discussion feature-request All issues for new features that have not been committed to feature All issues for new features that have been committed to and removed needs-discussion feature-request All issues for new features that have not been committed to labels Apr 26, 2023
@spewu
Copy link

spewu commented Jun 4, 2024

This would be perfect for us. For some specific queues, we leverage the visibility of messages in the queue, to postpone processing them. The current implementation means that KEDA will keep scaling up, because there are invisible messages in the queue, but the jobs it creates have nothing to do, since there are no visible messages in the queue.

So, being able to specify that for this particular trigger, we only want to consider the visible messages, would allow this to work for us without continuously creating useless jobs in the cluster.

@dgancho
Copy link

dgancho commented Jul 1, 2024

Hi @tomkerkhove et al, is there any known solution/workaround here or a path forward? The issue still persists and blocks from using Azure Container App Jobs as advertised.

@tomkerkhove
Copy link
Member Author

The PR is open as you can see above your comment.

Azure Container App Jobs as advertised

Mind elaborating on this? =)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature All issues for new features that have been committed to
Projects
Status: Proposed
Development

No branches or pull requests

3 participants