* Clarify some questions about BIOS update (phosphor-ipmi-flash)
@ 2019-08-05 11:56 Andrew MS1 Peng
2019-08-05 16:17 ` Patrick Venture
0 siblings, 1 reply; 10+ messages in thread
From: Andrew MS1 Peng @ 2019-08-05 11:56 UTC (permalink / raw)
To: Patrick Venture; +Cc: openbmc, Duke KH Du, Yonghui YH21 Liu, Lisa YJ19 Liu
[-- Attachment #1: Type: text/plain, Size: 512 bytes --]
Hi Patrick,
I have some questions about BIOS update via phosphor-ipmi-flash with L2A, please help to clarify. Thanks.
1. How can I update the status (success or failure) after upgrade BIOS firmware? Currently, the update status only appear "running".
2. It was failed when triggered "phosphor-ipmi-flash-bios-update.target" since the job-mode is null string. It can work well when I add string "replace" to job-mode. How can I config (json file or configure parameter) to support it?
Regards,
Andrew
[-- Attachment #2: Type: text/html, Size: 3930 bytes --]
^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: Clarify some questions about BIOS update (phosphor-ipmi-flash)
2019-08-05 11:56 Clarify some questions about BIOS update (phosphor-ipmi-flash) Andrew MS1 Peng
@ 2019-08-05 16:17 ` Patrick Venture
2019-08-05 16:32 ` Patrick Venture
0 siblings, 1 reply; 10+ messages in thread
From: Patrick Venture @ 2019-08-05 16:17 UTC (permalink / raw)
To: Andrew MS1 Peng; +Cc: openbmc, Duke KH Du, Yonghui YH21 Liu, Lisa YJ19 Liu
On Mon, Aug 5, 2019 at 4:56 AM Andrew MS1 Peng <pengms1@lenovo.com> wrote:
>
> Hi Patrick,
>
>
>
> I have some questions about BIOS update via phosphor-ipmi-flash with L2A, please help to clarify. Thanks.
>
>
>
Thanks for giving it a shot! I don't have any systems using this for
BIOS update, so I'm curious where there may be issues lurking :)
> 1. How can I update the status (success or failure) after upgrade BIOS firmware? Currently, the update status only appear "running".
>
I see, the update mechanism doesn't have a verification option. A bit
of an oversight. I'll have something up for review later today. Then
it'll be something you configure similarly to the verifySystemd json
entry.
> 2. It was failed when triggered "phosphor-ipmi-flash-bios-update.target" since the job-mode is null string. It can work well when I add string "replace" to job-mode. How can I config (json file or configure parameter) to support it?
I'll take a look! The parameter for job-mode, I thought, was
optional? Since we don't specify it for other jobs (verification,
etc). The string should be empty by default and then not added as a
null-string:
https://github.com/openbmc/phosphor-ipmi-flash/blob/f0c71df2514927e933f6fc33d9669ab0663a0706/bmc/update_systemd.cpp#L49
I don't think I have a unit-test that verifies the parameters are sent
down properly, so I'll also add that today AND I'll add an optional
mode parameter to the json.
Sending for review today.
...
>
>
>
> Regards,
> Andrew
^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: Clarify some questions about BIOS update (phosphor-ipmi-flash)
2019-08-05 16:17 ` Patrick Venture
@ 2019-08-05 16:32 ` Patrick Venture
2019-08-05 16:39 ` Patrick Venture
0 siblings, 1 reply; 10+ messages in thread
From: Patrick Venture @ 2019-08-05 16:32 UTC (permalink / raw)
To: Andrew MS1 Peng; +Cc: openbmc, Duke KH Du, Yonghui YH21 Liu, Lisa YJ19 Liu
On Mon, Aug 5, 2019 at 9:17 AM Patrick Venture <venture@google.com> wrote:
>
> On Mon, Aug 5, 2019 at 4:56 AM Andrew MS1 Peng <pengms1@lenovo.com> wrote:
> >
> > Hi Patrick,
> >
> >
> >
> > I have some questions about BIOS update via phosphor-ipmi-flash with L2A, please help to clarify. Thanks.
> >
> >
> >
>
> Thanks for giving it a shot! I don't have any systems using this for
> BIOS update, so I'm curious where there may be issues lurking :)
>
>
> > 1. How can I update the status (success or failure) after upgrade BIOS firmware? Currently, the update status only appear "running".
> >
>
> I see, the update mechanism doesn't have a verification option. A bit
> of an oversight. I'll have something up for review later today. Then
> it'll be something you configure similarly to the verifySystemd json
> entry.
>
> > 2. It was failed when triggered "phosphor-ipmi-flash-bios-update.target" since the job-mode is null string. It can work well when I add string "replace" to job-mode. How can I config (json file or configure parameter) to support it?
>
> I'll take a look! The parameter for job-mode, I thought, was
> optional? Since we don't specify it for other jobs (verification,
> etc). The string should be empty by default and then not added as a
> null-string:
>
> https://github.com/openbmc/phosphor-ipmi-flash/blob/f0c71df2514927e933f6fc33d9669ab0663a0706/bmc/update_systemd.cpp#L49
I see it is required! Since I always configure with reboot, I never
saw this failure. I'll default it to "replace"
>
> I don't think I have a unit-test that verifies the parameters are sent
> down properly, so I'll also add that today AND I'll add an optional
> mode parameter to the json.
>
> Sending for review today.
>
> ...
>
> >
> >
> >
> > Regards,
> > Andrew
^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: Clarify some questions about BIOS update (phosphor-ipmi-flash)
2019-08-05 16:32 ` Patrick Venture
@ 2019-08-05 16:39 ` Patrick Venture
2019-08-05 17:39 ` Patrick Venture
0 siblings, 1 reply; 10+ messages in thread
From: Patrick Venture @ 2019-08-05 16:39 UTC (permalink / raw)
To: Andrew MS1 Peng; +Cc: openbmc, Duke KH Du, Yonghui YH21 Liu, Lisa YJ19 Liu
On Mon, Aug 5, 2019 at 9:32 AM Patrick Venture <venture@google.com> wrote:
>
> On Mon, Aug 5, 2019 at 9:17 AM Patrick Venture <venture@google.com> wrote:
> >
> > On Mon, Aug 5, 2019 at 4:56 AM Andrew MS1 Peng <pengms1@lenovo.com> wrote:
> > >
> > > Hi Patrick,
> > >
> > >
> > >
> > > I have some questions about BIOS update via phosphor-ipmi-flash with L2A, please help to clarify. Thanks.
> > >
> > >
> > >
> >
> > Thanks for giving it a shot! I don't have any systems using this for
> > BIOS update, so I'm curious where there may be issues lurking :)
> >
> >
> > > 1. How can I update the status (success or failure) after upgrade BIOS firmware? Currently, the update status only appear "running".
> > >
> >
> > I see, the update mechanism doesn't have a verification option. A bit
> > of an oversight. I'll have something up for review later today. Then
> > it'll be something you configure similarly to the verifySystemd json
> > entry.
> >
> > > 2. It was failed when triggered "phosphor-ipmi-flash-bios-update.target" since the job-mode is null string. It can work well when I add string "replace" to job-mode. How can I config (json file or configure parameter) to support it?
> >
> > I'll take a look! The parameter for job-mode, I thought, was
> > optional? Since we don't specify it for other jobs (verification,
> > etc). The string should be empty by default and then not added as a
> > null-string:
> >
> > https://github.com/openbmc/phosphor-ipmi-flash/blob/f0c71df2514927e933f6fc33d9669ab0663a0706/bmc/update_systemd.cpp#L49
>
> I see it is required! Since I always configure with reboot, I never
> saw this failure. I'll default it to "replace"
First patch in the series generated from your email:
https://gerrit.openbmc-project.xyz/c/openbmc/phosphor-ipmi-flash/+/24165
Next: json configuration option to control the job-mode.
Then merge verification and update objects, s.t. you can specify the
status path.
*** A later patchset can have the status be not a path, but a dbus
property or dbus path -- if someone wants to implement a new object
for this, it's somewhat trivial.
>
> >
> > I don't think I have a unit-test that verifies the parameters are sent
> > down properly, so I'll also add that today AND I'll add an optional
> > mode parameter to the json.
> >
> > Sending for review today.
> >
> > ...
> >
> > >
> > >
> > >
> > > Regards,
> > > Andrew
^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: Clarify some questions about BIOS update (phosphor-ipmi-flash)
2019-08-05 16:39 ` Patrick Venture
@ 2019-08-05 17:39 ` Patrick Venture
2019-08-05 20:39 ` Patrick Venture
0 siblings, 1 reply; 10+ messages in thread
From: Patrick Venture @ 2019-08-05 17:39 UTC (permalink / raw)
To: Andrew MS1 Peng; +Cc: openbmc, Duke KH Du, Yonghui YH21 Liu, Lisa YJ19 Liu
On Mon, Aug 5, 2019 at 9:39 AM Patrick Venture <venture@google.com> wrote:
>
> On Mon, Aug 5, 2019 at 9:32 AM Patrick Venture <venture@google.com> wrote:
> >
> > On Mon, Aug 5, 2019 at 9:17 AM Patrick Venture <venture@google.com> wrote:
> > >
> > > On Mon, Aug 5, 2019 at 4:56 AM Andrew MS1 Peng <pengms1@lenovo.com> wrote:
> > > >
> > > > Hi Patrick,
> > > >
> > > >
> > > >
> > > > I have some questions about BIOS update via phosphor-ipmi-flash with L2A, please help to clarify. Thanks.
> > > >
> > > >
> > > >
> > >
> > > Thanks for giving it a shot! I don't have any systems using this for
> > > BIOS update, so I'm curious where there may be issues lurking :)
> > >
> > >
> > > > 1. How can I update the status (success or failure) after upgrade BIOS firmware? Currently, the update status only appear "running".
> > > >
> > >
> > > I see, the update mechanism doesn't have a verification option. A bit
> > > of an oversight. I'll have something up for review later today. Then
> > > it'll be something you configure similarly to the verifySystemd json
> > > entry.
> > >
> > > > 2. It was failed when triggered "phosphor-ipmi-flash-bios-update.target" since the job-mode is null string. It can work well when I add string "replace" to job-mode. How can I config (json file or configure parameter) to support it?
This patch will enable you to specify "mode" in the json. *** note: i
will set some time aside shortly to document writing json
configurations.
https://gerrit.openbmc-project.xyz/c/openbmc/phosphor-ipmi-flash/+/24167
> > >
> > > I'll take a look! The parameter for job-mode, I thought, was
> > > optional? Since we don't specify it for other jobs (verification,
> > > etc). The string should be empty by default and then not added as a
> > > null-string:
> > >
> > > https://github.com/openbmc/phosphor-ipmi-flash/blob/f0c71df2514927e933f6fc33d9669ab0663a0706/bmc/update_systemd.cpp#L49
> >
> > I see it is required! Since I always configure with reboot, I never
> > saw this failure. I'll default it to "replace"
>
> First patch in the series generated from your email:
>
> https://gerrit.openbmc-project.xyz/c/openbmc/phosphor-ipmi-flash/+/24165
>
> Next: json configuration option to control the job-mode.
> Then merge verification and update objects, s.t. you can specify the
> status path.
>
> *** A later patchset can have the status be not a path, but a dbus
> property or dbus path -- if someone wants to implement a new object
> for this, it's somewhat trivial.
>
> >
> > >
> > > I don't think I have a unit-test that verifies the parameters are sent
> > > down properly, so I'll also add that today AND I'll add an optional
> > > mode parameter to the json.
> > >
> > > Sending for review today.
> > >
> > > ...
> > >
> > > >
> > > >
> > > >
> > > > Regards,
> > > > Andrew
^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: Clarify some questions about BIOS update (phosphor-ipmi-flash)
2019-08-05 17:39 ` Patrick Venture
@ 2019-08-05 20:39 ` Patrick Venture
2019-08-07 11:47 ` 答复: [External] " Andrew MS1 Peng
0 siblings, 1 reply; 10+ messages in thread
From: Patrick Venture @ 2019-08-05 20:39 UTC (permalink / raw)
To: Andrew MS1 Peng; +Cc: openbmc, Duke KH Du, Yonghui YH21 Liu, Lisa YJ19 Liu
On Mon, Aug 5, 2019 at 10:39 AM Patrick Venture <venture@google.com> wrote:
>
> On Mon, Aug 5, 2019 at 9:39 AM Patrick Venture <venture@google.com> wrote:
> >
> > On Mon, Aug 5, 2019 at 9:32 AM Patrick Venture <venture@google.com> wrote:
> > >
> > > On Mon, Aug 5, 2019 at 9:17 AM Patrick Venture <venture@google.com> wrote:
> > > >
> > > > On Mon, Aug 5, 2019 at 4:56 AM Andrew MS1 Peng <pengms1@lenovo.com> wrote:
> > > > >
> > > > > Hi Patrick,
> > > > >
> > > > >
> > > > >
> > > > > I have some questions about BIOS update via phosphor-ipmi-flash with L2A, please help to clarify. Thanks.
> > > > >
> > > > >
> > > > >
> > > >
> > > > Thanks for giving it a shot! I don't have any systems using this for
> > > > BIOS update, so I'm curious where there may be issues lurking :)
> > > >
> > > >
> > > > > 1. How can I update the status (success or failure) after upgrade BIOS firmware? Currently, the update status only appear "running".
> > > > >
> > > >
> > > > I see, the update mechanism doesn't have a verification option. A bit
> > > > of an oversight. I'll have something up for review later today. Then
> > > > it'll be something you configure similarly to the verifySystemd json
> > > > entry.
> > > >
> > > > > 2. It was failed when triggered "phosphor-ipmi-flash-bios-update.target" since the job-mode is null string. It can work well when I add string "replace" to job-mode. How can I config (json file or configure parameter) to support it?
>
> This patch will enable you to specify "mode" in the json. *** note: i
> will set some time aside shortly to document writing json
> configurations.
>
> https://gerrit.openbmc-project.xyz/c/openbmc/phosphor-ipmi-flash/+/24167
>
> > > >
> > > > I'll take a look! The parameter for job-mode, I thought, was
> > > > optional? Since we don't specify it for other jobs (verification,
> > > > etc). The string should be empty by default and then not added as a
> > > > null-string:
> > > >
> > > > https://github.com/openbmc/phosphor-ipmi-flash/blob/f0c71df2514927e933f6fc33d9669ab0663a0706/bmc/update_systemd.cpp#L49
> > >
> > > I see it is required! Since I always configure with reboot, I never
> > > saw this failure. I'll default it to "replace"
> >
> > First patch in the series generated from your email:
> >
> > https://gerrit.openbmc-project.xyz/c/openbmc/phosphor-ipmi-flash/+/24165
> >
> > Next: json configuration option to control the job-mode.
> > Then merge verification and update objects, s.t. you can specify the
> > status path.
This patchset adds support for a new update configuration that lets
you read the status from a file.
https://gerrit.openbmc-project.xyz/24171
> >
> > *** A later patchset can have the status be not a path, but a dbus
> > property or dbus path -- if someone wants to implement a new object
> > for this, it's somewhat trivial.
> >
> > >
> > > >
> > > > I don't think I have a unit-test that verifies the parameters are sent
> > > > down properly, so I'll also add that today AND I'll add an optional
> > > > mode parameter to the json.
> > > >
> > > > Sending for review today.
> > > >
> > > > ...
> > > >
> > > > >
> > > > >
> > > > >
> > > > > Regards,
> > > > > Andrew
^ permalink raw reply [flat|nested] 10+ messages in thread
* 答复: [External] Re: Clarify some questions about BIOS update (phosphor-ipmi-flash)
2019-08-05 20:39 ` Patrick Venture
@ 2019-08-07 11:47 ` Andrew MS1 Peng
2019-08-07 15:36 ` Patrick Venture
0 siblings, 1 reply; 10+ messages in thread
From: Andrew MS1 Peng @ 2019-08-07 11:47 UTC (permalink / raw)
To: Patrick Venture; +Cc: openbmc
Hi Patrick,
The triggered "phosphor-ipmi-flash-bios-update.target" seems to been solved, but the BIOS update status has not yet been fixed.
Could we add checking the contents of the file in SystemdUpdateMechanism::status() function of update_systemd.cpp such as verification mechanism SystemdWithStatusFile::status() of general_systemd.cpp?
Regards,
Andrew
-----邮件原件-----
发件人: Patrick Venture <venture@google.com>
发送时间: 2019年8月6日 4:39
收件人: Andrew MS1 Peng <pengms1@lenovo.com>
抄送: openbmc@lists.ozlabs.org; Duke KH Du <dukh@lenovo.com>; Yonghui YH21 Liu <liuyh21@lenovo.com>; Lisa YJ19 Liu <liuyj19@lenovo.com>
主题: [External] Re: Clarify some questions about BIOS update (phosphor-ipmi-flash)
On Mon, Aug 5, 2019 at 10:39 AM Patrick Venture <venture@google.com> wrote:
>
> On Mon, Aug 5, 2019 at 9:39 AM Patrick Venture <venture@google.com> wrote:
> >
> > On Mon, Aug 5, 2019 at 9:32 AM Patrick Venture <venture@google.com> wrote:
> > >
> > > On Mon, Aug 5, 2019 at 9:17 AM Patrick Venture <venture@google.com> wrote:
> > > >
> > > > On Mon, Aug 5, 2019 at 4:56 AM Andrew MS1 Peng <pengms1@lenovo.com> wrote:
> > > > >
> > > > > Hi Patrick,
> > > > >
> > > > >
> > > > >
> > > > > I have some questions about BIOS update via phosphor-ipmi-flash with L2A, please help to clarify. Thanks.
> > > > >
> > > > >
> > > > >
> > > >
> > > > Thanks for giving it a shot! I don't have any systems using
> > > > this for BIOS update, so I'm curious where there may be issues
> > > > lurking :)
> > > >
> > > >
> > > > > 1. How can I update the status (success or failure) after upgrade BIOS firmware? Currently, the update status only appear "running".
> > > > >
> > > >
> > > > I see, the update mechanism doesn't have a verification option.
> > > > A bit of an oversight. I'll have something up for review later
> > > > today. Then it'll be something you configure similarly to the
> > > > verifySystemd json entry.
> > > >
> > > > > 2. It was failed when triggered "phosphor-ipmi-flash-bios-update.target" since the job-mode is null string. It can work well when I add string "replace" to job-mode. How can I config (json file or configure parameter) to support it?
>
> This patch will enable you to specify "mode" in the json. *** note: i
> will set some time aside shortly to document writing json
> configurations.
>
> https://gerrit.openbmc-project.xyz/c/openbmc/phosphor-ipmi-flash/+/241
> 67
>
> > > >
> > > > I'll take a look! The parameter for job-mode, I thought, was
> > > > optional? Since we don't specify it for other jobs
> > > > (verification, etc). The string should be empty by default and
> > > > then not added as a
> > > > null-string:
> > > >
> > > > https://github.com/openbmc/phosphor-ipmi-flash/blob/f0c71df25149
> > > > 27e933f6fc33d9669ab0663a0706/bmc/update_systemd.cpp#L49
> > >
> > > I see it is required! Since I always configure with reboot, I
> > > never saw this failure. I'll default it to "replace"
> >
> > First patch in the series generated from your email:
> >
> > https://gerrit.openbmc-project.xyz/c/openbmc/phosphor-ipmi-flash/+/2
> > 4165
> >
> > Next: json configuration option to control the job-mode.
> > Then merge verification and update objects, s.t. you can specify the
> > status path.
This patchset adds support for a new update configuration that lets you read the status from a file.
https://gerrit.openbmc-project.xyz/24171
> >
> > *** A later patchset can have the status be not a path, but a dbus
> > property or dbus path -- if someone wants to implement a new object
> > for this, it's somewhat trivial.
> >
> > >
> > > >
> > > > I don't think I have a unit-test that verifies the parameters
> > > > are sent down properly, so I'll also add that today AND I'll add
> > > > an optional mode parameter to the json.
> > > >
> > > > Sending for review today.
> > > >
> > > > ...
> > > >
> > > > >
> > > > >
> > > > >
> > > > > Regards,
> > > > > Andrew
^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [External] Re: Clarify some questions about BIOS update (phosphor-ipmi-flash)
2019-08-07 11:47 ` 答复: [External] " Andrew MS1 Peng
@ 2019-08-07 15:36 ` Patrick Venture
2019-08-08 8:21 ` 答复: " Andrew MS1 Peng
0 siblings, 1 reply; 10+ messages in thread
From: Patrick Venture @ 2019-08-07 15:36 UTC (permalink / raw)
To: Andrew MS1 Peng; +Cc: openbmc
On Wed, Aug 7, 2019 at 4:47 AM Andrew MS1 Peng <pengms1@lenovo.com> wrote:
>
> Hi Patrick,
>
> The triggered "phosphor-ipmi-flash-bios-update.target" seems to been solved, but the BIOS update status has not yet been fixed.
> Could we add checking the contents of the file in SystemdUpdateMechanism::status() function of update_systemd.cpp such as verification mechanism SystemdWithStatusFile::status() of general_systemd.cpp?
You can just use the type "fileSystemdUpdate" which let you check a
path for the status.
The three parameters for that update type are the same as the
verification type that matches:
- path - the filesystem path containing the status
- unit - the target "phosphor-ipmi-flash-bios-update.target"
- mode - optional job-mode parameter, defaults to "replace"
>
> Regards,
> Andrew
>
> -----邮件原件-----
> 发件人: Patrick Venture <venture@google.com>
> 发送时间: 2019年8月6日 4:39
> 收件人: Andrew MS1 Peng <pengms1@lenovo.com>
> 抄送: openbmc@lists.ozlabs.org; Duke KH Du <dukh@lenovo.com>; Yonghui YH21 Liu <liuyh21@lenovo.com>; Lisa YJ19 Liu <liuyj19@lenovo.com>
> 主题: [External] Re: Clarify some questions about BIOS update (phosphor-ipmi-flash)
>
> On Mon, Aug 5, 2019 at 10:39 AM Patrick Venture <venture@google.com> wrote:
> >
> > On Mon, Aug 5, 2019 at 9:39 AM Patrick Venture <venture@google.com> wrote:
> > >
> > > On Mon, Aug 5, 2019 at 9:32 AM Patrick Venture <venture@google.com> wrote:
> > > >
> > > > On Mon, Aug 5, 2019 at 9:17 AM Patrick Venture <venture@google.com> wrote:
> > > > >
> > > > > On Mon, Aug 5, 2019 at 4:56 AM Andrew MS1 Peng <pengms1@lenovo.com> wrote:
> > > > > >
> > > > > > Hi Patrick,
> > > > > >
> > > > > >
> > > > > >
> > > > > > I have some questions about BIOS update via phosphor-ipmi-flash with L2A, please help to clarify. Thanks.
> > > > > >
> > > > > >
> > > > > >
> > > > >
> > > > > Thanks for giving it a shot! I don't have any systems using
> > > > > this for BIOS update, so I'm curious where there may be issues
> > > > > lurking :)
> > > > >
> > > > >
> > > > > > 1. How can I update the status (success or failure) after upgrade BIOS firmware? Currently, the update status only appear "running".
> > > > > >
> > > > >
> > > > > I see, the update mechanism doesn't have a verification option.
> > > > > A bit of an oversight. I'll have something up for review later
> > > > > today. Then it'll be something you configure similarly to the
> > > > > verifySystemd json entry.
> > > > >
> > > > > > 2. It was failed when triggered "phosphor-ipmi-flash-bios-update.target" since the job-mode is null string. It can work well when I add string "replace" to job-mode. How can I config (json file or configure parameter) to support it?
> >
> > This patch will enable you to specify "mode" in the json. *** note: i
> > will set some time aside shortly to document writing json
> > configurations.
> >
> > https://gerrit.openbmc-project.xyz/c/openbmc/phosphor-ipmi-flash/+/241
> > 67
> >
> > > > >
> > > > > I'll take a look! The parameter for job-mode, I thought, was
> > > > > optional? Since we don't specify it for other jobs
> > > > > (verification, etc). The string should be empty by default and
> > > > > then not added as a
> > > > > null-string:
> > > > >
> > > > > https://github.com/openbmc/phosphor-ipmi-flash/blob/f0c71df25149
> > > > > 27e933f6fc33d9669ab0663a0706/bmc/update_systemd.cpp#L49
> > > >
> > > > I see it is required! Since I always configure with reboot, I
> > > > never saw this failure. I'll default it to "replace"
> > >
> > > First patch in the series generated from your email:
> > >
> > > https://gerrit.openbmc-project.xyz/c/openbmc/phosphor-ipmi-flash/+/2
> > > 4165
> > >
> > > Next: json configuration option to control the job-mode.
> > > Then merge verification and update objects, s.t. you can specify the
> > > status path.
>
> This patchset adds support for a new update configuration that lets you read the status from a file.
>
> https://gerrit.openbmc-project.xyz/24171
>
> > >
> > > *** A later patchset can have the status be not a path, but a dbus
> > > property or dbus path -- if someone wants to implement a new object
> > > for this, it's somewhat trivial.
> > >
> > > >
> > > > >
> > > > > I don't think I have a unit-test that verifies the parameters
> > > > > are sent down properly, so I'll also add that today AND I'll add
> > > > > an optional mode parameter to the json.
> > > > >
> > > > > Sending for review today.
> > > > >
> > > > > ...
> > > > >
> > > > > >
> > > > > >
> > > > > >
> > > > > > Regards,
> > > > > > Andrew
^ permalink raw reply [flat|nested] 10+ messages in thread
* 答复: [External] Re: Clarify some questions about BIOS update (phosphor-ipmi-flash)
2019-08-07 15:36 ` Patrick Venture
@ 2019-08-08 8:21 ` Andrew MS1 Peng
2019-08-08 17:25 ` Patrick Venture
0 siblings, 1 reply; 10+ messages in thread
From: Andrew MS1 Peng @ 2019-08-08 8:21 UTC (permalink / raw)
To: Patrick Venture; +Cc: openbmc
Hi Patrick,
1. Following your suggestion to modify the json file, but encountered an error message as "Parsing json failed ". Please help to take a look at it, thanks.
A part of config-bios.json(default) content and journalctl log was below.
"update": {
"type": "systemd",
"unit": "phosphor-ipmi-flash-bios-update.target"
}
Aug 07 06:54:22 hr630 ipmid[507]: Registering OEM:[0X00C2CF], Cmd:[0X80] for Blob Commands
Aug 07 06:54:22 hr630 ipmid[507]: config loaded: /flash/bios
Aug 07 06:54:22 hr630 ipmid[507]: config loaded: /flash/image
A part of config-bios.json(modification) content and journalctl log was below.
"update": {
"type": "fileSystemdUpdate",
"unit": "phosphor-ipmi-flash-bios-update.target"
"path": "/tmp/bios.update"
}
Aug 07 06:33:12 hr630 ipmid[498]: Registering OEM:[0X00C2CF], Cmd:[0X80] for Blob Commands
Aug 07 06:33:12 hr630 ipmid[498]: Parsing json failed
Aug 07 06:33:12 hr630 ipmid[498]: config loaded: /flash/image
2. Which one is the best way to modify json file in OpenBmc, patch or overwrite? I added lenovo-bios-update.bb to handle service and script. I encountered an error message as "file conflicts" when I added new file of json to overwrite original file. What should I do to fix it?
The error message is below.
Error: Transaction check error:
file /usr/share/phosphor-ipmi-flash/config-bios.json conflicts between attempted installs of phosphor-ipmi-flash-1.0+git999-r1.armv6 and lenovo-bios-update-1.0-r0.armv6
A part of lenovo-bios-update.bb content is below.
SRC_URI += " \
file://bios-verify.service \
file://bios-verify.sh \
file://bios-update.service \
file://bios-update.sh \
file://config-bios.json \
"
DEPENDS += "systemd"
RDEPENDS_${PN} = "bash"
SYSTEMD_PACKAGES = "${PN}"
SYSTEMD_SERVICE_${PN} = "bios-verify.service bios-update.service"
do_install() {
install -d ${D}/usr/sbin
install -d ${D}/usr/share/phosphor-ipmi-flash/
install -m 0755 ${S}bios-verify.sh ${D}/${sbindir}/
install -m 0755 ${S}bios-update.sh ${D}/${sbindir}/
install -m 0644 ${S}config-bios.json ${D}/usr/share/phosphor-ipmi-flash/
}
FILES_${PN} = " \
/usr/share/phosphor-ipmi-flash/config-bios.json \
${sbindir}/bios-update.sh \
${sbindir}/bios-verify.sh \
"
Regards,
Andrew
-----邮件原件-----
发件人: Patrick Venture <venture@google.com>
发送时间: 2019年8月7日 23:36
收件人: Andrew MS1 Peng <pengms1@lenovo.com>
抄送: openbmc@lists.ozlabs.org
主题: Re: [External] Re: Clarify some questions about BIOS update (phosphor-ipmi-flash)
On Wed, Aug 7, 2019 at 4:47 AM Andrew MS1 Peng <pengms1@lenovo.com> wrote:
>
> Hi Patrick,
>
> The triggered "phosphor-ipmi-flash-bios-update.target" seems to been solved, but the BIOS update status has not yet been fixed.
> Could we add checking the contents of the file in SystemdUpdateMechanism::status() function of update_systemd.cpp such as verification mechanism SystemdWithStatusFile::status() of general_systemd.cpp?
You can just use the type "fileSystemdUpdate" which let you check a path for the status.
The three parameters for that update type are the same as the verification type that matches:
- path - the filesystem path containing the status
- unit - the target "phosphor-ipmi-flash-bios-update.target"
- mode - optional job-mode parameter, defaults to "replace"
>
> Regards,
> Andrew
>
> -----邮件原件-----
> 发件人: Patrick Venture <venture@google.com>
> 发送时间: 2019年8月6日 4:39
> 收件人: Andrew MS1 Peng <pengms1@lenovo.com>
> 抄送: openbmc@lists.ozlabs.org; Duke KH Du <dukh@lenovo.com>; Yonghui
> YH21 Liu <liuyh21@lenovo.com>; Lisa YJ19 Liu <liuyj19@lenovo.com>
> 主题: [External] Re: Clarify some questions about BIOS update
> (phosphor-ipmi-flash)
>
> On Mon, Aug 5, 2019 at 10:39 AM Patrick Venture <venture@google.com> wrote:
> >
> > On Mon, Aug 5, 2019 at 9:39 AM Patrick Venture <venture@google.com> wrote:
> > >
> > > On Mon, Aug 5, 2019 at 9:32 AM Patrick Venture <venture@google.com> wrote:
> > > >
> > > > On Mon, Aug 5, 2019 at 9:17 AM Patrick Venture <venture@google.com> wrote:
> > > > >
> > > > > On Mon, Aug 5, 2019 at 4:56 AM Andrew MS1 Peng <pengms1@lenovo.com> wrote:
> > > > > >
> > > > > > Hi Patrick,
> > > > > >
> > > > > >
> > > > > >
> > > > > > I have some questions about BIOS update via phosphor-ipmi-flash with L2A, please help to clarify. Thanks.
> > > > > >
> > > > > >
> > > > > >
> > > > >
> > > > > Thanks for giving it a shot! I don't have any systems using
> > > > > this for BIOS update, so I'm curious where there may be issues
> > > > > lurking :)
> > > > >
> > > > >
> > > > > > 1. How can I update the status (success or failure) after upgrade BIOS firmware? Currently, the update status only appear "running".
> > > > > >
> > > > >
> > > > > I see, the update mechanism doesn't have a verification option.
> > > > > A bit of an oversight. I'll have something up for review
> > > > > later today. Then it'll be something you configure similarly
> > > > > to the verifySystemd json entry.
> > > > >
> > > > > > 2. It was failed when triggered "phosphor-ipmi-flash-bios-update.target" since the job-mode is null string. It can work well when I add string "replace" to job-mode. How can I config (json file or configure parameter) to support it?
> >
> > This patch will enable you to specify "mode" in the json. *** note:
> > i will set some time aside shortly to document writing json
> > configurations.
> >
> > https://gerrit.openbmc-project.xyz/c/openbmc/phosphor-ipmi-flash/+/2
> > 41
> > 67
> >
> > > > >
> > > > > I'll take a look! The parameter for job-mode, I thought, was
> > > > > optional? Since we don't specify it for other jobs
> > > > > (verification, etc). The string should be empty by default
> > > > > and then not added as a
> > > > > null-string:
> > > > >
> > > > > https://github.com/openbmc/phosphor-ipmi-flash/blob/f0c71df251
> > > > > 49
> > > > > 27e933f6fc33d9669ab0663a0706/bmc/update_systemd.cpp#L49
> > > >
> > > > I see it is required! Since I always configure with reboot, I
> > > > never saw this failure. I'll default it to "replace"
> > >
> > > First patch in the series generated from your email:
> > >
> > > https://gerrit.openbmc-project.xyz/c/openbmc/phosphor-ipmi-flash/+
> > > /2
> > > 4165
> > >
> > > Next: json configuration option to control the job-mode.
> > > Then merge verification and update objects, s.t. you can specify
> > > the status path.
>
> This patchset adds support for a new update configuration that lets you read the status from a file.
>
> https://gerrit.openbmc-project.xyz/24171
>
> > >
> > > *** A later patchset can have the status be not a path, but a dbus
> > > property or dbus path -- if someone wants to implement a new
> > > object for this, it's somewhat trivial.
> > >
> > > >
> > > > >
> > > > > I don't think I have a unit-test that verifies the parameters
> > > > > are sent down properly, so I'll also add that today AND I'll
> > > > > add an optional mode parameter to the json.
> > > > >
> > > > > Sending for review today.
> > > > >
> > > > > ...
> > > > >
> > > > > >
> > > > > >
> > > > > >
> > > > > > Regards,
> > > > > > Andrew
^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [External] Re: Clarify some questions about BIOS update (phosphor-ipmi-flash)
2019-08-08 8:21 ` 答复: " Andrew MS1 Peng
@ 2019-08-08 17:25 ` Patrick Venture
0 siblings, 0 replies; 10+ messages in thread
From: Patrick Venture @ 2019-08-08 17:25 UTC (permalink / raw)
To: Andrew MS1 Peng; +Cc: openbmc
On Thu, Aug 8, 2019 at 1:21 AM Andrew MS1 Peng <pengms1@lenovo.com> wrote:
>
> Hi Patrick,
>
> 1. Following your suggestion to modify the json file, but encountered an error message as "Parsing json failed ". Please help to take a look at it, thanks.
> A part of config-bios.json(default) content and journalctl log was below.
> "update": {
> "type": "systemd",
> "unit": "phosphor-ipmi-flash-bios-update.target"
> }
>
> Aug 07 06:54:22 hr630 ipmid[507]: Registering OEM:[0X00C2CF], Cmd:[0X80] for Blob Commands
> Aug 07 06:54:22 hr630 ipmid[507]: config loaded: /flash/bios
> Aug 07 06:54:22 hr630 ipmid[507]: config loaded: /flash/image
>
>
> A part of config-bios.json(modification) content and journalctl log was below.
> "update": {
> "type": "fileSystemdUpdate",
> "unit": "phosphor-ipmi-flash-bios-update.target" <--- missing comma.
> "path": "/tmp/bios.update"
> }
>
> Aug 07 06:33:12 hr630 ipmid[498]: Registering OEM:[0X00C2CF], Cmd:[0X80] for Blob Commands
> Aug 07 06:33:12 hr630 ipmid[498]: Parsing json failed
> Aug 07 06:33:12 hr630 ipmid[498]: config loaded: /flash/image
>
> 2. Which one is the best way to modify json file in OpenBmc, patch or overwrite? I added lenovo-bios-update.bb to handle service and script. I encountered an error message as "file conflicts" when I added new file of json to overwrite original file. What should I do to fix it?
> The error message is below.
> Error: Transaction check error:
> file /usr/share/phosphor-ipmi-flash/config-bios.json conflicts between attempted installs of phosphor-ipmi-flash-1.0+git999-r1.armv6 and lenovo-bios-update-1.0-r0.armv6
>
> A part of lenovo-bios-update.bb content is below.
> SRC_URI += " \
> file://bios-verify.service \
> file://bios-verify.sh \
> file://bios-update.service \
> file://bios-update.sh \
> file://config-bios.json \
> "
> DEPENDS += "systemd"
> RDEPENDS_${PN} = "bash"
> SYSTEMD_PACKAGES = "${PN}"
> SYSTEMD_SERVICE_${PN} = "bios-verify.service bios-update.service"
> do_install() {
> install -d ${D}/usr/sbin
> install -d ${D}/usr/share/phosphor-ipmi-flash/
> install -m 0755 ${S}bios-verify.sh ${D}/${sbindir}/
> install -m 0755 ${S}bios-update.sh ${D}/${sbindir}/
> install -m 0644 ${S}config-bios.json ${D}/usr/share/phosphor-ipmi-flash/
> }
> FILES_${PN} = " \
> /usr/share/phosphor-ipmi-flash/config-bios.json \
> ${sbindir}/bios-update.sh \
> ${sbindir}/bios-verify.sh \
> "
> Regards,
> Andrew
>
>
> -----邮件原件-----
> 发件人: Patrick Venture <venture@google.com>
> 发送时间: 2019年8月7日 23:36
> 收件人: Andrew MS1 Peng <pengms1@lenovo.com>
> 抄送: openbmc@lists.ozlabs.org
> 主题: Re: [External] Re: Clarify some questions about BIOS update (phosphor-ipmi-flash)
>
> On Wed, Aug 7, 2019 at 4:47 AM Andrew MS1 Peng <pengms1@lenovo.com> wrote:
> >
> > Hi Patrick,
> >
> > The triggered "phosphor-ipmi-flash-bios-update.target" seems to been solved, but the BIOS update status has not yet been fixed.
> > Could we add checking the contents of the file in SystemdUpdateMechanism::status() function of update_systemd.cpp such as verification mechanism SystemdWithStatusFile::status() of general_systemd.cpp?
>
> You can just use the type "fileSystemdUpdate" which let you check a path for the status.
>
> The three parameters for that update type are the same as the verification type that matches:
> - path - the filesystem path containing the status
> - unit - the target "phosphor-ipmi-flash-bios-update.target"
> - mode - optional job-mode parameter, defaults to "replace"
>
> >
> > Regards,
> > Andrew
> >
> > -----邮件原件-----
> > 发件人: Patrick Venture <venture@google.com>
> > 发送时间: 2019年8月6日 4:39
> > 收件人: Andrew MS1 Peng <pengms1@lenovo.com>
> > 抄送: openbmc@lists.ozlabs.org; Duke KH Du <dukh@lenovo.com>; Yonghui
> > YH21 Liu <liuyh21@lenovo.com>; Lisa YJ19 Liu <liuyj19@lenovo.com>
> > 主题: [External] Re: Clarify some questions about BIOS update
> > (phosphor-ipmi-flash)
> >
> > On Mon, Aug 5, 2019 at 10:39 AM Patrick Venture <venture@google.com> wrote:
> > >
> > > On Mon, Aug 5, 2019 at 9:39 AM Patrick Venture <venture@google.com> wrote:
> > > >
> > > > On Mon, Aug 5, 2019 at 9:32 AM Patrick Venture <venture@google.com> wrote:
> > > > >
> > > > > On Mon, Aug 5, 2019 at 9:17 AM Patrick Venture <venture@google.com> wrote:
> > > > > >
> > > > > > On Mon, Aug 5, 2019 at 4:56 AM Andrew MS1 Peng <pengms1@lenovo.com> wrote:
> > > > > > >
> > > > > > > Hi Patrick,
> > > > > > >
> > > > > > >
> > > > > > >
> > > > > > > I have some questions about BIOS update via phosphor-ipmi-flash with L2A, please help to clarify. Thanks.
> > > > > > >
> > > > > > >
> > > > > > >
> > > > > >
> > > > > > Thanks for giving it a shot! I don't have any systems using
> > > > > > this for BIOS update, so I'm curious where there may be issues
> > > > > > lurking :)
> > > > > >
> > > > > >
> > > > > > > 1. How can I update the status (success or failure) after upgrade BIOS firmware? Currently, the update status only appear "running".
> > > > > > >
> > > > > >
> > > > > > I see, the update mechanism doesn't have a verification option.
> > > > > > A bit of an oversight. I'll have something up for review
> > > > > > later today. Then it'll be something you configure similarly
> > > > > > to the verifySystemd json entry.
> > > > > >
> > > > > > > 2. It was failed when triggered "phosphor-ipmi-flash-bios-update.target" since the job-mode is null string. It can work well when I add string "replace" to job-mode. How can I config (json file or configure parameter) to support it?
> > >
> > > This patch will enable you to specify "mode" in the json. *** note:
> > > i will set some time aside shortly to document writing json
> > > configurations.
> > >
> > > https://gerrit.openbmc-project.xyz/c/openbmc/phosphor-ipmi-flash/+/2
> > > 41
> > > 67
> > >
> > > > > >
> > > > > > I'll take a look! The parameter for job-mode, I thought, was
> > > > > > optional? Since we don't specify it for other jobs
> > > > > > (verification, etc). The string should be empty by default
> > > > > > and then not added as a
> > > > > > null-string:
> > > > > >
> > > > > > https://github.com/openbmc/phosphor-ipmi-flash/blob/f0c71df251
> > > > > > 49
> > > > > > 27e933f6fc33d9669ab0663a0706/bmc/update_systemd.cpp#L49
> > > > >
> > > > > I see it is required! Since I always configure with reboot, I
> > > > > never saw this failure. I'll default it to "replace"
> > > >
> > > > First patch in the series generated from your email:
> > > >
> > > > https://gerrit.openbmc-project.xyz/c/openbmc/phosphor-ipmi-flash/+
> > > > /2
> > > > 4165
> > > >
> > > > Next: json configuration option to control the job-mode.
> > > > Then merge verification and update objects, s.t. you can specify
> > > > the status path.
> >
> > This patchset adds support for a new update configuration that lets you read the status from a file.
> >
> > https://gerrit.openbmc-project.xyz/24171
> >
> > > >
> > > > *** A later patchset can have the status be not a path, but a dbus
> > > > property or dbus path -- if someone wants to implement a new
> > > > object for this, it's somewhat trivial.
> > > >
> > > > >
> > > > > >
> > > > > > I don't think I have a unit-test that verifies the parameters
> > > > > > are sent down properly, so I'll also add that today AND I'll
> > > > > > add an optional mode parameter to the json.
> > > > > >
> > > > > > Sending for review today.
> > > > > >
> > > > > > ...
> > > > > >
> > > > > > >
> > > > > > >
> > > > > > >
> > > > > > > Regards,
> > > > > > > Andrew
^ permalink raw reply [flat|nested] 10+ messages in thread
end of thread, other threads:[~2019-08-08 17:25 UTC | newest]
Thread overview: 10+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2019-08-05 11:56 Clarify some questions about BIOS update (phosphor-ipmi-flash) Andrew MS1 Peng
2019-08-05 16:17 ` Patrick Venture
2019-08-05 16:32 ` Patrick Venture
2019-08-05 16:39 ` Patrick Venture
2019-08-05 17:39 ` Patrick Venture
2019-08-05 20:39 ` Patrick Venture
2019-08-07 11:47 ` 答复: [External] " Andrew MS1 Peng
2019-08-07 15:36 ` Patrick Venture
2019-08-08 8:21 ` 答复: " Andrew MS1 Peng
2019-08-08 17:25 ` Patrick Venture
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.