brooklyn-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From neykov <...@git.apache.org>
Subject [GitHub] brooklyn-server issue #451: Adds ConfigParametersYamlTest.testPortSetAsAttri...
Date Thu, 24 Nov 2016 10:00:22 GMT
Github user neykov commented on the issue:

    https://github.com/apache/brooklyn-server/pull/451
  
    Wondered how this test is passing. Turns out that we are [creating port sensors](https://github.com/apache/brooklyn-server/blob/master/core/src/main/java/org/apache/brooklyn/core/objs/BasicSpecParameter.java#L256)
in addition to the config keys from the yaml. And those get [set on the entity](https://github.com/apache/brooklyn-server/blob/master/core/src/main/java/org/apache/brooklyn/core/objs/proxy/InternalEntityFactory.java#L276)
just like config keys.
    
    The sensors aren't serialized as part of the entity type. That's fine though as the sensor
sets a value for itself on start which gets serialized (along with the sensor that set it).


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

Mime
View raw message