blockdev-test: Clean up bogus drive_add argument
authorMarkus Armbruster <armbru@redhat.com>
Thu, 2 Oct 2014 14:51:33 +0000 (16:51 +0200)
committerStefan Hajnoczi <stefanha@redhat.com>
Sat, 4 Oct 2014 18:28:13 +0000 (19:28 +0100)
The first argument should be a PCI address, which pci-addr=auto isn't.
Doesn't really matter, as drive_add ignores its first argument when
its second argument has if=none.  Clean it up anyway.

Signed-off-by: Markus Armbruster <armbru@redhat.com>
Reviewed-by: Eric Blake <eblake@redhat.com>
Message-id: 1412261496-24455-4-git-send-email-armbru@redhat.com
Signed-off-by: Stefan Hajnoczi <stefanha@redhat.com>
tests/drive_del-test.c

index a5de2395d63a1c7e47d92e769513f12c4419e236..32516a251cda965d78330d2444e10868e4fc68a2 100644 (file)
@@ -83,7 +83,7 @@ static void test_after_failed_device_add(void)
      */
     response = qmp("{'execute': 'human-monitor-command',"
                    " 'arguments': {"
-                   "   'command-line': 'drive_add pci-addr=auto if=none,id=drive0'"
+                   "   'command-line': 'drive_add 0 if=none,id=drive0'"
                    "}}");
     g_assert(response);
     g_assert_cmpstr(qdict_get_try_str(response, "return"), ==, "OK\r\n");