Uploaded image for project: 'Camel'
  1. Camel
  2. CAMEL-9636

CamelBlueprintTestSupport - initialization error in case of empty cm:property-placeholder

        Details

        • Patch Info:
          Patch Available
        • Estimated Complexity:
          Unknown

          Description

          Test execution hangs with the specific use case of passing a configuration similar to:

            <cm:property-placeholder persistent-id="my-placeholders" update-strategy="reload">
  </cm:property-placeholder>

          The will pass value for that pid only at Test level inside the method useOverridePropertiesWithConfigAdmin

          The fix makes the attached test case succeed.

            Issue Links

            relates to

            Bug - A problem which impairs or prevents the functions of the product. CAMEL-9313 CamelBlueprintTestSupport - can't initialize ConfigAdmin configurations

            • Major - Major loss of function.
            • Resolved

              Activity

              Ascending order - Click to sort in descending order
              Hide
              Permalink
              githubbot ASF GitHub Bot added a comment -

              GitHub user paoloantinori opened a pull request:

              https://github.com/apache/camel/pull/860

              CAMEL-9636 - camel-test-blueprint - support for empty properties

              You can merge this pull request into a Git repository by running:

              $ git pull https://github.com/paoloantinori/camel camel-2.15.x

              Alternatively you can review and apply these changes as the patch at:

              https://github.com/apache/camel/pull/860.patch

              To close this pull request, make a commit to your master/trunk branch
              with (at least) the following in the commit message:

              This closes #860

              commit 0766a067850c03817fdb3a99e3059ac4f7b0f82d
              Author: Paolo Antinori <pantinor@redhat.com>
              Date: 2016-02-23T16:50:45Z

              CAMEL-9636 - camel-test-blueprint - support for empty properties

              Show
              githubbot ASF GitHub Bot added a comment - GitHub user paoloantinori opened a pull request: https://github.com/apache/camel/pull/860 CAMEL-9636 - camel-test-blueprint - support for empty properties You can merge this pull request into a Git repository by running: $ git pull https://github.com/paoloantinori/camel camel-2.15.x Alternatively you can review and apply these changes as the patch at: https://github.com/apache/camel/pull/860.patch To close this pull request, make a commit to your master/trunk branch with (at least) the following in the commit message: This closes #860 commit 0766a067850c03817fdb3a99e3059ac4f7b0f82d Author: Paolo Antinori <pantinor@redhat.com> Date: 2016-02-23T16:50:45Z CAMEL-9636 - camel-test-blueprint - support for empty properties
              Hide
              Permalink
              paolo.antinori Paolo Antinori added a comment -

              PR for 2.25.x branch https://github.com/apache/camel/pull/860

              Show
              paolo.antinori Paolo Antinori added a comment - PR for 2.25.x branch https://github.com/apache/camel/pull/860
              Hide
              Permalink
              githubbot ASF GitHub Bot added a comment -

              Github user asfgit closed the pull request at:

              https://github.com/apache/camel/pull/860

              Show
              githubbot ASF GitHub Bot added a comment - Github user asfgit closed the pull request at: https://github.com/apache/camel/pull/860
              Hide
              Permalink
              gzres Grzegorz Grzybek added a comment -

              Fix for 2.15.x: https://github.com/apache/camel/commit/28bf2484c88fcbd33774aad4c67745f7901f0a03
              Fix for 2.16.x: https://github.com/apache/camel/commit/a1acec615e400a008d6d083d6a84ad68ebb7ab43
              FIx for 2.17.x: https://github.com/apache/camel/commit/e15603a977b825a89fcad20b8da0a8ba59d982d3

              The fix is to detect problems early and do not simply fail on timeout while waiting for CamelContext OSGi service. One of the failures of Blueprint may be inability to resolve property placeholders. We've provided ConfigAdminNoDefaultValuesBlueprintCreationTest to show this problem.

              Show
              gzres Grzegorz Grzybek added a comment - Fix for 2.15.x: https://github.com/apache/camel/commit/28bf2484c88fcbd33774aad4c67745f7901f0a03 Fix for 2.16.x: https://github.com/apache/camel/commit/a1acec615e400a008d6d083d6a84ad68ebb7ab43 FIx for 2.17.x: https://github.com/apache/camel/commit/e15603a977b825a89fcad20b8da0a8ba59d982d3 The fix is to detect problems early and do not simply fail on timeout while waiting for CamelContext OSGi service. One of the failures of Blueprint may be inability to resolve property placeholders. We've provided ConfigAdminNoDefaultValuesBlueprintCreationTest to show this problem.

                People

                • Assignee:
                  gzres Grzegorz Grzybek
                  Reporter:
                  paolo.antinori Paolo Antinori
                • Votes:
                  0 Vote for this issue
                  Watchers:
                  3 Start watching this issue

                  Dates

                  • Created:
                    Updated:
                    Resolved:

                    Development