Discussion:
[Linaro-validation] about lmbench test suite path
lyang0
2017-11-01 14:09:01 UTC
Permalink
Hi,

I'm using a job definition template to submit lava jobs, basically, I
use something like

path: automated/linux/{test_suite}/{test_suite}.yaml

but I find that lmbench it doesn't use this template.

can I send a patch to rename "automated/linux/lmbench/lmbench-memory.yaml"

to "automated/linux/lmbench-memory/lmbench-memory.yaml "

just like ltp-realtime does

***@pek-lyang0-d1:/tmp/tt/test-definitions$ ls
automated/linux/ltp-realtime/ltp-realtime.yaml
automated/linux/ltp-realtime/ltp-realtime.yaml


Lei
Naresh Kamboju
2017-11-02 06:00:52 UTC
Permalink
Hi Lei,
Hi,
I'm using a job definition template to submit lava jobs, basically, I use
something like
path: automated/linux/{test_suite}/{test_suite}.yaml
but I find that lmbench it doesn't use this template.
can I send a patch to rename "automated/linux/lmbench/lmbench-memory.yaml"
to "automated/linux/lmbench-memory/lmbench-memory.yaml "
Please send a patch.
Our team will review.

- Naresh
just like ltp-realtime does
automated/linux/ltp-realtime/ltp-realtime.yaml
automated/linux/ltp-realtime/ltp-realtime.yaml
Lei
Chase Qi
2017-11-02 07:28:09 UTC
Permalink
Post by Naresh Kamboju
Hi Lei,
Hi,
I'm using a job definition template to submit lava jobs, basically, I use
something like
path: automated/linux/{test_suite}/{test_suite}.yaml
We use relative path directly. What is the benefits in using a variable
here? Can you share your whole job template?
Post by Naresh Kamboju
but I find that lmbench it doesn't use this template.
lmbench-memory is part of lmbench. We could have more then one sub-tests in
the same folder, or more then one yaml files using the same script, but
with different params and slightly different file name. It is not just
lmbench. There are more tests like this(about 6 tests). It is fine to use a
separate test folder for each sub-test like ltp-realtime does, but you
cannot assume test case writer always uses the same name for both test
folder and file names, in most case, yes, but not always.
Post by Naresh Kamboju
can I send a patch to rename "automated/linux/lmbench/
lmbench-memory.yaml"
to "automated/linux/lmbench-memory/lmbench-memory.yaml "
This change will break our(or other teams') existing test job templates and
test plans. Please share your use case, lets see it is worth to do so.

Thanks,
Chase
Post by Naresh Kamboju
Please send a patch.
Our team will review.
- Naresh
just like ltp-realtime does
automated/linux/ltp-realtime/ltp-realtime.yaml
automated/linux/ltp-realtime/ltp-realtime.yaml
Lei
lyang0
2017-11-02 07:52:01 UTC
Permalink
Post by Naresh Kamboju
Hi Lei,
Post by lyang0
Hi,
I'm using a job definition template to submit lava jobs,
basically, I use
Post by lyang0
something like
path: automated/linux/{test_suite}/{test_suite}.yaml
We use relative path directly. What is the benefits in using a
variable here? Can you share your whole job template?
I use

- test:
timeout:
minutes: 100
definitions:
- repository: https://git.linaro.org/qa/test-definitions.git
from: git
path: automated/linux/{test_suite}/{test_suite}.yaml
name: test
parameters:
DEVICE: /dev/sda
PARTITION: /dev/sda1
SKIP_INSTALL: true
skip_install:
- deps


Ok, I'm fine with not changing anything. I can use a dictionary to fix
the value

Lei
Post by Naresh Kamboju
Post by lyang0
but I find that lmbench it doesn't use this template.
lmbench-memory is part of lmbench. We could have more then one
sub-tests in the same folder, or more then one yaml files using the
same script, but with different params and slightly different file
name. It is not just lmbench. There are more tests like this(about 6
tests). It is fine to use a separate test folder for each sub-test
like ltp-realtime does, but you cannot assume test case writer always
uses the same name for both test folder and file names, in most case,
yes, but not always.
Post by lyang0
can I send a patch to rename
"automated/linux/lmbench/lmbench-memory.yaml"
Post by lyang0
to "automated/linux/lmbench-memory/lmbench-memory.yaml "
This change will break our(or other teams') existing test job
templates and test plans. Please share your use case, lets see it is
worth to do so.
Thanks,
Chase
Please send a patch.
Our team will review.
- Naresh
Post by lyang0
just like ltp-realtime does
automated/linux/ltp-realtime/ltp-realtime.yaml
automated/linux/ltp-realtime/ltp-realtime.yaml
Lei
Chase Qi
2017-11-02 08:01:17 UTC
Permalink
Post by Chase Qi
Post by Naresh Kamboju
Hi Lei,
Hi,
I'm using a job definition template to submit lava jobs, basically, I use
something like
path: automated/linux/{test_suite}/{test_suite}.yaml
We use relative path directly. What is the benefits in using a variable
here? Can you share your whole job template?
I use
minutes: 100
- repository: https://git.linaro.org/qa/test-definitions.git
from: git
path: automated/linux/{test_suite}/{test_suite}.yaml
name: test
DEVICE: /dev/sda
PARTITION: /dev/sda1
SKIP_INSTALL: true
- deps
Ok, I'm fine with not changing anything. I can use a dictionary to fix the
value
Great! Thanks for your understanding!
Post by Chase Qi
Lei
Post by Naresh Kamboju
but I find that lmbench it doesn't use this template.
lmbench-memory is part of lmbench. We could have more then one sub-tests in
the same folder, or more then one yaml files using the same script, but with
different params and slightly different file name. It is not just lmbench.
There are more tests like this(about 6 tests). It is fine to use a separate
test folder for each sub-test like ltp-realtime does, but you cannot assume
test case writer always uses the same name for both test folder and file
names, in most case, yes, but not always.
Post by Naresh Kamboju
can I send a patch to rename
"automated/linux/lmbench/lmbench-memory.yaml"
to "automated/linux/lmbench-memory/lmbench-memory.yaml "
This change will break our(or other teams') existing test job templates and
test plans. Please share your use case, lets see it is worth to do so.
Thanks,
Chase
Post by Naresh Kamboju
Please send a patch.
Our team will review.
- Naresh
just like ltp-realtime does
automated/linux/ltp-realtime/ltp-realtime.yaml
automated/linux/ltp-realtime/ltp-realtime.yaml
Lei
Loading...