Age | Commit message (Collapse) | Author | Files | Lines |
|
Split the container build into two stages:
* builder
Builds collectd and containers all the build artifacts
* final
Containers collectd binary and all the dependencies needed to run.
Does not contain build artifacts, which contribute to a very large
container size.
The main advantage is that the final container is much smaller (~700MB vs
1.2GB).
Change-Id: Ia75734096233b7ecab9d81df06084c24bde2bf90
Signed-off-by: Emma Foley <efoley@redhat.com>
|
|
|
|
|
|
* Add tasks to use collectd_config
* Move generated configs to combine with existing and additional configs
* Move vars to defaults dir
Some of the vars are needed for the collectd_config role, and need to
be set before the other values are set.
JIRA: BAROMETER-180
Signed-off-by: Emma Foley <efoley@redhat.com>
Change-Id: Id5a8b8289c5e120eae7eb5bcf946c5d4dd9fdd48
|
|
* config_files: Rename templates/master to templates/latest
Change-Id: I5db6491695bf35a87851f779603dc16454c117d5
Signed-off-by: Emma Foley <efoley@redhat.com>
|
|
By adding the "en_default_all" tag to each task in the
default_read_import task, there is no need to have different tasks for
that tag vs when individual plugin tags are enabled.
JIRA: BAROMETER-180
Change-Id: I15cc77959bbd25516e9da10af064abeea5527ea9
Signed-off-by: Emma Foley <efoley@redhat.com>
|
|
* Add instruction to docs/ for using
docker/barometer-collectd-experimental/Dockerfile to build collectd
version 6
* Add a new COLLECTD_FLAVOR to src/collectd/Makefile to configure
tags for a collectd 6 build.
* Allow passing COLLECTD_CONFIG_CMD_ARGS to the collectd build
For testing with new plugins for collectd-6, as un-ported plugins are
disabled by default.
This new option lets the ./configure command be run with extra args,
e.g. --enable-cpu --enable-<my-newly-ported-plugin>, which means that
plugin can be enabled for the PR that is being tested.
* Update flavor_image_name selection to include collectd-6
Change-Id: I55ee0bbcb5d89357b1ef11ab17c5b029603ecdf7
Signed-off-by: Emma Foley <efoley@redhat.com>
|
|
This is an optional build arg that overrides the defaults that are set
by COLLECTD_FLAVOR
This should let the user pull from an arbitrary collectd branch (e.g. a
WIP branch or a PR branch from upstream)
Change-Id: I6d0ede2b8b57f28e152e53eddae378ac6c722706
Signed-off-by: Emma Foley <efoley@redhat.com>
|
|
* Move collectd_apply_pull_request.sh to src/collectd/, which is
where the Makefile expects this script to be anyway
* Use ARG instead of ENV in collectd container
for COLLECTD_FLAVOR, COLLECTD_PULL_REQUESTS, WITH_DPDK
* Move the experimental sample configs to src/collectd
The configs are copied into the src/collectd directory within the
container anyway, so this skips a step in the container build.
* Remove initial package installation in containers
Move the installation of these packages to the package install steps
in the build script in systems/ instead.
* Remove unnecessary WORKDIR command
The WORKDIR {repo_dir} is unnecessary, since the following commands
are using absolute paths rather than relative paths.
Signed-off-by: Emma Foley <efoley@redhat.com>
Change-Id: I52b96a651cb5d9b644f11f5407cabf3fb21cb6a9
|
|
Capabilities plugin was interoduced in collectd-5.11, which is part of
the barometer-collectd container, so it can be included by default.
Change-Id: Ia91d883907025ab8618b39147e14d47a4fc541c3
Signed-off-by: Emma Foley <efoley@redhat.com>
|
|
The dpdk_telemetry plugin superceeds the dpdk{stats,events} plugins, and
these cannot be enabled together. Since the stable container on CentOS 8
uses DPDK 19.11, the dpdk_telemetry plugin should be used instead.
Change-Id: I20e087b40ff90ba95741aaff641ef1a745d77dd8
Signed-off-by: Emma Foley <efoley@redhat.com>
|
|
Updated the script to take the PR list from the environment
Updated the container to allow users to pass COLLECTD_PULL_REQUESTS as a
build-arg when building the container so they don't have to hard code
values to change the PRs that are applied
Change-Id: Ie820709d5f57712dc246b6728c772e086e53ab5b
Signed-off-by: Emma Foley <efoley@redhat.com>
|
|
* Use CentOS 8 instead of CentOS7
* Remove collectd-openstack-plugins
* Update the list of PRs that are applied
logparser and capabilities were merged
* Move the experimental configs to collectd_sample_configs{,-latest}
Change-Id: Ic1133ee7172e092ad4713c94886f306bc4d8e6ec
Signed-off-by: Emma Foley <efoley@redhat.com>
|
|
Collectd changed the upstream default to main, and Barometer added a few
changes to incorporate this. One change was renaming the
`barometer-collectd-master` container to `barometer-collectd-latest`, since
`barometer-collectd-main` didn't make sense semantically, as we use the
stable container as the "main" reference.
At the time, we didn't have the changes made in releng to build the new
container, so container renaming was paused.
Now that the change is merged in releng (and has been for a while), and
the containers are being updated, this change should happen, so that the
container gets published.
Signed-off-by: Emma Foley <efoley@redhat.com>
Change-Id: I4ee91c289b9e7477157bea148452be58343dc733
|
|
Update collectd to 5.12
Add in CentOS as base for the barometer-collectd container
Remove openstack plugins - they are not used anymore
Use packaged DPDK (v18.11 in CentOS 7 amd v19.11 in CentOS 8)
Install packages from package manager instead of building libpqos,
pmu-tools and librdkafka
Combine install commands into one for CentOS8
(systems/centos/8/build_base_machine.sh)
These changes reduce the size of the container from 1.7GB to ~825MB.
Change-Id: I7e474f9578cdbcc7cce8c4fbc5e297eab89fd371
Signed-off-by: Emma Foley <efoley@redhat.com>
|
|
The succeeded and failed checks are no longer supported for use as filters.
Signed-off-by: Emma Foley <efoley@redhat.com>
Change-Id: If5bdae81b34c1e63d7a2bda349beee5823e2550d
|
|
Enabling dpdk_telemetry plugin with One Click Install.
Added the config. templates and the related Documentation.
Signed-off-by: Jabir Kanhira Kadavathu <jabir.kanhira.kadavathu@intel.com>
Change-Id: Ibecd96d4102cd76a71d31613879ae57950f5108f
|
|
This reverts commit 59338794c8ae8ff82c00dc14ea47a9279cc213d4.
The reason for reverting is that there is not yet a container image available
on Dockerhub for barometer-collectd-latest
Signed-off-by: Emma Foley <efoley@redhat.com>
Change-Id: Ia6e37af99787a6c02eb368fc7bfdd9d584716f2b
|
|
Change-Id: I0dfff659b4e05e3e619ea62f1771db79f7a8f0df
Signed-off-by: Emma Foley <efoley@redhat.com>
|
|
The branch names have been updated, and main has replaced
master, which will no longer be updated.
Change-Id: Idfce21554594f15a831c8b7ec769eb85d52068b1
Signed-off-by: Emma Foley <efoley@redhat.com>
|
|
|
|
* changes:
[docs] Update Docker install guide
Update collectd container to use 5.11
|
|
The one-click install was not working with SNMP. The manual install
also wasn't working due to the ``Instance true`` config option in
the config. ``Instance true`` is not a valid config option for snmp
agent.
This option appears in the stable container config, and also in the
ansible config, but not in the master container. Removing this
unsupported option allows the config to be read and collectd to be
started.
JIRA: BAROMETER-134
Signed-off-by: Emma Foley <efoley@redhat.com>
Change-Id: If154b3190d82d1cd3638a9c258d2b48bb0efe9af
|
|
Signed-off-by: Emma Foley <efoley@redhat.com>
Change-Id: I1867ec05011a58b16a0d88bd9bd3fdfd7ac5d2e3
|
|
It's widely based on the former Functest features container.
As barometer seems leveraging on python2.7 and depends on Functest, it
inherits from opnfv/functest-core:hunter (latest py2 release).
pika seems missing in requirements.
INSTALLER_IP should be protected (it raises exceptions to the upper
program).
Change-Id: I50d1cf0a7a1233447245d852f79775c8e9fa8c99
Signed-off-by: Cédric Ollivier <cedric.ollivier@orange.com>
|
|
SNMP config has been causing the one click install to fail.
Disabling it temperorily so that it doesn't block Iruya release
JIRA: BAROMETER-134
Change-Id: Ia7944eb6461497c0cd17a2f7ba5b2c2018ffd2a2
Signed-off-by: Emma Foley <efoley@redhat.com>
|
|
|
|
Signed-off-by: jabirkkclt <jabir.kanhira.kadavathu@intel.com>
Change-Id: Id29ba83455d2cea21ec634d3e0eee38d84de465b
|
|
on CentOS 7. Mount the /sys/fs/resctrl from host to container to avoid issues with intel_rdt plugin."
|
|
container. Update one-click install of experimental container to add capabilities plugin."
|
|
CentOS 7. Mount the /sys/fs/resctrl from host to container to avoid
issues with intel_rdt plugin.
Change-Id: I04d22fccb950aabbde774c1850a5b44ee57908f5
Signed-off-by: jabirkkclt <jabir.kanhira.kadavathu@intel.com>
Signed-off-by: Kamil Wiatrowski <kamilx.wiatrowski@intel.com>
|
|
container. Update one-click install of experimental container to add capabilities plugin.
Signed-off-by: jabirkkclt <jabir.kanhira.kadavathu@intel.com>
Change-Id: I1753c4d8af279ad9c7de29e7a198dbd8161c516d
|
|
Signed-off-by: jabirkkclt <jabir.kanhira.kadavathu@intel.com>
Change-Id: I2e4934518f46afb060748ab08fd2ea9bfc0271d6
|
|
|
|
Signed-off-by: jabirkkclt <jabir.kanhira.kadavathu@intel.com>
Change-Id: I1579a2bd0ef2280134fc13183ebafeb444b752f9
|
|
Signed-off-by: jabirkkclt <jabir.kanhira.kadavathu@intel.com>
Change-Id: Ie368f2d32ce1291036f6fff95cd13b0dbdeced24
|
|
should be same host.
Signed-off-by: jabirkkclt <jabir.kanhira.kadavathu@intel.com>
Change-Id: I82732de3eacd7abda5d63059d988452e8141201b
|
|
Signed-off-by: Emma Foley <efoley@redhat.com>
Change-Id: Ia62933a4b824f1280a693fc8004d29c67dc5e2ac
|
|
Add a variable to playbooks to select type of collectd container.
Container flavor can be stable, master or experimental.
Change-Id: Ie591ca3b7816210c377c5bffbb54117f8ee764aa
Signed-off-by: Kamil Wiatrowski <kamilx.wiatrowski@intel.com>
|
|
|
|
Updated intel_pmu.conf for ansible installation to monitor
cores 0-4 by default.
Add sample conf files for plugins supported by master container:
pcie_errors and write_prometheus.
Change-Id: Ifedff2f3b099f5d7b5a88d14909c3879d38e4e64
Signed-off-by: Kamil Wiatrowski <kamilx.wiatrowski@intel.com>
|
|
Experimental container is based on master branch and includes
set of pull requests with experimental features.
By default one addtional PR is included - logparser.
Change-Id: I455122b23b6de382f6ef75a790f7b052784b59ac
Signed-off-by: Radoslaw Jablonski <radoslawx.jablonski@intel.com>
Signed-off-by: Kamil Wiatrowski <kamilx.wiatrowski@intel.com>
|
|
Change-Id: I54a2d46b8acecea6f6278f130d7567acc813c324
Signed-off-by: Kamil Wiatrowski <kamilx.wiatrowski@intel.com>
|
|
Previously 'zookeeper' docker container should be installed
manually by user - now it is a part of ansible one click install
for VES.
Change-Id: If3873cffa51ddf04d2c3769705259aaf5928cc45
Signed-off-by: Radoslaw Jablonski <radoslawx.jablonski@intel.com>
Signed-off-by: Kamil Wiatrowski <kamilx.wiatrowski@intel.com>
|
|
Previously 'zookeeper_host_ip' was set to 'localhost' instead
of IP address - this caused error during starting kafka container.
Change-Id: I55cf6e3375e3d08df2baf69f678aaab74f4aa408
Signed-off-by: Radoslaw Jablonski <radoslawx.jablonski@intel.com>
|
|
On Ubuntu 18 mcelog is no longer available through apt from
standard repositories. It can be downloaded directly, and make
on target machine. Mcelog version 161 is used.
Change-Id: Ie24990869190f86efa26994081d4f887f879c867
Signed-off-by: Kamil Wiatrowski <kamilx.wiatrowski@intel.com>
|
|
Add dpdk plugins to ansible installation with default
configuration.
Change-Id: I1f7949962aff5119141b5712bb6b32db8f45b287
Signed-off-by: Kamil Wiatrowski <kamilx.wiatrowski@intel.com>
|
|
|
|
Build dpdk plugins in collectd container.
Change-Id: Ifdca827a419d6d3d4d9c6070d3a581372df8a2f9
Signed-off-by: Kamil Wiatrowski <kamilx.wiatrowski@intel.com>
|
|
Added a note about risks with mixing IP addresses and hostnames
in inventory file.
If the same host will be called in inventory file using both its
hostname and IP address then problems may occur during step
where dependencies are installed (ansible thinks that there
are 2 independent hosts and will try install dependencies
for both of them in the same time - it may fail be because
of apt/yum synchronization locks).
Signed-off-by: Radoslaw Jablonski <radoslawx.jablonski@intel.com>
Change-Id: I2cd1ea293fadfb12f78c03028cafec993dadbe2d
|