Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Create online commands for deploying/undeploying #42

Closed
rhatlapa opened this issue Dec 17, 2015 · 4 comments
Closed

Create online commands for deploying/undeploying #42

rhatlapa opened this issue Dec 17, 2015 · 4 comments
Assignees
Milestone

Comments

@rhatlapa
Copy link

There should be created command, which will allow to easily create deployment operation, which can be used for both: server in standalone and in domain mode.

@rhatlapa rhatlapa self-assigned this Dec 17, 2015
@rhatlapa
Copy link
Author

rhatlapa commented Feb 1, 2016

I would like to discuss how to create undeploy command for domain (its behaviour)

There is mainly one concern and that is deployment in repository vs being deployed to some specific server groups.

The deployment can't be removed from deployment repository till it is used by some server group.

Here is the problematic part, which is to decide whether it should be only undeployed/removed from specified server group or also from the deployment repository.

@rhatlapa
Copy link
Author

rhatlapa commented Feb 1, 2016

The easiest way seems to me making builder over CLI operation undeploy in this case. Where it would allow to specify whether it should be undeployed from all relevant server groups or just from specific ones.

@Ladicek
Copy link
Contributor

Ladicek commented Feb 2, 2016

The undeploy command can have options specific to managed domain that would specify the intended behavior.

The key here is the default behavior where no such options are provided -- should it undeploy from all server groups and remove from content repository? Seems quite brutal, but probably closest to the standalone case. Or any other options?

@Ladicek
Copy link
Contributor

Ladicek commented Feb 17, 2016

Fixed by #57.

@Ladicek Ladicek closed this as completed Feb 17, 2016
@Ladicek Ladicek added this to the 0.9.5 milestone Feb 17, 2016
jtymel pushed a commit to jtymel/creaper that referenced this issue Nov 21, 2016
refactor: CredentialRef contains CredentialRefBuilder
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

No branches or pull requests

2 participants