This is an automated email from the ASF dual-hosted git repository. astefanutti pushed a commit to branch release-1.4.x in repository https://gitbox.apache.org/repos/asf/camel-k.git
commit faf7b3b3f6345fdbf40981727fd3bebfbe04f6dc Author: Pasquale Congiusti <pasquale.congiu...@gmail.com> AuthorDate: Tue Apr 27 17:19:00 2021 +0200 chore(trait): polishing PR --- .../camel-k-dev/1.4.0/camel.apache.org_integrations.yaml | 15 --------------- deploy/traits.yaml | 2 +- docs/modules/traits/pages/error-handler.adoc | 2 +- .../kamelets/error-handler/error-handler.kamelet.yaml | 3 +-- .../error-handler/incremental-id-source.kamelet.yaml | 2 +- pkg/apis/camel/v1/integration_types_support.go | 2 +- pkg/trait/error_handler.go | 2 +- 7 files changed, 6 insertions(+), 22 deletions(-) diff --git a/deploy/olm-catalog/camel-k-dev/1.4.0/camel.apache.org_integrations.yaml b/deploy/olm-catalog/camel-k-dev/1.4.0/camel.apache.org_integrations.yaml index 0b1b88a..96e26b9 100644 --- a/deploy/olm-catalog/camel-k-dev/1.4.0/camel.apache.org_integrations.yaml +++ b/deploy/olm-catalog/camel-k-dev/1.4.0/camel.apache.org_integrations.yaml @@ -88,21 +88,6 @@ spec: items: type: string type: array - errorHandler: - description: ErrorHandlerSpec respresents an integration error handler - to be used as default at runtime - properties: - parameters: - description: ErrorHandlerParameters is an unstructured object - representing the configuration parameters available for an error - handler - type: object - x-kubernetes-preserve-unknown-fields: true - type: - type: string - uri: - type: string - type: object flows: items: description: Flow is an unstructured object representing a Camel diff --git a/deploy/traits.yaml b/deploy/traits.yaml index e09c4a5..a252fa8 100755 --- a/deploy/traits.yaml +++ b/deploy/traits.yaml @@ -284,7 +284,7 @@ traits: type: bool description: Can be used to enable or disable a trait. All traits share this common property. - - name: ',omitempty' + - name: error-handler-ref type: string description: The error handler ref name found in application properties - name: gc diff --git a/docs/modules/traits/pages/error-handler.adoc b/docs/modules/traits/pages/error-handler.adoc index 149eaee..978e76f 100755 --- a/docs/modules/traits/pages/error-handler.adoc +++ b/docs/modules/traits/pages/error-handler.adoc @@ -27,7 +27,7 @@ The following configuration options are available: | bool | Can be used to enable or disable a trait. All traits share this common property. -| error-handler.,omitempty +| error-handler.error-handler-ref | string | The error handler ref name found in application properties diff --git a/examples/kamelets/error-handler/error-handler.kamelet.yaml b/examples/kamelets/error-handler/error-handler.kamelet.yaml index 9ace959..a5aa060 100644 --- a/examples/kamelets/error-handler/error-handler.kamelet.yaml +++ b/examples/kamelets/error-handler/error-handler.kamelet.yaml @@ -22,7 +22,7 @@ metadata: spec: definition: title: "Error Log Sink" - description: "Consume events from a channel" + description: "Error logger channel" required: - message properties: @@ -35,7 +35,6 @@ spec: from: uri: kamelet:source steps: -# - to: my-dlc - set-body: constant: "{{message}}" - to: "log:error-sink" diff --git a/examples/kamelets/error-handler/incremental-id-source.kamelet.yaml b/examples/kamelets/error-handler/incremental-id-source.kamelet.yaml index 5e3181b..ee475da 100644 --- a/examples/kamelets/error-handler/incremental-id-source.kamelet.yaml +++ b/examples/kamelets/error-handler/incremental-id-source.kamelet.yaml @@ -24,7 +24,7 @@ metadata: spec: definition: title: "Incremental ID Source" - description: "Produces periodic events with an incremental ID" + description: "Produces periodic events with an incremental ID and fails when the ID contains any 0 character" properties: period: title: Period diff --git a/pkg/apis/camel/v1/integration_types_support.go b/pkg/apis/camel/v1/integration_types_support.go index 6efdc24..f7e505e 100644 --- a/pkg/apis/camel/v1/integration_types_support.go +++ b/pkg/apis/camel/v1/integration_types_support.go @@ -136,7 +136,7 @@ func (in *IntegrationSpec) GetConfigurationProperty(property string) string { for _, confSpec := range in.Configuration { if confSpec.Type == "property" && strings.HasPrefix(confSpec.Value, property) { splitConf := strings.Split(confSpec.Value, "=") - if len(splitConf) > 0 { + if len(splitConf) > 1 { return splitConf[1] } } diff --git a/pkg/trait/error_handler.go b/pkg/trait/error_handler.go index dd6e218..a72a94a 100644 --- a/pkg/trait/error_handler.go +++ b/pkg/trait/error_handler.go @@ -29,7 +29,7 @@ import ( type errorHandlerTrait struct { BaseTrait `property:",squash"` // The error handler ref name found in application properties - ErrorHandlerRef string `property:",omitempty"` + ErrorHandlerRef string `property:"error-handler-ref" json:"error-handler-ref,omitempty"` } func newErrorHandlerTrait() Trait {