Hi Donald, On Thu, Oct 26, 2017 at 3:02 PM, Donald Hunter (donaldh) <[email protected]> wrote:
> Hi Michael, > > > > Please don’t just remove unimgr. That’s not helpful to unimgr. The first I > heard there was a problem relating to unimgr was yesterday when I read your > email after you forwarded it to the unimgr-dev list. > > > > What’s the problem that you think is caused by unimgr and I’ll see if we > can resolve it. > I've no idea actually - but only based on https://jira.opendaylight.org/browse/NETCONF-479, my understanding is that Tomas Cere and Vratko Polak understand this better. My main point really is that, whatever needs to be done in unimgr and/or netconf (which personally I unforatuntely don't have the cycles to help out more with), is not an excuse to keep failing distribution build jobs for many, many other projects... I'd there again like to propose to TEMPORARILY (!) remove unimgr from distribution, until this is solved - what's the harm? > Cheers, > > Donald. > > > > *From: *<[email protected]> on behalf of Michael > Vorburger <[email protected]> > *Date: *Thursday, 26 October 2017 at 12:44 > *To: *"Vratko Polak -X (vrpolak - PANTHEON TECHNOLOGIES at Cisco)" < > [email protected]>, "Tomas Cere -X (tcere - PANTHEON TECHNOLOGIES at > Cisco)" <[email protected]> > *Cc: *controller-dev <[email protected]>, " > [email protected]" <[email protected]>, > Tom Pantelis <[email protected]> > *Subject: *Re: [unimgr-dev] [controller-dev] SingleFeatureTest (SFT) > failure on odl-integration-compatible-with-all due to > ConflictingModificationAppliedException: Node was created by other > transaction. > > > > On Tue, Oct 24, 2017 at 11:44 PM, Michael Vorburger <[email protected]> > wrote: > > +unimgr-dev: > > > > On Mon, Oct 23, 2017 at 6:02 PM, Vratko Polak -X (vrpolak - PANTHEON > TECHNOLOGIES at Cisco) <[email protected]> wrote: > > Previous story: [2]. > > > > It's not that rare - just hit me again (on https://git.opendaylight. > org/gerrit/#/c/64674/), had to override once more - and find this > annoying.. > > > > > who would have to do what > > > > Ideally, Netconf developers would unify their features, > > which does not seem to get done anytime soon [3]. > > > If I understand [3] correctly, Tomas Cere doesn't even consider this a > netconf issue, but asks for "unimgr should move towards > odl-netconf-topology" (instead of odl-netconf-connector-ssh, because "There > is no reason to pull in odl-netconf-connector-ssh unless you are using > config subsystem still"). Is this something the unimgr project would be > willing to do? > > > > If not, or if unimgr-dev, assuming I understand things correctly, why > don't we just kick unimgr project out of distribution?! I'll raise a patch > proposing this when it next hits me, if it's not resolved by then. > > > > this just happened AGAIN on https://jenkins.opendaylight.org/releng/job/ > genius-distribution-check-oxygen/481/console for https://git.opendaylight. > org/gerrit/#/c/60303/ ... it's a real PITA IMHO! > > > > I'd therefore like to suggest https://git.opendaylight.org/ > gerrit/#/c/64761/ - objections, anyone? > > > > There is a workaround in Int/Dist [4] prepared, > > but it keeps SFT unstable, this time due to > > (lack of) Karaf 4 memory efficiency [5]. > > > > Current road to stability seem to be > > fixing various ODL project features (like [6]) > > to be less taxing on Karaf 4 bundle resolver, > > and then merging [4]. > > > > > It would be nice if the exception included some context like the path. > > > > I have rebased my old [7]. > > In this case, it is multiple netconf features trying to update > > operational topology status for "controller-config" device. > > > > > How does SFT even pick this up to fail the test? > > > > In general, a "caused by" thrown by featuresService.installFeature. > > In this case there was an ISE (visible in surefire report [8] > > at 12:27:25,742) coming from here [9] (at least in Nitrogen), > > not sure which component gets to throw it. > > > > Vratko. > > > > [2] https://lists.opendaylight.org/pipermail/release/2017- > October/012728.html > > [3] https://jira.opendaylight.org/browse/NETCONF-479 > > [4] https://git.opendaylight.org/gerrit/64410 > > [5] https://jira.opendaylight.org/browse/ODLPARENT-125 > > [6] https://jira.opendaylight.org/browse/INTDIST-92 > > [7] https://git.opendaylight.org/gerrit/48118 > > [8] https://logs.opendaylight.org/releng/jenkins092/infrautils- > distribution-check-oxygen/144/distribution/features/singles/ > odl-integration-compatible-with-all/target/surefire- > reports/org.opendaylight.odlparent.featuretest. > SingleFeatureTest-output.txt.gz > > [9] https://github.com/opendaylight/netconf/blob/ > release/nitrogen/netconf/sal-netconf-connector/src/main/ > java/org/opendaylight/netconf/sal/connect/netconf/sal/ > NetconfDeviceTopologyAdapter.java#L243 > > > > *From:* [email protected] [mailto: > [email protected]] *On Behalf Of *Tom Pantelis > *Sent:* 23 October, 2017 15:24 > *To:* Michael Vorburger <[email protected]> > *Cc:* controller-dev <[email protected]> > *Subject:* Re: [controller-dev] SingleFeatureTest (SFT) failure on > odl-integration-compatible-with-all due to > ConflictingModificationAppliedException: > Node was created by other transaction. > > > > > > > > On Mon, Oct 23, 2017 at 8:41 AM, Michael Vorburger <[email protected]> > wrote: > > Hello, > > > > Any idea who would have to do what to precent SFT from (only > occassionally?!) failing on odl-integration-compatible-with-all due to > ConflictingModificationAppliedException: Node was created by other > transaction, as seen on https://logs.opendaylight.org/releng/jenkins092/ > infrautils-distribution-check-oxygen/144/console.log.gz for > https://git.opendaylight.org/gerrit/#/c/63466/ ? > > > > It would be nice if the exception included some context like the path. > Does this test install all netconf features? I've seen such sporadic issues > with the callhome feature when it's installed with all the other netconf > features. > > > > How does SFT even pick this up to fail the test? Log scraping? Or was it > a "caused by" thrown on blueprint startup? > > > > > Tx, > > M. > > -- > > Michael Vorburger, Red Hat > [email protected] | IRC: vorburger @freenode | ~ = http://vorburger.ch > > > _______________________________________________ > controller-dev mailing list > [email protected] > https://lists.opendaylight.org/mailman/listinfo/controller-dev > > > > > > >
_______________________________________________ controller-dev mailing list [email protected] https://lists.opendaylight.org/mailman/listinfo/controller-dev
