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

NO AUTO Makes container wait for neo4j database ready #3065

Merged
merged 1 commit into from Jul 21, 2022

Conversation

ncordon
Copy link
Contributor

@ncordon ncordon commented Jul 19, 2022

Neo4j 5.x reconciler is eventually consistent now, which means on startup we cannot assume neo4j will be ready and we have to wait for it to be.

The inspiration for this came from the official neo4j-docker test utils. Thanks @gfx54b for the help.

@ncordon ncordon added bug 5.0 team-cypher-surface Cypher Surface team should review the PR labels Jul 19, 2022
@nadja-muller nadja-muller self-assigned this Jul 20, 2022
@ncordon ncordon changed the title Makes container wait for neo4j database ready NO AUTO Makes container wait for neo4j database ready Jul 20, 2022
@ncordon ncordon merged commit a34ab7d into dev Jul 21, 2022
@ncordon ncordon deleted the dev-solves-database-wait branch July 21, 2022 08:03
AzuObs pushed a commit that referenced this pull request Jul 28, 2022
* Remove deprecated procedures

* Remove references of deprecated procedures from all documentation

* Add documentation about deprecation removals

* Bring back apoc.warmup.run (deprecated 5.0 only)

* Add deprecation documentation around the removed APOC config setting

* Convert old tests of deprecated functions into ones of their replacements

* NO AUTO Makes container wait for neo4j database ready (#3065)

* Fix failing test (looking for now removed procedure)

* Add updated mongodb back into navigation

Co-authored-by: Nacho Cordón <ncordon@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
5.0 bug team-cypher-surface Cypher Surface team should review the PR
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants