Skip to content

Commit

Permalink
Updated Python wrapper application folder owner to default 8888 (#3674)
Browse files Browse the repository at this point in the history
* Updated folder owner to default 8888

* Added docs
  • Loading branch information
axsaucedo authored Oct 18, 2021
1 parent 3a162d0 commit c41ddaf
Show file tree
Hide file tree
Showing 3 changed files with 16 additions and 0 deletions.
10 changes: 10 additions & 0 deletions doc/source/reference/upgrading.md
Original file line number Diff line number Diff line change
Expand Up @@ -6,6 +6,16 @@ If you were running our Openshift 0.4.2 certified operator and are looking to up

Make sure you also [read the CHANGELOG](./changelog.html) to see the detailed features and bug-fixes in each version.


## Upgrading to 1.12

### Updated Python wrapper folder configurations

* The default running user in Seldon Core is 8888
* Some servers like the MLFlow Server v1 installs installations in runtime
* Access required to modify files in the local folder are required so the application folder should be writable
* The default base image now changes the owner of the /microservice folder to user 8888

## Upgrading to 1.11

### Python S2I Wrapper
Expand Down
3 changes: 3 additions & 0 deletions wrappers/s2i/python/Dockerfile
Original file line number Diff line number Diff line change
Expand Up @@ -31,4 +31,7 @@ RUN cd /microservice/python && pip install .
RUN mkdir -p /.conda && chmod a+rwx /.conda
RUN yum -y update-minimal --security --sec-severity=Important --sec-severity=Critical

# Default user is 8888 so changing for permissiosn to modify (needed in mlflow server)
RUN chown -R 8888 /microservice

EXPOSE 5000
3 changes: 3 additions & 0 deletions wrappers/s2i/python/Dockerfile.gpu
Original file line number Diff line number Diff line change
Expand Up @@ -46,4 +46,7 @@ RUN cd /microservice/python && pip install .

RUN mkdir -p /.conda && chmod a+rwx /.conda

# Default user is 8888 so changing for permissiosn to modify (needed in mlflow server)
RUN chown -R 8888 /microservice

EXPOSE 5000

0 comments on commit c41ddaf

Please sign in to comment.