This group is locked. No changes can be made to the group while it is locked.
Date
1 - 5 of 5
No port stats today
|
Luis Gomez <ecelgp@...>
Hi all,
Just let you know port stats does not work today: https://jenkins.opendaylight.org/integration/job/integration-master-csit-base-of13/1693/robot/report/log.html BR/Luis
|
|
|
|
Ed Warnicke (eaw) <eaw@...>
Weird… I just built openflowplugin and integration locally from master:
toggle quoted messageShow quoted text
oberon:openflowplugin hagbard$ git log -1 --oneline
aaf29e2 Added a missing nullcheck in stats converter
oberon:integration hagbard$ git log -1 --oneline
76da7a2 BUG-1277 Reflection of OFJava's ByteBufUtils move on OFPlugin
and I see port stats (and flows stats, and table stats etc).
There was a small bug with a lack of a null check that was fixed here:
Is there a way to drill more deeply into whats happening in the integration job?
Ed
On Jul 3, 2014, at 3:38 PM, Luis Gomez <ecelgp@...> wrote:
Hi all,
|
|
|
|
Luis Gomez <ecelgp@...>
It is fixed now and by looking the time I would say this patch did the work: BR/Luis
On Jul 3, 2014, at 4:30 PM, Ed Warnicke (eaw) <eaw@...> wrote:
|
|
|
|
Michal Rehak -X (mirehak - Pantheon Technologies SRO@Cisco) <mirehak@...>
Hi Luis, actually you might be right. I don't think that content of 8666 did the job as it exposes internal OFPlugin statistics (message counts) through jconsole (JMX). Although I can not exclude this for 100%. But recently I found out that the easiest and tricky way how to reintroduce a bug is to merge multiple independent changes and let gerrit merge them to master. Here jenkins merge job will always publish artifacts of code before merge. So in git there is everything ok but artifacts in nexus could be broken. And some irrelevant later change (based on fixed master state) would in fact fix stuff in nexus. Till now I thought that rebasing before merge is waste of time. Regards, Michal From: openflowjava-dev-bounces@... [openflowjava-dev-bounces@...] on behalf of Luis Gomez [ecelgp@...]
Sent: Friday, July 04, 2014 20:05 To: Ed Warnicke (eaw) Cc: openflowjava-dev; openflowplugin-dev Subject: Re: [openflowjava-dev] No port stats today It is fixed now and by looking the time I would say this patch did the work:
BR/Luis
On Jul 3, 2014, at 4:30 PM, Ed Warnicke (eaw) <eaw@...> wrote:
|
|
|
|
Luis Gomez <ecelgp@...>
Agree, I think the existing project merge job does not use the latest code available in GIT but the base code was used when patch was pushed. This leads to weird situations where as you say code is OK but artifacts are not. Probably the only way to fix this is rebasing changes just before merging them as you also comment. BR/Luis On Jul 6, 2014, at 11:52 PM, Michal Rehak -X (mirehak - Pantheon Technologies SRO at Cisco) <mirehak@...> wrote:
|
|
|