Solr installer should set facts for solr reset about ansible-hydra

[acozine]

Oh, interesting. Can you give me a bit more context? It looks like this is happening on the deploy role. Are you running that role on a pre-existing box to deploy fresh code? Are you running it in the context of building a box? Which OS? What version of Solr are you trying to restart?

[mark-dce]

Yes - I'm running it in a deploy role. I'm copying in my playbook here since the vars are in a private repo to keep them secure

# Config file to bulid a Ubuntu 16.04 LTS Xenial Xerus server for
# projects based on https://github.com/curationexperts/epigaea
# run the playbook with the command:
# "ansible-playbook -i hosts sample-ubuntu-16.yml"
# 
- hosts: localhost
  connection: local
  gather_facts: False
  vars_files:
    - redaacted-vars.yml
  roles:
    - launch_ec2

- hosts: newboxes
  name: prep
  user: ubuntu
  gather_facts: false
  pre_tasks:
      - name: install python2 for ansible
        raw: sudo apt-get -y install python-simplejson

- hosts: newboxes
  name: configuration
  user: ubuntu
  vars_files:
    - redacted-vars.yml
  gather_facts: true
  roles:
    - { role: packages }
    - { role: mount_disks }
    - { role: set_timezone }
    - { role: system_setup }
    - { role: capistrano_setup }
    - { role: ruby }
    - { role: fedora }
    - { role: solr }
    - { role: frontend }
    - { role: sufia_dependencies/install }
    - { role: sufia_dependencies/config }
# sufia_dependencies must precede marmotta, which requires postgresql
    - { role: webserver/install }
    - { role: webserver/config }
    - { role: deploy }

[sanfordd]

Does this mean we don't want to remove Solr 4 support as per #244 if there's still use cases for it?

[mark-dce]

@sanfordd - no, I'm not trying to comment about specific versions, you could just update my example to

when: {{ solr_main_version | default('not_installed') }} == 5

or something similar.

My main point is that I shouldn't have to explicitly define both solr_version and solr_main_version since the latter is derivable from the former. As long as the main version is a single digit, it looks like {{ solr_version[0] }} should probably work. You could do something more complex and find the index of the first period in solr_version and then grab everything up to the period: {{ solr_version[0:index_of_first_period_in_verison_nubmer] }}

[bess]

Fixed in ansible-samvera