Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Adding separate pod/deployment option to documentation #4092

Closed
saeid93 opened this issue May 11, 2022 · 4 comments · Fixed by #4144
Closed

Adding separate pod/deployment option to documentation #4092

saeid93 opened this issue May 11, 2022 · 4 comments · Fixed by #4144

Comments

@saeid93
Copy link
Contributor

saeid93 commented May 11, 2022

Hi Seldon team,

I followed the https://docs.seldon.io/projects/seldon-core/en/latest/examples/transformers-v2-protocol.html and https://docs.seldon.io/projects/seldon-core/en/latest/examples/graph-metadata.html to make a simple inference graph and I first ended up with the following scheme:

apiVersion: machinelearning.seldon.io/v1
kind: SeldonDeployment
metadata:
  name: linear-pipeline-single-pod
spec:
  name: linear-pipeline
  predictors:
  - componentSpecs:
    - spec:
        containers:
        - image: sdghafouri/linearmodel:nodeone
          name: node-one
        - image: sdghafouri/linearmodel:nodetwo
          name: node-two
        - image: sdghafouri/linearmodel:nodethree
          name: node-three
    graph:
      name: node-one
      type: MODEL
      children:
      - name: node-two
        type: MODEL
        children:
        - name: node-three
          type: MODEL
          children: []
    labels:
      sidecar.istio.io/inject: "true"
    name: example

which was pretty similar to most of the examples in the documentation, however, for my usecase I needed to have three different pods for each nodes instead of having them in the same pod and I struggled to find a solution to this, I finally ended up with the following which I think is doing what I need:

apiVersion: machinelearning.seldon.io/v1
kind: SeldonDeployment
metadata:
  name: linear-pipeline-separate-pods
spec:
  name: linear-pipeline
  annotations:
    seldon.io/engine-separate-pod: "true"
  predictors:
  - componentSpecs:
    - spec:
        containers:
        - image: sdghafouri/linearmodel:nodeone
          name: node-one
          imagePullPolicy: Always
    - spec:
        containers:
        - image: sdghafouri/linearmodel:nodetwo
          name: node-two
          imagePullPolicy: Always
    - spec:
        containers:
          - image: sdghafouri/linearmodel:nodethree
            name: node-three
            imagePullPolicy: Always
    graph:
      name: node-one
      type: MODEL
      children:
      - name: node-two
        type: MODEL
        children:
        - name: node-three
          type: MODEL
          children: []
    labels:
      sidecar.istio.io/inject: "true"
    name: example

I couldn't find a solid explanation of the differences between the two cases in the documentation and the availability of the second method (separate pods) and I accidentally find my answer when I was checking how to replicate each node in the https://docs.seldon.io/projects/seldon-core/en/latest/graph/scaling.html Therefore I think it would be nice to add some explanation somewhere in the document about the distinction and availability of both options and maybe the pros and cons (e.g. I think the per node scaling is only available in the separated pod options).

Thank you!

@axsaucedo
Copy link
Contributor

Thank you very much for adding this feedback on the documentation - we would be quite keen to add a note to highlight this, there are some points that are a bit hidden so may be worth rethinking where this may fit best - would you be interseted to add this as a documentation contribution? Recently @edshee restructured the docs so he could best placed to point where in the docs it could be most appropriate if you would be interested to contribute

@saeid93
Copy link
Contributor Author

saeid93 commented May 11, 2022

your welcome! Sure, if you could let me know which directory it should be added I'll be glad to contribute to the documentation.

@ukclivecox
Copy link
Contributor

You could add to doc/source/graph

@saeid93
Copy link
Contributor Author

saeid93 commented May 23, 2022

@cliveseldon Thank you, I'll add it this week.

saeid93 added a commit to saeid93/seldon-core that referenced this issue Jun 9, 2022
axsaucedo pushed a commit that referenced this issue Jun 10, 2022
* Create graph-modes.md

Describing the #4092

* link added and typo fixed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants