Skip to content
This repository has been archived by the owner on Nov 9, 2022. It is now read-only.

Improve group and host deployment support #312

Closed
grtjn opened this issue Nov 21, 2014 · 3 comments
Closed

Improve group and host deployment support #312

grtjn opened this issue Nov 21, 2014 · 3 comments

Comments

@grtjn
Copy link
Contributor

grtjn commented Nov 21, 2014

Recent changes allow adding @group on appservers and databases. It also creates groups if they don't exist yet. Unfortunately it doesn't assign hosts to those new groups, so databases assigned to those new groups end up without forests.

I recommend improving config by wrapping the relevant appservers inside gr:groups/gr:group with a gr:group-name that shows in which group those should be added. This provides space for setting group properties as well. Additionally a ho:hosts/ho:host structure needs to be added that allows setting host properties which also includes assigning a host to a group.

It is possible to rewrite configs on the fly to support backwards compatibility, and put appservers and database forest in Default group (or the one of the current host/app-server) if not explicitly specified.

@grtjn
Copy link
Contributor Author

grtjn commented Nov 21, 2014

I have worked on this, and managed to get bootstrap and wipe working with this. Works nicely with the previous work on assigning servers and database forests to (hosts within) groups.

Only one catch that admin:group-delete refuses to delete a group even after saving config. Apparently a restart is necessary to refresh results returned by xdmp:groups and xdmp:group-hosts, which is used inside the admin function to verify if admin-config has been saved properly. I worked around that by bailing out early, and telling the user to restart and wipe again.

I will open a pull request for review..

@grtjn
Copy link
Contributor Author

grtjn commented Nov 21, 2014

It does extend on #307, so maybe review that first..

grtjn added a commit to grtjn/roxy that referenced this issue Dec 9, 2014
grtjn added a commit to grtjn/roxy that referenced this issue Dec 10, 2014
grtjn added a commit to grtjn/roxy that referenced this issue Dec 10, 2014
grtjn added a commit to grtjn/roxy that referenced this issue Dec 23, 2014
paxtonhare added a commit that referenced this issue Jan 12, 2015
Fixed #312: added support for group and host properties, backwards compatible
@grtjn
Copy link
Contributor Author

grtjn commented Jan 20, 2015

Fixed in dev

@grtjn grtjn closed this as completed Jan 20, 2015
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

2 participants