## What changes were proposed in this pull request?

In Ambari 2.7 we moved LDAP configuration into the Ambari DB and introduced 
common naming pattern. However a typo has been made in 
`ambari.ldap.advance.collision_behavior` This should be renamed to 
`ambari.ldap.advanced.collision_behavior`

## How was this patch tested?

In addition to executing unit tests in `ambari-server` I simulated an Ambari 
server upgrade from 2.7.1 to 2.7.2:

1. had an Ambari 2.7.1 in my test environment
2. I configured LDAP using the 2.7.1 `setupSecurity.py`
3. updated `/var/lib/ambari-server/resources/version to 2.7.2
4. run `ambari-server upgrade`

In the DB I found the correct property name; in the log I found the trace:
```
2018-08-27 13:21:55,770  INFO [main] SchemaUpgradeHelper:232 - Executing DML 
changes.
2018-08-27 13:21:55,781  DEBUG [main] UpgradeCatalog272:80 - Executing: UPDATE 
ambari_configuration SET property_name = 
'ambari.ldap.advanced.collision_behavior' WHERE category_name = 
'ldap-configuration' AND property_name = 
'ambari.ldap.advance.collision_behavior'
2018-08-27 13:21:55,783  INFO [main] UpgradeCatalog272:82 - Renamed 1 record 
with incorrect LDAP configuration property name
```

Besides this I tested if the new `ambari-server setup-ldap` works as expected.

[ Full content available at: https://github.com/apache/ambari/pull/2174 ]
This message was relayed via gitbox.apache.org for [email protected]

Reply via email to