← Back to team overview

group.of.nepali.translators team mailing list archive

[Bug 1883320] Re: [kvm] change check_enabled to /dev/kvm

 

This bug was fixed in the package sosreport - 3.9.1-1ubuntu0.20.04.1

---------------
sosreport (3.9.1-1ubuntu0.20.04.1) focal; urgency=medium

  * New 3.9.1 upstream release. (LP: #1884293) (LP: #1883320)
    This maintenance release includes:
    - New plugins: sos_extras, ovirt_engine_backup, console,
      validation_framework.
    - lxd plugin collections have been overhauled.
    - Fixed handling of the namespace pattern for the networking
      plugin.
    - A basic path is now defined in Policy for all subclasses.

    Plugin API Enhancements:
    - Enablement checks have been extended to include architecture
      constraints.
    - SoSPredicate has been extended to include architecture constraints,
      as well as negative constraints for all elements.
    - Plugins will now capture service status information for all services
      defined in the services class attr.

    Further release information and tarballs are available at:
    - https://github.com/sosreport/sos/releases/tag/3.9.1

  * Former patches now fixed upstream:
    - d/p/0001-unittest-py3-fix.patch
    - d/p/0002-lxd-drop-db-collection-and-introduce-lxd-buginfo.patch

  * Other specific modifications:
    - d/p/0001-lshw-command.patch
    - d/p/0002-lds-substitute-oidc-conf.patch
    - d/p/0003-kvm-change-trigger-to-dev-kvm.patch

 -- Eric Desrochers <slashd@xxxxxxxxxx>  Tue, 16 Jun 2020 12:18:58 +0000

** Changed in: sosreport (Ubuntu Focal)
       Status: Fix Committed => Fix Released

-- 
You received this bug notification because you are a member of नेपाली
भाषा समायोजकहरुको समूह, which is subscribed to Xenial.
Matching subscriptions: Ubuntu 16.04 Bugs
https://bugs.launchpad.net/bugs/1883320

Title:
  [kvm] change check_enabled to /dev/kvm

Status in sosreport package in Ubuntu:
  Fix Released
Status in sosreport source package in Xenial:
  New
Status in sosreport source package in Bionic:
  Fix Released
Status in sosreport source package in Eoan:
  Fix Released
Status in sosreport source package in Focal:
  Fix Released
Status in sosreport source package in Groovy:
  Fix Released

Bug description:
  [Impact]

  The KVM plugin get triggered in a container
  (e.g lxd) because of "/sys/module/kvm"
  inheritance from the kernel host.

  Not only it's a waste of sosreport time,
  but running it inside a container may
  unintentionally reveal details from its
  host. Which is a undesired behaviour.

  Switching to /dev/kvm, is more appropriate
  and follow current standard as used by tool
  such as cpu-checker (kvm-ok) for instance.

  And taking benefit of this change to get rid
  of the check_enabled() overwrite in favour of
  using "files=" trigger.

  [Test Case]

  * Run sosreport inside a LXD container
  * The container will contain KVM information from its host (Iff its host as KVM installed)

  [Regression Potential]

  We only change the way the plugin gets triggered to use a most
  conventional way.

  If /dev/kvm exist, then exercise the kvm plugin.

  No regression expected, but worse case scenario, it will only be
  isolated and impact the KVM plugin to collect data properly as
  expected.

  It won't cause any harm to others plugins nor sosreport core
  functionalities itself.

  The commit went through some upstream Travis testing which used a
  shell script called "simple.sh" to verify sosreport collection.

  BTW, this same exact script is part of the sosreport autopkgtest, and
  will be run as part of the SRU to push this change.

  [Other Information]

  * Upstream bug:
  https://github.com/sosreport/sos/issues/2062

  * Upstream commit:
  https://github.com/sosreport/sos/pull/2063/commits/dc58a038c18ff3e838d883d4b31aad5dac7ea9e1

To manage notifications about this bug go to:
https://bugs.launchpad.net/ubuntu/+source/sosreport/+bug/1883320/+subscriptions