You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@ambari.apache.org by Jaimin Jetly <ja...@hortonworks.com> on 2015/06/10 04:00:28 UTC

Review Request 35283: With HBase master HA Ambari sometimes displays incorrect dashboard information

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

Review request for Ambari, Srimanth Gunturi and Yusaku Sako.


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


Repository: ambari


Description
-------

Behavior of service component endpoint for metric related fields inside ServiceComponent map was undefined in HA state of the service component. It randoml picked the host component and rolled up the metric values instead of picking up the active master host component. This mostly affected HBase service as web ui was using those serviceComponent field.

As part of this patch, ambari-web now no longer looks at any serviceComponent fields for any service. Instead web code figures out the active host component and gets the metric values from the host component.


Diffs
-----

  ambari-server/src/main/resources/common-services/YARN/2.1.0.2.0/YARN_metrics.json 92c32d8 
  ambari-web/app/assets/data/metrics/HBASE/definition.json 3374f2d 
  ambari-web/app/controllers/global/update_controller.js af7b69e 
  ambari-web/app/mappers/service_metrics_mapper.js 9c39fc6 
  ambari-web/app/messages.js 88c716d 
  ambari-web/app/models/service/hbase.js 662577f 
  ambari-web/app/models/service/mapreduce2.js 388b236 
  ambari-web/app/models/service/yarn.js eb33886 
  ambari-web/app/styles/enhanced_service_dashboard.less 0ff9c88 
  ambari-web/test/controllers/global/update_controller_test.js f55136b 
  ambari-web/test/mappers/service_mapper_test.js 1ab4cf9 

Diff: https://reviews.apache.org/r/35283/diff/


Testing
-------

tested the cluster dashboard and service summary pages with the patch on a live cluster with and without HA enabled for NameNode, ResourceManager and Hbase master.

verified that all existing unit tests passess with the patch


Thanks,

Jaimin Jetly


Re: Review Request 35283: With HBase master HA Ambari sometimes displays incorrect dashboard information

Posted by Yusaku Sako <yu...@hortonworks.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/35283/#review87751
-----------------------------------------------------------

Ship it!


Ship It!

- Yusaku Sako


On June 11, 2015, 8:31 p.m., Jaimin Jetly wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/35283/
> -----------------------------------------------------------
> 
> (Updated June 11, 2015, 8:31 p.m.)
> 
> 
> Review request for Ambari, Srimanth Gunturi and Yusaku Sako.
> 
> 
> Bugs: AMBARI-11821
>     https://issues.apache.org/jira/browse/AMBARI-11821
> 
> 
> Repository: ambari
> 
> 
> Description
> -------
> 
> Behavior of service component endpoint for metric related fields inside ServiceComponent map was undefined in HA state of the service component. It randoml picked the host component and rolled up the metric values instead of picking up the active master host component. This mostly affected HBase service as web ui was using those serviceComponent field.
> 
> As part of this patch, ambari-web now no longer looks at any serviceComponent fields for any service. Instead web code figures out the active host component and gets the metric values from the host component.
> 
> 
> Diffs
> -----
> 
>   ambari-server/src/main/resources/common-services/HBASE/0.96.0.2.0/metrics.json ae15651 
>   ambari-server/src/main/resources/common-services/YARN/2.1.0.2.0/YARN_metrics.json 92c32d8 
>   ambari-server/src/main/resources/stacks/HDP/2.3/services/HBASE/metrics.json a309ec7 
>   ambari-web/app/assets/data/metrics/HBASE/definition.json 3374f2d 
>   ambari-web/app/controllers/global/update_controller.js af7b69e 
>   ambari-web/app/mappers/service_metrics_mapper.js 9c39fc6 
>   ambari-web/app/messages.js 88c716d 
>   ambari-web/app/models/service/hbase.js 662577f 
>   ambari-web/app/models/service/mapreduce2.js 388b236 
>   ambari-web/app/models/service/yarn.js eb33886 
>   ambari-web/test/controllers/global/update_controller_test.js f55136b 
>   ambari-web/test/mappers/service_mapper_test.js 1ab4cf9 
> 
> Diff: https://reviews.apache.org/r/35283/diff/
> 
> 
> Testing
> -------
> 
> tested the cluster dashboard and service summary pages with the patch on a live cluster with and without HA enabled for NameNode, ResourceManager and Hbase master.
> 
> verified that all existing unit tests passess with the patch
> 
> 
> Thanks,
> 
> Jaimin Jetly
> 
>


Re: Review Request 35283: With HBase master HA Ambari sometimes displays incorrect dashboard information

Posted by Jaimin Jetly <ja...@hortonworks.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/35283/
-----------------------------------------------------------

(Updated June 11, 2015, 8:31 p.m.)


Review request for Ambari, Srimanth Gunturi and Yusaku Sako.


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


Repository: ambari


Description
-------

Behavior of service component endpoint for metric related fields inside ServiceComponent map was undefined in HA state of the service component. It randoml picked the host component and rolled up the metric values instead of picking up the active master host component. This mostly affected HBase service as web ui was using those serviceComponent field.

As part of this patch, ambari-web now no longer looks at any serviceComponent fields for any service. Instead web code figures out the active host component and gets the metric values from the host component.


