Skip to content

Conversation

@gerlowskija
Copy link
Contributor

Ambari 2.7 included a few changes in the configuration structure which
caused issues with our Solr mpack. This commit tweaks some of our
configuration references to work against the more recent 2.7.0+ config
structure.

Ambari 2.7 included a few changes in the configuration structure which
caused issues with our Solr mpack.  This commit tweaks some of our
configuration references to work against the more recent 2.7.0+ config
structure.
Copy link
Contributor

@acesar acesar left a comment

Choose a reason for hiding this comment

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

LGTM

{
"stack_name": "HDP",
"stack_version": "2.6"
"stack_version": "3.0"
Copy link
Contributor

Choose a reason for hiding this comment

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

This file has the property min_ambari_version;
do we need to change it? I think not unless one of the Solr versions get deprecated, right?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

My understanding is that our "min_ambari_version" is informed by the oldest SOLR-service version that we bundle in here. 5.5.5 is our oldest, and it's used for HDP 2.4, which in turn is intended for Ambari 2.2.1.0 (https://hortonworks.com/wp-content/uploads/2016/03/ReleaseNotes_3_3_2016.pdf)

So based on that, I'm not sure we can increase our min_ambari_version, at least not unless we want to get rid of the SOLR 5.5.5 stack version. (In fact, I'm not sure why the min_ambari_version is as high as it is. I'd think it would need to be 2.2.1.0. Maybe I'm misunderstanding how these versions fit together. Will do a little more digging.)

@gerlowskija gerlowskija merged commit 78bb852 into branch_7x Aug 3, 2018
@gerlowskija gerlowskija deleted the use_ambari_270 branch August 3, 2018 13:12
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.

3 participants