GitHub user ahgittin opened a pull request:
https://github.com/apache/brooklyn-server/pull/887
Config on entity dynamic goes on type after persistence
Fixes two issues noted with rebind of dynamic config. See first commit
first for failing test, then the others which fix it and comment.
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/ahgittin/brooklyn-server
config-on-entity-dynamic-goes-on-type-after-persistence
Alternatively you can review and apply these changes as the patch at:
https://github.com/apache/brooklyn-server/pull/887.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 #887
----
commit d353d2352e746339ad546758e87cb438427234ce
Author: Alex Heneveld <[email protected]>
Date: 2017-11-09T10:20:03Z
failing test for config persistence edge case
commit 82dfdd27ba34d13b8bdb3fcd9a5d84f0a463bf09
Author: Alex Heneveld <[email protected]>
Date: 2017-11-09T10:33:38Z
compare query and declared types when looking up config
fixes one of the two checks in the previously failing test by
preferring the query type if more specific when looking up config.
also introduces a warning if the query type is incompatible.
commit 1980b005a4081d7374af90ac2454f51d1763d20b
Author: Alex Heneveld <[email protected]>
Date: 2017-11-09T11:12:17Z
don't persist anonymous keys
another way of fixing the failing-before-this-commit test.
adding anonymous keys to the type means rebinded entities differ from
original.
see comments, we still have that discrepancy for non-anonymous keys, but
there it seems more acceptable.
commit 31ff1855fbd0334457e64b621376cd349c545992
Author: Alex Heneveld <[email protected]>
Date: 2017-11-09T11:34:20Z
more comments and test for persisted non-type config
in particular the fact that such added config shifts to be on the type
after rebind,
and tidy for other affected tests
----
---
