https://gerrit.openbmc-project.xyz/#/c/openbmc/phosphor-dbus-interfaces/+/16468/ needs your review

[Yong Li]

Hi Brad,

Thanks for your mail. For this change, It should be better to add it into 2.6.
Based on this change, I will start to add the implementation in the IPMI watchdog command.

Thanks,
Yong
-----Original Message-----
From: Brad Bishop <bradleyb at fuzziesquirrel.com> 
Sent: Wednesday, January 9, 2019 11:35 AM
To: Yong Li <yong.b.li at linux.intel.com>
Cc: Tanous, Ed <ed.tanous at intel.com>
Subject: Re: https://gerrit.openbmc-project.xyz/#/c/openbmc/phosphor-dbus-interfaces/+/16468/ needs your review


> On Jan 8, 2019, at 10:16 PM, Yong Li <yong.b.li at linux.intel.com> wrote:
> 
> HI Brad,
>  
> Sorry to disturbing, but the below code review is idle for serval days:
> https://gerrit.openbmc-project.xyz/#/c/openbmc/phosphor-dbus-interface
> s/+/16468/
>  
> several reviewers had “+1”, could you help to review or merge it please?
>  
> Thanks,
> Yong

Hi Yong

Please copy the list with this sort of thing.  In fact you should just always copy the list.  Thanks.

I’m not merging much right now because we are in the middle of trying to release 2.6:
https://lists.ozlabs.org/pipermail/openbmc/2019-January/014503.html

If you think it should be in 2.6, we can probably do that.  Do you need it in 2.6?

I expect to start merging code again after the three days are over and we branch.

thx - brad=