Petals SE Camel

Add ConsumesExt MONIT traces

Details

  • Type: Improvement Request Improvement Request
  • Status: Open Open
  • Priority: Major Major
  • Resolution: Unresolved
  • Affects Version/s: 0.5.0
  • Fix Version/s: 0.5.1, 1.5.0-BC
  • Component/s: None
  • Security Level: Public
  • Description:

    The idea is to have MONIT traces for flows starting from Camel (when it consumes something in the bus).

    Not sure this can be done easily because we have to plug into Camel's infrastructure...

  • Environment:
    -

Issue Links

Activity

Victor NOËL made changes - Tue, 22 Sep 2015 - 09:42:22 +0200
Field Original Value New Value
Link This issue depends on PETALSSECAMEL-2 [ PETALSSECAMEL-2 ]
Hide
Victor NOËL added a comment - Tue, 29 Sep 2015 - 17:07:58 +0200

We could at least initialize the a flow if there is no flow attributes presently available (most certainly because the exchange never entered in Camel from Petals) when the exchange is sent from Camel to Petals.

We should log a warning in that case to be sure we don't overlook another problem: this wouldn't solve this issue, but at least it would make it less problematic (see for example PETALSSECAMEL-10 for a case where it would be nice).

Show
Victor NOËL added a comment - Tue, 29 Sep 2015 - 17:07:58 +0200 We could at least initialize the a flow if there is no flow attributes presently available (most certainly because the exchange never entered in Camel from Petals) when the exchange is sent from Camel to Petals. We should log a warning in that case to be sure we don't overlook another problem: this wouldn't solve this issue, but at least it would make it less problematic (see for example PETALSSECAMEL-10 for a case where it would be nice).
Christophe DENEUX made changes - Tue, 29 Sep 2015 - 17:21:04 +0200
Link This issue blocks PETALSSECAMEL-10 [ PETALSSECAMEL-10 ]
Victor NOËL made changes - Wed, 30 Sep 2015 - 15:25:19 +0200
Status New [ 10000 ] Open [ 10002 ]
Priority Major [ 3 ]
Hide
Victor NOËL added a comment - Wed, 30 Sep 2015 - 15:27:31 +0200

A first shot has been committed in https://github.com/petalslink/petals-se-camel/commit/da06d89edcef18a20dbe9fba39bf355bd2793514 and https://github.com/petalslink/petals-se-camel/commit/134af0dbd1e4e51b22af3eb6fb5cde1c26d3f5a2 (for Petals 4.3.x).

For now it considers that if there is no flow attributes available in the execution context, it means that we are acting as a BC.
A warning is logged to keep some traces of that assumption and then ConsumesExtFlow... MONIT traces are logged (ConsumesFlow... in Petals 4.3.x) around the call to the Petals service.

Show
Victor NOËL added a comment - Wed, 30 Sep 2015 - 15:27:31 +0200 A first shot has been committed in https://github.com/petalslink/petals-se-camel/commit/da06d89edcef18a20dbe9fba39bf355bd2793514 and https://github.com/petalslink/petals-se-camel/commit/134af0dbd1e4e51b22af3eb6fb5cde1c26d3f5a2 (for Petals 4.3.x). For now it considers that if there is no flow attributes available in the execution context, it means that we are acting as a BC. A warning is logged to keep some traces of that assumption and then ConsumesExtFlow... MONIT traces are logged (ConsumesFlow... in Petals 4.3.x) around the call to the Petals service.
Victor NOËL made changes - Wed, 30 Sep 2015 - 16:11:28 +0200
Fix Version/s 0.5.1 [ 10559 ]
Fix Version/s 1.0.0 [ 10543 ]
Victor NOËL made changes - Thu, 1 Oct 2015 - 09:30:37 +0200
Status Open [ 10002 ] In Progress [ 10003 ]
Hide
Victor NOËL added a comment - Thu, 1 Oct 2015 - 09:31:27 +0200

A preliminary fix is in v0.5.1 (and thus will be in 1.0.0 at least).

Show
Victor NOËL added a comment - Thu, 1 Oct 2015 - 09:31:27 +0200 A preliminary fix is in v0.5.1 (and thus will be in 1.0.0 at least).
Victor NOËL made changes - Thu, 1 Oct 2015 - 09:31:27 +0200
Status In Progress [ 10003 ] Resolved [ 10004 ]
Fix Version/s 1.0.0 [ 10543 ]
Resolution Fixed [ 1 ]
Hide
Victor NOËL added a comment - Thu, 1 Oct 2015 - 09:31:37 +0200

Reopening because there is more work to do

Show
Victor NOËL added a comment - Thu, 1 Oct 2015 - 09:31:37 +0200 Reopening because there is more work to do
Victor NOËL made changes - Thu, 1 Oct 2015 - 09:31:37 +0200
Status Resolved [ 10004 ] Open [ 10002 ]
Resolution Fixed [ 1 ]
Christophe DENEUX made changes - Thu, 1 Oct 2015 - 11:12:11 +0200
Fix Version/s 1.0.1 [ 10577 ]
Hide
Victor NOËL added a comment - Tue, 13 Oct 2015 - 10:28:19 +0200

