Age | Commit message (Collapse) | Author | Files | Lines |
|
* Update docs/submodules/nfvbench from branch 'master'
- [NFVBENCH-68] Skip bonding interfaces for interface discovery
Change-Id: I446f02366a7d1801dc6740657bfea23f73bc27c7
Signed-off-by: Yichen Wang <yicwang@cisco.com>
|
|
* Update docs/submodules/releng-xci from branch 'master'
- Merge "xci: osa: Do not wipe cinder volumes"
- xci: osa: Do not wipe cinder volumes
When a volume is removed, cinder will fill the entire disk with 0s. This
can slow down the volume removal process. Since we do not care what
happens with the volume after it's being removed, we don't need to
wipe it at all.
Change-Id: I5ef1b01a18cdf2694252b450ca4f1c6a37090f71
Link: https://docs.openstack.org/cinder/latest/sample_config.html
Signed-off-by: Markos Chandras <mchandras@suse.de>
|
|
* Update docs/submodules/apex from branch 'master'
- Typo fix for disabling ipxe
JIRA: APEX-535
Change-Id: If1d074d01246407b322d5a4bc27dfde35349e9db
Signed-off-by: Dan Radez <dradez@redhat.com>
|
|
* Update docs/submodules/sfc from branch 'master'
- Bump SHAs for Pike
Change-Id: I0b0aabffb2a5995364051fa34d218b2ebc649d4e
Signed-off-by: Manuel Buil <mbuil@suse.com>
|
|
* Update docs/submodules/releng from branch 'master'
- Merge "Jenkins not reporting lint results to gerrit"
- Jenkins not reporting lint results to gerrit
I set -x in a test and then it did report.
not sure what's going on here,
I thought this was fixed.
May as well cat the violation log as we debug
Change-Id: I142fb3855ebc000ee90aa24e0ecb336bb9b313df
Signed-off-by: agardner <agardner@linuxfoundation.org>
|
|
* Update docs/submodules/releng from branch 'master'
- Merge "Update Pharos Dashboard Deploy Job and Disable"
- Update Pharos Dashboard Deploy Job and Disable
Copying the keys required for talking to JIRA and disabling the job
until chages can be tested to ensure a deploy runs successfully.
Change-Id: Ib3602338cfe06d5507f5d3f5b853906feb39f4c5
Signed-off-by: Trevor Bramwell <tbramwell@linuxfoundation.org>
|
|
* Update docs/submodules/releng from branch 'master'
- Merge "jjb: xci: Use the 'ts' utility to print timestamps on executed steps."
- jjb: xci: Use the 'ts' utility to print timestamps on executed steps.
We can use the 'ts' utility to print timestamps on every executed
command so we can keep track on the duration of each task and the
job overall.
Change-Id: I77d5d66c0146f943d805612ca5b47e2eed3ffc41
Signed-off-by: Markos Chandras <mchandras@suse.de>
|
|
* Update docs/submodules/releng from branch 'master'
- Merge "Fix SFC Email PTL Macro"
- Fix SFC Email PTL Macro
Includes the name of the PTL but should be just the email.
Change-Id: If6ef3b7c15defd44c6f86a24d6250840c37dd24b
Signed-off-by: Trevor Bramwell <tbramwell@linuxfoundation.org>
|
|
* Update docs/submodules/releng from branch 'master'
- Merge "Update Functest jobs"
- Update Functest jobs
Add support for functest tier with multijob
Add publishers separate for fetures and
all other functest tiers
Run functest features suits in a mutijob
Since there are no arm specific requirements,
remove arm-daily builders and jobs
JIRA: FUNCTEST-917
Change-Id: I92e3d7ca378dd6d293520c679c741de4bf336639
Signed-off-by: Delia Popescu <delia.popescu@enea.com>
|
|
* Update docs/submodules/barometer from branch 'master'
- Update mcelog.conf.erb template
The <Memory> tag is required for MCELog.
This was not updated in the puppet templates used by Apex.
Co-Authored-By: Aaron Smith <aasmith@redhat.com>
Co-Authored-By: Gordon Kelly <gordon.kelly@intel.com>
Change-Id: I6b0793b33099ef448316b9abcd4f0293d7518e29
Signed-off-by: Emma Foley <emma.l.foley@intel.com>
|
|
* Update docs/submodules/fuel from branch 'master'
- [noHA] Add dedicated cinder volume for virtual cmp
Replace loop device LVM-backed cinder volume with a dedicated
/dev/vdb drive.
This is another small step towards bringing noHA to baremetal.
Change-Id: I80f9c2bee42e933a36ab7a8f9b4c5247d1652b42
Signed-off-by: Alexandru Avadanii <Alexandru.Avadanii@enea.com>
|
|
* Update docs/submodules/releng-xci from branch 'master'
- Update the .gitignore in roles
THe prepare-functest role changed the name but .gitignore is still
using the old name
Change-Id: I67e962704a62756663abdc721fb8b9a9ac8648a2
Signed-off-by: Manuel Buil <mbuil@suse.com>
|
|
* Update docs/submodules/releng-xci from branch 'master'
- xci: roles: configure-network: Determine host NIC from Ansible facts
Hardcoding the interface as a variable is very fragile since it varies
from host to host. We could use the Ansible facts to find out the
interface name and then use that to configure all the VLANs and
networking.
Change-Id: Ie7e2409d638625b9bede23b6c1fe33dc36f81840
Signed-off-by: Markos Chandras <mchandras@suse.de>
|
|
* Update docs/submodules/functest from branch 'master'
- Merge "Use constants for Functest config file"
- Use constants for Functest config file
env_file can't be defined in a config file simply because it's an
entry point (Jenkins jobs, end users)
Change-Id: Ie23c0ef90efc839d60f0f3a9754c58746b1f3a00
Signed-off-by: Cédric Ollivier <cedric.ollivier@orange.com>
|
|
* Update docs/submodules/yardstick from branch 'master'
- Merge "Fix error in address input format in "_ip_range_action_partial""
- Fix error in address input format in "_ip_range_action_partial"
IP address format introduced in [1] should be unicode instead of string.
"ipaddress.IPv4Address(min_value)" doesn't parse correctly the input
parameter unless the parameter is in unicode format; this is valid both
for Python version 2 and 3.
Execution error if the parameter is a string:
>>> int(ipaddress.IPv4Address('10.0.3.2'))
Traceback (most recent call last):
File "<stdin>", line 1, in <module>
File "/usr/local/lib/python2.7/dist-packages/ipaddress.py",
line 1391, in __init__
self._check_packed_address(address, 4)
File "/usr/local/lib/python2.7/dist-packages/ipaddress.py",
line 554, in _check_packed_address
expected_len, self._version))
ipaddress.AddressValueError: '10.0.3.2' (len 8 != 4) is not permitted
as an IPv4 address. Did you pass in a bytes (str in Python 2) instead
of a unic
[1]https://github.com/opnfv/yardstick/blob/e5775e7efbc55f116b4d4ac11ff87b8d8553247e/yardstick/network_services/traffic_profile/traffic_profile.py#L87-L88
JIRA: YARDSTICK-996
Change-Id: Ic727a79044834b181c99789f0f5efc21c68f0ff2
Signed-off-by: Rodolfo Alonso Hernandez <rodolfo.alonso.hernandez@intel.com>
|
|
* Update docs/submodules/vswitchperf from branch 'master'
- Merge "LTD: Expand requirements of Soak/Long Duration Tests"
- LTD: Expand requirements of Soak/Long Duration Tests
In Project discussion on Jan 17, 2018 including review of RFC 8204,
we identified the following five areas to improve the Soak tests.
See the JIRA ticket for details.
JIRA: VSPERF-557
Change-Id: I5314c283a383c57d99863fcf7262c4f28e1c12a8
Signed-off-by: Al Morton <acmorton@att.com>
Reviewed-by: Martin Klozik <martinx.klozik@intel.com>
Reviewed-by: Maryam Tahhan <maryam.tahhan@intel.com>
Reviewed-by: Christian Trautman <ctrautma@redhat.com>
Reviewed-by: Sridhar Rao <sridhar.rao@spirent.com>
Reviewed-by: Trevor Cooper <trevor.cooper@intel.com>
Reviewed-by: Richard Elias <richardx.elias@intel.com>
|
|
* Update docs/submodules/samplevnf from branch 'master'
- Merge "Fixed the calculation of dropped packets and other changes"
- Fixed the calculation of dropped packets and other changes
* Calculation of aboslute and relative dropped packets was wrong and is
now fixed.
* ./createrapid.py has now an extra option to select custom availability
zones for the different VMs.
* Fixed a typo to create the flavor: hw:cpu_thread_policy
* The generator config files have now the packet id & signature fields.
See PROX for more details on this.
* ./runrapid has now a new option --configonly. Default is False. If set
to True, all configuration files will be copied to the VMS, but the
tests are not started. This allows you to go to the VMs and manually
start PROX.
* In addition to the average latency, maximum latency is now also
displayed.
Change-Id: I3bbd7897deb6e452068c07ef68d3959ffa4a4b50
Signed-off-by: Luc Provoost <luc.provoost@intel.com>
|
|
* Update docs/submodules/samplevnf from branch 'master'
- Merge "Add support for comments in configuration variables"
- Add support for comments in configuration variables
This feature will enable the possibility to have many cores
configured in a prox config file, and enable/disable them
through variables.
For instance, a [core $var1] section in a config file will
result in [core 1] if $var1 = 1; the whole section and section
content will be ignored if $var1=#
Before this implementation, [#core 1] or [core #] was already
treated as a commented out section (the whole section was
commented). But there was no way to define a variable $var = #
to comment a section through a variable.
Note that in today's implementation any non numerical
(except s, h, t, -) characteter in the [core] section header
(and not only #) will cause the section to be ignored.
It would probably be better to consider # (as maybe N/A and none)
as comments, and everything else as error. This is however not
supported by the change request.
Change-Id: Id4e2b27a1f9b6d595e0b442dcd971ad44a502031
Signed-off-by: Xavier Simonart <xavier.simonart@intel.com>
|
|
* Update docs/submodules/samplevnf from branch 'master'
- Merge "Fix latency accuracy and dumping latencies to file"
- Fix latency accuracy and dumping latencies to file
- Also changed lat_info.tx_packet_index to uint64_t,
so also did it for task_lat.prev_tx_packet_index and n_loss.
- Adjusted format strings accordingly, and fixed some other formats.
- Adjusted overflow increment to 2^32 (i.e. UINT32_MAX + 1).
- Replaced hard-coded 64 with ACCURACY_BUFFER_SIZE (still hard-coded
in handle_gen.c).
Change-Id: Ia59f36e17c0797a2a958dbe3b2ac420263473524
Signed-off-by: Xavier Simonart <xavier.simonart@intel.com>
Signed-off-by: Patrice Buriez <patrice.buriez@intel.com>
|
|
* Update docs/submodules/barometer from branch 'master'
- Merge "docs: update userguide with zookeeper instance tags and fix typos"
- docs: update userguide with zookeeper instance tags and fix typos
Change-Id: I916f6e9388a7ac25ff0b5631ec49fa25157c909f
Signed-off-by: Gordon Kelly <gordon.kelly@intel.com>
|
|
* Update docs/submodules/barometer from branch 'master'
- Merge "docker: add command to run run_collectd script on bring up"
- docker: add command to run run_collectd script on bring up
Change-Id: I509ee90f29115171e65b77dbba3081ae69aa0c8c
Signed-off-by: Gordon Kelly <gordon.kelly@intel.com>
|
|
* Update docs/submodules/yardstick from branch 'master'
- Merge "Make files pep8 compliant"
- Make files pep8 compliant
This change makes files pep8 compliant before adding the
reuse context feature
JIRA: YARDSTICK-886
Change-Id: Iae7daaa159a9ddbb5809a9a7ac74f8a53683089a
Signed-off-by: Emma Foley <emma.l.foley@intel.com>
|
|
|
|
* Update docs/submodules/yardstick from branch 'master'
- Merge "Refactor benchmark.core.test_plugin.PluginTestCase"
- Refactor benchmark.core.test_plugin.PluginTestCase
Tests in benchmark.core.test_plugin.PluginTestCase are not covering
correctly the functionality of the tested class.
YARDSTICK-851
Change-Id: I0b4cda7dbf109776a202167abfde8c9eb268db12
Signed-off-by: Rodolfo Alonso Hernandez <rodolfo.alonso.hernandez@intel.com>
|
|
* Update docs/submodules/sfc from branch 'master'
- Merge "Switch to os.environ for getting OpenStack env vars"
- Switch to os.environ for getting OpenStack env vars
CONST shouldn't be selected to get env vars.
Now it may raise KeyErrors if env is not well configured.
Change-Id: I5e3f525640809778c5349fdc1c3796227894d0cd
Signed-off-by: Cédric Ollivier <cedric.ollivier@orange.com>
|
|
* Update docs/submodules/functest from branch 'master'
- Merge "Unify security group operations"
- Unify security group operations
It updates juju_epc to conform with cloudify_ims and
cloudify_vrouter regarding security group operations.
Change-Id: I2bc4a16ec17f2836564df82d3ceddd3bc73ece7a
Signed-off-by: Cédric Ollivier <cedric.ollivier@orange.com>
|
|
* Update docs/submodules/armband from branch 'master'
- u/fuel: Bump & rebase for cinder disk volume
JIRA: FUEL-330
Change-Id: If089aa41a77a9694ae99dfee79a105ac92917416
Signed-off-by: Alexandru Avadanii <Alexandru.Avadanii@enea.com>
|
|
* Update docs/submodules/fuel from branch 'master'
- [HA] [cinder] Fix rootfs part name for sda > 2TB
On systems with more than 2TB on /dev/sda, MaaS automatically creates
a 1MB partition with bios_grub label as /dev/sda1.
Account for this and expect rootfs to reside on /dev/sda2 (assume
all compute nodes have the same behavior wrt this issue).
Ideally, later we will switch to explicit partitioning via MaaS, or to
a new baremetal provisioner like Ironic.
Requires bumping Pharos git submodule to pick up the new j2 filter.
Fixes: f6f8a56
JIRA: FUEL-330
Change-Id: I12f5046cf69bf2f52c8b8dc331b90048df643646
Signed-off-by: Alexandru Avadanii <Alexandru.Avadanii@enea.com>
|
|
* Update docs/submodules/pharos from branch 'master'
- gen_config_lib: Add storage_size_num filter
New filter for converting PDF disk_capacity strings into a numeric
only string (to bypass scientific notation), parsable as float.
Change-Id: I779bec7db03ef1b3ce1bfd6ca239a3e7b4021eb2
Signed-off-by: Alexandru Avadanii <Alexandru.Avadanii@enea.com>
|
|
* Update docs/submodules/compass4nfv from branch 'master'
- Merge "add a multus with 2 fannel interfaces installation"
- add a multus with 2 fannel interfaces installation
Support deploying multus multiple flannel CNI plugins by setting
environment "kube_network_plugin" to "2flannel".
Change-Id: I23a3d42452b4a5e0858a18934f508649f5961e7a
Signed-off-by: Di Xu <di.xu@arm.com>
|
|
* Update docs/submodules/releng from branch 'master'
- Move generation of artifacts.opnfv.org to jenkins
Move generation of artifacts.opnfv.org/index.html
and sub pages into a jenkins job.
Script was run on cron on master previously.
run on lf-build2
Run job once per hour (it takes a long time to run)
Block if job is already running
Change-Id: Icc03cac61a802e036e4af8a9b9e3a244e147abea
Signed-off-by: agardner <agardner@linuxfoundation.org>
|
|
* Update docs/submodules/barometer from branch 'master'
- compose: modify zookeeper tag and add container names
Change-Id: I676940bb0d5c7d9e181d27b15130e6ec4a4ececd
Signed-off-by: Gordon Kelly <gordon.kelly@intel.com>
|
|
* Update docs/submodules/functest from branch 'master'
- Merge "Leverage of snaps creds to get the right region name"
- Leverage of snaps creds to get the right region name
Else it gets the defaut value as it's unset in env.
Change-Id: Ic2a9cdaed7891fe2723f960782c81c0cbb04e8f0
Signed-off-by: Cédric Ollivier <cedric.ollivier@orange.com>
|
|
* Update docs/submodules/functest from branch 'master'
- Merge "Print stdout and stderr when calling commands over SSH"
- Print stdout and stderr when calling commands over SSH
It will help debugging possible issues vs APEX [1].
[1] https://build.opnfv.org/ci/view/functest/job/functest-apex-baremetal-daily-master/792/console
Change-Id: I610f5b0a7774440c51fa086fad4f61c9be0571dc
Signed-off-by: Cédric Ollivier <cedric.ollivier@orange.com>
|
|
* Update docs/submodules/functest from branch 'master'
- Create a second user in cloudify_vrouter
It mainly bypasses a bug when a similar network name has already been
created.
We can't apply the same operations for cloudify_vims because the key
must be created by the _member_ user.
Snaps haven't allowed that since [1] has been merged
[1] https://gerrit.opnfv.org/gerrit/#/c/52157/
Change-Id: I90131487d0e08a4d95c684efb16a8009c3709a8e
Signed-off-by: Cédric Ollivier <cedric.ollivier@orange.com>
|
|
* Update docs/submodules/releng-xci from branch 'master'
- Correct the BIFROST_IRONIC_VERSION variable
By mistake the BIFROST_IRONIC_VERSION variable was
written BIFROST_IRONIC_CLIENT_VERSION
Change-Id: Ie35a4938a2fb38c633dd149c1d18f1da20a4e82a
Signed-off-by: Dimitrios Markou <mardim@intracom-telecom.com>
|
|
* Update docs/submodules/fuel from branch 'master'
- [HA] [cinder] Switch loop to free space on sda
On cmp nodes, we assing 30G (fixed) to rootfs, use the rest for
cinder.
Note: AArch64 gets one extra partition (the EFI System Partition or
ESP) on /dev/sda1 via MaaS.
JIRA: FUEL-330
Change-Id: I2a36107d074532b627bd2349cafc0c8ee61f500f
Signed-off-by: Alexandru Avadanii <Alexandru.Avadanii@enea.com>
- [salt.sh] Armband extra repo for forked reclass
- add Armband nightly/extra DEB repository;
- install forked and updated reclass 1.5.2 which includes:
* better error reporting;
* support for deleting existing keys during list interpolation;
* various other improvements and optimizations;
While at it, update copyright year for patches.
JIRA: FUEL-345
Change-Id: I00d8b625fe191648e7ea34b3dd4c8375691384e6
Signed-off-by: Alexandru Avadanii <Alexandru.Avadanii@enea.com>
|
|
* Update docs/submodules/releng from branch 'master'
- Merge "Add functest-kubernetes repo to lint verification"
- Add functest-kubernetes repo to lint verification
Change-Id: Ia1e05db344e6e6b4a773bf17157488716ac0c305
Signed-off-by: Jose Lausuch <jalausuch@suse.com>
|
|
* Update docs/submodules/sfc from branch 'master'
- Update XCI deployment files
We are now aligned with master so we don't need a-r-r file anymore or the
OpenStack versions file
Change-Id: Ida0b8f0414870d5fa3fecdc79564def345187b2e
Signed-off-by: Manuel Buil <mbuil@suse.com>
|
|
* Update docs/submodules/snaps from branch 'master'
- openstack: nova_utils_tests: Wait for volume detachment
The 'test_add_remove_volum' is supposed to wait for volume attachment
and detachments. However, the code for the latter was missing. Moreover,
we remove some code that checks for attached volumes since that should
never happen when detaching the volume.
Change-Id: I9ed54a467062d0ddc4efeddf03af7ebe6173e789
Signed-off-by: Markos Chandras <mchandras@suse.de>
|
|
* Update docs/submodules/snaps from branch 'master'
- openstack: nova_utils_tests: Increase timeout for API calls
Patch I2245adb84d0e6d4e3350d17d0c2e44baf5202d51 increased the timeout
for the API calls to 2 minutes which was then somewhat reverted in
Iea3aeab59c378917fbd175d673113e8d30e2e4b9. However, the problem still
persists on slow systems and 30 seconds may not be enough to confirm
a volume attachment to the VM.
Change-Id: I860b2a797917e13772e95c5d756919e9792b63a6
Signed-off-by: Markos Chandras <mchandras@suse.de>
|
|
* Update docs/submodules/joid from branch 'master'
- modfiied to reflect the supported version only.
Change-Id: Ib8fd3588376a160a4e1c9260ed7a6b51635f6152
Signed-off-by: Narinder Gupta <narinder.gupta@canonical.com>
|
|
* Update docs/submodules/sfc from branch 'master'
- Take into account last Functest config updates
It follows the next changes (Functest) which renamed the related
config entry [1].
[1] https://gerrit.opnfv.org/gerrit/#/c/51283/5/functest/ci/config_functest.yaml
Change-Id: Ieae5baff13bb70aa35b01c2358ab4865cdb9dfb0
Signed-off-by: Cédric Ollivier <cedric.ollivier@orange.com>
|
|
* Update docs/submodules/yardstick from branch 'master'
- Merge "Bug fix for Ansible destroy VM script"
- Bug fix for Ansible destroy VM script
Modified the destroy script to ignore errors as 2 possible
problems can cause a play to fail.
1. A VM can be running but undefined.
2. A VM can be destroyed but still running.
JIRA: YARDSTICK-1006
Change-Id: Ib66f48401bfefb598f553cfef31bfa2682f5a584
Signed-off-by: John O Loughlin <john.oloughlin@intel.com>
|
|
* Update docs/submodules/releng from branch 'master'
- Merge "Rename var XCI_LOOP to CI_LOOP to align with Functest"
- Rename var XCI_LOOP to CI_LOOP to align with Functest
Change-Id: If5c5b828312dc588515589b8075b9754537b8f86
Signed-off-by: Fatih Degirmenci <fdegir@gmail.com>
|
|
* Update docs/submodules/releng from branch 'master'
- Merge "jjb: xci: Fix bifrost blockers for XCI jobs"
- jjb: xci: Fix bifrost blockers for XCI jobs
The bifrost regexp was wrong leading to conflicts betwen the XCI
and bifrost jobs. Moreover, we need to add the same restrictions for
jobs/node on bifrost jobs similar to the XCI ones. This finally fixes
the 'throttle' property to use the 'xci-verify-virtual' category.
Change-Id: I6d3f8fdd82738c1d079b1a3fdef134753eb4281a
Signed-off-by: Markos Chandras <mchandras@suse.de>
|
|
* Update docs/submodules/releng-xci from branch 'master'
- Merge "xci: xci-deploy.sh: Calculate and print the total deployment time"
- xci: xci-deploy.sh: Calculate and print the total deployment time
It's useful to know how XCI performs in terms of duration time so
use the bash $SECONDS variable to calculate the total amount of time
it takes for a full XCI deployment. Moreover, use the 'ts' command to
print a timestamp for every command executed. This will help developers
identify tasks which take a lot of time to complete and possibly
optimize them.
Change-Id: I75a8cb8c05957bb205e63210d6cafaf54d989f32
Signed-off-by: Markos Chandras <mchandras@suse.de>
|
|
* Update docs/submodules/sfc from branch 'master'
- Merge "Switch from CONST to env.get()"
- Switch from CONST to env.get()
env.get() ensures that default values are reused if unset.
It also avoids desynchronization between os.environ and CONST.
It follows the change "Get properly env vars or their default values"
[1] applied in Functest.
It also fixes minor issues on shebangs and takes several pylint rules
into account in this module:
- http://pylint-messages.wikidot.com/messages:w0702
- https://www.python.org/dev/peps/pep-0282/
[1] https://gerrit.opnfv.org/gerrit/#/c/52221/
Change-Id: Ia7ad168b2cbbade8ea3890fb95dc01bcdfc50468
Signed-off-by: Cédric Ollivier <cedric.ollivier@orange.com>
|
|
* Update docs/submodules/releng-xci from branch 'master'
- Merge "xci: playbooks: run-functest: Collect functest results"
- xci: playbooks: run-functest: Collect functest results
Add a new volume to the docker container which will hold the results
from the functest run.
Change-Id: I70541a8337c9198a97304312adfab74d4dd06b70
Signed-off-by: Markos Chandras <mchandras@suse.de>
|
|
* Update docs/submodules/releng-xci from branch 'master'
- xci: Switch VM disk cache to 'unsafe' and use 'iothreads' for I/O
According to the docs[1]
"writeback: This mode causes the hypervisor to interact with the disk
image file or block device with neither O_DSYNC nor O_DIRECT semantics.
The host page cache is used and writes are reported to the guest as
completed when they are placed in the host page cache. The normal page
cache management will handle commitment to the storage device.
Additionally, the guest's virtual storage adapter is informed of the
writeback cache, so the guest would be expected to send down flush
commands as needed to manage data integrity. Analogous to a raid
controller with RAM cache."
and
"writeback: This mode informs the guest of the presence of a write
cache, and relies on the guest to send flush commands as needed to
maintain data integrity within its disk image. This is a common
storage design which is completely accounted for within modern file
systems. This mode exposes the guest to data loss in the unlikely case
of a host failure, because there is a window of time between the time
a write is reported as completed, and that write being committed to the
storage device."
"unsafe: This mode is similar to writeback caching except for the
following: the guest flush commands are ignored, nullifying the data
integrity control of these flush commands, and resulting in a higher
risk of data loss because of host failure. The name “unsafe” should
serve as a warning that there is a much higher potential for data
loss because of a host failure than with the other modes. As the
guest terminates, the cached data is flushed at that time."
It's beneficial to use the host page cache to cache I/O from the guest
instead of waiting for data to reach the actual disk device. We do not
normally care about data integrity so data loss is not a problem.
Moreover, we drop the cache configuration from the flavor files since
it's independent of the flavor that's being deployed.
[1] https://www.suse.com/documentation/sles-12/singlehtml/book_virt/book_virt.html#cha.cachemodes
Change-Id: I118ffdf84b1be672185b3eff60fe5d0b5f1a590d
Signed-off-by: Markos Chandras <mchandras@suse.de>
|