Re: [KVM_AUTOTEST][RFC] pre_command chaining

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi Michael,

you are right, it is possible. But if I specify "pre_command = true" at the top of my config file, this command will be executed even if no additional command is added into the queue. (tests with pre_commands are not selected)

That is my reason why I'd like to see this two lines change into the framework.

Still you are right that it's basically a "cosmetic" modification for simplification the config file.

Dne 10.7.2009 17:27, Michael Goldish napsal(a):
----- "Lukáš Doktor"<ldoktor@xxxxxxxxxx>  wrote:

Hi,

the way how kvm_autotest currently handle pre_command/post_command it
don't allow to specify more than one command. BASH can handle this
itself with a small change in the framework , as shown in the
attachment.

Why do you say the framework doesn't allow chaining pre_commands?
What's wrong with:
pre_command = "command0"
pre_command += "&&  command1"
pre_command += "&&  command2"

In .cfg file we just change variable from:
   pre_command = "command"
to:
   pre_commane += "command&&"
produce:
   $(command&&  true)

Framework adds the last command true, which enclose whole command.
This
way we can chain infinite pre/post_commands without losing the return

value (if something go wrong, other commands are not executed and
return
value is preserve.

example:
in cfg:
   pre_command += "echo A&&"
   pre_command += "echo B&&"
   pre_command += "echo C&&"
framework params.get("pre_command"):
   "echo A&&  echo B&&  echo C&&"
framework process_command execute on the host:
   "echo A&&  echo B&&  echo C&&  true"

regards,
Lukáš Doktor

In any case, the proposed solution does not allow the user to use
pre_command in the most straightforward way:
pre_command = "command"
because that would get translated into:
"command true"
So the user must append&&  to the command, which makes little sense.

There could be other solutions, like

1. Specifying "pre_command = true" at the top of the config file, and
then using:
pre_command += "&&  command0"
pre_command += "&&  command1"

"pre_command = command" will also work fine in this case.

2. Removing the final "&&" from the command, if any, so that if the
user enters:
pre_command = "command0&&"
pre_command += "command1&&"
the framework will run:
"command0&&  command1" instead of "command0&&  command1&&".

In any case, can you provide an example where it's impossible or
difficult to do command chaining without changing the framework?
--
To unsubscribe from this list: send the line "unsubscribe kvm" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html

--
To unsubscribe from this list: send the line "unsubscribe kvm" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html

[Index of Archives]     [KVM ARM]     [KVM ia64]     [KVM ppc]     [Virtualization Tools]     [Spice Development]     [Libvirt]     [Libvirt Users]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite Questions]     [Linux Kernel]     [Linux SCSI]     [XFree86]
  Powered by Linux