Diffs (updated)
-----

  ambari-server/src/main/resources/common-services/HBASE/0.96.0.2.0/metrics.json ae15651 
  ambari-server/src/main/resources/common-services/YARN/2.1.0.2.0/YARN_metrics.json 92c32d8 
  ambari-server/src/main/resources/stacks/HDP/2.3/services/HBASE/metrics.json a309ec7 
  ambari-web/app/assets/data/metrics/HBASE/definition.json 3374f2d 
  ambari-web/app/controllers/global/update_controller.js af7b69e 
  ambari-web/app/mappers/service_metrics_mapper.js 9c39fc6 
  ambari-web/app/messages.js 88c716d 
  ambari-web/app/models/service/hbase.js 662577f 
  ambari-web/app/models/service/mapreduce2.js 388b236 
  ambari-web/app/models/service/yarn.js eb33886 
  ambari-web/test/controllers/global/update_controller_test.js f55136b 
  ambari-web/test/mappers/service_mapper_test.js 1ab4cf9 

Diff: https://reviews.apache.org/r/35283/diff/


Testing
-------

tested the cluster dashboard and service summary pages with the patch on a live cluster with and without HA enabled for NameNode, ResourceManager and Hbase master.

verified that all existing unit tests passess with the patch


Thanks,

Jaimin Jetly


Re: Review Request 35283: With HBase master HA Ambari sometimes displays incorrect dashboard information

Posted by Jaimin Jetly <ja...@hortonworks.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/35283/
-----------------------------------------------------------

(Updated June 11, 2015, 8:24 p.m.)


Review request for Ambari, Srimanth Gunturi and Yusaku Sako.


Changes
-------

Reopening the issue as it was causing a JS error on dashboard page.
Attaching another version of the patch. This adds missing metric being referenced in the UI in metrics.json file for HBase.


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


Repository: ambari


Description
-------

Behavior of service component endpoint for metric related fields inside ServiceComponent map was undefined in HA state of the service component. It randoml picked the host component and rolled up the metric values instead of picking up the active master host component. This mostly affected HBase service as web ui was using those serviceComponent field.

As part of this patch, ambari-web now no longer looks at any serviceComponent fields for any service. Instead web code figures out the active host component and gets the metric values from the host component.


Diffs (updated)
-----

  ambari-server/src/main/resources/common-services/HBASE/0.96.0.2.0/metrics.json ae15651 
  ambari-server/src/main/resources/common-services/YARN/2.1.0.2.0/YARN_metrics.json 92c32d8 
  ambari-server/src/main/resources/stacks/HDP/2.3/services/HBASE/metrics.json a309ec7 
  ambari-web/app/assets/data/metrics/HBASE/definition.json 3374f2d 
  ambari-web/app/controllers/global/update_controller.js af7b69e 
  ambari-web/app/mappers/service_metrics_mapper.js 9c39fc6 
  ambari-web/app/messages.js 88c716d 
  ambari-web/app/models/service/hbase.js 662577f 
  ambari-web/app/models/service/mapreduce2.js 388b236 
  ambari-web/app/models/service/yarn.js eb33886 
  ambari-web/test/controllers/global/update_controller_test.js f55136b 
  ambari-web/test/mappers/service_mapper_test.js 1ab4cf9 

Diff: https://reviews.apache.org/r/35283/diff/


Testing
-------

tested the cluster dashboard and service summary pages with the patch on a live cluster with and without HA enabled for NameNode, ResourceManager and Hbase master.

verified that all existing unit tests passess with the patch


Thanks,

Jaimin Jetly


Re: Review Request 35283: With HBase master HA Ambari sometimes displays incorrect dashboard information

Posted by Jaimin Jetly <ja...@hortonworks.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/35283/
-----------------------------------------------------------

(Updated June 10, 2015, 2:03 a.m.)


Review request for Ambari, Srimanth Gunturi and Yusaku Sako.


Changes
-------

Removed enhanced_service_dashboard.less changes in this patch which is supposed to be part of another patch


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


Repository: ambari


Description
-------

Behavior of service component endpoint for metric related fields inside ServiceComponent map was undefined in HA state of the service component. It randoml picked the host component and rolled up the metric values instead of picking up the active master host component. This mostly affected HBase service as web ui was using those serviceComponent field.

As part of this patch, ambari-web now no longer looks at any serviceComponent fields for any service. Instead web code figures out the active host component and gets the metric values from the host component.


Diffs (updated)
-----

  ambari-server/src/main/resources/common-services/YARN/2.1.0.2.0/YARN_metrics.json 92c32d8 
  ambari-web/app/assets/data/metrics/HBASE/definition.json 3374f2d 
  ambari-web/app/controllers/global/update_controller.js af7b69e 
  ambari-web/app/mappers/service_metrics_mapper.js 9c39fc6 
  ambari-web/app/messages.js 88c716d 
  ambari-web/app/models/service/hbase.js 662577f 
  ambari-web/app/models/service/mapreduce2.js 388b236 
  ambari-web/app/models/service/yarn.js eb33886 
  ambari-web/test/controllers/global/update_controller_test.js f55136b 
  ambari-web/test/mappers/service_mapper_test.js 1ab4cf9 

Diff: https://reviews.apache.org/r/35283/diff/


Testing
-------

tested the cluster dashboard and service summary pages with the patch on a live cluster with and without HA enabled for NameNode, ResourceManager and Hbase master.

verified that all existing unit tests passess with the patch


Thanks,

Jaimin Jetly