[Feature] Integrate ndctl package into Ubuntu

[XiongZhang]

ndctl is Utility library for managing the libnvdimm (non-volatile memory device) sub-system in the Linux kernel.

In order to support Intel Apache Pass, ndctl is essential and is the base of APP which use Apache Pass.

Source Code: https://github.com/pmem/ndctl

Target Release: 18.10

[quanxian (quanxian-wang)]

any update?

[quanxian (quanxian-wang)]

move it into 18.04

[Narinder Gupta (narindergupta)]

I have created snap for 16.04 LTS and it does gets deployed successfully for me.
ubuntu@etsi-ubuntu-jh:~$ sudo snap install ndctl --devmode --edge
ndctl (edge) 57.1 from 'tppdev' installed
ubuntu@etsi-ubuntu-jh:~$
ubuntu@etsi-ubuntu-jh:~$
ubuntu@etsi-ubuntu-jh:~$ ndctl --help

 usage: ndctl [--version] [--help] COMMAND [ARGS]

 See 'ndctl help COMMAND' for more information on a specific command.
 ndctl --list-cmds to see all available commands

ubuntu@etsi-ubuntu-jh:~$

[quanxian (quanxian-wang)]

Currently ndctl version is v58.4. v59 will be available soon.

[Jeff Lane  (bladernr)]

Nish, any update to this?

[David Britton (dpb)]

I'm not aware of Nish working on this. I'll circle with him when he is back.

[Nish Aravamudan (nacc)]

On Feb 8, 2018 15:10, "David Britton" <email address hidden> wrote:

I'm not aware of Nish working on this. I'll circle with him when he is
back.

Aiui, Narinder had packaged it as a snap, which should be sufficient for
all supported Ubuntu.

-Nish

[David Britton (dpb)]

Will track any progress here: https://bugs.launchpad.net/ubuntu/+bug/1752378

[quanxian (quanxian-wang)]

AEP(Apache Pass) testing has been blocked by this package integration. Hopefully it could be integrated into 18.04.

Another user package tracking number is
LP#1694877 for ixpdimm

[quanxian (quanxian-wang)]

ndctl has been integrated 18.04, it should be not blocker now.

[David Britton (dpb)]

Hi Quanxian -- where has it been integrated? Andreas and my team are working on this, are you seeing something else?

[[e-mail] (pragyan)]

Hi David we looked at https://bugs.launchpad.net/ubuntu/+bug/1752378 and might have assumed mvml and ndctl is a done deal. If not please correct us.

[Joseph Salisbury (jsalisbury)]

Can we move this bug to the "Linux" package and make it public?

[Andreas Hasenack (ahasenack)]

Note that ndctl is in cosmic already, so it will be part of ubuntu 18.10:

https://launchpad.net/ubuntu/+source/ndctl

The SRU for 18.04 is being worked on in https://bugs.launchpad.net/bugs/1781268

[quanxian (quanxian-wang)]

yes can move it to be public.

[[e-mail] (pragyan)]

Hi Andreas
ndctl v62 is released. Can you confirm if you will package this as part of Ubuntu 18.10.
Thank you
Pragyan

[Andreas Hasenack (ahasenack)]

I can try, considering feature freeze is next week, but I have yet to get back a test report from you guys about the current package in cosmic (ndctl 61.2). Since I don't have the hardware, there isn't much I can do to test it, other than the automated tests I wrote in debian/tests (DEP8 tests). Could you please give me some feedback?

[Andreas Hasenack (ahasenack)]

I'm getting this build error during make check with version 62:
/bin/bash ../libtool --tag=CC --mode=link gcc -D DEF_CONF_FILE='"/etc/ndctl/monitor.conf"' -D DEF_TMPFS_DIR='"/run/ndctl"' -Wall -Wchar-subscripts -Wformat-security -Wmissing-declarations -Wmissing-prototypes -Wnested-externs -Wpointer-arith -Wshadow -Wsign-compare -Wstrict-prototypes -Wtype-limits -Wmaybe-uninitialized -Wdeclaration-after-statement -Wunused-result -D_FORTIFY_SOURCE=2 -O2 -fvisibility=hidden -ffunction-sections -fdata-sections -g -O2 -fdebug-prefix-map=/home/ubuntu/git/ndctl=. -fstack-protector-strong -Wformat -Werror=format-security -Wl,--gc-sections -Wl,--as-needed -Wl,-Bsymbolic-functions -Wl,-z,relro -Wl,-z,now -o list-smart-dimm list-smart-dimm.o ../util/json.o ../ndctl/lib/libndctl.la ../daxctl/lib/libdaxctl.la -ljson-c ../libutil.a
libtool: link: gcc -D DEF_CONF_FILE=\"/etc/ndctl/monitor.conf\" -D DEF_TMPFS_DIR=\"/run/ndctl\" -Wall -Wchar-subscripts -Wformat-security -Wmissing-declarations -Wmissing-prototypes -Wnested-externs -Wpointer-arith -Wshadow -Wsign-compare -Wstrict-prototypes -Wtype-limits -Wmaybe-uninitialized -Wdeclaration-after-statement -Wunused-result -D_FORTIFY_SOURCE=2 -O2 -fvisibility=hidden -ffunction-sections -fdata-sections -g -O2 -fdebug-prefix-map=/home/ubuntu/git/ndctl=. -fstack-protector-strong -Wformat -Werror=format-security -Wl,--gc-sections -Wl,--as-needed -Wl,-Bsymbolic-functions -Wl,-z -Wl,relro -Wl,-z -Wl,now -o .libs/list-smart-dimm list-smart-dimm.o ../util/json.o ../ndctl/lib/.libs/libndctl.so ../daxctl/lib/.libs/libdaxctl.so -ljson-c ../libutil.a
/usr/bin/ld: ../util/json.o: undefined reference to symbol 'uuid_is_null@@UUID_1.0'
/usr/bin/ld: //lib/x86_64-linux-gnu/libuuid.so.1: error adding symbols: DSO missing from command line
collect2: error: ld returned 1 exit status
make[3]: *** [Makefile:890: list-smart-dimm] Error 1
make[2]: *** [Makefile:1436: check-am] Error 2
make[1]: *** [Makefile:773: check-recursive] Error 1
make: *** [Makefile:1067: check] Error 2

We do have uuid-dev installed:
uuid-dev:amd64 2.32-0.1ubuntu1

[Andreas Hasenack (ahasenack)]

I'm going to close this bug because ndctl is in Cosmic. From now on bugs should be filed using https://bugs.launchpad.net/ubuntu/+source/ndctl/+filebug

P.S.: I filed an issue upstream about the build error from my previous comment, with a patch.