search

intel / cloud-native-ai-pipeline

AI cloud native pipeline for confidential and sustainable computing
https://intel.github.io/cloud-native-ai-pipeline/
Apache License 2.0
37 stars 15 forks source link

container: Add redis + DTO image for DSA. #126

Closed pingzhaozz closed 1 year ago

pingzhaozz commented 1 year ago

This PR is to add the container for dsa enabled redis.

leyao-daily commented 1 year ago

Thanks for your changes, and

  1. For redis.conf, I checked and found just a few changes have been made. Is these changes necessary for redis dsa enabled?
  2. Why remove resource and I don't find anything changed in deployment.
pingzhaozz commented 1 year ago

Thanks for your changes, and

  1. For redis.conf, I checked and found just a few changes have been made. Is these changes necessary for redis dsa enabled?
  2. Why remove resource and I don't find anything changed in deployment.

redis.conf enable the multi io threads for better dsa offloading. I move the resource field in value.yaml to deployment.yaml because dsa has following resources:

        resources:
          limits:
            dsa.intel.com/wq-user-shared: 2

To avoid duplicate & override, better to keep 1 location