chef-solr configs are packed within tar.gz

Details

  • Type: Bug Bug
  • Status: Closed Closed
  • Priority: Major Major
  • Resolution: Fixed
  • Affects Version/s: None
  • Fix Version/s: 0.10.6
  • Component/s: None
  • Labels:
    None

Description

Trying to fix CHEF-2346 but illustrating my change is difficult due to the configs being packed in chef-solr/solr/solr-home.tar.gz and chef-solr/solr/solr-jetty.tar.gz.

Issue Links

is duplicated by

Bug - A problem which impairs or prevents the functions of the product. CHEF-2973 undefined method `collect' in check_server_side_cookbooks

  • Major - Major loss of function.
  • Closed - The issue is considered finished, the resolution is correct. Issues which are closed can be reopened.

Activity

Ascending order - Click to sort in descending order
Hide
Permalink
Matthew Kent added a comment -

Proposed solution in https://github.com/mdkent/chef/commits/CHEF-2553

Took the approach of creating a tar_solr rake task that's a prerequisite to rake gem (and thus rake package). I dropped the files from the gemspec and add them dynamically.

One issue I see with this approach is that gem build chef-solr.gemspec won't produce a complete package as it's not using rake. Not sure about the idea of putting ruby in the gemspec but it could bail out if $0 == "gem" or otherwise produce a warning. Still seems an improvement over the current situation though.

Show
Matthew Kent added a comment - Proposed solution in https://github.com/mdkent/chef/commits/CHEF-2553 Took the approach of creating a tar_solr rake task that's a prerequisite to rake gem (and thus rake package). I dropped the files from the gemspec and add them dynamically. One issue I see with this approach is that gem build chef-solr.gemspec won't produce a complete package as it's not using rake. Not sure about the idea of putting ruby in the gemspec but it could bail out if $0 == "gem" or otherwise produce a warning. Still seems an improvement over the current situation though.
Hide
Permalink
Bryan McLellan [Opscode] added a comment -

Merged to opscode/master. Thanks Matthew!

Note to future selves: this is a pretty big build change and we need to be wary of the result.

Show
Bryan McLellan [Opscode] added a comment - Merged to opscode/master. Thanks Matthew! Note to future selves: this is a pretty big build change and we need to be wary of the result.
Hide
Permalink
Matthew Kent added a comment -

From IRC: chef-solr.gemspec is broken now, it specifies chef-solr-rebuild which was removed in CHEF-2530.

Please pull from https://github.com/mdkent/chef/commits/CHEF-2553-gemspec

Show
Matthew Kent added a comment - From IRC: chef-solr.gemspec is broken now, it specifies chef-solr-rebuild which was removed in CHEF-2530 . Please pull from https://github.com/mdkent/chef/commits/CHEF-2553-gemspec
Hide
Permalink
Daniel DeLeo added a comment -

Merged the gemspec fix. Thanks again!

Show
Daniel DeLeo added a comment - Merged the gemspec fix. Thanks again!

People

  • Assignee:
    Matthew Kent
    Reporter:
    Matthew Kent
Vote (0)
Watch (1)

Dates

  • Created:
    Updated:
    Resolved: