Friday, 2016-10-28

*** iceyao has joined #openstack-zun00:35
*** yanyanhu has joined #openstack-zun01:24
*** sudipto has joined #openstack-zun03:03
*** sheel has joined #openstack-zun03:10
*** sudipto has quit IRC03:24
*** GB21 has joined #openstack-zun04:37
*** mkrai has joined #openstack-zun04:37
*** GB21 has quit IRC04:42
*** sheel has quit IRC05:20
*** mkrai has quit IRC05:26
*** GB21 has joined #openstack-zun05:56
*** pksingh has joined #openstack-zun06:36
*** GB21 has quit IRC06:47
*** zhurong has joined #openstack-zun06:57
*** GB21 has joined #openstack-zun06:59
*** coolsvap has joined #openstack-zun06:59
*** zhurong_ has joined #openstack-zun06:59
*** zhurong has quit IRC07:02
*** zhurong_ has quit IRC07:03
*** shubhams has left #openstack-zun07:05
*** pksingh has quit IRC07:14
*** harlowja has joined #openstack-zun07:17
*** GB21 has quit IRC07:19
*** GB21 has joined #openstack-zun07:33
eliqiaomkrai, sure, I commented on it.07:50
openstackgerritPradeep Kumar Singh proposed openstack/zun: Add status_reason column to 'container' table  https://review.openstack.org/39136608:05
*** sheel has joined #openstack-zun08:40
*** vikasc has quit IRC08:42
openstackgerritDeepak proposed openstack/zun-ui: Enable DeprecationWarning in test environments  https://review.openstack.org/39075708:48
*** iceyao has quit IRC08:57
*** iceyao has joined #openstack-zun08:57
*** janki has joined #openstack-zun09:04
*** iceyao_ has joined #openstack-zun09:21
*** iceyao has quit IRC09:25
*** GB21 has quit IRC09:29
*** pksingh has joined #openstack-zun09:34
pksingheliqiao: Hi , are u there?09:35
eliqiaoyes pksingh09:44
eliqiaowhat can I help?09:44
pksingheliqiao: its about https://review.openstack.org/#/c/391366/09:45
pksingheliqiao: so you want me to write the UTs, correct09:45
*** yanyanhu has quit IRC09:46
eliqiaopksingh: yes. and I don't think you finish that bug09:46
eliqiaowhich is reported by myself, right?09:46
eliqiaodo you have following patches ?09:46
pksinghyes, https://bugs.launchpad.net/zun/+bug/163476009:46
openstackLaunchpad bug 1634760 in Zun "Add fail reason to container object" [Wishlist,In progress] - Assigned to Pradeep Kumar Singh (pradeep-singh-u)09:46
pksingheliqiao: will fix above bug in next patch09:46
pksingheliqiao: is it ok?09:47
*** yanyanhu has joined #openstack-zun09:47
eliqiaookay, so please don't use closes-bug, you can use partial(something like this)09:47
pksingheliqiao: i mentioned the same in commit message09:47
eliqiaopksingh: okay feel free to split patches.09:47
pksingheliqiao: there are two bugs for the same09:48
pksingheliqiao: so i thought to submit two patches09:48
eliqiaosure, I see. but closes-bugs tag is not corrext.09:48
*** yanyanhu has quit IRC09:48
eliqiao2 bugs?09:48
eliqiaoreally?09:48
pksinghhttps://bugs.launchpad.net/zun/+bug/163476009:49
openstackLaunchpad bug 1634760 in Zun "Add fail reason to container object" [Wishlist,In progress] - Assigned to Pradeep Kumar Singh (pradeep-singh-u)09:49
pksinghhttps://bugs.launchpad.net/zun/+bug/161439809:49
openstackLaunchpad bug 1614398 in Zun "Show status reason for container" [High,In progress] - Assigned to Pradeep Kumar Singh (pradeep-singh-u)09:49
pksingheliqiao: so do you feel it ok? i will add unit test to patch09:51
eliqiaopksingh: this patch https://review.openstack.org/#/c/391366/ should closes bug 1634760, not 161439809:51
openstackbug 1634760 in Zun "Add fail reason to container object" [Wishlist,In progress] https://launchpad.net/bugs/1634760 - Assigned to Pradeep Kumar Singh (pradeep-singh-u)09:51
eliqiaookay, correct closes-bug and add more ut for this patch, then submit another patch on top of this one to closes 161439809:52
pksingheliqiao: ok sure thanks :)09:52
eliqiaopksingh: thanks the the ping :)09:54
*** harlowja has quit IRC09:55
*** harlowja has joined #openstack-zun09:55
openstackgerritPradeep Kumar Singh proposed openstack/zun: Add status_reason column to 'container' table  https://review.openstack.org/39136609:56
*** vikasc has joined #openstack-zun10:00
*** pksingh has quit IRC10:08
*** zhurong has joined #openstack-zun10:10
*** zhurong has quit IRC10:12
*** iceyao_ has quit IRC10:12
*** zhurong_ has joined #openstack-zun10:16
*** janki has quit IRC10:30
*** zhurong_ has quit IRC10:45
*** zhurong has joined #openstack-zun10:45
*** zhurong has quit IRC10:50
*** harlowja has quit IRC11:04
*** GB21 has joined #openstack-zun11:59
*** harlowja has joined #openstack-zun12:08
openstackgerritPradeep Kumar Singh proposed openstack/zun: Add status_reason column to 'container' table  https://review.openstack.org/39136612:12
*** vikasc has quit IRC12:20
*** hehx has quit IRC12:20
*** harlowja has quit IRC12:33
*** hehx has joined #openstack-zun12:36
*** janki has joined #openstack-zun12:36
*** vikasc has joined #openstack-zun12:36
*** iceyao has joined #openstack-zun12:48
*** GB21 has quit IRC12:55
*** janki has quit IRC13:11
*** harlowja has joined #openstack-zun13:26
openstackgerritabdul nizamuddin proposed openstack/zun-ui: Drop MANIFEST.in - it's not needed by pbr  https://review.openstack.org/39147513:39
openstackgerritabdul nizamuddin proposed openstack/zun-ui: Don't include openstack/common in flake8 exclude list  https://review.openstack.org/39148814:02
*** sheel has quit IRC14:10
*** harlowja has quit IRC14:22
*** harlowja has joined #openstack-zun14:33
*** harlowja has quit IRC14:35
*** harlowja has joined #openstack-zun14:38
*** harlowja has quit IRC14:42
*** harlowja has joined #openstack-zun15:01
*** harlowja has quit IRC15:07
*** zhurong has joined #openstack-zun16:00
*** zhurong has quit IRC16:04
*** iceyao has quit IRC17:06
*** iceyao has joined #openstack-zun17:07
*** pksingh has joined #openstack-zun17:13
*** pksingh has quit IRC17:14
*** pksingh has joined #openstack-zun17:15
*** pksingh has left #openstack-zun17:16
*** pksingh has joined #openstack-zun17:17
*** pksingh has quit IRC17:20
*** pksingh has joined #openstack-zun17:21
*** pksingh has quit IRC17:22
*** coolsvap has quit IRC17:46
*** iceyao has quit IRC17:52
*** sheel has joined #openstack-zun18:09

Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!