All of lore.kernel.org
 help / color / mirror / Atom feed
From: "Tony Lee (李文富)" <Tony.Lee@quantatw.com>
To: Rahul Maheshwari <rahulmaheshwari01@gmail.com>
Cc: "openbmc@lists.ozlabs.org" <openbmc@lists.ozlabs.org>
Subject: RE: Test Case "Verify IPMI Root User Password Change" in openbmc-test-automation
Date: Thu, 5 Dec 2019 12:31:26 +0000	[thread overview]
Message-ID: <293a022b44784f86bee3a83fbdafd03b@quantatw.com> (raw)
In-Reply-To: <CAAMkS11DHQvaX+XHBGXDv-3j9__ZS66B1UPhpieUsnfWu+vVgQ@mail.gmail.com>

Hi Rahul,

Got it, it seems I can only skip it first.
There is another question about test_ipmi_user.robot.
Cases which creating a new user success, and then execute a external ipmi command with
this new user account will fail on our system. For example, test case "Verify Setting Valid Password For IPMI User".
The key is that the default channel in the Keyword "Set Channel Access" is 1.
"eth0" corresponding to channel 1 is used for debug purposes, and
"eth1" corresponding to channel 2 is used for ipmi remote access in our system.
So once I set the default channel to 2 in the Keyword "Set Channel Access", those cases succeed.

I have a solution: add variable ${CHANNEL} default is 1 in lib/resource.robot.
Then, let ${channel}=${CHANNEL} in Keyword "Set Channel Access".
Let user to specify channel while running, if not, the default is 1.

Example: robot -v OPENBMC_HOST:x.x.x.x -v CHANNEL:2 ipmi/test_ipmi_user.robot

Is this approach sounds fine to you or there are other suggestions?

Thanks
Tony

From: Rahul Maheshwari <rahulmaheshwari01@gmail.com> 
Sent: Thursday, December 5, 2019 5:47 PM
To: Tony Lee (李文富) <Tony.Lee@quantatw.com>
Cc: openbmc@lists.ozlabs.org
Subject: Re: Test Case "Verify IPMI Root User Password Change" in openbmc-test-automation

Hi Tony,
We have also faced the same problem. So to avoid such problems, we decided to set BMC password to 8 characters length before running our bucket. As far as skipping such test case, we can do that by using `-e Verify_IPMI_Root_User_Password_Change` while running.

Example:
robot -v OPENBMC_HOST:x.x.x.x -e Verify_IPMI_Root_User_Password_Change ipmi/test_ipmi_user.robot

Thanks
Rahul

On Thu, Dec 5, 2019 at 8:05 AM Tony Lee (李文富) <mailto:Tony.Lee@quantatw.com> wrote:
Hi Rahul,

Reference to https://github.com/openbmc/openbmc-test-automation/issues/1920.

Case "Verify IPMI Root User Password Change" will change root user password and
unable to reset default password due to the length is limited to 8.
As a result, cases will also fail after this because the password has been changed to 0penBmc1.
Should we remove this test case? There seems to be no other way to fix it.

Thanks
Best Regards,
Tony


  reply	other threads:[~2019-12-05 12:31 UTC|newest]

Thread overview: 4+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2019-12-05  2:35 Test Case "Verify IPMI Root User Password Change" in openbmc-test-automation Tony Lee (李文富)
2019-12-05  9:46 ` Rahul Maheshwari
2019-12-05 12:31   ` Tony Lee (李文富) [this message]
2019-12-05 15:05     ` Rahul Maheshwari

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=293a022b44784f86bee3a83fbdafd03b@quantatw.com \
    --to=tony.lee@quantatw.com \
    --cc=openbmc@lists.ozlabs.org \
    --cc=rahulmaheshwari01@gmail.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
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.