One of the most problematic things arrising with this issue is the following:

  • Currently, when using Camel as a BC for incoming external events (i.e., having a route with a from that is not from petals), the flow are initialised at the moment the Petals service is called by the Camel SE.
  • So whatever happens before in the Camel SE (such as processing, execution, logging, etc), is NOT associated by the Camel SE to the current flow.

The solution would thus be to:

  • Detect that a route is starting but not from Petals
  • To initialise the flow just before and log the ConsumesExt MONIT trace.

This is not easy, as we have to plug in Camel infrastructure.
This is maybe not desired as it goes a bit agains SOA principles by doing the bindings to external events inside Camel instead of isolating it in a dedicated service.
This is maybe better if we can do it anyway because Camel is really helpful in many tricky situation where we need to interact with systems other than those supported by the Petals BC.

Show
Victor NOËL added a comment - Tue, 13 Oct 2015 - 10:28:19 +0200 One of the most problematic things arrising with this issue is the following:
  • Currently, when using Camel as a BC for incoming external events (i.e., having a route with a from that is not from petals), the flow are initialised at the moment the Petals service is called by the Camel SE.
  • So whatever happens before in the Camel SE (such as processing, execution, logging, etc), is NOT associated by the Camel SE to the current flow.
The solution would thus be to:
  • Detect that a route is starting but not from Petals
  • To initialise the flow just before and log the ConsumesExt MONIT trace.
This is not easy, as we have to plug in Camel infrastructure. This is maybe not desired as it goes a bit agains SOA principles by doing the bindings to external events inside Camel instead of isolating it in a dedicated service. This is maybe better if we can do it anyway because Camel is really helpful in many tricky situation where we need to interact with systems other than those supported by the Petals BC.
Victor NOËL made changes - Tue, 13 Oct 2015 - 10:31:01 +0200
Link This issue depends on PETALSDISTRIB-119 [ PETALSDISTRIB-119 ]
Christophe DENEUX made changes - Mon, 23 May 2016 - 12:50:16 +0200
Fix Version/s 1.0.2 [ 10656 ]
Fix Version/s 1.0.1 [ 10577 ]
Christophe DENEUX made changes - Tue, 20 Sep 2016 - 17:12:05 +0200
Fix Version/s 1.0.2 [ 10656 ]
Fix Version/s 1.0.3 [ 10691 ]
Hide
Christophe DENEUX added a comment - Tue, 20 Sep 2016 - 17:12:28 +0200 - edited

Postponed to 1.5.0

Show
Christophe DENEUX added a comment - Tue, 20 Sep 2016 - 17:12:28 +0200 - edited Postponed to 1.5.0
Victor NOËL made changes - Mon, 18 Sep 2017 - 13:53:28 +0200
Fix Version/s 1.1.0 [ 10750 ]
Fix Version/s 1.0.3 [ 10691 ]
Christophe DENEUX made changes - Thu, 4 Jan 2018 - 12:08:54 +0100
Fix Version/s 1.1.1 [ 10771 ]
Fix Version/s 1.1.0 [ 10750 ]
Christophe DENEUX made changes - Thu, 8 Feb 2018 - 19:18:31 +0100
Fix Version/s 1.1.2 [ 10814 ]
Fix Version/s 1.1.1 [ 10771 ]
Christophe DENEUX made changes - Thu, 26 Jul 2018 - 17:13:08 +0200
Assignee Victor NOËL [ vnoel ]
Fix Version/s 1.2.1 [ 10874 ]
Fix Version/s 1.2.0 [ 10814 ]
Christophe DENEUX made changes - Wed, 12 Apr 2023 - 10:08:53 +0200
Fix Version/s 1.4.0-BC [ 11112 ]
Fix Version/s 1.3.0-BC [ 10874 ]
Christophe DENEUX made changes - Wed, 25 Feb 2026 - 12:32:14 +0100
Fix Version/s 1.5.0-BC [ 11251 ]
Fix Version/s 1.4.0-BC [ 11112 ]
Transition Status Change Time Execution Times Last Executer Last Execution Date
New New Open Open
8d 5h 43m
1
Victor NOËL
Wed, 30 Sep 2015 - 15:25:19 +0200
Open Open In Progress In Progress
18h 5m
1
Victor NOËL
Thu, 1 Oct 2015 - 09:30:37 +0200
In Progress In Progress Resolved Resolved
50s
1
Victor NOËL
Thu, 1 Oct 2015 - 09:31:27 +0200
Resolved Resolved Open Open
10s
1
Victor NOËL
Thu, 1 Oct 2015 - 09:31:37 +0200

People

Dates

  • Created:
    Tue, 22 Sep 2015 - 09:42:11 +0200
    Updated:
    Wed, 25 Feb 2026 - 12:32:24 +0100