2012-05-17 15:08:53 +00:00
|
|
|
/*
|
|
|
|
* virsh-edit.c: Implementation of generic virsh *-edit intelligence
|
|
|
|
*
|
|
|
|
* Copyright (C) 2012 Red Hat, Inc.
|
|
|
|
*
|
2012-07-27 09:39:53 +00:00
|
|
|
* This library is free software; you can redistribute it and/or
|
|
|
|
* modify it under the terms of the GNU Lesser General Public
|
|
|
|
* License as published by the Free Software Foundation; either
|
|
|
|
* version 2.1 of the License, or (at your option) any later version.
|
|
|
|
*
|
|
|
|
* This library is distributed in the hope that it will be useful,
|
|
|
|
* but WITHOUT ANY WARRANTY; without even the implied warranty of
|
|
|
|
* MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU
|
|
|
|
* Lesser General Public License for more details.
|
|
|
|
*
|
|
|
|
* You should have received a copy of the GNU Lesser General Public
|
|
|
|
* License along with this library; If not, see
|
|
|
|
* <http://www.gnu.org/licenses/>.
|
2012-05-17 15:08:53 +00:00
|
|
|
*
|
|
|
|
* Usage:
|
|
|
|
* Define macros:
|
|
|
|
* EDIT_GET_XML - expression which produces a pointer to XML string, e.g:
|
|
|
|
* #define EDIT_GET_XML virDomainGetXMLDesc(dom, flags)
|
|
|
|
*
|
|
|
|
* EDIT_NOT_CHANGED - this action is taken if the XML wasn't changed.
|
|
|
|
* Note, that you don't want to jump to cleanup but edit_cleanup label
|
|
|
|
* where temporary variables are free()-d and temporary file is deleted:
|
|
|
|
* #define EDIT_NOT_CHANGED vshPrint(ctl, _("Domain %s XML not changed"), \
|
|
|
|
* virDomainGetName(dom)); \
|
|
|
|
* ret = true; goto edit_cleanup;
|
|
|
|
* Note that this is a statement.
|
|
|
|
*
|
|
|
|
* EDIT_DEFINE - expression which redefines the object. The edited XML from
|
|
|
|
* user is in 'doc_edited' variable. Don't overwrite the pointer to the
|
|
|
|
* object, as we may iterate once more over and therefore the pointer
|
|
|
|
* would be invalid. Hence assign object to a different variable.
|
|
|
|
* Moreover, this needs to be an expression where:
|
|
|
|
* - 0 is taken as error (our virDefine* APIs often return NULL on error)
|
|
|
|
* - everything else is taken as success
|
|
|
|
* For example:
|
|
|
|
* #define EDIT_DEFINE (dom_edited = virDomainDefineXML(ctl->conn, doc_edited))
|
|
|
|
*
|
|
|
|
* EDIT_FREE - statement which vir*Free()-s object defined by EDIT_DEFINE, e.g:
|
|
|
|
* #define EDIT_FREE if (dom_edited) virDomainFree(dom_edited);
|
|
|
|
*
|
|
|
|
* Michal Privoznik <mprivozn@redhat.com>
|
|
|
|
*/
|
|
|
|
|
|
|
|
#ifndef EDIT_GET_XML
|
|
|
|
# error Missing EDIT_GET_XML definition
|
|
|
|
#endif
|
|
|
|
|
|
|
|
#ifndef EDIT_NOT_CHANGED
|
|
|
|
# error Missing EDIT_NOT_CHANGED definition
|
|
|
|
#endif
|
|
|
|
|
|
|
|
#ifndef EDIT_DEFINE
|
|
|
|
# error Missing EDIT_DEFINE definition
|
|
|
|
#endif
|
|
|
|
|
|
|
|
#ifndef EDIT_FREE
|
|
|
|
# error Missing EDIT_FREE definition
|
|
|
|
#endif
|
|
|
|
|
|
|
|
do {
|
|
|
|
char *tmp = NULL;
|
|
|
|
char *doc = NULL;
|
|
|
|
char *doc_edited = NULL;
|
|
|
|
char *doc_reread = NULL;
|
2012-06-07 03:13:57 +00:00
|
|
|
const char *msg = NULL;
|
2012-05-17 15:08:53 +00:00
|
|
|
|
|
|
|
/* Get the XML configuration of the object. */
|
|
|
|
doc = (EDIT_GET_XML);
|
|
|
|
if (!doc)
|
|
|
|
goto edit_cleanup;
|
|
|
|
|
|
|
|
/* Create and open the temporary file. */
|
virsh: use common namespacing
Convert the exported items in virsh.h to use a common 'vsh' prefix.
* tools/virsh.h (VIRSH_MAX_XML_FILE): Rename...
(VSH_MAX_XML_FILE): ...and parenthesize.
(DIFF_MSEC, CTRL_CLOSE_BRACKET): Delete.
(vshUsage, vshInit, vshDeinit, vshParseArgv): Remove prototype.
(editWriteToTempFile, editFile, editReadBackFile, prettyCapacity)
(virshReportError): Rename...
(vshEditWriteToTempFile, vshEditFile, vshEditReadBackFile)
(vshPrettyCapacity, vshReportError): ...into vsh namespace.
(jobWatchTimeoutFunc): Move to virsh-domain.c.
* tools/virsh.c (vshCommandRun): Inline former DIFF_MSEC.
(main): Inline former CTRL_CLOSE_BRACKET.
(vshUsage, vshInit, vshDeinit, vshParseArgv): Make static.
(prettyCapacity, virshReportError, editWriteToTempFile, editFile):
Fix naming, and adjust usage.
(vshAskReedit, vshCommandRun, vshEventLoop, vshInit): Adjust
usage.
* tools/virsh-domain.c (cmdAttachDevice, cmdCPUCompare)
(cmdCPUBaseline, cmdCreate, cmdDefine, cmdDetachDevice)
(cmdUpdateDevice, cmdDesc, cmdUndefine, cmdStart, cmdVcpucount)
(cmdAttachDevice, cmdDomjobinfo): Likewise.
* tools/virsh-edit.c (do): Likewise.
* tools/virsh-interface.c (cmdInterfaceDefine): Likewise.
* tools/virsh-network.c (cmdNetworkCreate, cmdNetworkDefine):
Likewise.
* tools/virsh-nodedev.c (cmdNodeDeviceCreate): Likewise.
* tools/virsh-nwfilter.c (cmdNWFilterDefine): Likewise.
* tools/virsh-pool.c (cmdPoolCreate, cmdPoolDefine)
(cmdPoolDiscoverSources, cmdPoolList): Likewise.
* tools/virsh-secret.c (cmdSecretDefine): Likewise.
* tools/virsh-snapshot.c (cmdSnapshotCreate, vshSnapshotCreate)
(vshLookupSnapshot, cmdSnapshotEdit, cmdSnapshotCurrent)
(vshGetSnapshotParent): Likewise.
* tools/virsh-volume.c (cmdVolCreate, cmdVolCreateFrom)
(cmdVolInfo, cmdVolList): Likewise.
2012-08-19 04:10:17 +00:00
|
|
|
tmp = vshEditWriteToTempFile(ctl, doc);
|
2012-05-17 15:08:53 +00:00
|
|
|
if (!tmp)
|
|
|
|
goto edit_cleanup;
|
|
|
|
|
2012-05-18 10:21:06 +00:00
|
|
|
reedit:
|
2012-05-17 15:08:53 +00:00
|
|
|
/* Start the editor. */
|
virsh: use common namespacing
Convert the exported items in virsh.h to use a common 'vsh' prefix.
* tools/virsh.h (VIRSH_MAX_XML_FILE): Rename...
(VSH_MAX_XML_FILE): ...and parenthesize.
(DIFF_MSEC, CTRL_CLOSE_BRACKET): Delete.
(vshUsage, vshInit, vshDeinit, vshParseArgv): Remove prototype.
(editWriteToTempFile, editFile, editReadBackFile, prettyCapacity)
(virshReportError): Rename...
(vshEditWriteToTempFile, vshEditFile, vshEditReadBackFile)
(vshPrettyCapacity, vshReportError): ...into vsh namespace.
(jobWatchTimeoutFunc): Move to virsh-domain.c.
* tools/virsh.c (vshCommandRun): Inline former DIFF_MSEC.
(main): Inline former CTRL_CLOSE_BRACKET.
(vshUsage, vshInit, vshDeinit, vshParseArgv): Make static.
(prettyCapacity, virshReportError, editWriteToTempFile, editFile):
Fix naming, and adjust usage.
(vshAskReedit, vshCommandRun, vshEventLoop, vshInit): Adjust
usage.
* tools/virsh-domain.c (cmdAttachDevice, cmdCPUCompare)
(cmdCPUBaseline, cmdCreate, cmdDefine, cmdDetachDevice)
(cmdUpdateDevice, cmdDesc, cmdUndefine, cmdStart, cmdVcpucount)
(cmdAttachDevice, cmdDomjobinfo): Likewise.
* tools/virsh-edit.c (do): Likewise.
* tools/virsh-interface.c (cmdInterfaceDefine): Likewise.
* tools/virsh-network.c (cmdNetworkCreate, cmdNetworkDefine):
Likewise.
* tools/virsh-nodedev.c (cmdNodeDeviceCreate): Likewise.
* tools/virsh-nwfilter.c (cmdNWFilterDefine): Likewise.
* tools/virsh-pool.c (cmdPoolCreate, cmdPoolDefine)
(cmdPoolDiscoverSources, cmdPoolList): Likewise.
* tools/virsh-secret.c (cmdSecretDefine): Likewise.
* tools/virsh-snapshot.c (cmdSnapshotCreate, vshSnapshotCreate)
(vshLookupSnapshot, cmdSnapshotEdit, cmdSnapshotCurrent)
(vshGetSnapshotParent): Likewise.
* tools/virsh-volume.c (cmdVolCreate, cmdVolCreateFrom)
(cmdVolInfo, cmdVolList): Likewise.
2012-08-19 04:10:17 +00:00
|
|
|
if (vshEditFile(ctl, tmp) == -1)
|
2012-05-17 15:08:53 +00:00
|
|
|
goto edit_cleanup;
|
|
|
|
|
|
|
|
/* Read back the edited file. */
|
virsh: use common namespacing
Convert the exported items in virsh.h to use a common 'vsh' prefix.
* tools/virsh.h (VIRSH_MAX_XML_FILE): Rename...
(VSH_MAX_XML_FILE): ...and parenthesize.
(DIFF_MSEC, CTRL_CLOSE_BRACKET): Delete.
(vshUsage, vshInit, vshDeinit, vshParseArgv): Remove prototype.
(editWriteToTempFile, editFile, editReadBackFile, prettyCapacity)
(virshReportError): Rename...
(vshEditWriteToTempFile, vshEditFile, vshEditReadBackFile)
(vshPrettyCapacity, vshReportError): ...into vsh namespace.
(jobWatchTimeoutFunc): Move to virsh-domain.c.
* tools/virsh.c (vshCommandRun): Inline former DIFF_MSEC.
(main): Inline former CTRL_CLOSE_BRACKET.
(vshUsage, vshInit, vshDeinit, vshParseArgv): Make static.
(prettyCapacity, virshReportError, editWriteToTempFile, editFile):
Fix naming, and adjust usage.
(vshAskReedit, vshCommandRun, vshEventLoop, vshInit): Adjust
usage.
* tools/virsh-domain.c (cmdAttachDevice, cmdCPUCompare)
(cmdCPUBaseline, cmdCreate, cmdDefine, cmdDetachDevice)
(cmdUpdateDevice, cmdDesc, cmdUndefine, cmdStart, cmdVcpucount)
(cmdAttachDevice, cmdDomjobinfo): Likewise.
* tools/virsh-edit.c (do): Likewise.
* tools/virsh-interface.c (cmdInterfaceDefine): Likewise.
* tools/virsh-network.c (cmdNetworkCreate, cmdNetworkDefine):
Likewise.
* tools/virsh-nodedev.c (cmdNodeDeviceCreate): Likewise.
* tools/virsh-nwfilter.c (cmdNWFilterDefine): Likewise.
* tools/virsh-pool.c (cmdPoolCreate, cmdPoolDefine)
(cmdPoolDiscoverSources, cmdPoolList): Likewise.
* tools/virsh-secret.c (cmdSecretDefine): Likewise.
* tools/virsh-snapshot.c (cmdSnapshotCreate, vshSnapshotCreate)
(vshLookupSnapshot, cmdSnapshotEdit, cmdSnapshotCurrent)
(vshGetSnapshotParent): Likewise.
* tools/virsh-volume.c (cmdVolCreate, cmdVolCreateFrom)
(cmdVolInfo, cmdVolList): Likewise.
2012-08-19 04:10:17 +00:00
|
|
|
doc_edited = vshEditReadBackFile(ctl, tmp);
|
2012-05-17 15:08:53 +00:00
|
|
|
if (!doc_edited)
|
|
|
|
goto edit_cleanup;
|
|
|
|
|
|
|
|
/* Compare original XML with edited. Has it changed at all? */
|
|
|
|
if (STREQ(doc, doc_edited)) {
|
|
|
|
EDIT_NOT_CHANGED;
|
|
|
|
}
|
|
|
|
|
2012-05-18 10:21:06 +00:00
|
|
|
redefine:
|
|
|
|
msg = NULL;
|
|
|
|
|
2012-05-17 15:08:53 +00:00
|
|
|
/* Now re-read the object XML. Did someone else change it while
|
|
|
|
* it was being edited? This also catches problems such as us
|
|
|
|
* losing a connection or the object going away.
|
|
|
|
*/
|
|
|
|
doc_reread = (EDIT_GET_XML);
|
|
|
|
if (!doc_reread)
|
|
|
|
goto edit_cleanup;
|
|
|
|
|
|
|
|
if (STRNEQ(doc, doc_reread)) {
|
2012-05-18 10:21:06 +00:00
|
|
|
msg = _("The XML configuration was changed by another user.");
|
|
|
|
VIR_FREE(doc);
|
|
|
|
doc = doc_reread;
|
|
|
|
doc_reread = NULL;
|
2012-05-17 15:08:53 +00:00
|
|
|
}
|
|
|
|
|
|
|
|
/* Everything checks out, so redefine the object. */
|
|
|
|
EDIT_FREE;
|
2012-05-18 10:21:06 +00:00
|
|
|
if (!msg && !(EDIT_DEFINE)) {
|
|
|
|
msg = _("Failed.");
|
|
|
|
}
|
|
|
|
|
|
|
|
if (msg) {
|
|
|
|
int c = vshAskReedit(ctl, msg);
|
|
|
|
switch (c) {
|
|
|
|
case 'y':
|
|
|
|
goto reedit;
|
|
|
|
break;
|
|
|
|
|
|
|
|
case 'f':
|
|
|
|
goto redefine;
|
|
|
|
break;
|
|
|
|
|
|
|
|
case 'n':
|
|
|
|
goto edit_cleanup;
|
|
|
|
break;
|
|
|
|
|
|
|
|
default:
|
|
|
|
vshError(ctl, "%s", msg);
|
|
|
|
break;
|
|
|
|
}
|
|
|
|
}
|
2012-05-17 15:08:53 +00:00
|
|
|
|
|
|
|
break;
|
|
|
|
|
|
|
|
edit_cleanup:
|
|
|
|
VIR_FREE(doc);
|
|
|
|
VIR_FREE(doc_edited);
|
|
|
|
VIR_FREE(doc_reread);
|
|
|
|
if (tmp) {
|
|
|
|
unlink (tmp);
|
|
|
|
VIR_FREE(tmp);
|
|
|
|
}
|
|
|
|
goto cleanup;
|
|
|
|
|
|
|
|
} while (0);
|
|
|
|
|
|
|
|
|
|
|
|
#undef EDIT_GET_XML
|
|
|
|
#undef EDIT_NOT_CHANGED
|
|
|
|
#undef EDIT_DEFINE
|
|
|
|
#undef EDIT_FREE
|