store icon indicating copy to clipboard operation
store copied to clipboard

Submission Request, imgmod function

Open psyhomb opened this issue 6 years ago • 15 comments

Hi guys, I would like to submit a review request for this Image Manipulation Function with hope it can be added to the OpenFaaS Function Store :slightly_smiling_face:

{
  "title": "Image Manipulation Function",
  "description": "Provide URL and use image manipulation function to download, scale, gray, flip, mirror, invert any online jpeg or png image",
  "image": "psyhomb/imgmod:latest",
  "name": "imgmod",
  "fprocess": "python3 index.py",
  "network": "func_functions",
  "repo_url": "https://github.com/psyhomb/serverless/tree/master/openfaas/imgmod",
  "environment": {
    "input_mode": "url",
    "output_mode": "image"
  },
  "labels": {
    "com.openfaas.ui.ext": "jpg"
  }
}

Thanks

psyhomb avatar Apr 21 '19 16:04 psyhomb

Anything new on this? :(

psyhomb avatar Apr 30 '19 20:04 psyhomb

Hi there, thanks for taking an interest. The usage information would really help here.

Alex

alexellis avatar Apr 30 '19 21:04 alexellis

Hi, I think everything has been explained in the README.md file.

Thanks

psyhomb avatar Apr 30 '19 21:04 psyhomb

This needs to run as a non root user and to write to /tmp/ only. Please can you take a look?

alexellis avatar Apr 30 '19 21:04 alexellis

Hey @alexellis, function is using in memory file object, in other words it doesn't require filesystem access, so I'm not quite sure what do you mean by it has to write to /tmp?

psyhomb avatar Apr 30 '19 22:04 psyhomb

Ok then please just make sure you use a non root user next.

alexellis avatar Apr 30 '19 22:04 alexellis

Will do that ASAP, thanks.

psyhomb avatar Apr 30 '19 22:04 psyhomb

Ok then please just make sure you use a non root user next.

Done, all processes running inside of container will be executed under non root user.

psyhomb avatar May 01 '19 00:05 psyhomb

Hey @alexellis is there anything else I have to change?

psyhomb avatar May 02 '19 16:05 psyhomb

Hey @alexellis what is going on with this submission, is this suppose to take this long, it's been almost a month?

psyhomb avatar May 18 '19 01:05 psyhomb

The OpenFaaS project is run by unpaid volunteers. As the maintainer I receive hundreds of GitHub notifications per week, so it's hard to keep track of all of them.

I'll check your PR again.

The best way to get on my radar is to message in a public channel on Slack or to DM me there.

alexellis avatar Jun 04 '19 18:06 alexellis

After deploying a function from the store, users will have no idea how to invoke your function. It's not your fault, but you could help them by providing a helper message when the user doesn't provide your expected input, or they give a non-JSON input.

What do you think?

alexellis avatar Jun 04 '19 18:06 alexellis

I'm also struggling to understand why the official Python template wasn't used for the function? I'd rather it was based upon something which is being maintained by the project.

Let us know on Slack if you need any technical support with templates or building custom modules.

Alex

-- Join Slack to connect with the community https://docs.openfaas.com

alexellis avatar Jun 04 '19 18:06 alexellis

After deploying a function from the store, users will have no idea how to invoke your function. It's not your fault, but you could help them by providing a helper message when the user doesn't provide your expected input, or they give a non-JSON input.

What do you think?

What about repo_url link to full documentation?

"repo_url": "https://github.com/psyhomb/serverless/tree/master/openfaas/imgmod",

psyhomb avatar Jun 05 '19 20:06 psyhomb

I'm also struggling to understand why the official Python template wasn't used for the function?

Development of imgmod function was started 2 years ago, at that time python3 template didn't even exist, I've also sent you python3 template suggestion on Slack at that time... so that's the reason why :slightly_smiling_face:.

I'd rather it was based upon something which is being maintained by the project.

This function is tested and it's working on OpenFaaS and AWS Lambda platform/service, is there any kind of issue you've detected?

psyhomb avatar Jun 05 '19 20:06 psyhomb