Skip to content

Commit

Permalink
Revert "#15: Install Drupal configset for Solr8 via custom Solr8 imag…
Browse files Browse the repository at this point in the history
…e. (#163)"

This reverts commit 1cff739.
  • Loading branch information
reynoldsalec authored Mar 16, 2023
1 parent 1cff739 commit 09e53d9
Show file tree
Hide file tree
Showing 4 changed files with 4 additions and 20 deletions.
3 changes: 0 additions & 3 deletions .github/workflows/build-pantheon-images.yml
Original file line number Diff line number Diff line change
Expand Up @@ -18,9 +18,6 @@ jobs:
- image: pantheon-index
tag: 3.6-3
context: recipes/pantheon/3.6-solr
- image: pantheon-index
tag: 8.8-4
context: recipes/pantheon/8.8-solr
- image: pantheon-appserver
tag: 8.1-4
context: recipes/pantheon/8.1-fpm
Expand Down
7 changes: 3 additions & 4 deletions lib/utils.js
Original file line number Diff line number Diff line change
Expand Up @@ -252,16 +252,15 @@ exports.getPantheonEnvironment = options => ({
* Helper to build index service
*/
exports.getPantheonIndex = options => {
if (options.solrTag === '8.8') {
if (options.solrTag === '8') {
return {
services: {
index: {
type: 'solr:8.8.2',
type: 'solr:8.11.2',
portforward: true,
overrides: {
image: `devwithlando/pantheon-index:${options.solrTag}`,
volumes: [
`${options.confDest}/jetty.xml:/opt/solr-8.8.2/server/etc/jetty.xml`,
`${options.confDest}/jetty.xml:/opt/solr-8.11.2/server/etc/jetty.xml`,
],
},
},
Expand Down
12 changes: 0 additions & 12 deletions recipes/pantheon/8.8-solr/Dockerfile

This file was deleted.

2 changes: 1 addition & 1 deletion recipes/pantheon/builder.js
Original file line number Diff line number Diff line change
Expand Up @@ -130,7 +130,7 @@ module.exports = {
// Set the search version
const searchVersion = _.toString(_.get(options, 'search.version', '3'));
// Set solrtag if search is set to solr8.
if (searchVersion === '8') options.solrTag = '8.8';
if (searchVersion === '8') options.solrTag = searchVersion;
options.database = `${dbService}:${dbVersion}`;
// Set correct things based on framework
options.defaultFiles.vhosts = `${options.framework}.conf.tpl`;
Expand Down

0 comments on commit 09e53d9

Please sign in to comment.