Github user aledsage commented on a diff in the pull request:

    https://github.com/apache/brooklyn-server/pull/151#discussion_r64374027
  
    --- Diff: 
camp/camp-brooklyn/src/test/java/org/apache/brooklyn/camp/brooklyn/EmptySoftwareProcessYamlTest.java
 ---
    @@ -60,6 +62,31 @@ public void testProvisioningProperties() throws 
Exception {
         }
     
         @Test(groups="Integration")
    +    public void testProvisioningPropertiesOverrideValues() throws 
Exception {
    +        Entity app = createAndStartApplication(
    +                "location:",
    +                "  localhost:",
    +                "    templateOptions:",
    +                "      testKey: testValue",
    --- End diff --
    
    I'd include in the test an assertion about what takes precedence. (and you 
can turn off all attempts to ssh to the machine). For example, have:
    
        location:
          useJcloudsSshInit: false
          waitForSshable: false
          minRam: 1G
          minCores: 1
          templateOptions:
            locKey1: locVal1
            locKey2: locVal2
        services:
        - type: EmptySoftwareProcess
          brooklyn.config:
            onbox.base.dir.skipResolution: true
            sshMonitoring.enabled: false
            provisioning.properties:
              minRam: 2G
              templateOptions:
                locKey1: overriddenLocVal1
                entKey1: entVal1



---
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 [email protected] or file a JIRA ticket
with INFRA.
---

Reply via email to