** Description changed:

  [Impact]
  The support for Intel NVDIMM technology requires both kernel and userspace 
components. The kernel components have landed in 4.15 and are available in 
Bionic. In order for users to take advantage of NVDIMMs in their deployments, 
the following userspace packages are also needed in Bionic:
  
  ndctl -- Utility library for managing the libnvdimm (non-volatile memory 
device) sub-system in the Linux kernel.
  Upstream project: https://github.com/pmem/ndctl
  
  PMDK -- persistent memory toolkit
  Upstream project: https://github.com/pmem/pmdk/
  
  This SRU is about ndctl; pmdk is being handled in bug #1781269
  
  [Test Case]
  * Install daxctl and ndctl
  sudo apt install daxctl ndctl
  
  * download the two dep8 test scripts:
  wget 
https://git.launchpad.net/~usd-import-team/ubuntu/+source/ndctl/plain/debian/tests/daxctl-commands
 
https://git.launchpad.net/~usd-import-team/ubuntu/+source/ndctl/plain/debian/tests/ndctl-commands
  
  * execute them and verify exit status, which should be zero:
  sh ./daxctl-commands; echo $?
  sh ./ndctl-commands; echo $?
  
  More detailed testing can be executed if the appropriate hardware is
  available.
  
  [Regression Potential]
+ These are NEW packages for bionic. They were accepted into the archive after 
a lengthy review. It is lintian clear, with appropriate overrides or an 
explanation in README.source.
  
-  * discussion of how regressions are most likely to manifest as a result
- of this change.
+ This package includes libraries, which could inadvertently become new
+ dependencies. If a package has support for ndctl, for example, but it
+ was always disabled because there was no ndctl in the archive, it could
+ now be inadvertently enabled merely by having the lib-dev package
+ installed.
  
-  * It is assumed that any SRU candidate patch is well-tested before
-    upload and has a low overall risk of regression, but it's important
-    to make the effort to think about what ''could'' happen in the
-    event of a regression.
- 
-  * This both shows the SRU team that the risks have been considered,
-    and provides guidance to testers in regression-testing the SRU.
+ Similarly, code paths in existing packages that would never have been
+ traversed because of the lack of ndctl support, now could be exposed.
  
  [Other Info]
- 
-  * Anything else you think is useful to include
-  * Anticipate questions from users, SRU, +1 maintenance, security teams and 
the Technical Board
-  * and address these questions in advance
+ The git master branch at 
https://git.launchpad.net/~ahasenack/ubuntu/+source/ndctl/log/?h=master shows 
all the work that was put into this package to make it suitable for Ubuntu 
inclusion.
+ The bug at https://bugs.launchpad.net/bugs/1752378 also contains a lengthy 
history and discussions about some of the finer details, and is linked to from 
files in debian/* where further explanation was necessary about some behavior.

** Description changed:

  [Impact]
  The support for Intel NVDIMM technology requires both kernel and userspace 
components. The kernel components have landed in 4.15 and are available in 
Bionic. In order for users to take advantage of NVDIMMs in their deployments, 
the following userspace packages are also needed in Bionic:
  
  ndctl -- Utility library for managing the libnvdimm (non-volatile memory 
device) sub-system in the Linux kernel.
  Upstream project: https://github.com/pmem/ndctl
  
  PMDK -- persistent memory toolkit
  Upstream project: https://github.com/pmem/pmdk/
  
  This SRU is about ndctl; pmdk is being handled in bug #1781269
  
  [Test Case]
  * Install daxctl and ndctl
  sudo apt install daxctl ndctl
  
  * download the two dep8 test scripts:
  wget 
https://git.launchpad.net/~usd-import-team/ubuntu/+source/ndctl/plain/debian/tests/daxctl-commands
 
https://git.launchpad.net/~usd-import-team/ubuntu/+source/ndctl/plain/debian/tests/ndctl-commands
  
  * execute them and verify exit status, which should be zero:
  sh ./daxctl-commands; echo $?
  sh ./ndctl-commands; echo $?
  
  More detailed testing can be executed if the appropriate hardware is
  available.
  
  [Regression Potential]
  These are NEW packages for bionic. They were accepted into the archive after 
a lengthy review. It is lintian clear, with appropriate overrides or an 
explanation in README.source.
  
  This package includes libraries, which could inadvertently become new
  dependencies. If a package has support for ndctl, for example, but it
  was always disabled because there was no ndctl in the archive, it could
- now be inadvertently enabled merely by having the lib-dev package
+ now be inadvertently enabled merely by having the lib*-dev package(s)
  installed.
  
  Similarly, code paths in existing packages that would never have been
  traversed because of the lack of ndctl support, now could be exposed.
  
  [Other Info]
  The git master branch at 
https://git.launchpad.net/~ahasenack/ubuntu/+source/ndctl/log/?h=master shows 
all the work that was put into this package to make it suitable for Ubuntu 
inclusion.
  The bug at https://bugs.launchpad.net/bugs/1752378 also contains a lengthy 
history and discussions about some of the finer details, and is linked to from 
files in debian/* where further explanation was necessary about some behavior.

** Changed in: ndctl (Ubuntu)
    Milestone: ubuntu-18.04.1 => bionic-updates

** Changed in: ndctl (Ubuntu)
       Status: Confirmed => In Progress

-- 
You received this bug notification because you are a member of Ubuntu
Bugs, which is subscribed to Ubuntu.
https://bugs.launchpad.net/bugs/1781268

Title:
  SRU ndctl into Bionic

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

-- 
ubuntu-bugs mailing list
ubuntu-bugs@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs

Reply via email to