Skip to content

Conversation

@likitha
Copy link

@likitha likitha commented Jul 2, 2015

…instance.

Provide permissions to template folder when mounted on management server.
@wilderrodrigues
Copy link
Contributor

Travis failed due to a timeout.

I will have a look at the code.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

instance write-access to a static, even when in a synchronised block of code does not make for a good pattern. There is no guarantee that some other (synchronised) block accesses it as well.

@remibergsma
Copy link
Contributor

Who wants to step in and finish this work? It seems the original author is not able to finish it. If no one steps in, we'll have to close the PR without merging it so please help :-).

remibergsma added a commit to remibergsma/cloudstack that referenced this pull request Aug 17, 2015
This closes apache#577
This closes apache#566
This closes apache#562
This closes apache#561
This closes apache#556
This closes apache#555
This closes apache#554
This closes apache#548
This closes apache#544
This closes apache#540
This closes apache#508
This closes apache#384
This closes apache#372
remibergsma added a commit to remibergsma/cloudstack that referenced this pull request Aug 17, 2015
This closes apache#577
This closes apache#566
This closes apache#562
This closes apache#561
This closes apache#556
This closes apache#555
This closes apache#554
This closes apache#548
This closes apache#544
This closes apache#540
This closes apache#384
This closes apache#372
remibergsma added a commit to remibergsma/cloudstack that referenced this pull request Aug 17, 2015
This closes apache#577
This closes apache#566
This closes apache#562
This closes apache#561
This closes apache#556
This closes apache#555
This closes apache#554
This closes apache#548
This closes apache#544
This closes apache#540
This closes apache#384
This closes apache#372
@rohityadavcloud
Copy link
Member

tag:vmware-pickup

@sureshanaparti
Copy link
Contributor

sureshanaparti commented Apr 13, 2017

Continued with the code changes here in a new PR #1875. This PR will be closed when PR #1875 is merged.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants