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

use official zookeeper image for solr-drupal recipe #195

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

mkalkbrenner
Copy link
Contributor

switch from bitnami to official for arm64 support

@rfay
Copy link
Member

rfay commented Feb 2, 2022

Thanks for doing this! I'm on vacation on a road trip with very limited internet, but hope to look soon.

@rfay
Copy link
Member

rfay commented Feb 11, 2022

BTW, ddev v1.19.0-alpha4 is out, so this should be easily testable there.

@rfay
Copy link
Member

rfay commented Mar 8, 2022

Just a note that after v1.19.0 release this is high priority to get in and also to make into a ddev get addon.

@rfay rfay changed the title use official zookeeper image use official zookeeper image for solr-drupal recipe Mar 16, 2022
Copy link
Member

@rfay rfay left a comment

Choose a reason for hiding this comment

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

Thanks for all the work on this.

I rebased and made some minor changes. The use of ddev-<project>-solr is no longer necessary in ddev v1.19.

  1. I wasn't able to figure out what the README instructions mean by Default Solr collection: techproducts -there's no entry in the Drupal config form by that name.
  2. I didn't know what to put in "solr core name" as that's not mentioned in the README.
  3. zookeeper wouldn't come up, so I got stopped there. See gist at https://gist.github.com/rfay/7238505ac7b7455d50ce3764880dfd11

Thanks for your pioneering of all this. Sorry for the slow review. My hope is to translate this into a ddev-get recipe right away, but I have to succeed at setup first.

@rfay
Copy link
Member

rfay commented Feb 24, 2023

Bitnami now has a multi-arch zookeeper image, https://hub.docker.com/r/bitnami/zookeeper/tags

@rfay
Copy link
Member

rfay commented Aug 4, 2023

This should be obsolete right? Just add pointer to your new add-on?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants