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

added a dependency so that dev_solr waits for dev_solr_initializer to… #180

Merged
merged 1 commit into from
Sep 22, 2023

Conversation

ekraffmiller
Copy link
Contributor

… complete

What this PR does / why we need it:

modifies docker-compose so that dev_solr service waits for dev_solr_initializer to complete

Which issue(s) this PR closes:

Special notes for your reviewer:

Suggestions on how to test this:

Call ./run-env.sh to start the dataverse container.
Go to dataverse (localhost:8000) and login as dataverseAdmin.
Try to create a dataverse. It should create the dataverse successfully.

Does this PR introduce a user interface change? If mockups are available, please link/include them here:

No

Is there a release notes update needed for this change?:

Additional documentation:

@ekraffmiller ekraffmiller linked an issue Sep 22, 2023 that may be closed by this pull request
@coveralls
Copy link

Coverage Status

coverage: 98.58% (+0.7%) from 97.93% when pulling d97f917 on 179-fix-solr-dev-env into a220c4f on develop.

@pdurbin
Copy link
Member

pdurbin commented Sep 22, 2023

A number of automated checks are failing (chromatic, e2e, and component) but I'm not sure if that's expected or not. We should check with @GPortas and @MellyGray

I did get on this branch and try the run-env script. Dataverse spun up fine and I was able to create a collection with no Solr error. Moving to QA.

I'm going to slap a size of 10 on this for the combined hours spent.

@pdurbin pdurbin added the Size: 10 A percentage of a sprint. 7 hours. label Sep 22, 2023
Copy link
Member

@pdurbin pdurbin left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Works on my machine!

By the way, here are the depends_on docs @poikilotherm linked us to: https://github.com/compose-spec/compose-spec/blob/master/spec.md#long-syntax-1 (thanks!)

Sending to QA but note that some automated tests are failing. I'm not sure why. 🤔

@kcondon kcondon self-assigned this Sep 22, 2023
@kcondon kcondon merged commit 82888c0 into develop Sep 22, 2023
3 of 6 checks passed
@kcondon kcondon deleted the 179-fix-solr-dev-env branch September 22, 2023 19:08
jayanthkomarraju pushed a commit to jayanthkomarraju/dataverse-frontend that referenced this pull request May 31, 2024
added a dependency so that dev_solr waits for dev_solr_initializer to…
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Size: 10 A percentage of a sprint. 7 hours.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Solr error when running dev dataverse container
4 participants