Skip to content
This repository has been archived by the owner on Mar 7, 2020. It is now read-only.

Modified Dockerfile to run as non-root #8

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

Conversation

padiazg
Copy link
Contributor

@padiazg padiazg commented Feb 22, 2019

Signed-off-by: Patricio Diaz [email protected]

This is a WIP to solve issue #7

I need to test the native build toolchain aspect to assure it accomplishes all the requirements.

@alexellis
Copy link
Member

Hi, is this still WIP?

@padiazg
Copy link
Contributor Author

padiazg commented Mar 6, 2019

I tested this template using this:

handler.py

import numpy as np
import json

def handle(req):
    e = np.random.random((2,2))
    l = e.tolist()
    return json.dumps(l)

requirements.txt

numpy

Deployed using

$ faas-cli up --skip-push

After calling it from the UI got

[
  [
    0.18303540222696302,
    0.3272620080124268
  ],
  [
    0.3718849181724705,
    0.2596801229379512
  ]
]

So I think we can remove the WIP label

@padiazg padiazg changed the title WIP: Modified Dockerfile to run as non-root Modified Dockerfile to run as non-root Mar 6, 2019
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