Skip to content
This repository has been archived by the owner on Jun 6, 2024. It is now read-only.

add 'enflame.com/dtu' to rest-server #5139

Closed
wants to merge 2 commits into from
Closed

add 'enflame.com/dtu' to rest-server #5139

wants to merge 2 commits into from

Conversation

beingj
Copy link
Contributor

@beingj beingj commented Dec 2, 2020

No description provided.

@ghost
Copy link

ghost commented Dec 2, 2020

CLA assistant check
All CLA requirements met.

@coveralls
Copy link

coveralls commented Dec 2, 2020

Coverage Status

Coverage remained the same at 33.999% when pulling e8fd0b6 on beingj:enflame into c15214f on microsoft:master.

Comment on lines +487 to +488
'enflame.com/dtu':
config.taskRoles[taskRole].resourcePerInstance.gpu,
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

what is this for? could you include a clear and detailed description explaining the reasons for the changes?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

enflame DTU(Deep Thinking Unit) is the AI process card from enflame.
With help of enflame k8s device plugin, the DTUs in worker nodes can be detected and allocated by K8S.
This commit adds the 'enflame.com/dtu' as another GPU resource to OpenPAI.

@beingj beingj requested a review from abuccts December 2, 2020 05:39
@mydmdm mydmdm linked an issue Dec 2, 2020 that may be closed by this pull request
4 tasks
@@ -484,6 +484,8 @@ const generateTaskRole = (
'github.com/fuse': 1,
'nvidia.com/gpu':
config.taskRoles[taskRole].resourcePerInstance.gpu,
'enflame.com/dtu':
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

  1. we don't have such card to test or use
  2. this pr won't work and it will break all nodes

pls feel free to keep a fork to support your new card, and pls do test your code before submitting a pr

@hzy46
Copy link
Contributor

hzy46 commented Jan 13, 2021

Close this. Related issues & PRs are tracked in #5138 now.

@hzy46 hzy46 closed this Jan 13, 2021
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.

add support to enflame DTO
4 participants