You are viewing a plain text version of this content. The canonical link for it is here.

Posted to dev@brooklyn.apache.org by aledsage <gi...@git.apache.org> on 2018/09/06 11:41:05 UTC

[GitHub] brooklyn-server pull request #986: BROOKLYN-600: cleanup entities on deploy-...

GitHub user aledsage opened a pull request:

    https://github.com/apache/brooklyn-server/pull/986

    BROOKLYN-600: cleanup entities on deploy-failure

    Fix for https://issues.apache.org/jira/browse/BROOKLYN-600
    
    Note the `LocalEntityManager. discardPremanaged(Entity)` is a bit more complex/long that it might need to be because I want to make sure it does no harm - fail if the entity or any child is already managed (because then should call `unmanage()` instead.

You can merge this pull request into a Git repository by running:

    $ git pull https://github.com/aledsage/brooklyn-server BROOKLYN-600

Alternatively you can review and apply these changes as the patch at:

    https://github.com/apache/brooklyn-server/pull/986.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 #986
    
----
commit a87e6692a54a10d389165fd82c93b3fd2f4ccc03
Author: Aled Sage <al...@...>
Date:   2018-09-06T11:38:05Z

    BROOKLYN-600: cleanup entities on deploy-failure

----


---

[GitHub] brooklyn-server pull request #986: BROOKLYN-600: cleanup entities on deploy-...

Posted by asfgit <gi...@git.apache.org>.

Github user asfgit closed the pull request at:

    https://github.com/apache/brooklyn-server/pull/986


---

[GitHub] brooklyn-server issue #986: BROOKLYN-600: cleanup entities on deploy-failure

Posted by duncangrant <gi...@git.apache.org>.

Github user duncangrant commented on the issue:

    https://github.com/apache/brooklyn-server/pull/986
  
    This looks good to me


---