Skip to content
This repository has been archived by the owner on Dec 15, 2021. It is now read-only.

Add the support for perl runtime. #1014

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

stanguturi
Copy link

  • Added the core implementation to support perl
    runtime for the kubeless.

  • Added few examples for using new perl runtime support.

  • Added detailed instructions/documentation in README.

Issue Ref: 936

Description:
Support for deploying perl functions is not present in Kubeless.

  * Added the core implementation to support perl
  runtime for the kubeless.

  * Added few examples for using new perl runtime support.

  * Added detailed instructions/documentation in README.
@stanguturi
Copy link
Author

I submitted a pull request but one check failed. https://circleci.com/gh/kubeless/kubeless/2418?utm_campaign=vcs-integration-link&utm_medium=referral&utm_source=github-build-link

I don't know how my patch will be causing that failure. Can I get any help?

Copy link
Contributor

@andresmgot andresmgot left a comment

Choose a reason for hiding this comment

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

Hi @stanguturi,

Thank you for this PR! For your information the runtimes code has been moved to this repository:

https://github.com/kubeless/runtimes

Can you send the PR there? The incubator folder will be fine while we figure it out the remaining TODOs in the runtime.

Also there is a couple of things missing like the runtime manifests and the Makefile tasks to deploy and test the examples. You can find an updated guide for adding runtimes here:

https://github.com/kubeless/runtimes/blob/master/DEVELOPER_GUIDE.md

The runtime code looks good so far so looking forward to the new addition! :)

be killed and an error is returned to the client.

## Default Route End-points
* Following are few default routes defined by `perl` framework in `kubelss`
Copy link
Contributor

Choose a reason for hiding this comment

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

there is a typo kubelss --> kubeless

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants