-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/62996/
-----------------------------------------------------------

Review request for Ambari, Dmytro Grinenko, Dmitro Lisnichenko, and Jonathan 
Hurley.


Bugs: AMBARI-22240
    https://issues.apache.org/jira/browse/AMBARI-22240


Repository: ambari


Description
-------

It looks like the consistent item here is that Kerberos is enabled, and that's 
where this bug surfaces.  It looks like when we switch the {{clusterconfig}} 
entities to {{setSelected(true)}}, that those values are not being saved into 
the database.  When the transaction completes, they're all zero!:

{noformat}
ambari=# select config_id, type_name, stack_id, version, version_tag, selected, 
unmapped from ambari.clusterconfig where type_name = 'cluster-env' order by 
version_tag;
 config_id |  type_name  | stack_id | version |     version_tag      | selected 
| unmapped
-----------+-------------+----------+---------+----------------------+----------+----------
        43 | cluster-env |        8 |       1 | version1             |        0 
|        0
        54 | cluster-env |        8 |       2 | version1507910265301 |        0 
|        0
       102 | cluster-env |        8 |       3 | version1507917459553 |        0 
|        0
       113 | cluster-env |        6 |       4 | version1507917549402 |        0 
|        0
{noformat}

The fix I currently have in place forces the db to flush writes, and it appears 
to be working.  Not 100% sure why that's the case, and why only when enabling 
Kerberos.  Testing various RU/EU/PU scenarios to make sure nothing else is 
broken.


Diffs
-----

  
ambari-server/src/main/java/org/apache/ambari/server/state/cluster/ClusterImpl.java
 6bde42c10b 


Diff: https://reviews.apache.org/r/62996/diff/1/


Testing
-------

Manual IN PROGRESS:
RU Full through Finalize (PENDING)
RU Full but Downgrade (PENDING)
EU Full through Finalize (DONE)
EU Full but Downgrade (PENDING)

EU Patch but Downgrade (PENDING)
EU Patch then Finalize (PENDING)
EU Patch then Finalize then Revert (PENDING)

Unit tests:
Verify no tests are broken with this change.


Thanks,

Nate Cole

Reply via email to