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

[BUG] eXist-db conf templates use the incorrect versions #54

Open
adamretter opened this issue Apr 19, 2023 · 2 comments
Open

[BUG] eXist-db conf templates use the incorrect versions #54

adamretter opened this issue Apr 19, 2023 · 2 comments

Comments

@adamretter
Copy link
Contributor

At present when exist_confxml_from_template: yes is used, the conf.xml is read from the templates folder.

Unfortunately the template to use is selected based only on the major version number of eXist-db (see: https://github.com/eXist-db/existdb-ansible-role/blob/v1.2/tasks/postinstall.yml#L227).

However eXist-db has changed its conf.xml files between minor versions to add new features; as is permissible by the Semver 2.0.0 specification.

One such example is configuration settings around Document and Collection Locks, and the Lock Table. This means that it is quite possible that Ansible will deploy a non-complete (or even incorrect) conf.xml file, which may end up with certain config settings in eXist-db being "undefined", which leads to "undefined" behaviour at runtime.

@chakl
Copy link
Collaborator

chakl commented Apr 19, 2023

However eXist-db has changed its conf.xml files between minor versions to add new features

Exactly. When this role was written (around eXist-db 4.7), core configuration files like conf.xml have changed without version numbers being updated. (not sure if this still is the case). This left us with guesswork about correct template files.

As a workaround, you can set exist_confxml_from_template: no and provide a desired conf.xml yourself.

This should be fixed upstream, ensuring that every change to default config files will result in a new version number.

@adamretter
Copy link
Contributor Author

@chakl I'm not sure I understand what you mean, could you explain further please? Each released version of eXist-db has a corresponding conf.xml file

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