[dpdk-ci] [PATCH v4 7/7] tests: add checkpatch

[Wei, FangfangX]

Hi Thomas,



My internet connection is working fine, and also configure http_proxy and https_proxy, but the connection is still timeout.

I think that's because the rpc client can't connect to your rpc server. Is there any other configuration should be configured?



According to my understanding, the "pwid=$($pwclient list -f '%{id}' -m "$msgid " in the script "send-patch-report.sh" is just trying to get the patchwork id, is that right?

Then I just define pwid="17772" in "send-patch-report.sh" directly, run the script again, nothing happened. The email hasn't been sent to test-report at dpdk.org.



I think the function about send-patch-report.sh is just to send patch report to test-report at dpdk.org<mailto:test-report at dpdk.org>. It can't generate the following content in patchwork as checkpatch. Am I right?

[cid:image001.png at 01D25227.03650540]





Best Regards

Fangfang Wei



-----Original Message-----
From: Thomas Monjalon [mailto:thomas.monjalon at 6wind.com]
Sent: Thursday, December 8, 2016 9:11 PM
To: Wei, FangfangX <fangfangx.wei at intel.com>
Cc: ci at dpdk.org; Xu, Qian Q <qian.q.xu at intel.com>; Liu, Yong <yong.liu at intel.com>; Chen, WeichunX <weichunx.chen at intel.com>
Subject: Re: [dpdk-ci] [PATCH v4 7/7] tests: add checkpatch



2016-12-08 09:02, Wei, FangfangX:

> Hi Thomas,

> Following is the debug information today, please help to review and give some suggestion.

>

> 1. replace "$sendmail -t" by "cat":

> >echo $report | tools/send-patch-report.sh -t "pw17762  [PATCH, 1/2] eth: get rid of goto's in rte_eth_dev_detach" -f "fangfangx.wei at intel.com<mailto:fangfangx.wei at intel.com>" -m "20161208014751.24285-2-stephen at networkplumber.org<mailto:20161208014751.24285-2-stephen at networkplumber.org>" -p "dev.dpdk.org" -l "Intel Per-patch compilation check" -s "SUCCESS" -d "Compilation OK"

>

> usage: pwclient [-h]

>                 {apply,git-am,get,info,projects,check-list,check-info,check-create,states,view,update,list,search}

>                 ...

> pwclient: error: No default project configured in ~/.pwclientrc

>

>

> There is no ~/.pwclientrc, should I copy config/pwclientrc to ~/.pwclientrc?



Yes

We could add a section about configuration in the README.



> 2. replace "$sendmail -t" by "cat" then copy config/pwclientrc to ~/.pwclientrc:

> >echo $report | tools/send-patch-report.sh -t "pw17762  [PATCH, 1/2] eth: get rid of goto's in rte_eth_dev_detach" -f "fangfangx.wei at intel.com<mailto:fangfangx.wei at intel.com>" -m "20161208014751.24285-2-stephen at networkplumber.org<mailto:20161208014751.24285-2-stephen at networkplumber.org>" -p "dev.dpdk.org" -l "Intel Per-patch compilation check" -s "SUCCESS" -d "Compilation OK"

>

> Traceback (most recent call last):

[...]

> socket.error: [Errno 110] Connection timed out



Is your Internet connection working fine? Do you need some proxy configuration?



> 3. The ci.config hasn't been loaded, because the environment $DPDK_CI_PWCLIENT and $DPDK_CI_MAILER are None.



They are not needed.

Some default values are used in send-patch-report.sh:

sendmail=${DPDK_CI_MAILER:-/usr/sbin/sendmail}

pwclient=${DPDK_CI_PWCLIENT:-$(dirname $(readlink -m $0))/pwclient}



> 4. replace "$sendmail -t" by "cat" then replace " pwid=$($pwclient list -f '%{id}' -m "$msgid " by "#pwid=$($pwclient list -f '%{id}' -m "$msgid")":

> >echo $report | tools/send-patch-report.sh -t "pw17762  [PATCH, 1/2] eth: get rid of goto's in rte_eth_dev_detach" -f "fangfangx.wei at intel.com<mailto:fangfangx.wei at intel.com>" -m "20161208014751.24285-2-stephen at networkplumber.org<mailto:20161208014751.24285-2-stephen at networkplumber.org>" -p "dev.dpdk.org" -l "Intel Per-patch compilation check" -s "SUCCESS" -d "Compilation OK"

