Add optional OpenFAAS chart

[gabor-boros]

This PR resolves #53 by adding an optional OpenFAAS chart to the dependencies.

Example configuration:

openfaas:
  enabled: true
  ingress:
    enabled: false
    hosts:
      - host: gateway.openfaas.local  # Replace with gateway.example.com if public-facing
        serviceName: gateway
        servicePort: 8080
        path: /
    annotations:
      kubernetes.io/ingress.class: nginx
      cert-manager.io/cluster-issuer: letsencrypt-prod
      nginx.ingress.kubernetes.io/ssl-redirect: "true" 
      nginx.ingress.kubernetes.io/force-ssl-redirect: "true"

[openedx-webhooks]

Thanks for the pull request, @gabor-boros! Please note that it may take us up to several weeks or months to complete a review and merge your PR.

Feel free to add as much of the following information to the ticket as you can:

• supporting documentation
• Open edX discussion forum threads
• timeline information ("this must be merged by XX date", and why that is)
• partner information ("this is a course on edx.org")
• any other information that can help Product understand the context for the PR

All technical communication about the code itself will be done via the GitHub pull request interface. As a reminder, our process documentation is here.

Please let us know once your PR is ready for our review and all tests are green.

[openedx-webhooks]

Thanks for the pull request, @gabor-boros! Please note that it may take us up to several weeks or months to complete a review and merge your PR.

Feel free to add as much of the following information to the ticket as you can:

• supporting documentation
• Open edX discussion forum threads
• timeline information ("this must be merged by XX date", and why that is)
• partner information ("this is a course on edx.org")
• any other information that can help Product understand the context for the PR

All technical communication about the code itself will be done via the GitHub pull request interface. As a reminder, our process documentation is here.

Please let us know once your PR is ready for our review and all tests are green.

[openedx-webhooks]

Thanks for the pull request, @gabor-boros! Please note that it may take us up to several weeks or months to complete a review and merge your PR.

Feel free to add as much of the following information to the ticket as you can:

• supporting documentation
• Open edX discussion forum threads
• timeline information ("this must be merged by XX date", and why that is)
• partner information ("this is a course on edx.org")
• any other information that can help Product understand the context for the PR

All technical communication about the code itself will be done via the GitHub pull request interface. As a reminder, our process documentation is here.

Please let us know once your PR is ready for our review and all tests are green.

[adzuci]

Could this statement link out to documentation showing some examples of how OpenFaas could be used or providing details on how to add a function?

[gabor-boros]

Although this could, I'm not convinced that the Harmony chart should contain "tutorial"/educational content. I mean, if someone enables OpenFAAS, probably the reason is that he/she knows why it is going to be enabled.

[adzuci]

I think OpenFaas is a great tool for hosting logic that glues things together. I am curious which things might make sense to use OpenFaas vs jobs vs something heavier like Argo Wrokflows and Argo Events. I also think this PR should include a change to https://github.com/openedx/openedx-k8s-harmony?tab=readme-ov-file#technology-stack-and-architecture detailing the newly togglable feature.

[jfavellar90]

Please add a new empty line at the end of the file

[gabor-boros]

👍 added!

[jfavellar90]

you need to bump the version of the harmony chart to trigger a new release

[gabor-boros]

Thank you! Would you like me to bump before or after review?

[jfavellar90]

@gabor-boros Hi, could you please rebase these changes please?

[mphilbrick211]

@gabor-boros Hi, could you please rebase these changes please?

Hi @gabor-boros! Friendly ping on this.

[gabor-boros]

@jfavellar90 I missed the ping here. Sorry. Will rebase tomorrow.

[gabor-boros]

@jfavellar90 Rebased and squashed the commits. Now it should be good for a final approval & merge.

[gabor-boros]

@jfavellar90 a gentle reminder here 😇

[openedx-webhooks]

@gabor-boros 🎉 Your pull request was merged! Please take a moment to answer a two question survey so we can improve your experience in the future.