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

fix-build-issue #1017

Merged
merged 9 commits into from
Apr 24, 2024
1 change: 0 additions & 1 deletion .env
Original file line number Diff line number Diff line change
Expand Up @@ -30,7 +30,6 @@ SOLR_COLLECTION_NAME=hydra-development
SOLR_CONFIGSET_NAME=hyku-1
SOLR_HOST=solr
SOLR_PORT=8983
SOLR_TAG=latest
SOLR_URL=http://solr:SolrRocks@solr:8983/solr/

# Comment out these 5 for single tenancy / Uncomment for multi
Expand Down
14 changes: 6 additions & 8 deletions .github/workflows/build-test-lint.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -17,21 +17,19 @@ on:

jobs:
build:
uses: scientist-softserv/actions/.github/workflows/build.yaml@v0.0.14
uses: scientist-softserv/actions/.github/workflows/build.yaml@v0.0.21
secrets: inherit
with:
platforms: "linux/amd64"
webTarget: hyku-base
solrTarget: hyku-solr
workerTarget: hyku-worker
test:
needs: build
uses: scientist-softserv/actions/.github/workflows/test.yaml@v0.0.14
with:
webTarget: hyku-base
workerTarget: hyku-worker
uses: scientist-softserv/actions/.github/workflows/test.yaml@v0.0.21
lint:
needs: build
uses: scientist-softserv/actions/.github/workflows/lint.yaml@v0.0.14
uses: scientist-softserv/actions/.github/workflows/lint.yaml@v0.0.21
with:
webTarget: hyku-base
workerTarget: hyku-worker
rubocop_cmd: 'bundle exec rubocop --parallel --format progress'

4 changes: 2 additions & 2 deletions .github/workflows/deploy.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -5,7 +5,7 @@ on:
inputs:
environment:
description: 'Deploy to Environment'
require: true
required: true
default: 'staging'
type: choice
options:
Expand All @@ -20,5 +20,5 @@ on:

jobs:
deploy:
uses: scientist-softserv/actions/.github/workflows/deploy.yaml@v0.0.15
uses: scientist-softserv/actions/.github/workflows/deploy.yaml@v0.0.21
secrets: inherit
2 changes: 1 addition & 1 deletion Gemfile
Original file line number Diff line number Diff line change
Expand Up @@ -17,7 +17,7 @@ gem 'blacklight_oai_provider', '~> 6.1', '>= 6.1.1'
gem 'blacklight_range_limit'
gem 'bolognese', '>= 1.9.10'
gem 'bootstrap-datepicker-rails'
gem 'bulkrax', '~> 5.0', github: 'samvera-labs/bulkrax', branch: 'main'
gem 'bulkrax', git: 'https://github.com/samvera-labs/bulkrax.git', tag: 'v5.3.0'
gem 'byebug', group: %i[development test]
gem 'capybara', group: %i[test]
gem 'capybara-screenshot', '~> 1.0', group: %i[test]
Expand Down
6 changes: 3 additions & 3 deletions Gemfile.lock
Original file line number Diff line number Diff line change
Expand Up @@ -20,8 +20,8 @@ GIT

GIT
remote: https://github.com/samvera-labs/bulkrax.git
revision: 2e4c9b79c37451fc2d147c0ae12cc4c10ceefa3c
branch: main
revision: f2439755f501caabedf4cc50618d1804e5b2293a
tag: v5.3.0
specs:
bulkrax (5.3.0)
bagit (~> 0.4)
Expand Down Expand Up @@ -1424,7 +1424,7 @@ DEPENDENCIES
blacklight_range_limit
bolognese (>= 1.9.10)
bootstrap-datepicker-rails
bulkrax (~> 5.0)!
bulkrax!
byebug
capybara
capybara-screenshot (~> 1.0)
Expand Down
3 changes: 2 additions & 1 deletion app/controllers/application_controller.rb
Original file line number Diff line number Diff line change
Expand Up @@ -34,7 +34,8 @@ class ApplicationController < ActionController::Base

# rubocop:disable Metrics/AbcSize
def global_request_logging
rl = ActiveSupport::Logger.new('log/request.log')
FileUtils.mkdir_p(Rails.root.join('log')) unless Dir.exist?(Rails.root.join('log'))
rl = ActiveSupport::Logger.new(Rails.root.join('log', 'request.log'))
if request.host&.match('blc.hykucommons')
http_request_header_keys = request.headers.env.keys.select { |header_name| header_name.match("^HTTP.*|^X-User.*") }
http_request_headers = request.headers.env.select { |header_name, _header_value| http_request_header_keys.index(header_name) }
Expand Down
7 changes: 4 additions & 3 deletions docker-compose.yml
Original file line number Diff line number Diff line change
Expand Up @@ -29,8 +29,8 @@ x-app-worker: &app-worker
- EXTRA_APK_PACKAGES=less vim bash openjdk11-jre ffmpeg rsync exiftool
- HYKU_BULKRAX_ENABLED=true
cache_from:
- ghcr.io/scientist-softserv/palni-palci:${TAG:-latest}
- ghcr.io/scientist-softserv/palni-palci/worker:${TAG:-latest}
- ghcr.io/scientist-softserv/palni-palci:latest
- ghcr.io/scientist-softserv/palni-palci/worker:latest
image: ghcr.io/scientist-softserv/palni-palci/worker:${TAG:-latest}
command: sh -l -c 'bundle && bundle exec sidekiq'
depends_on:
Expand Down Expand Up @@ -85,10 +85,11 @@ services:
timeout: "8s"

solr:
image: ghcr.io/samvera/hyku/solr:${SOLR_TAG:-latest}
image: ghcr.io/scientist-softserv/palni-palci/solr:${TAG:-latest}
build:
context: solr
dockerfile: Dockerfile
target: hyku-solr
environment:
- OOM=script
- SOLR_ADMIN_USER=solr
Expand Down
2 changes: 1 addition & 1 deletion solr/Dockerfile
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
FROM solr:8.3
FROM solr:8.3 as hyku-solr
ENV SOLR_USER="solr" \
SOLR_GROUP="solr"
USER root
Expand Down
Loading