>

> Subject: |SUCCESS| pw17762 [PATCH, 1/2] eth: get rid of goto's in

> rte_eth_dev_detach

> In-Reply-To: 20161208014751.24285-2-stephen at networkplumber.org<mailto:20161208014751.24285-2-stephen at networkplumber.org>

> References: 20161208014751.24285-2-stephen at networkplumber.org<mailto:20161208014751.24285-2-stephen at networkplumber.org>

> To: test-report at dpdk.org<mailto:test-report at dpdk.org>

> Cc: fangfangx.wei at intel.com<mailto:fangfangx.wei at intel.com>

>

> Test-Label: Intel Per-patch compilation check

> Test-Status: SUCCESS

> http://dpdk.org/patch/?

>

> _Compilation OK_

>

> Test-Label: Intel Per-patch compilation check Test-Status: SUCCESS

> http://www.dpdk.org/dev/patchwork/patch/17762 Submitter: Stephen

> Hemminger <stephen at networkplumber.org<mailto:stephen at networkplumber.org>> Date: Wed, 7 Dec 2016 17:47:50

> -0800 DPDK git baseline: Repo:dpdk, Branch:master,

> CommitID:c431384c8fbf8503693bcae1bdcd58d6fa459b8a Patch17762-17762 -->

> compile pass Build Summary: 18 Builds Done, 18 Successful, 0 Failures

> Test environment and configuration as below: OS: FreeBSD10.3_64 Kernel

> Version:10.3-RELEASE CPU info:CPU: Intel(R) Xeon(R) CPU E5-2699 v4 @

> 2.20GHz (2194.97-MHz K8-class CPU) GCC Version:gcc (FreeBSD Ports

> Collection) 4.8.5 Clang Version:3.4.1 x86_64-native-bsdapp-clang

> x86_64-native-bsdapp-gcc OS: RHEL7.2_64 Kernel

> Version:3.10.0-327.el7.x86_64 CPU info:Intel(R) Xeon(R) CPU E5-2699 v4

> @ 2.20GHz GCC Version:gcc (GCC) 4.8.5 20150623 (Red Hat 4.8.5-4) Clang

> Version:3.4.2 i686-native-linuxapp-gcc x86_64-native-linuxapp-gcc

> x86_64-native-linuxapp-gcc-shared OS: UB1604_64 Kernel

> Version:4.4.0-47-generic CPU info:Intel(R) Xeon(R) CPU E5-2699 v3 @

> 2.30GHz GCC Version:gcc (Ubuntu 5.4.0-6ubuntu1~16.04.4) 5.4.0 20160609

> Clang Version:3.8.0 i686-native-linuxapp-gcc

> x86_64-native-linuxapp-clang x86_64-native-linuxapp-gcc-shared

> x86_64-native-linuxapp-gcc OS: CentOS7_64 Kernel

> Version:3.10.0-327.el7.x86_64 CPU info:Intel(R) Xeon(R) CPU E5-2699 v3

> @ 2.30GHz GCC Version:gcc (GCC) 4.8.5 20150623 (Red Hat 4.8.5-4) Clang

> Version:3.4.2 i686-native-linuxapp-gcc x86_64-native-linuxapp-clang

> x86_64-native-linuxapp-gcc-shared x86_64-native-linuxapp-gcc OS:

> FC24_64 Kernel Version:4.8.6-201.fc24.x86_64 CPU info:Intel(R) Xeon(R)

> CPU E5-2699 v3 @ 2.30GHz GCC Version:gcc (GCC) 6.2.1 20160916 (Red Hat

> 6.2.1-2) Clang Version:3.8.0 x86_64-native-linuxapp-gcc

> i686-native-linuxapp-gcc x86_64-native-linuxapp-clang

> x86_64-native-linuxapp-gcc-shared x86_64-native-linuxapp-gcc-debug



You have a wrapping issue ;)

You can remove Test-Label, Test-Status and other lines as they are already printed above.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://dpdk.org/ml/archives/ci/attachments/20161209/77f2c00f/attachment-0001.html>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: image001.png
Type: image/png
Size: 7624 bytes
Desc: image001.png
URL: <http://dpdk.org/ml/archives/ci/attachments/20161209/77f2c00f/attachment-0001.png>