Github user rdowner commented on the issue:

    https://github.com/apache/brooklyn-server/pull/777
  
    @ahgittin this appears to go against our published deprecation policy and 
is known to break some blueprints.
    
    Your PR comments are making references to previous discussions, but it's 
not clear where those are, and I haven't been able to find them. Example: `as 
comments there suggested` in the first comment; `the new "primary" model` is 
mentioned in your last commit but no explanation of this model; `causes 
important blueprints to fail` but does not say which ones. So unfortunately I 
don't yet understand why this breaking change is needed.
    
    I think we should prepare a response (explanation, workaround, new best 
practice recommendation) to the fact that we've broken some of our user's 
blueprints, with a clear justification of why it had to be done. I'd like to 
see this discussed on the mailing list, and noted in [the release 
notes](https://github.com/apache/brooklyn-docs/blob/master/guide/misc/release-notes.md),
 and documentation too.
    
    Thanks


---

Reply via email to