You are viewing a plain text version of this content. The canonical link for it is here.
Posted to reviews@ambari.apache.org by Sangeeta Ravindran <sa...@gmail.com> on 2017/04/20 19:53:59 UTC
Re: Review Request 58590: Provide an optional attribute for custom
commands to allow using command name in operations window title
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/58590/
-----------------------------------------------------------
(Updated April 20, 2017, 7:53 p.m.)
Review request for Ambari, Di Li and Tim Thorpe.
Summary (updated)
-----------------
Provide an optional attribute for custom commands to allow using command name in operations window title
Bugs: AMBARI-20616
https://issues.apache.org/jira/browse/AMBARI-20616
Repository: ambari
Description
-------
Some services in our stack have custom commands where the name is a combination of an operation and a component name.
For e.g. <name>Remove_Logical_myservice_mycomponent</name>
When the custom command is being executed, the operations window title is displayed as "Remove Logical myservice mycomponent mycomponent"
It would be useful to provide an optional attribute e.g. useNameInOps for custom commands to allow using the command name itself as the title displayed in the operations window.
<customCommand>
<name>Remove_Logical_myservice_mycomponent</name>
<useNameInOps>true</useNameInOps>
<commandScript>
<script>scripts/mycomponent.py</script>
<scriptType>PYTHON</scriptType>
<timeout>10800</timeout>
</commandScript>
</customCommand>
Diffs
-----
ambari-server/src/main/java/org/apache/ambari/server/actionmanager/HostRoleCommand.java 87a6edf
ambari-server/src/main/java/org/apache/ambari/server/controller/AmbariCustomCommandExecutionHelper.java 17bc718
ambari-server/src/main/java/org/apache/ambari/server/controller/internal/TaskResourceProvider.java 20b0417
ambari-server/src/main/java/org/apache/ambari/server/orm/entities/HostRoleCommandEntity.java 86feceb
ambari-server/src/main/java/org/apache/ambari/server/state/CustomCommandDefinition.java 280a59b
ambari-server/src/main/java/org/apache/ambari/server/upgrade/UpgradeCatalog300.java 633d837
ambari-server/src/main/resources/Ambari-DDL-Derby-CREATE.sql fd49b94
ambari-server/src/main/resources/Ambari-DDL-MySQL-CREATE.sql 9fc3209
ambari-server/src/main/resources/Ambari-DDL-Oracle-CREATE.sql 310208d
ambari-server/src/main/resources/Ambari-DDL-Postgres-CREATE.sql c052104
ambari-server/src/main/resources/Ambari-DDL-SQLAnywhere-CREATE.sql 5a58ef8
ambari-server/src/main/resources/Ambari-DDL-SQLServer-CREATE.sql 8c7a31d
ambari-server/src/main/resources/properties.json c2545fe
ambari-server/src/test/java/org/apache/ambari/server/controller/internal/TaskResourceProviderTest.java 8a52b8a
ambari-server/src/test/java/org/apache/ambari/server/orm/entities/HostRoleCommandEntityTest.java PRE-CREATION
ambari-server/src/test/java/org/apache/ambari/server/upgrade/UpgradeCatalog300Test.java fec041c
ambari-web/app/controllers/wizard/step9_controller.js 4697dad
ambari-web/app/utils/ajax/ajax.js 2b5e52a
ambari-web/app/utils/helper.js 03a2e82
ambari-web/app/utils/host_progress_popup.js c615cae
ambari-web/app/views/wizard/step9/hostLogPopupBody_view.js d983252
ambari-web/test/utils/helper_test.js 4b9ec36
Diff: https://reviews.apache.org/r/58590/diff/1/
Testing
-------
Manual Testing.
Added new test cases.
Thanks,
Sangeeta Ravindran
Re: Review Request 58590: Provide an optional attribute for custom
commands to allow using command name in operations window title
Posted by Sangeeta Ravindran <sa...@gmail.com>.
> On April 20, 2017, 8:06 p.m., Tim Thorpe wrote:
> > Has upgrade been tested?
Hello Tim,
Yes I upgraded an ambari 2.4 cluster and verified that the new column gets added to the host_role_command table.
- Sangeeta
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/58590/#review172542
-----------------------------------------------------------
On April 24, 2017, 11:33 p.m., Sangeeta Ravindran wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/58590/
> -----------------------------------------------------------
>
> (Updated April 24, 2017, 11:33 p.m.)
>
>
> Review request for Ambari, Di Li and Tim Thorpe.
>
>
> Bugs: AMBARI-20616
> https://issues.apache.org/jira/browse/AMBARI-20616
>
>
> Repository: ambari
>
>
> Description
> -------
>
> Some services in our stack have custom commands where the name is a combination of an operation and a component name.
>
> For e.g. <name>Remove_Logical_myservice_mycomponent</name>
>
> When the custom command is being executed, the operations window title is displayed as "Remove Logical myservice mycomponent mycomponent"
>
> It would be useful to provide an optional attribute e.g. opsDisplayName for custom commands to allow specifying the string for the title displayed in the operations window.
>
> <customCommand>
> <name>Remove_Logical_myservice_mycomponent</name>
> <opsDisplayName>true</opsDisplayName>
> <commandScript>
> <script>scripts/mycomponent.py</script>
> <scriptType>PYTHON</scriptType>
> <timeout>10800</timeout>
> </commandScript>
> </customCommand>
>
>
> Diffs
> -----
>
> ambari-server/src/main/java/org/apache/ambari/server/actionmanager/HostRoleCommand.java 87a6edf
> ambari-server/src/main/java/org/apache/ambari/server/controller/AmbariCustomCommandExecutionHelper.java 17bc718
> ambari-server/src/main/java/org/apache/ambari/server/controller/internal/TaskResourceProvider.java 20b0417
> ambari-server/src/main/java/org/apache/ambari/server/orm/entities/HostRoleCommandEntity.java 86feceb
> ambari-server/src/main/java/org/apache/ambari/server/state/CustomCommandDefinition.java 280a59b
> ambari-server/src/main/java/org/apache/ambari/server/upgrade/UpgradeCatalog300.java 633d837
> ambari-server/src/main/resources/Ambari-DDL-Derby-CREATE.sql fd49b94
> ambari-server/src/main/resources/Ambari-DDL-MySQL-CREATE.sql 9fc3209
> ambari-server/src/main/resources/Ambari-DDL-Oracle-CREATE.sql 310208d
> ambari-server/src/main/resources/Ambari-DDL-Postgres-CREATE.sql c052104
> ambari-server/src/main/resources/Ambari-DDL-SQLAnywhere-CREATE.sql 5a58ef8
> ambari-server/src/main/resources/Ambari-DDL-SQLServer-CREATE.sql 8c7a31d
> ambari-server/src/main/resources/properties.json c2545fe
> ambari-server/src/test/java/org/apache/ambari/server/controller/internal/TaskResourceProviderTest.java 8a52b8a
> ambari-server/src/test/java/org/apache/ambari/server/orm/entities/HostRoleCommandEntityTest.java PRE-CREATION
> ambari-server/src/test/java/org/apache/ambari/server/upgrade/UpgradeCatalog300Test.java fec041c
> ambari-web/app/controllers/wizard/step9_controller.js 4697dad
> ambari-web/app/utils/ajax/ajax.js 2b5e52a
> ambari-web/app/utils/helper.js 03a2e82
> ambari-web/app/utils/host_progress_popup.js c615cae
> ambari-web/app/views/wizard/step9/hostLogPopupBody_view.js d983252
> ambari-web/test/utils/helper_test.js 4b9ec36
>
>
> Diff: https://reviews.apache.org/r/58590/diff/2/
>
>
> Testing
> -------
>
> Manual Testing.
> Added new test cases.
>
>
> Thanks,
>
> Sangeeta Ravindran
>
>
Re: Review Request 58590: Provide an optional attribute for custom
commands to allow using command name in operations window title
Posted by Tim Thorpe <tt...@ca.ibm.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/58590/#review172542
-----------------------------------------------------------
Has upgrade been tested?
- Tim Thorpe
On April 20, 2017, 7:53 p.m., Sangeeta Ravindran wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/58590/
> -----------------------------------------------------------
>
> (Updated April 20, 2017, 7:53 p.m.)
>
>
> Review request for Ambari, Di Li and Tim Thorpe.
>
>
> Bugs: AMBARI-20616
> https://issues.apache.org/jira/browse/AMBARI-20616
>
>
> Repository: ambari
>
>
> Description
> -------
>
> Some services in our stack have custom commands where the name is a combination of an operation and a component name.
>
> For e.g. <name>Remove_Logical_myservice_mycomponent</name>
>
> When the custom command is being executed, the operations window title is displayed as "Remove Logical myservice mycomponent mycomponent"
>
> It would be useful to provide an optional attribute e.g. useNameInOps for custom commands to allow using the command name itself as the title displayed in the operations window.
>
> <customCommand>
> <name>Remove_Logical_myservice_mycomponent</name>
> <useNameInOps>true</useNameInOps>
> <commandScript>
> <script>scripts/mycomponent.py</script>
> <scriptType>PYTHON</scriptType>
> <timeout>10800</timeout>
> </commandScript>
> </customCommand>
>
>
> Diffs
> -----
>
> ambari-server/src/main/java/org/apache/ambari/server/actionmanager/HostRoleCommand.java 87a6edf
> ambari-server/src/main/java/org/apache/ambari/server/controller/AmbariCustomCommandExecutionHelper.java 17bc718
> ambari-server/src/main/java/org/apache/ambari/server/controller/internal/TaskResourceProvider.java 20b0417
> ambari-server/src/main/java/org/apache/ambari/server/orm/entities/HostRoleCommandEntity.java 86feceb
> ambari-server/src/main/java/org/apache/ambari/server/state/CustomCommandDefinition.java 280a59b
> ambari-server/src/main/java/org/apache/ambari/server/upgrade/UpgradeCatalog300.java 633d837
> ambari-server/src/main/resources/Ambari-DDL-Derby-CREATE.sql fd49b94
> ambari-server/src/main/resources/Ambari-DDL-MySQL-CREATE.sql 9fc3209
> ambari-server/src/main/resources/Ambari-DDL-Oracle-CREATE.sql 310208d
> ambari-server/src/main/resources/Ambari-DDL-Postgres-CREATE.sql c052104
> ambari-server/src/main/resources/Ambari-DDL-SQLAnywhere-CREATE.sql 5a58ef8
> ambari-server/src/main/resources/Ambari-DDL-SQLServer-CREATE.sql 8c7a31d
> ambari-server/src/main/resources/properties.json c2545fe
> ambari-server/src/test/java/org/apache/ambari/server/controller/internal/TaskResourceProviderTest.java 8a52b8a
> ambari-server/src/test/java/org/apache/ambari/server/orm/entities/HostRoleCommandEntityTest.java PRE-CREATION
> ambari-server/src/test/java/org/apache/ambari/server/upgrade/UpgradeCatalog300Test.java fec041c
> ambari-web/app/controllers/wizard/step9_controller.js 4697dad
> ambari-web/app/utils/ajax/ajax.js 2b5e52a
> ambari-web/app/utils/helper.js 03a2e82
> ambari-web/app/utils/host_progress_popup.js c615cae
> ambari-web/app/views/wizard/step9/hostLogPopupBody_view.js d983252
> ambari-web/test/utils/helper_test.js 4b9ec36
>
>
> Diff: https://reviews.apache.org/r/58590/diff/1/
>
>
> Testing
> -------
>
> Manual Testing.
> Added new test cases.
>
>
> Thanks,
>
> Sangeeta Ravindran
>
>
Re: Review Request 58590: Provide an optional attribute for custom
commands to allow using command name in operations window title
Posted by Alejandro Fernandez <af...@hortonworks.com>.
> On April 20, 2017, 9:32 p.m., Alejandro Fernandez wrote:
> > ambari-server/src/main/java/org/apache/ambari/server/actionmanager/HostRoleCommand.java
> > Lines 72 (patched)
> > <https://reviews.apache.org/r/58590/diff/1/?file=1695868#file1695868line72>
> >
> > I would suggest to actually have a field to overwrite the display name to use in order to make it more flexible.
> >
> > The reason being that the custom command name is used to find the function in python, which is not necessarily very human readable.
Sangeeta, what do you think about this? I think it will be more flexible in the long run.
- Alejandro
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/58590/#review172549
-----------------------------------------------------------
On April 20, 2017, 7:53 p.m., Sangeeta Ravindran wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/58590/
> -----------------------------------------------------------
>
> (Updated April 20, 2017, 7:53 p.m.)
>
>
> Review request for Ambari, Di Li and Tim Thorpe.
>
>
> Bugs: AMBARI-20616
> https://issues.apache.org/jira/browse/AMBARI-20616
>
>
> Repository: ambari
>
>
> Description
> -------
>
> Some services in our stack have custom commands where the name is a combination of an operation and a component name.
>
> For e.g. <name>Remove_Logical_myservice_mycomponent</name>
>
> When the custom command is being executed, the operations window title is displayed as "Remove Logical myservice mycomponent mycomponent"
>
> It would be useful to provide an optional attribute e.g. useNameInOps for custom commands to allow using the command name itself as the title displayed in the operations window.
>
> <customCommand>
> <name>Remove_Logical_myservice_mycomponent</name>
> <useNameInOps>true</useNameInOps>
> <commandScript>
> <script>scripts/mycomponent.py</script>
> <scriptType>PYTHON</scriptType>
> <timeout>10800</timeout>
> </commandScript>
> </customCommand>
>
>
> Diffs
> -----
>
> ambari-server/src/main/java/org/apache/ambari/server/actionmanager/HostRoleCommand.java 87a6edf
> ambari-server/src/main/java/org/apache/ambari/server/controller/AmbariCustomCommandExecutionHelper.java 17bc718
> ambari-server/src/main/java/org/apache/ambari/server/controller/internal/TaskResourceProvider.java 20b0417
> ambari-server/src/main/java/org/apache/ambari/server/orm/entities/HostRoleCommandEntity.java 86feceb
> ambari-server/src/main/java/org/apache/ambari/server/state/CustomCommandDefinition.java 280a59b
> ambari-server/src/main/java/org/apache/ambari/server/upgrade/UpgradeCatalog300.java 633d837
> ambari-server/src/main/resources/Ambari-DDL-Derby-CREATE.sql fd49b94
> ambari-server/src/main/resources/Ambari-DDL-MySQL-CREATE.sql 9fc3209
> ambari-server/src/main/resources/Ambari-DDL-Oracle-CREATE.sql 310208d
> ambari-server/src/main/resources/Ambari-DDL-Postgres-CREATE.sql c052104
> ambari-server/src/main/resources/Ambari-DDL-SQLAnywhere-CREATE.sql 5a58ef8
> ambari-server/src/main/resources/Ambari-DDL-SQLServer-CREATE.sql 8c7a31d
> ambari-server/src/main/resources/properties.json c2545fe
> ambari-server/src/test/java/org/apache/ambari/server/controller/internal/TaskResourceProviderTest.java 8a52b8a
> ambari-server/src/test/java/org/apache/ambari/server/orm/entities/HostRoleCommandEntityTest.java PRE-CREATION
> ambari-server/src/test/java/org/apache/ambari/server/upgrade/UpgradeCatalog300Test.java fec041c
> ambari-web/app/controllers/wizard/step9_controller.js 4697dad
> ambari-web/app/utils/ajax/ajax.js 2b5e52a
> ambari-web/app/utils/helper.js 03a2e82
> ambari-web/app/utils/host_progress_popup.js c615cae
> ambari-web/app/views/wizard/step9/hostLogPopupBody_view.js d983252
> ambari-web/test/utils/helper_test.js 4b9ec36
>
>
> Diff: https://reviews.apache.org/r/58590/diff/1/
>
>
> Testing
> -------
>
> Manual Testing.
> Added new test cases.
>
>
> Thanks,
>
> Sangeeta Ravindran
>
>
Re: Review Request 58590: Provide an optional attribute for custom
commands to allow using command name in operations window title
Posted by Sangeeta Ravindran <sa...@gmail.com>.
> On April 20, 2017, 9:32 p.m., Alejandro Fernandez wrote:
> > ambari-server/src/main/java/org/apache/ambari/server/actionmanager/HostRoleCommand.java
> > Lines 72 (patched)
> > <https://reviews.apache.org/r/58590/diff/1/?file=1695868#file1695868line72>
> >
> > I would suggest to actually have a field to overwrite the display name to use in order to make it more flexible.
> >
> > The reason being that the custom command name is used to find the function in python, which is not necessarily very human readable.
>
> Alejandro Fernandez wrote:
> Sangeeta, what do you think about this? I think it will be more flexible in the long run.
>
> Sangeeta Ravindran wrote:
> Hi Alejandro,
>
> Yes I think, it would be better to have a field to allow custom commands to overwrite the display name. I have created an updated patch based on your suggestion and will attach it shortly.
I've added the updated patch.
- Sangeeta
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/58590/#review172549
-----------------------------------------------------------
On April 24, 2017, 11:33 p.m., Sangeeta Ravindran wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/58590/
> -----------------------------------------------------------
>
> (Updated April 24, 2017, 11:33 p.m.)
>
>
> Review request for Ambari, Di Li and Tim Thorpe.
>
>
> Bugs: AMBARI-20616
> https://issues.apache.org/jira/browse/AMBARI-20616
>
>
> Repository: ambari
>
>
> Description
> -------
>
> Some services in our stack have custom commands where the name is a combination of an operation and a component name.
>
> For e.g. <name>Remove_Logical_myservice_mycomponent</name>
>
> When the custom command is being executed, the operations window title is displayed as "Remove Logical myservice mycomponent mycomponent"
>
> It would be useful to provide an optional attribute e.g. opsDisplayName for custom commands to allow specifying the string for the title displayed in the operations window.
>
> <customCommand>
> <name>Remove_Logical_myservice_mycomponent</name>
> <opsDisplayName>true</opsDisplayName>
> <commandScript>
> <script>scripts/mycomponent.py</script>
> <scriptType>PYTHON</scriptType>
> <timeout>10800</timeout>
> </commandScript>
> </customCommand>
>
>
> Diffs
> -----
>
> ambari-server/src/main/java/org/apache/ambari/server/actionmanager/HostRoleCommand.java 87a6edf
> ambari-server/src/main/java/org/apache/ambari/server/controller/AmbariCustomCommandExecutionHelper.java 17bc718
> ambari-server/src/main/java/org/apache/ambari/server/controller/internal/TaskResourceProvider.java 20b0417
> ambari-server/src/main/java/org/apache/ambari/server/orm/entities/HostRoleCommandEntity.java 86feceb
> ambari-server/src/main/java/org/apache/ambari/server/state/CustomCommandDefinition.java 280a59b
> ambari-server/src/main/java/org/apache/ambari/server/upgrade/UpgradeCatalog300.java 633d837
> ambari-server/src/main/resources/Ambari-DDL-Derby-CREATE.sql fd49b94
> ambari-server/src/main/resources/Ambari-DDL-MySQL-CREATE.sql 9fc3209
> ambari-server/src/main/resources/Ambari-DDL-Oracle-CREATE.sql 310208d
> ambari-server/src/main/resources/Ambari-DDL-Postgres-CREATE.sql c052104
> ambari-server/src/main/resources/Ambari-DDL-SQLAnywhere-CREATE.sql 5a58ef8
> ambari-server/src/main/resources/Ambari-DDL-SQLServer-CREATE.sql 8c7a31d
> ambari-server/src/main/resources/properties.json c2545fe
> ambari-server/src/test/java/org/apache/ambari/server/controller/internal/TaskResourceProviderTest.java 8a52b8a
> ambari-server/src/test/java/org/apache/ambari/server/orm/entities/HostRoleCommandEntityTest.java PRE-CREATION
> ambari-server/src/test/java/org/apache/ambari/server/upgrade/UpgradeCatalog300Test.java fec041c
> ambari-web/app/controllers/wizard/step9_controller.js 4697dad
> ambari-web/app/utils/ajax/ajax.js 2b5e52a
> ambari-web/app/utils/helper.js 03a2e82
> ambari-web/app/utils/host_progress_popup.js c615cae
> ambari-web/app/views/wizard/step9/hostLogPopupBody_view.js d983252
> ambari-web/test/utils/helper_test.js 4b9ec36
>
>
> Diff: https://reviews.apache.org/r/58590/diff/2/
>
>
> Testing
> -------
>
> Manual Testing.
> Added new test cases.
>
>
> Thanks,
>
> Sangeeta Ravindran
>
>
Re: Review Request 58590: Provide an optional attribute for custom
commands to allow using command name in operations window title
Posted by Sangeeta Ravindran <sa...@gmail.com>.
> On April 20, 2017, 9:32 p.m., Alejandro Fernandez wrote:
> > ambari-server/src/main/java/org/apache/ambari/server/actionmanager/HostRoleCommand.java
> > Lines 72 (patched)
> > <https://reviews.apache.org/r/58590/diff/1/?file=1695868#file1695868line72>
> >
> > I would suggest to actually have a field to overwrite the display name to use in order to make it more flexible.
> >
> > The reason being that the custom command name is used to find the function in python, which is not necessarily very human readable.
>
> Alejandro Fernandez wrote:
> Sangeeta, what do you think about this? I think it will be more flexible in the long run.
Hi Alejandro,
Yes I think, it would be better to have a field to allow custom commands to overwrite the display name. I have created an updated patch based on your suggestion and will attach it shortly.
- Sangeeta
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/58590/#review172549
-----------------------------------------------------------
On April 20, 2017, 7:53 p.m., Sangeeta Ravindran wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/58590/
> -----------------------------------------------------------
>
> (Updated April 20, 2017, 7:53 p.m.)
>
>
> Review request for Ambari, Di Li and Tim Thorpe.
>
>
> Bugs: AMBARI-20616
> https://issues.apache.org/jira/browse/AMBARI-20616
>
>
> Repository: ambari
>
>
> Description
> -------
>
> Some services in our stack have custom commands where the name is a combination of an operation and a component name.
>
> For e.g. <name>Remove_Logical_myservice_mycomponent</name>
>
> When the custom command is being executed, the operations window title is displayed as "Remove Logical myservice mycomponent mycomponent"
>
> It would be useful to provide an optional attribute e.g. useNameInOps for custom commands to allow using the command name itself as the title displayed in the operations window.
>
> <customCommand>
> <name>Remove_Logical_myservice_mycomponent</name>
> <useNameInOps>true</useNameInOps>
> <commandScript>
> <script>scripts/mycomponent.py</script>
> <scriptType>PYTHON</scriptType>
> <timeout>10800</timeout>
> </commandScript>
> </customCommand>
>
>
> Diffs
> -----
>
> ambari-server/src/main/java/org/apache/ambari/server/actionmanager/HostRoleCommand.java 87a6edf
> ambari-server/src/main/java/org/apache/ambari/server/controller/AmbariCustomCommandExecutionHelper.java 17bc718
> ambari-server/src/main/java/org/apache/ambari/server/controller/internal/TaskResourceProvider.java 20b0417
> ambari-server/src/main/java/org/apache/ambari/server/orm/entities/HostRoleCommandEntity.java 86feceb
> ambari-server/src/main/java/org/apache/ambari/server/state/CustomCommandDefinition.java 280a59b
> ambari-server/src/main/java/org/apache/ambari/server/upgrade/UpgradeCatalog300.java 633d837
> ambari-server/src/main/resources/Ambari-DDL-Derby-CREATE.sql fd49b94
> ambari-server/src/main/resources/Ambari-DDL-MySQL-CREATE.sql 9fc3209
> ambari-server/src/main/resources/Ambari-DDL-Oracle-CREATE.sql 310208d
> ambari-server/src/main/resources/Ambari-DDL-Postgres-CREATE.sql c052104
> ambari-server/src/main/resources/Ambari-DDL-SQLAnywhere-CREATE.sql 5a58ef8
> ambari-server/src/main/resources/Ambari-DDL-SQLServer-CREATE.sql 8c7a31d
> ambari-server/src/main/resources/properties.json c2545fe
> ambari-server/src/test/java/org/apache/ambari/server/controller/internal/TaskResourceProviderTest.java 8a52b8a
> ambari-server/src/test/java/org/apache/ambari/server/orm/entities/HostRoleCommandEntityTest.java PRE-CREATION
> ambari-server/src/test/java/org/apache/ambari/server/upgrade/UpgradeCatalog300Test.java fec041c
> ambari-web/app/controllers/wizard/step9_controller.js 4697dad
> ambari-web/app/utils/ajax/ajax.js 2b5e52a
> ambari-web/app/utils/helper.js 03a2e82
> ambari-web/app/utils/host_progress_popup.js c615cae
> ambari-web/app/views/wizard/step9/hostLogPopupBody_view.js d983252
> ambari-web/test/utils/helper_test.js 4b9ec36
>
>
> Diff: https://reviews.apache.org/r/58590/diff/1/
>
>
> Testing
> -------
>
> Manual Testing.
> Added new test cases.
>
>
> Thanks,
>
> Sangeeta Ravindran
>
>
Re: Review Request 58590: Provide an optional attribute for custom
commands to allow using command name in operations window title
Posted by Alejandro Fernandez <af...@hortonworks.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/58590/#review172549
-----------------------------------------------------------
ambari-server/src/main/java/org/apache/ambari/server/actionmanager/HostRoleCommand.java
Lines 72 (patched)
<https://reviews.apache.org/r/58590/#comment245651>
I would suggest to actually have a field to overwrite the display name to use in order to make it more flexible.
The reason being that the custom command name is used to find the function in python, which is not necessarily very human readable.
- Alejandro Fernandez
On April 20, 2017, 7:53 p.m., Sangeeta Ravindran wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/58590/
> -----------------------------------------------------------
>
> (Updated April 20, 2017, 7:53 p.m.)
>
>
> Review request for Ambari, Di Li and Tim Thorpe.
>
>
> Bugs: AMBARI-20616
> https://issues.apache.org/jira/browse/AMBARI-20616
>
>
> Repository: ambari
>
>
> Description
> -------
>
> Some services in our stack have custom commands where the name is a combination of an operation and a component name.
>
> For e.g. <name>Remove_Logical_myservice_mycomponent</name>
>
> When the custom command is being executed, the operations window title is displayed as "Remove Logical myservice mycomponent mycomponent"
>
> It would be useful to provide an optional attribute e.g. useNameInOps for custom commands to allow using the command name itself as the title displayed in the operations window.
>
> <customCommand>
> <name>Remove_Logical_myservice_mycomponent</name>
> <useNameInOps>true</useNameInOps>
> <commandScript>
> <script>scripts/mycomponent.py</script>
> <scriptType>PYTHON</scriptType>
> <timeout>10800</timeout>
> </commandScript>
> </customCommand>
>
>
> Diffs
> -----
>
> ambari-server/src/main/java/org/apache/ambari/server/actionmanager/HostRoleCommand.java 87a6edf
> ambari-server/src/main/java/org/apache/ambari/server/controller/AmbariCustomCommandExecutionHelper.java 17bc718
> ambari-server/src/main/java/org/apache/ambari/server/controller/internal/TaskResourceProvider.java 20b0417
> ambari-server/src/main/java/org/apache/ambari/server/orm/entities/HostRoleCommandEntity.java 86feceb
> ambari-server/src/main/java/org/apache/ambari/server/state/CustomCommandDefinition.java 280a59b
> ambari-server/src/main/java/org/apache/ambari/server/upgrade/UpgradeCatalog300.java 633d837
> ambari-server/src/main/resources/Ambari-DDL-Derby-CREATE.sql fd49b94
> ambari-server/src/main/resources/Ambari-DDL-MySQL-CREATE.sql 9fc3209
> ambari-server/src/main/resources/Ambari-DDL-Oracle-CREATE.sql 310208d
> ambari-server/src/main/resources/Ambari-DDL-Postgres-CREATE.sql c052104
> ambari-server/src/main/resources/Ambari-DDL-SQLAnywhere-CREATE.sql 5a58ef8
> ambari-server/src/main/resources/Ambari-DDL-SQLServer-CREATE.sql 8c7a31d
> ambari-server/src/main/resources/properties.json c2545fe
> ambari-server/src/test/java/org/apache/ambari/server/controller/internal/TaskResourceProviderTest.java 8a52b8a
> ambari-server/src/test/java/org/apache/ambari/server/orm/entities/HostRoleCommandEntityTest.java PRE-CREATION
> ambari-server/src/test/java/org/apache/ambari/server/upgrade/UpgradeCatalog300Test.java fec041c
> ambari-web/app/controllers/wizard/step9_controller.js 4697dad
> ambari-web/app/utils/ajax/ajax.js 2b5e52a
> ambari-web/app/utils/helper.js 03a2e82
> ambari-web/app/utils/host_progress_popup.js c615cae
> ambari-web/app/views/wizard/step9/hostLogPopupBody_view.js d983252
> ambari-web/test/utils/helper_test.js 4b9ec36
>
>
> Diff: https://reviews.apache.org/r/58590/diff/1/
>
>
> Testing
> -------
>
> Manual Testing.
> Added new test cases.
>
>
> Thanks,
>
> Sangeeta Ravindran
>
>
Re: Review Request 58590: Provide an optional attribute for custom
commands to specify title used in operations window title
Posted by Alejandro Fernandez <af...@hortonworks.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/58590/#review172882
-----------------------------------------------------------
Fix it, then Ship it!
ambari-server/src/main/resources/Ambari-DDL-Oracle-CREATE.sql
Lines 387 (patched)
<https://reviews.apache.org/r/58590/#comment245908>
VARCHAR2
- Alejandro Fernandez
On April 24, 2017, 11:35 p.m., Sangeeta Ravindran wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/58590/
> -----------------------------------------------------------
>
> (Updated April 24, 2017, 11:35 p.m.)
>
>
> Review request for Ambari, Di Li and Tim Thorpe.
>
>
> Bugs: AMBARI-20616
> https://issues.apache.org/jira/browse/AMBARI-20616
>
>
> Repository: ambari
>
>
> Description
> -------
>
> Some services in our stack have custom commands where the name is a combination of an operation and a component name.
>
> For e.g. <name>Remove_Logical_myservice_mycomponent</name>
>
> When the custom command is being executed, the operations window title is displayed as "Remove Logical myservice mycomponent mycomponent"
>
> It would be useful to provide an optional attribute e.g. opsDisplayName for custom commands to allow specifying the string for the title displayed in the operations window.
>
> <customCommand>
> <name>Remove_Logical_myservice_mycomponent</name>
> <opsDisplayName>true</opsDisplayName>
> <commandScript>
> <script>scripts/mycomponent.py</script>
> <scriptType>PYTHON</scriptType>
> <timeout>10800</timeout>
> </commandScript>
> </customCommand>
>
>
> Diffs
> -----
>
> ambari-server/src/main/java/org/apache/ambari/server/actionmanager/HostRoleCommand.java 87a6edf
> ambari-server/src/main/java/org/apache/ambari/server/controller/AmbariCustomCommandExecutionHelper.java 17bc718
> ambari-server/src/main/java/org/apache/ambari/server/controller/internal/TaskResourceProvider.java 20b0417
> ambari-server/src/main/java/org/apache/ambari/server/orm/entities/HostRoleCommandEntity.java 86feceb
> ambari-server/src/main/java/org/apache/ambari/server/state/CustomCommandDefinition.java 280a59b
> ambari-server/src/main/java/org/apache/ambari/server/upgrade/UpgradeCatalog300.java 633d837
> ambari-server/src/main/resources/Ambari-DDL-Derby-CREATE.sql fd49b94
> ambari-server/src/main/resources/Ambari-DDL-MySQL-CREATE.sql 9fc3209
> ambari-server/src/main/resources/Ambari-DDL-Oracle-CREATE.sql 310208d
> ambari-server/src/main/resources/Ambari-DDL-Postgres-CREATE.sql c052104
> ambari-server/src/main/resources/Ambari-DDL-SQLAnywhere-CREATE.sql 5a58ef8
> ambari-server/src/main/resources/Ambari-DDL-SQLServer-CREATE.sql 8c7a31d
> ambari-server/src/main/resources/properties.json c2545fe
> ambari-server/src/test/java/org/apache/ambari/server/controller/internal/TaskResourceProviderTest.java 8a52b8a
> ambari-server/src/test/java/org/apache/ambari/server/orm/entities/HostRoleCommandEntityTest.java PRE-CREATION
> ambari-server/src/test/java/org/apache/ambari/server/upgrade/UpgradeCatalog300Test.java fec041c
> ambari-web/app/controllers/wizard/step9_controller.js 4697dad
> ambari-web/app/utils/ajax/ajax.js 2b5e52a
> ambari-web/app/utils/helper.js 03a2e82
> ambari-web/app/utils/host_progress_popup.js c615cae
> ambari-web/app/views/wizard/step9/hostLogPopupBody_view.js d983252
> ambari-web/test/utils/helper_test.js 4b9ec36
>
>
> Diff: https://reviews.apache.org/r/58590/diff/2/
>
>
> Testing
> -------
>
> Manual Testing.
> Added new test cases.
>
>
> Thanks,
>
> Sangeeta Ravindran
>
>
Re: Review Request 58590: Provide an optional attribute for custom
commands to specify title used in operations window title
Posted by Alejandro Fernandez <af...@hortonworks.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/58590/#review172964
-----------------------------------------------------------
Ship it!
Ship It!
- Alejandro Fernandez
On April 25, 2017, 4:51 a.m., Sangeeta Ravindran wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/58590/
> -----------------------------------------------------------
>
> (Updated April 25, 2017, 4:51 a.m.)
>
>
> Review request for Ambari, Di Li and Tim Thorpe.
>
>
> Bugs: AMBARI-20616
> https://issues.apache.org/jira/browse/AMBARI-20616
>
>
> Repository: ambari
>
>
> Description
> -------
>
> Some services in our stack have custom commands where the name is a combination of an operation and a component name.
>
> For e.g. <name>Remove_Logical_myservice_mycomponent</name>
>
> When the custom command is being executed, the operations window title is displayed as "Remove Logical myservice mycomponent mycomponent"
>
> It would be useful to provide an optional attribute e.g. opsDisplayName for custom commands to allow specifying the string for the title displayed in the operations window.
>
> <customCommand>
> <name>Remove_Logical_myservice_mycomponent</name>
> <opsDisplayName>true</opsDisplayName>
> <commandScript>
> <script>scripts/mycomponent.py</script>
> <scriptType>PYTHON</scriptType>
> <timeout>10800</timeout>
> </commandScript>
> </customCommand>
>
>
> Diffs
> -----
>
> ambari-server/src/main/java/org/apache/ambari/server/actionmanager/HostRoleCommand.java 87a6edf
> ambari-server/src/main/java/org/apache/ambari/server/controller/AmbariCustomCommandExecutionHelper.java 17bc718
> ambari-server/src/main/java/org/apache/ambari/server/controller/internal/TaskResourceProvider.java 20b0417
> ambari-server/src/main/java/org/apache/ambari/server/orm/entities/HostRoleCommandEntity.java 86feceb
> ambari-server/src/main/java/org/apache/ambari/server/state/CustomCommandDefinition.java 280a59b
> ambari-server/src/main/java/org/apache/ambari/server/upgrade/UpgradeCatalog300.java 633d837
> ambari-server/src/main/resources/Ambari-DDL-Derby-CREATE.sql fd49b94
> ambari-server/src/main/resources/Ambari-DDL-MySQL-CREATE.sql 9fc3209
> ambari-server/src/main/resources/Ambari-DDL-Oracle-CREATE.sql 310208d
> ambari-server/src/main/resources/Ambari-DDL-Postgres-CREATE.sql c052104
> ambari-server/src/main/resources/Ambari-DDL-SQLAnywhere-CREATE.sql 5a58ef8
> ambari-server/src/main/resources/Ambari-DDL-SQLServer-CREATE.sql 8c7a31d
> ambari-server/src/main/resources/properties.json c2545fe
> ambari-server/src/test/java/org/apache/ambari/server/controller/internal/TaskResourceProviderTest.java 8a52b8a
> ambari-server/src/test/java/org/apache/ambari/server/orm/entities/HostRoleCommandEntityTest.java PRE-CREATION
> ambari-server/src/test/java/org/apache/ambari/server/upgrade/UpgradeCatalog300Test.java fec041c
> ambari-web/app/controllers/wizard/step9_controller.js 4697dad
> ambari-web/app/utils/ajax/ajax.js 5d7108f
> ambari-web/app/utils/helper.js 03a2e82
> ambari-web/app/utils/host_progress_popup.js c615cae
> ambari-web/app/views/wizard/step9/hostLogPopupBody_view.js d983252
> ambari-web/test/utils/helper_test.js 4b9ec36
>
>
> Diff: https://reviews.apache.org/r/58590/diff/3/
>
>
> Testing
> -------
>
> Manual Testing.
> Added new test cases.
>
>
> Thanks,
>
> Sangeeta Ravindran
>
>
Re: Review Request 58590: Provide an optional attribute for custom
commands to specify title used in operations window title
Posted by Tim Thorpe <tt...@ca.ibm.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/58590/#review172908
-----------------------------------------------------------
Ship it!
Ship It!
- Tim Thorpe
On April 25, 2017, 4:51 a.m., Sangeeta Ravindran wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/58590/
> -----------------------------------------------------------
>
> (Updated April 25, 2017, 4:51 a.m.)
>
>
> Review request for Ambari, Di Li and Tim Thorpe.
>
>
> Bugs: AMBARI-20616
> https://issues.apache.org/jira/browse/AMBARI-20616
>
>
> Repository: ambari
>
>
> Description
> -------
>
> Some services in our stack have custom commands where the name is a combination of an operation and a component name.
>
> For e.g. <name>Remove_Logical_myservice_mycomponent</name>
>
> When the custom command is being executed, the operations window title is displayed as "Remove Logical myservice mycomponent mycomponent"
>
> It would be useful to provide an optional attribute e.g. opsDisplayName for custom commands to allow specifying the string for the title displayed in the operations window.
>
> <customCommand>
> <name>Remove_Logical_myservice_mycomponent</name>
> <opsDisplayName>true</opsDisplayName>
> <commandScript>
> <script>scripts/mycomponent.py</script>
> <scriptType>PYTHON</scriptType>
> <timeout>10800</timeout>
> </commandScript>
> </customCommand>
>
>
> Diffs
> -----
>
> ambari-server/src/main/java/org/apache/ambari/server/actionmanager/HostRoleCommand.java 87a6edf
> ambari-server/src/main/java/org/apache/ambari/server/controller/AmbariCustomCommandExecutionHelper.java 17bc718
> ambari-server/src/main/java/org/apache/ambari/server/controller/internal/TaskResourceProvider.java 20b0417
> ambari-server/src/main/java/org/apache/ambari/server/orm/entities/HostRoleCommandEntity.java 86feceb
> ambari-server/src/main/java/org/apache/ambari/server/state/CustomCommandDefinition.java 280a59b
> ambari-server/src/main/java/org/apache/ambari/server/upgrade/UpgradeCatalog300.java 633d837
> ambari-server/src/main/resources/Ambari-DDL-Derby-CREATE.sql fd49b94
> ambari-server/src/main/resources/Ambari-DDL-MySQL-CREATE.sql 9fc3209
> ambari-server/src/main/resources/Ambari-DDL-Oracle-CREATE.sql 310208d
> ambari-server/src/main/resources/Ambari-DDL-Postgres-CREATE.sql c052104
> ambari-server/src/main/resources/Ambari-DDL-SQLAnywhere-CREATE.sql 5a58ef8
> ambari-server/src/main/resources/Ambari-DDL-SQLServer-CREATE.sql 8c7a31d
> ambari-server/src/main/resources/properties.json c2545fe
> ambari-server/src/test/java/org/apache/ambari/server/controller/internal/TaskResourceProviderTest.java 8a52b8a
> ambari-server/src/test/java/org/apache/ambari/server/orm/entities/HostRoleCommandEntityTest.java PRE-CREATION
> ambari-server/src/test/java/org/apache/ambari/server/upgrade/UpgradeCatalog300Test.java fec041c
> ambari-web/app/controllers/wizard/step9_controller.js 4697dad
> ambari-web/app/utils/ajax/ajax.js 5d7108f
> ambari-web/app/utils/helper.js 03a2e82
> ambari-web/app/utils/host_progress_popup.js c615cae
> ambari-web/app/views/wizard/step9/hostLogPopupBody_view.js d983252
> ambari-web/test/utils/helper_test.js 4b9ec36
>
>
> Diff: https://reviews.apache.org/r/58590/diff/3/
>
>
> Testing
> -------
>
> Manual Testing.
> Added new test cases.
>
>
> Thanks,
>
> Sangeeta Ravindran
>
>
Re: Review Request 58590: Provide an optional attribute for custom
commands to specify title used in operations window title
Posted by Di Li <di...@ca.ibm.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/58590/#review172913
-----------------------------------------------------------
Ship it!
Ship It!
- Di Li
On April 25, 2017, 4:51 a.m., Sangeeta Ravindran wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/58590/
> -----------------------------------------------------------
>
> (Updated April 25, 2017, 4:51 a.m.)
>
>
> Review request for Ambari, Di Li and Tim Thorpe.
>
>
> Bugs: AMBARI-20616
> https://issues.apache.org/jira/browse/AMBARI-20616
>
>
> Repository: ambari
>
>
> Description
> -------
>
> Some services in our stack have custom commands where the name is a combination of an operation and a component name.
>
> For e.g. <name>Remove_Logical_myservice_mycomponent</name>
>
> When the custom command is being executed, the operations window title is displayed as "Remove Logical myservice mycomponent mycomponent"
>
> It would be useful to provide an optional attribute e.g. opsDisplayName for custom commands to allow specifying the string for the title displayed in the operations window.
>
> <customCommand>
> <name>Remove_Logical_myservice_mycomponent</name>
> <opsDisplayName>true</opsDisplayName>
> <commandScript>
> <script>scripts/mycomponent.py</script>
> <scriptType>PYTHON</scriptType>
> <timeout>10800</timeout>
> </commandScript>
> </customCommand>
>
>
> Diffs
> -----
>
> ambari-server/src/main/java/org/apache/ambari/server/actionmanager/HostRoleCommand.java 87a6edf
> ambari-server/src/main/java/org/apache/ambari/server/controller/AmbariCustomCommandExecutionHelper.java 17bc718
> ambari-server/src/main/java/org/apache/ambari/server/controller/internal/TaskResourceProvider.java 20b0417
> ambari-server/src/main/java/org/apache/ambari/server/orm/entities/HostRoleCommandEntity.java 86feceb
> ambari-server/src/main/java/org/apache/ambari/server/state/CustomCommandDefinition.java 280a59b
> ambari-server/src/main/java/org/apache/ambari/server/upgrade/UpgradeCatalog300.java 633d837
> ambari-server/src/main/resources/Ambari-DDL-Derby-CREATE.sql fd49b94
> ambari-server/src/main/resources/Ambari-DDL-MySQL-CREATE.sql 9fc3209
> ambari-server/src/main/resources/Ambari-DDL-Oracle-CREATE.sql 310208d
> ambari-server/src/main/resources/Ambari-DDL-Postgres-CREATE.sql c052104
> ambari-server/src/main/resources/Ambari-DDL-SQLAnywhere-CREATE.sql 5a58ef8
> ambari-server/src/main/resources/Ambari-DDL-SQLServer-CREATE.sql 8c7a31d
> ambari-server/src/main/resources/properties.json c2545fe
> ambari-server/src/test/java/org/apache/ambari/server/controller/internal/TaskResourceProviderTest.java 8a52b8a
> ambari-server/src/test/java/org/apache/ambari/server/orm/entities/HostRoleCommandEntityTest.java PRE-CREATION
> ambari-server/src/test/java/org/apache/ambari/server/upgrade/UpgradeCatalog300Test.java fec041c
> ambari-web/app/controllers/wizard/step9_controller.js 4697dad
> ambari-web/app/utils/ajax/ajax.js 5d7108f
> ambari-web/app/utils/helper.js 03a2e82
> ambari-web/app/utils/host_progress_popup.js c615cae
> ambari-web/app/views/wizard/step9/hostLogPopupBody_view.js d983252
> ambari-web/test/utils/helper_test.js 4b9ec36
>
>
> Diff: https://reviews.apache.org/r/58590/diff/3/
>
>
> Testing
> -------
>
> Manual Testing.
> Added new test cases.
>
>
> Thanks,
>
> Sangeeta Ravindran
>
>
Re: Review Request 58590: Provide an optional attribute for custom
commands to specify title used in operations window title
Posted by Sangeeta Ravindran <sa...@gmail.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/58590/
-----------------------------------------------------------
(Updated April 25, 2017, 4:51 a.m.)
Review request for Ambari, Di Li and Tim Thorpe.
Bugs: AMBARI-20616
https://issues.apache.org/jira/browse/AMBARI-20616
Repository: ambari
Description
-------
Some services in our stack have custom commands where the name is a combination of an operation and a component name.
For e.g. <name>Remove_Logical_myservice_mycomponent</name>
When the custom command is being executed, the operations window title is displayed as "Remove Logical myservice mycomponent mycomponent"
It would be useful to provide an optional attribute e.g. opsDisplayName for custom commands to allow specifying the string for the title displayed in the operations window.
<customCommand>
<name>Remove_Logical_myservice_mycomponent</name>
<opsDisplayName>true</opsDisplayName>
<commandScript>
<script>scripts/mycomponent.py</script>
<scriptType>PYTHON</scriptType>
<timeout>10800</timeout>
</commandScript>
</customCommand>
Diffs (updated)
-----
ambari-server/src/main/java/org/apache/ambari/server/actionmanager/HostRoleCommand.java 87a6edf
ambari-server/src/main/java/org/apache/ambari/server/controller/AmbariCustomCommandExecutionHelper.java 17bc718
ambari-server/src/main/java/org/apache/ambari/server/controller/internal/TaskResourceProvider.java 20b0417
ambari-server/src/main/java/org/apache/ambari/server/orm/entities/HostRoleCommandEntity.java 86feceb
ambari-server/src/main/java/org/apache/ambari/server/state/CustomCommandDefinition.java 280a59b
ambari-server/src/main/java/org/apache/ambari/server/upgrade/UpgradeCatalog300.java 633d837
ambari-server/src/main/resources/Ambari-DDL-Derby-CREATE.sql fd49b94
ambari-server/src/main/resources/Ambari-DDL-MySQL-CREATE.sql 9fc3209
ambari-server/src/main/resources/Ambari-DDL-Oracle-CREATE.sql 310208d
ambari-server/src/main/resources/Ambari-DDL-Postgres-CREATE.sql c052104
ambari-server/src/main/resources/Ambari-DDL-SQLAnywhere-CREATE.sql 5a58ef8
ambari-server/src/main/resources/Ambari-DDL-SQLServer-CREATE.sql 8c7a31d
ambari-server/src/main/resources/properties.json c2545fe
ambari-server/src/test/java/org/apache/ambari/server/controller/internal/TaskResourceProviderTest.java 8a52b8a
ambari-server/src/test/java/org/apache/ambari/server/orm/entities/HostRoleCommandEntityTest.java PRE-CREATION
ambari-server/src/test/java/org/apache/ambari/server/upgrade/UpgradeCatalog300Test.java fec041c
ambari-web/app/controllers/wizard/step9_controller.js 4697dad
ambari-web/app/utils/ajax/ajax.js 5d7108f
ambari-web/app/utils/helper.js 03a2e82
ambari-web/app/utils/host_progress_popup.js c615cae
ambari-web/app/views/wizard/step9/hostLogPopupBody_view.js d983252
ambari-web/test/utils/helper_test.js 4b9ec36
Diff: https://reviews.apache.org/r/58590/diff/3/
Changes: https://reviews.apache.org/r/58590/diff/2-3/
Testing
-------
Manual Testing.
Added new test cases.
Thanks,
Sangeeta Ravindran
Re: Review Request 58590: Provide an optional attribute for custom
commands to specify title used in operations window title
Posted by Sangeeta Ravindran <sa...@gmail.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/58590/
-----------------------------------------------------------
(Updated April 24, 2017, 11:35 p.m.)
Review request for Ambari, Di Li and Tim Thorpe.
Summary (updated)
-----------------
Provide an optional attribute for custom commands to specify title used in operations window title
Bugs: AMBARI-20616
https://issues.apache.org/jira/browse/AMBARI-20616
Repository: ambari
Description
-------
Some services in our stack have custom commands where the name is a combination of an operation and a component name.
For e.g. <name>Remove_Logical_myservice_mycomponent</name>
When the custom command is being executed, the operations window title is displayed as "Remove Logical myservice mycomponent mycomponent"
It would be useful to provide an optional attribute e.g. opsDisplayName for custom commands to allow specifying the string for the title displayed in the operations window.
<customCommand>
<name>Remove_Logical_myservice_mycomponent</name>
<opsDisplayName>true</opsDisplayName>
<commandScript>
<script>scripts/mycomponent.py</script>
<scriptType>PYTHON</scriptType>
<timeout>10800</timeout>
</commandScript>
</customCommand>
Diffs
-----
ambari-server/src/main/java/org/apache/ambari/server/actionmanager/HostRoleCommand.java 87a6edf
ambari-server/src/main/java/org/apache/ambari/server/controller/AmbariCustomCommandExecutionHelper.java 17bc718
ambari-server/src/main/java/org/apache/ambari/server/controller/internal/TaskResourceProvider.java 20b0417
ambari-server/src/main/java/org/apache/ambari/server/orm/entities/HostRoleCommandEntity.java 86feceb
ambari-server/src/main/java/org/apache/ambari/server/state/CustomCommandDefinition.java 280a59b
ambari-server/src/main/java/org/apache/ambari/server/upgrade/UpgradeCatalog300.java 633d837
ambari-server/src/main/resources/Ambari-DDL-Derby-CREATE.sql fd49b94
ambari-server/src/main/resources/Ambari-DDL-MySQL-CREATE.sql 9fc3209
ambari-server/src/main/resources/Ambari-DDL-Oracle-CREATE.sql 310208d
ambari-server/src/main/resources/Ambari-DDL-Postgres-CREATE.sql c052104
ambari-server/src/main/resources/Ambari-DDL-SQLAnywhere-CREATE.sql 5a58ef8
ambari-server/src/main/resources/Ambari-DDL-SQLServer-CREATE.sql 8c7a31d
ambari-server/src/main/resources/properties.json c2545fe
ambari-server/src/test/java/org/apache/ambari/server/controller/internal/TaskResourceProviderTest.java 8a52b8a
ambari-server/src/test/java/org/apache/ambari/server/orm/entities/HostRoleCommandEntityTest.java PRE-CREATION
ambari-server/src/test/java/org/apache/ambari/server/upgrade/UpgradeCatalog300Test.java fec041c
ambari-web/app/controllers/wizard/step9_controller.js 4697dad
ambari-web/app/utils/ajax/ajax.js 2b5e52a
ambari-web/app/utils/helper.js 03a2e82
ambari-web/app/utils/host_progress_popup.js c615cae
ambari-web/app/views/wizard/step9/hostLogPopupBody_view.js d983252
ambari-web/test/utils/helper_test.js 4b9ec36
Diff: https://reviews.apache.org/r/58590/diff/2/
Testing
-------
Manual Testing.
Added new test cases.
Thanks,
Sangeeta Ravindran
Re: Review Request 58590: Provide an optional attribute for custom
commands to allow using command name in operations window title
Posted by Sangeeta Ravindran <sa...@gmail.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/58590/
-----------------------------------------------------------
(Updated April 24, 2017, 11:33 p.m.)
Review request for Ambari, Di Li and Tim Thorpe.
Bugs: AMBARI-20616
https://issues.apache.org/jira/browse/AMBARI-20616
Repository: ambari
Description (updated)
-------
Some services in our stack have custom commands where the name is a combination of an operation and a component name.
For e.g. <name>Remove_Logical_myservice_mycomponent</name>
When the custom command is being executed, the operations window title is displayed as "Remove Logical myservice mycomponent mycomponent"
It would be useful to provide an optional attribute e.g. opsDisplayName for custom commands to allow specifying the string for the title displayed in the operations window.
<customCommand>
<name>Remove_Logical_myservice_mycomponent</name>
<opsDisplayName>true</opsDisplayName>
<commandScript>
<script>scripts/mycomponent.py</script>
<scriptType>PYTHON</scriptType>
<timeout>10800</timeout>
</commandScript>
</customCommand>
Diffs (updated)
-----
ambari-server/src/main/java/org/apache/ambari/server/actionmanager/HostRoleCommand.java 87a6edf
ambari-server/src/main/java/org/apache/ambari/server/controller/AmbariCustomCommandExecutionHelper.java 17bc718
ambari-server/src/main/java/org/apache/ambari/server/controller/internal/TaskResourceProvider.java 20b0417
ambari-server/src/main/java/org/apache/ambari/server/orm/entities/HostRoleCommandEntity.java 86feceb
ambari-server/src/main/java/org/apache/ambari/server/state/CustomCommandDefinition.java 280a59b
ambari-server/src/main/java/org/apache/ambari/server/upgrade/UpgradeCatalog300.java 633d837
ambari-server/src/main/resources/Ambari-DDL-Derby-CREATE.sql fd49b94
ambari-server/src/main/resources/Ambari-DDL-MySQL-CREATE.sql 9fc3209
ambari-server/src/main/resources/Ambari-DDL-Oracle-CREATE.sql 310208d
ambari-server/src/main/resources/Ambari-DDL-Postgres-CREATE.sql c052104
ambari-server/src/main/resources/Ambari-DDL-SQLAnywhere-CREATE.sql 5a58ef8
ambari-server/src/main/resources/Ambari-DDL-SQLServer-CREATE.sql 8c7a31d
ambari-server/src/main/resources/properties.json c2545fe
ambari-server/src/test/java/org/apache/ambari/server/controller/internal/TaskResourceProviderTest.java 8a52b8a
ambari-server/src/test/java/org/apache/ambari/server/orm/entities/HostRoleCommandEntityTest.java PRE-CREATION
ambari-server/src/test/java/org/apache/ambari/server/upgrade/UpgradeCatalog300Test.java fec041c
ambari-web/app/controllers/wizard/step9_controller.js 4697dad
ambari-web/app/utils/ajax/ajax.js 2b5e52a
ambari-web/app/utils/helper.js 03a2e82
ambari-web/app/utils/host_progress_popup.js c615cae
ambari-web/app/views/wizard/step9/hostLogPopupBody_view.js d983252
ambari-web/test/utils/helper_test.js 4b9ec36
Diff: https://reviews.apache.org/r/58590/diff/2/
Changes: https://reviews.apache.org/r/58590/diff/1-2/
Testing
-------
Manual Testing.
Added new test cases.
Thanks,
Sangeeta Ravindran
Re: Review Request 58590: Provide an optional attribute for custom
commands to allow using command name in operations window title
Posted by Sangeeta Ravindran <sa...@gmail.com>.
> On April 21, 2017, 5:45 a.m., Attila Doroszlai wrote:
> > ambari-server/src/test/java/org/apache/ambari/server/orm/entities/HostRoleCommandEntityTest.java
> > Lines 24-25 (patched)
> > <https://reviews.apache.org/r/58590/diff/1/?file=1695882#file1695882line24>
> >
> > Unused imports
Thanks. I've removed the unused imports.
- Sangeeta
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/58590/#review172586
-----------------------------------------------------------
On April 24, 2017, 11:33 p.m., Sangeeta Ravindran wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/58590/
> -----------------------------------------------------------
>
> (Updated April 24, 2017, 11:33 p.m.)
>
>
> Review request for Ambari, Di Li and Tim Thorpe.
>
>
> Bugs: AMBARI-20616
> https://issues.apache.org/jira/browse/AMBARI-20616
>
>
> Repository: ambari
>
>
> Description
> -------
>
> Some services in our stack have custom commands where the name is a combination of an operation and a component name.
>
> For e.g. <name>Remove_Logical_myservice_mycomponent</name>
>
> When the custom command is being executed, the operations window title is displayed as "Remove Logical myservice mycomponent mycomponent"
>
> It would be useful to provide an optional attribute e.g. opsDisplayName for custom commands to allow specifying the string for the title displayed in the operations window.
>
> <customCommand>
> <name>Remove_Logical_myservice_mycomponent</name>
> <opsDisplayName>true</opsDisplayName>
> <commandScript>
> <script>scripts/mycomponent.py</script>
> <scriptType>PYTHON</scriptType>
> <timeout>10800</timeout>
> </commandScript>
> </customCommand>
>
>
> Diffs
> -----
>
> ambari-server/src/main/java/org/apache/ambari/server/actionmanager/HostRoleCommand.java 87a6edf
> ambari-server/src/main/java/org/apache/ambari/server/controller/AmbariCustomCommandExecutionHelper.java 17bc718
> ambari-server/src/main/java/org/apache/ambari/server/controller/internal/TaskResourceProvider.java 20b0417
> ambari-server/src/main/java/org/apache/ambari/server/orm/entities/HostRoleCommandEntity.java 86feceb
> ambari-server/src/main/java/org/apache/ambari/server/state/CustomCommandDefinition.java 280a59b
> ambari-server/src/main/java/org/apache/ambari/server/upgrade/UpgradeCatalog300.java 633d837
> ambari-server/src/main/resources/Ambari-DDL-Derby-CREATE.sql fd49b94
> ambari-server/src/main/resources/Ambari-DDL-MySQL-CREATE.sql 9fc3209
> ambari-server/src/main/resources/Ambari-DDL-Oracle-CREATE.sql 310208d
> ambari-server/src/main/resources/Ambari-DDL-Postgres-CREATE.sql c052104
> ambari-server/src/main/resources/Ambari-DDL-SQLAnywhere-CREATE.sql 5a58ef8
> ambari-server/src/main/resources/Ambari-DDL-SQLServer-CREATE.sql 8c7a31d
> ambari-server/src/main/resources/properties.json c2545fe
> ambari-server/src/test/java/org/apache/ambari/server/controller/internal/TaskResourceProviderTest.java 8a52b8a
> ambari-server/src/test/java/org/apache/ambari/server/orm/entities/HostRoleCommandEntityTest.java PRE-CREATION
> ambari-server/src/test/java/org/apache/ambari/server/upgrade/UpgradeCatalog300Test.java fec041c
> ambari-web/app/controllers/wizard/step9_controller.js 4697dad
> ambari-web/app/utils/ajax/ajax.js 2b5e52a
> ambari-web/app/utils/helper.js 03a2e82
> ambari-web/app/utils/host_progress_popup.js c615cae
> ambari-web/app/views/wizard/step9/hostLogPopupBody_view.js d983252
> ambari-web/test/utils/helper_test.js 4b9ec36
>
>
> Diff: https://reviews.apache.org/r/58590/diff/2/
>
>
> Testing
> -------
>
> Manual Testing.
> Added new test cases.
>
>
> Thanks,
>
> Sangeeta Ravindran
>
>
Re: Review Request 58590: Provide an optional attribute for custom
commands to allow using command name in operations window title
Posted by Attila Doroszlai <ad...@hortonworks.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/58590/#review172586
-----------------------------------------------------------
ambari-server/src/test/java/org/apache/ambari/server/orm/entities/HostRoleCommandEntityTest.java
Lines 24-25 (patched)
<https://reviews.apache.org/r/58590/#comment245682>
Unused imports
- Attila Doroszlai
On April 20, 2017, 9:53 p.m., Sangeeta Ravindran wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/58590/
> -----------------------------------------------------------
>
> (Updated April 20, 2017, 9:53 p.m.)
>
>
> Review request for Ambari, Di Li and Tim Thorpe.
>
>
> Bugs: AMBARI-20616
> https://issues.apache.org/jira/browse/AMBARI-20616
>
>
> Repository: ambari
>
>
> Description
> -------
>
> Some services in our stack have custom commands where the name is a combination of an operation and a component name.
>
> For e.g. <name>Remove_Logical_myservice_mycomponent</name>
>
> When the custom command is being executed, the operations window title is displayed as "Remove Logical myservice mycomponent mycomponent"
>
> It would be useful to provide an optional attribute e.g. useNameInOps for custom commands to allow using the command name itself as the title displayed in the operations window.
>
> <customCommand>
> <name>Remove_Logical_myservice_mycomponent</name>
> <useNameInOps>true</useNameInOps>
> <commandScript>
> <script>scripts/mycomponent.py</script>
> <scriptType>PYTHON</scriptType>
> <timeout>10800</timeout>
> </commandScript>
> </customCommand>
>
>
> Diffs
> -----
>
> ambari-server/src/main/java/org/apache/ambari/server/actionmanager/HostRoleCommand.java 87a6edf
> ambari-server/src/main/java/org/apache/ambari/server/controller/AmbariCustomCommandExecutionHelper.java 17bc718
> ambari-server/src/main/java/org/apache/ambari/server/controller/internal/TaskResourceProvider.java 20b0417
> ambari-server/src/main/java/org/apache/ambari/server/orm/entities/HostRoleCommandEntity.java 86feceb
> ambari-server/src/main/java/org/apache/ambari/server/state/CustomCommandDefinition.java 280a59b
> ambari-server/src/main/java/org/apache/ambari/server/upgrade/UpgradeCatalog300.java 633d837
> ambari-server/src/main/resources/Ambari-DDL-Derby-CREATE.sql fd49b94
> ambari-server/src/main/resources/Ambari-DDL-MySQL-CREATE.sql 9fc3209
> ambari-server/src/main/resources/Ambari-DDL-Oracle-CREATE.sql 310208d
> ambari-server/src/main/resources/Ambari-DDL-Postgres-CREATE.sql c052104
> ambari-server/src/main/resources/Ambari-DDL-SQLAnywhere-CREATE.sql 5a58ef8
> ambari-server/src/main/resources/Ambari-DDL-SQLServer-CREATE.sql 8c7a31d
> ambari-server/src/main/resources/properties.json c2545fe
> ambari-server/src/test/java/org/apache/ambari/server/controller/internal/TaskResourceProviderTest.java 8a52b8a
> ambari-server/src/test/java/org/apache/ambari/server/orm/entities/HostRoleCommandEntityTest.java PRE-CREATION
> ambari-server/src/test/java/org/apache/ambari/server/upgrade/UpgradeCatalog300Test.java fec041c
> ambari-web/app/controllers/wizard/step9_controller.js 4697dad
> ambari-web/app/utils/ajax/ajax.js 2b5e52a
> ambari-web/app/utils/helper.js 03a2e82
> ambari-web/app/utils/host_progress_popup.js c615cae
> ambari-web/app/views/wizard/step9/hostLogPopupBody_view.js d983252
> ambari-web/test/utils/helper_test.js 4b9ec36
>
>
> Diff: https://reviews.apache.org/r/58590/diff/1/
>
>
> Testing
> -------
>
> Manual Testing.
> Added new test cases.
>
>
> Thanks,
>
> Sangeeta Ravindran
>
>