faas-netes icon indicating copy to clipboard operation
faas-netes copied to clipboard

Add queue-worker chart

Open welteki opened this issue 2 years ago • 3 comments

Signed-off-by: Han Verstraete (OpenFaaS Ltd) [email protected]

Description

Add separate chart for the queue-worker

Motivation and Context

Allow people to set up named queues

How Has This Been Tested?

  1. Use the chart to setup an additional queue
helm upgrade slow-queue --install chart/queue-worker \
  --namespace openfaas \
  --set maxInflight=5 \
  --set nats.stream.name=slow-queue \
  --set nats.consumer.durableName=slow-queue-workers
  1. Deploy a function with the correct annotations
faas-cli store deploy sleep --annotation com.openfaas.queue=slow-queue
  1. Invoke the function an check the logs for the slow-queue
echo "" | faas-cli invoke sleep --async 

stern slow-queue -n openfaas

Screenshot 2022-07-22 at 20 23 10

Types of changes

  • [ ] Bug fix (non-breaking change which fixes an issue)
  • [x] New feature (non-breaking change which adds functionality)
  • [ ] Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

  • [x] My code follows the code style of this project.
  • [x] My change requires a change to the documentation.
  • [ ] I have updated the documentation accordingly.
  • [x] I've read the CONTRIBUTION guide
  • [x] I have signed-off my commits with git commit -s
  • [ ] I have added tests to cover my changes.
  • [ ] All new and existing tests passed.

welteki avatar Jul 22 '22 18:07 welteki

Perhaps we should add --set upstreamTimeout=15m or similar to the example?

alexellis avatar Aug 02 '22 14:08 alexellis

FYI this needs a rebase

alexellis avatar Sep 08 '22 12:09 alexellis

The PR has been rebased.

welteki avatar Sep 08 '22 12:09 welteki