Skip to content

Commit

Permalink
Fix/41 correction of solr ports v2
Browse files Browse the repository at this point in the history
Merge branch 'development' into 'master'

Closes #41

See merge request sdbm/sdbmss!41
  • Loading branch information
emeryr-upenn committed Sep 12, 2022
2 parents ce96c5f + d826c02 commit 8714c09
Show file tree
Hide file tree
Showing 10 changed files with 20 additions and 10 deletions.
7 changes: 6 additions & 1 deletion README-development.md
Original file line number Diff line number Diff line change
Expand Up @@ -50,7 +50,12 @@ Generate separate keys (for each) by running rails task secret.

URL for SOLR server, using relative location of docker service (using name 'solr' from docker-compose.yml)

SOLR_URL=http://solr:8982/solr/development
NB: The SOLR port here and everywhere else in the code is `8983`. Previous versions
of the code had `8982` in some cases. This was wrong and created a conflict
between Jetty (which was set to `8983` and the docker-compose values, which
had `8982`).

SOLR_URL=http://solr:8983/solr/development

**3. Build and Run (First Time)**

Expand Down
7 changes: 6 additions & 1 deletion README.md
Original file line number Diff line number Diff line change
Expand Up @@ -75,7 +75,12 @@ DEPRECATED -- USING ANSIBLE

URL for SOLR server, using relative location of docker service (using name 'solr' from docker-compose.yml)

SOLR_URL=http://solr:8982/solr/development
NB: The SOLR port here and everywhere else in the code is `8983`. Previous versions
of the code had `8982` in some cases. This was wrong and created a conflict
between Jetty (which was set to `8983` and the docker-compose values, which
had `8982`).

SOLR_URL=http://solr:8983/solr/development

**3. (Optional) Create a file VERSION in the root folder of the cloned repository.**

Expand Down
2 changes: 1 addition & 1 deletion config/SolrMarc/config.properties
Original file line number Diff line number Diff line change
Expand Up @@ -28,7 +28,7 @@ solr.indexer.properties = index.properties
# NOTE:
# When using the solr:marc:index rake task, this value will be automatically
# looked up from your solr.yml, and the value here will NOT be used.
solr.hosturl = http://localhost:8982/solr/update
solr.hosturl = http://localhost:8983/solr/update

# MARC handling parameters
marc.to_utf_8 = true
Expand Down
2 changes: 1 addition & 1 deletion config/sunspot.yml
Original file line number Diff line number Diff line change
Expand Up @@ -6,7 +6,7 @@
production:
solr:
# hostname: localhost
port: 8982
port: 8983
log_level: WARNING
# path: /solr/production
# read_timeout: 2
Expand Down
2 changes: 1 addition & 1 deletion docker-compose-dev.yml
Original file line number Diff line number Diff line change
Expand Up @@ -42,7 +42,7 @@ services:
context: .
dockerfile: Dockerfile
ports:
- "8982:8982"
- "8983:8983"
# logging:
# driver: "journald"
links:
Expand Down
2 changes: 1 addition & 1 deletion docker-compose.yml
Original file line number Diff line number Diff line change
Expand Up @@ -63,7 +63,7 @@ services:
image: 'gitlab.library.upenn.edu/sdbm/sdbmss/sdbmss_web:${SDBMSS_IMAGE_TAG}'
restart: always
ports:
- "8982:8982"
- "8983:8983"
# env_file: .docker-environment
# logging:
# driver: "journald"
Expand Down
2 changes: 1 addition & 1 deletion lib/sdbmss/util.rb
Original file line number Diff line number Diff line change
Expand Up @@ -507,7 +507,7 @@ def int? s
# really can take 5 secs, if not more.
def wait_for_solr_to_be_current
host = ENV['SOLR_URL'].present? ? URI(ENV['SOLR_URL']).host : 'localhost'
uri = "http://#{host}:8982/solr/admin/cores?action=STATUS&core=test"
uri = "http://#{host}:8983/solr/admin/cores?action=STATUS&core=test"

current = false
count = 0
Expand Down
2 changes: 1 addition & 1 deletion run_tests.sh
100755 → 100644
Original file line number Diff line number Diff line change
Expand Up @@ -5,7 +5,7 @@
# Do NOT use this when running in a Docker container!

export RAILS_ENV=test
export SOLR_URL="http://127.0.0.1:8982/solr/test"
export SOLR_URL="http://127.0.0.1:8983/solr/test"

bundle exec god -c sdbmss_test.god -l log/god_test.log

Expand Down
2 changes: 1 addition & 1 deletion sdbmss_nginx/nginx.conf
Original file line number Diff line number Diff line change
Expand Up @@ -18,7 +18,7 @@ http {

upstream solr {
least_conn;
server solr:8982 weight=10 max_fails=3 fail_timeout=30s;
server solr:8983 weight=10 max_fails=3 fail_timeout=30s;
}

server {
Expand Down
2 changes: 1 addition & 1 deletion solr/conf/scripts.conf
Original file line number Diff line number Diff line change
Expand Up @@ -15,7 +15,7 @@

user=
solr_hostname=localhost
solr_port=8982
solr_port=8983
rsyncd_port=18983
data_dir=
webapp_name=solr
Expand Down

0 comments on commit 8714c09

Please sign in to comment.