You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@ambari.apache.org by Matt <mi...@gmail.com> on 2016/02/09 19:13:20 UTC

Review Request 43365: assign_master_components.js breaks next step in certain case

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

Review request for Ambari.


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


Repository: ambari


Description
-------

There was noAction onSecondary (Cancel button). Implemented onSecondary, and modified the callback for the same.


Diffs
-----

  ambari-web/app/mixins/wizard/assign_master_components.js 7dc267e 

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


Testing
-------

All unit tests passed:
  10364 tests complete (9 seconds)
  121 tests pending

[INFO] ------------------------------------------------------------------------
[INFO] BUILD SUCCESS
[INFO] ------------------------------------------------------------------------
[INFO] Total time: 01:15 min
[INFO] Finished at: 2016-02-09T10:11:05-08:00
[INFO] Final Memory: 14M/429M
[INFO] ------------------------------------------------------------------------


Thanks,

Matt


Re: Review Request 43365: assign_master_components.js breaks next step in certain case

Posted by Matt <mi...@gmail.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/43365/
-----------------------------------------------------------

(Updated Feb. 9, 2016, 10:45 a.m.)


Review request for Ambari.


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


Repository: ambari


Description
-------

There was noAction onSecondary (Cancel button). Implemented onSecondary, and modified the callback for the same.


Diffs (updated)
-----

  ambari-web/app/mixins/wizard/assign_master_components.js 7dc267e 

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


Testing (updated)
-------

All unit tests passed:
  10384 tests complete (9 seconds)
  121 tests pending

[INFO] ------------------------------------------------------------------------
[INFO] BUILD SUCCESS
[INFO] ------------------------------------------------------------------------
[INFO] Total time: 43.184 s
[INFO] Finished at: 2016-02-09T10:43:13-08:00
[INFO] Final Memory: 15M/429M
[INFO] ------------------------------------------------------------------------


Thanks,

Matt