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

Job deletion handler should consider DeletedFinalStateUnknown #58

Closed
hzxuzhonghu opened this issue Apr 2, 2019 · 4 comments
Closed
Labels
kind/bug Categorizes issue or PR as related to a bug.

Comments

@hzxuzhonghu
Copy link
Collaborator

in case delete event miss, there is a risk for memory leak

@hzxuzhonghu hzxuzhonghu added the bug label Apr 2, 2019
@TommyLike
Copy link
Contributor

👍

@TommyLike
Copy link
Contributor

@hzxuzhonghu can you add more detail regarding memory leak?

@asifdxtreme asifdxtreme added kind/bug Categorizes issue or PR as related to a bug. and removed bug labels Apr 8, 2019
@k82cn
Copy link
Member

k82cn commented Apr 27, 2019

is that fixed?

wangyuqing4 pushed a commit to wangyuqing4/volcano-1 that referenced this issue Apr 30, 2019
【Issue volcano-sh#58】新增mxnet在volcano上使用cpu资源训练mnist数据集示例

单号:  无

特性/模块名称: volcano-example

修改原因:

1. 新增mxnet在volcano上使用cpu资源训练mnist数据集示例

修改内容:

1. 新增mxnet在volcano上使用cpu资源训练mnist数据集示例

自验情况 自验通过

验证结果

![image](/uploads/0ff52de96b993b2d201353e0752d04bf/image.png)
![image](/uploads/fad0cae6a2767b1c894a986b12cdb2c3/image.png)
![image](/uploads/580d5aa4f9712f2439293da150caf222/image.png)
![image](/uploads/158c50ffc00a03bbfd15ab941fa69837/image.png)
![image](/uploads/5a537ab8b16f0459eeccdea5c5d08503/image.png)
![image](/uploads/3c10962abf5b5ec5a3de7da289121e3b/image.png)
![image](/uploads/18d3c4296987c1c8d37fbd7cafac5ab5/image.png)
![image](/uploads/9e7607ffb7c84241e6c75cbe0afd65f4/image.png)
![image](/uploads/5f70a5c28edfa49e2743f29e14e58408/image.png)
![image](/uploads/d7ca3c252952936e8d92d1ee30c60237/image.png)


Issues info:
Issue ID: 58
Title: 新增mxnet在volcano上使用cpu资源训练mnist数据集示例
Issue url: CBU-PaaS/Community/volcano/volcano#58


See merge request CBU-PaaS/Community/volcano/volcano!101
@hzxuzhonghu
Copy link
Collaborator Author

I think so.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/bug Categorizes issue or PR as related to a bug.
Projects
None yet
Development

No branches or pull requests

4 participants