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

provide pre-install hook for applications #2309

Open
line-o opened this issue Nov 23, 2018 · 8 comments
Open

provide pre-install hook for applications #2309

line-o opened this issue Nov 23, 2018 · 8 comments
Labels
bug issue confirmed as bug needs documentation Signals issues or PRs that will require an update to the documentation repo

Comments

@line-o
Copy link
Member

line-o commented Nov 23, 2018

What is the problem

Install an app A in version x and modify some data.

Now update A to a new version y.

In order to be able to backup or migrate the data in the running instance of A the installation process has to have a hook where:

  • The application versions of the current and the new one are known.
  • No data was deleted.
  • a script is run when defined in repo.xml

What did you expect

I expected that the <setup/> hook would provide this. But it seems that the current data is removed too early.

Describe how to reproduce or add a test

see above

Context information

Please always add the following information

  • affected versions eXist-db 3.x.x and 4.x.x have not tested on the release candidates yet
  • Java 8
  • tested on Linux and MacOs
  • 64 bit
  • built from source and started with bin/startup.sh
@triage-new-issues triage-new-issues bot added the triage issue needs to be investigated label Nov 23, 2018
@duncdrum duncdrum added the needs documentation Signals issues or PRs that will require an update to the documentation repo label Nov 23, 2018
@triage-new-issues triage-new-issues bot removed the triage issue needs to be investigated label Nov 23, 2018
@duncdrum
Copy link
Contributor

there is a setup hook for repo.xml??
needs documentation

@line-o
Copy link
Member Author

line-o commented Nov 23, 2018

I know, found it by reading source code :)

https://github.com/eXist-db/exist/blob/develop/src/org/exist/repo/Deployment.java#L354

@duncdrum
Copy link
Contributor

see eXist-db/documentation#164

@adamretter
Copy link
Member

@line-o @duncdrum Can we close this now?

@duncdrum
Copy link
Contributor

duncdrum commented Dec 4, 2018

@adamretter afaik its still not working, and not-documented why would we close this?

@adamretter
Copy link
Member

I thought from the conversation above it was fixed

@duncdrum duncdrum reopened this Dec 4, 2018
@duncdrum
Copy link
Contributor

duncdrum commented Dec 4, 2018

no a) the hook exists but isn't working, b) the existence of the hook is not documented.

@duncdrum
Copy link
Contributor

duncdrum commented May 14, 2019

@joewiz might be of interest
see eXist-db/public-repo#39

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug issue confirmed as bug needs documentation Signals issues or PRs that will require an update to the documentation repo
Projects
None yet
Development

No branches or pull requests

3 participants