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

          Activity

          Hide
          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
          paolo.antinori Paolo Antinori added a comment -
          Show
          paolo.antinori Paolo Antinori added a comment - PR for 2.25.x branch https://github.com/apache/camel/pull/860
          Hide
          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
          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