LTP-realtime run_c_files.sh: too few arguments (at least two)

[Mike Holmes]

There are a number of test cases that are not being called correctly

/opt/ltp/testcases/realtime/scripts/run_c_files.sh: too few arguments (at least two)

http://validation.linaro.org/scheduler/job/86554/log_file#L_59_488

[Mike Holmes (mike-holmes)]

Test cases still have missing arguments
https://validation.linaro.org/dashboard/attachment/650027/view

[Gary S. Robertson (gary-robertson)]

In stand-alone testing during data gathering for the LCA14 RT presentation I noticed this happening during ltp-realtime tests.
The problem is in the test directories in the respective run_auto.sh scripts. run_c_files.sh is now expecting to be passed a profile selector as its first parameter. The run_auto.sh scripts for the offending tests are lacking the profile selector setup and are not passing this to the run_c_files.sh script. The 'default' profile in the 'profiles' subdirectory also needs to be updated to contain information about these tests. I did not have time to do a proper correction of these issues, but adding the 'default' profile selector to the run_auto.sh scripts in the 'realtime/func' tests did resolve the issue of run_c_files.sh aborting and allowed the tests to do a single run iteration with no pass/fail criteria.

[Gary S. Robertson (gary-robertson)]

See files changed per my previous comment in the following links:

http://people.linaro.org/~gary.robertson/LCA14/test-scripts/ltp-realtime-be-scripts.tbz

http://people.linaro.org/~gary.robertson/LCA14/test-scripts/ltp-realtime-scripts.tbz

These files correct the problem with missing arguments but dont use the 'profile' capability to best advantage for each of the tests. To see what was changed you should diff these scripts against those built into the OE rootfs (LE and BE) which LNG is using in their testing.

[Ciprian Barbu (ciprian-barbu)]

Sent patch to gerrit:

https://review.linaro.org/#/c/1332/

And sent patch for ltp upstream:

http://sourceforge.net/p/ltp/mailman/message/32082777/

[Mike Holmes (mike-holmes)]

Patch merged in Linaro, Ciprain are there any upstream updates ?

[Ciprian Barbu (ciprian-barbu)]

I haven't received a reply to my patch and I don't see the change in the repo. I will send a reminder to the mailing list.

[Hongbo Zhang (hongbo-zhang)]

Glad to see Ciprian make patch for the bug which was assigned to me before,
thanks.
I didn't have time to handle the bugs in my hand during the past days, and
will try to spend time slots on them since this week.

On 21 March 2014 18:30, Ciprian Barbu <email address hidden> wrote:

> I haven't received a reply to my patch and I don't see the change in the
> repo. I will send a reminder to the mailing list.
>
> --
> You received this bug notification because you are a member of Linaro
> Networking Group, which is subscribed to linaro-networking.
> Matching subscriptions: LNG all
> https://bugs.launchpad.net/bugs/1252788
>
> Title:
> LTP-realtime run_c_files.sh: too few arguments (at least two)
>
> Status in Linaro networking Group:
> Confirmed
>
> Bug description:
>
> There are a number of test cases that are not being called correctly
>
> /opt/ltp/testcases/realtime/scripts/run_c_files.sh: too few arguments
> (at least two)
>
> http://validation.linaro.org/scheduler/job/86554/log_file#L_59_488
>
> To manage notifications about this bug go to:
> https://bugs.launchpad.net/linaro-networking/+bug/1252788/+subscriptions
>

[Ciprian Barbu (ciprian-barbu)]

Patch applied upstream:

https://github.com/linux-test-project/ltp/commit/79c6f33cd8c2c473de1fb7afc98e92ac59f93aa9

We can close it now.