You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@cloudstack.apache.org by rhtyd <gi...@git.apache.org> on 2016/08/02 07:18:03 UTC

[GitHub] cloudstack pull request #1620: oobm: simply change password transactional lo...

GitHub user rhtyd opened a pull request:

    https://github.com/apache/cloudstack/pull/1620

    oobm: simply change password transactional logic

    - Simplifies change password transactional logic without using pessmistic locks
    - Adds a re-enter password field in the UI to valid ipmi/oobm password
    
    Signed-off-by: Rohit Yadav <ro...@shapeblue.com>

You can merge this pull request into a Git repository by running:

    $ git pull https://github.com/shapeblue/cloudstack 4.9-oobm-password-fix

Alternatively you can review and apply these changes as the patch at:

    https://github.com/apache/cloudstack/pull/1620.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

    This closes #1620
    
----
commit 8ac3c883cd61fa0e18498aaae4e4f519743295c4
Author: Rohit Yadav <ro...@shapeblue.com>
Date:   2016-07-11T11:36:09Z

    oobm: simply change password transactional logic
    
    - Simplifies change password transactional logic without using pessmistic locks
    - Adds a re-enter password field in the UI to valid ipmi/oobm password
    
    Signed-off-by: Rohit Yadav <ro...@shapeblue.com>

----


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] cloudstack pull request #1620: oobm: simply change password transactional lo...

Posted by rhtyd <gi...@git.apache.org>.
GitHub user rhtyd reopened a pull request:

    https://github.com/apache/cloudstack/pull/1620

    oobm: simply change password transactional logic

    - Simplifies change password transactional logic without using pessmistic locks
    - Adds a re-enter password field in the UI to valid ipmi/oobm password

You can merge this pull request into a Git repository by running:

    $ git pull https://github.com/shapeblue/cloudstack 4.9-oobm-password-fix

Alternatively you can review and apply these changes as the patch at:

    https://github.com/apache/cloudstack/pull/1620.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

    This closes #1620
    
----
commit 8ac3c883cd61fa0e18498aaae4e4f519743295c4
Author: Rohit Yadav <ro...@shapeblue.com>
Date:   2016-07-11T11:36:09Z

    oobm: simply change password transactional logic
    
    - Simplifies change password transactional logic without using pessmistic locks
    - Adds a re-enter password field in the UI to valid ipmi/oobm password
    
    Signed-off-by: Rohit Yadav <ro...@shapeblue.com>

----


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] cloudstack issue #1620: oobm: simply change password transactional logic

Posted by rhtyd <gi...@git.apache.org>.
Github user rhtyd commented on the issue:

    https://github.com/apache/cloudstack/pull/1620
  
    Trillian test results wrt oobm:
    
    Tests out-of-band management background powerstate sync ... === TestName: test_oobm_background_powerstate_sync | Status : SUCCESS ===
    ok
    Tests out-of-band management configuration with valid data ... === TestName: test_oobm_configure_default_driver | Status : SUCCESS ===
    ok
    Tests out-of-band management configuration with invalid driver ... === TestName: test_oobm_configure_invalid_driver | Status : SUCCESS ===
    ok
    Tests out-of-band management host disable feature with ... === TestName: test_oobm_disable_feature_invalid | Status : SUCCESS ===
    ok
    Tests out-of-band management host disable feature with ... === TestName: test_oobm_disable_feature_valid | Status : SUCCESS ===
    ok
    Tests out-of-band management host enable feature with ... === TestName: test_oobm_enable_feature_invalid | Status : SUCCESS ===
    ok
    Tests out-of-band management host enable feature with ... === TestName: test_oobm_enable_feature_valid | Status : SUCCESS ===
    ok
    Tests out-of-band management enable/disable feature at cluster ... === TestName: test_oobm_enabledisable_across_clusterzones | Status : SUCCESS ===
    ok
    Tests out-of-band management issue power cycle action ... === TestName: test_oobm_issue_power_cycle | Status : SUCCESS ===
    ok
    Tests out-of-band management issue power off action ... === TestName: test_oobm_issue_power_off | Status : SUCCESS ===
    ok
    Tests out-of-band management issue power on action ... === TestName: test_oobm_issue_power_on | Status : SUCCESS ===
    ok
    Tests out-of-band management issue power reset action ... === TestName: test_oobm_issue_power_reset | Status : SUCCESS ===
    ok
    Tests out-of-band management issue power soft action ... === TestName: test_oobm_issue_power_soft | Status : SUCCESS ===
    ok
    Tests out-of-band management issue power action ... === TestName: test_oobm_issue_power_status | Status : SUCCESS ===
    ok
    Tests out-of-band management ownership expiry across multi-mgmt server ... === TestName: test_oobm_multiple_mgmt_server_ownership | Status : SUCCESS ===
    ok
    Tests out-of-band management change password feature ... === TestName: test_oobm_zchange_password | Status : SUCCESS ===
    ok
    
    ----------------------------------------------------------------------
    Ran 16 tests in 207.725s
    
    I'll add regression result results below;


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] cloudstack issue #1620: oobm: simply change password transactional logic

Posted by blueorangutan <gi...@git.apache.org>.
Github user blueorangutan commented on the issue:

    https://github.com/apache/cloudstack/pull/1620
  
    Trillian test result (trillian-pr1638-29-xenserver-65sp1-cs410): Test completed. 41 look ok, 11 have errors
    +-------------------------+-------------------------+-------------------------+
    |          Test           |         Result          |          Time           |
    +=========================+=========================+=========================+
    | test_DeployVmAntiAffini | Success                 | 282.569                 |
    | tyGroup_in_project      |                         |                         |
    +-------------------------+-------------------------+-------------------------+
    | test_create_pvlan_netwo | Success                 | 5.224                   |
    | rk                      |                         |                         |
    +-------------------------+-------------------------+-------------------------+
    | test_DeployVmAntiAffini | Success                 | 136.402                 |
    | tyGroup                 |                         |                         |
    +-------------------------+-------------------------+-------------------------+
    | test_dedicatePublicIpRa | marvin.cloudstackExcept | 0.245                   |
    | nge                     | ion.CloudstackAPIExcept |                         |
    |                         | ion                     |                         |
    +-------------------------+-------------------------+-------------------------+
    | test_deploy_vgpu_enable | Skipped                 | 0.004                   |
    | d_vm                    |                         |                         |
    +-------------------------+-------------------------+-------------------------+
    | test_deploy_vm_from_iso | exceptions.Exception    | 519.082                 |
    +-------------------------+-------------------------+-------------------------+
    | test_01_scale_vm        | Success                 | 5.209                   |
    +-------------------------+-------------------------+-------------------------+
    | test_00_deploy_vm_root_ | Success                 | 6.418                   |
    | resize                  |                         |                         |
    +-------------------------+-------------------------+-------------------------+
    | test_01_deploy_vm_root_ | Success                 | 6.271                   |
    | resize                  |                         |                         |
    +-------------------------+-------------------------+-------------------------+
    | test_02_deploy_vm_root_ | Success                 | 6.509                   |
    | resize                  |                         |                         |
    +-------------------------+-------------------------+-------------------------+
    | test_01_router_internal | Success                 | 0.321                   |
    | _basic                  |                         |                         |
    +-------------------------+-------------------------+-------------------------+
    | test_02_router_internal | Success                 | 0.548                   |
    | _adv                    |                         |                         |
    +-------------------------+-------------------------+-------------------------+
    | test_03_restart_network | Success                 | 120.830                 |
    | _cleanup                |                         |                         |
    +-------------------------+-------------------------+-------------------------+
    | test_04_restart_network | Success                 | 5.404                   |
    | _wo_cleanup             |                         |                         |
    +-------------------------+-------------------------+-------------------------+
    | test_05_router_basic    | Success                 | 0.032                   |
    +-------------------------+-------------------------+-------------------------+
    | test_06_router_advanced | Success                 | 0.063                   |
    +-------------------------+-------------------------+-------------------------+
    | test_07_stop_router     | Success                 | 15.174                  |
    +-------------------------+-------------------------+-------------------------+
    | test_08_start_router    | Success                 | 50.420                  |
    +-------------------------+-------------------------+-------------------------+
    | test_09_reboot_router   | Success                 | 65.510                  |
    +-------------------------+-------------------------+-------------------------+
    | test_deployvm_firstfit  | Success                 | 181.469                 |
    +-------------------------+-------------------------+-------------------------+
    | test_deployvm_userconce | Success                 | 95.964                  |
    | ntrated                 |                         |                         |
    +-------------------------+-------------------------+-------------------------+
    | test_deployvm_userdispe | Success                 | 45.720                  |
    | rsing                   |                         |                         |
    +-------------------------+-------------------------+-------------------------+
    | test_deployvm_userdata  | Success                 | 126.146                 |
    +-------------------------+-------------------------+-------------------------+
    | test_deployvm_userdata_ | Success                 | 25.496                  |
    | post                    |                         |                         |
    +-------------------------+-------------------------+-------------------------+
    | test_01_sys_vm_start    | Success                 | 0.146                   |
    +-------------------------+-------------------------+-------------------------+
    | test_02_sys_template_re | Success                 | 0.103                   |
    | ady                     |                         |                         |
    +-------------------------+-------------------------+-------------------------+
    | test_01_create_disk_off | Success                 | 0.119                   |
    | ering                   |                         |                         |
    +-------------------------+-------------------------+-------------------------+
    | test_02_create_sparse_t | Success                 | 0.077                   |
    | ype_disk_offering       |                         |                         |
    +-------------------------+-------------------------+-------------------------+
    | test_04_create_fat_type | Success                 | 0.071                   |
    | _disk_offering          |                         |                         |
    +-------------------------+-------------------------+-------------------------+
    | test_02_edit_disk_offer | Success                 | 0.086                   |
    | ing                     |                         |                         |
    +-------------------------+-------------------------+-------------------------+
    | test_03_delete_disk_off | Success                 | 0.040                   |
    | ering                   |                         |                         |
    +-------------------------+-------------------------+-------------------------+
    | test_default_role_delet | Success                 | 6.490                   |
    | ion                     |                         |                         |
    +-------------------------+-------------------------+-------------------------+
    | test_role_account_acls  | Success                 | 8.130                   |
    +-------------------------+-------------------------+-------------------------+
    | test_role_account_acls_ | Success                 | 7.846                   |
    | multiple_mgmt_servers   |                         |                         |
    +-------------------------+-------------------------+-------------------------+
    | test_role_inuse_deletio | Success                 | 6.309                   |
    | n                       |                         |                         |
    +-------------------------+-------------------------+-------------------------+
    | test_role_lifecycle_cre | Success                 | 6.329                   |
    | ate                     |                         |                         |
    +-------------------------+-------------------------+-------------------------+
    | test_role_lifecycle_del | Success                 | 11.347                  |
    | ete                     |                         |                         |
    +-------------------------+-------------------------+-------------------------+
    | test_role_lifecycle_lis | Success                 | 6.332                   |
    | t                       |                         |                         |
    +-------------------------+-------------------------+-------------------------+
    | test_role_lifecycle_upd | Success                 | 11.459                  |
    | ate                     |                         |                         |
    +-------------------------+-------------------------+-------------------------+
    | test_role_lifecycle_upd | Success                 | 6.292                   |
    | ate_role_inuse          |                         |                         |
    +-------------------------+-------------------------+-------------------------+
    | test_rolepermission_lif | Success                 | 6.557                   |
    | ecycle_concurrent_updat |                         |                         |
    | es                      |                         |                         |
    +-------------------------+-------------------------+-------------------------+
    | test_rolepermission_lif | Success                 | 6.329                   |
    | ecycle_create           |                         |                         |
    +-------------------------+-------------------------+-------------------------+
    | test_rolepermission_lif | Success                 | 6.292                   |
    | ecycle_delete           |                         |                         |
    +-------------------------+-------------------------+-------------------------+
    | test_rolepermission_lif | Success                 | 6.408                   |
    | ecycle_list             |                         |                         |
    +-------------------------+-------------------------+-------------------------+
    | test_rolepermission_lif | Success                 | 6.634                   |
    | ecycle_update           |                         |                         |
    +-------------------------+-------------------------+-------------------------+
    | test_01_create_service_ | Success                 | 0.087                   |
    | offering                |                         |                         |
    +-------------------------+-------------------------+-------------------------+
    | test_02_edit_service_of | Success                 | 0.078                   |
    | fering                  |                         |                         |
    +-------------------------+-------------------------+-------------------------+
    | test_03_delete_service_ | Success                 | 0.042                   |
    | offering                |                         |                         |
    +-------------------------+-------------------------+-------------------------+
    | test_04_change_offering | Success                 | 126.087                 |
    | _small                  |                         |                         |
    +-------------------------+-------------------------+-------------------------+
    | test_UpdateConfigParamW | Success                 | 0.145                   |
    | ithScope                |                         |                         |
    +-------------------------+-------------------------+-------------------------+
    | ContextSuite context=Te | exceptions.AttributeErr | 0                       |
    | stDedicateGuestVlanRang | or                      |                         |
    | e>:setup                |                         |                         |
    +-------------------------+-------------------------+-------------------------+
    | test_01_snapshot_root_d | Success                 | 66.466                  |
    | isk                     |                         |                         |
    +-------------------------+-------------------------+-------------------------+
    | test_01_internallb_roun | exceptions.Exception    | 161.948                 |
    | drobin_1VPC_3VM_HTTP_po |                         |                         |
    | rt80                    |                         |                         |
    +-------------------------+-------------------------+-------------------------+
    | test_01_internallb_roun | exceptions.Exception    | 172.282                 |
    | drobin_1VPC_3VM_HTTP_po |                         |                         |
    | rt80                    |                         |                         |
    +-------------------------+-------------------------+-------------------------+
    | test_02_internallb_roun | exceptions.Exception    | 272.445                 |
    | drobin_1RVPC_3VM_HTTP_p |                         |                         |
    | ort80                   |                         |                         |
    +-------------------------+-------------------------+-------------------------+
    | test_02_internallb_roun | exceptions.Exception    | 282.629                 |
    | drobin_1RVPC_3VM_HTTP_p |                         |                         |
    | ort80                   |                         |                         |
    +-------------------------+-------------------------+-------------------------+
    | test_03_vpc_internallb_ | exceptions.Exception    | 161.681                 |
    | haproxy_stats_on_all_in |                         |                         |
    | terfaces                |                         |                         |
    +-------------------------+-------------------------+-------------------------+
    | test_03_vpc_internallb_ | exceptions.Exception    | 166.779                 |
    | haproxy_stats_on_all_in |                         |                         |
    | terfaces                |                         |                         |
    +-------------------------+-------------------------+-------------------------+
    | test_04_rvpc_internallb | exceptions.Exception    | 247.162                 |
    | _haproxy_stats_on_all_i |                         |                         |
    | nterfaces               |                         |                         |
    +-------------------------+-------------------------+-------------------------+
    | test_04_rvpc_internallb | exceptions.Exception    | 252.263                 |
    | _haproxy_stats_on_all_i |                         |                         |
    | nterfaces               |                         |                         |
    +-------------------------+-------------------------+-------------------------+
    | test_01_list_sec_storag | Success                 | 0.092                   |
    | e_vm                    |                         |                         |
    +-------------------------+-------------------------+-------------------------+
    | test_02_list_cpvm_vm    | Success                 | 0.119                   |
    +-------------------------+-------------------------+-------------------------+
    | test_03_ssvm_internals  | Success                 | 3.824                   |
    +-------------------------+-------------------------+-------------------------+
    | test_04_cpvm_internals  | Success                 | 0.936                   |
    +-------------------------+-------------------------+-------------------------+
    | test_05_stop_ssvm       | Success                 | 138.763                 |
    +-------------------------+-------------------------+-------------------------+
    | test_06_stop_cpvm       | Success                 | 196.832                 |
    +-------------------------+-------------------------+-------------------------+
    | test_07_reboot_ssvm     | Success                 | 148.905                 |
    +-------------------------+-------------------------+-------------------------+
    | test_08_reboot_cpvm     | Success                 | 146.425                 |
    +-------------------------+-------------------------+-------------------------+
    | test_09_destroy_ssvm    | Success                 | 198.831                 |
    +-------------------------+-------------------------+-------------------------+
    | test_10_destroy_cpvm    | Success                 | 201.686                 |
    +-------------------------+-------------------------+-------------------------+
    | test_01_create_iso      | Success                 | 66.416                  |
    +-------------------------+-------------------------+-------------------------+
    | test_02_edit_iso        | Success                 | 0.079                   |
    +-------------------------+-------------------------+-------------------------+
    | test_03_delete_iso      | Success                 | 95.187                  |
    +-------------------------+-------------------------+-------------------------+
    | test_04_extract_Iso     | Success                 | 5.127                   |
    +-------------------------+-------------------------+-------------------------+
    | test_05_iso_permissions | Success                 | 0.056                   |
    +-------------------------+-------------------------+-------------------------+
    | test_06_copy_iso        | Skipped                 | 0                       |
    +-------------------------+-------------------------+-------------------------+
    | test_07_list_default_is | Success                 | 0.050                   |
    | o                       |                         |                         |
    +-------------------------+-------------------------+-------------------------+
    | test_01_list_volumes    | Success                 | 0.025                   |
    +-------------------------+-------------------------+-------------------------+
    | test_02_list_templates  | Success                 | 0.040                   |
    +-------------------------+-------------------------+-------------------------+
    | test_03_list_snapshots  | Success                 | 0.074                   |
    +-------------------------+-------------------------+-------------------------+
    | test_static_role_accoun | Skipped                 | 0.020                   |
    | t_acls                  |                         |                         |
    +-------------------------+-------------------------+-------------------------+
    | test_01_create_lb_rule_ | Success                 | 208.003                 |
    | src_nat                 |                         |                         |
    +-------------------------+-------------------------+-------------------------+
    | test_02_create_lb_rule_ | Success                 | 207.487                 |
    | non_nat                 |                         |                         |
    +-------------------------+-------------------------+-------------------------+
    | test_assign_and_removal | Success                 | 148.681                 |
    | _lb                     |                         |                         |
    +-------------------------+-------------------------+-------------------------+
    | login_test_saml_user    | Success                 | 21.730                  |
    +-------------------------+-------------------------+-------------------------+
    | test_01_create_template | Success                 | 70.600                  |
    +-------------------------+-------------------------+-------------------------+
    | test_CreateTemplateWith | Success                 | 115.968                 |
    | DuplicateName           |                         |                         |
    +-------------------------+-------------------------+-------------------------+
    | test_02_edit_template   | Success                 | 90.217                  |
    +-------------------------+-------------------------+-------------------------+
    | test_03_delete_template | Success                 | 5.106                   |
    +-------------------------+-------------------------+-------------------------+
    | test_04_extract_templat | Success                 | 5.164                   |
    | e                       |                         |                         |
    +-------------------------+-------------------------+-------------------------+
    | test_05_template_permis | Success                 | 0.054                   |
    | sions                   |                         |                         |
    +-------------------------+-------------------------+-------------------------+
    | test_06_copy_template   | Skipped                 | 0                       |
    +-------------------------+-------------------------+-------------------------+
    | test_07_list_public_tem | Success                 | 0.031                   |
    | plates                  |                         |                         |
    +-------------------------+-------------------------+-------------------------+
    | test_08_list_system_tem | Success                 | 0.023                   |
    | plates                  |                         |                         |
    +-------------------------+-------------------------+-------------------------+
    | test_nic_secondaryip_ad | Success                 | 152.613                 |
    | d_remove                |                         |                         |
    +-------------------------+-------------------------+-------------------------+
    | test_network_acl        | Success                 | 121.390                 |
    +-------------------------+-------------------------+-------------------------+
    | test_delete_account     | Success                 | 248.156                 |
    +-------------------------+-------------------------+-------------------------+
    | test_01_port_fwd_on_src | Success                 | 83.818                  |
    | _nat                    |                         |                         |
    +-------------------------+-------------------------+-------------------------+
    | test_02_port_fwd_on_non | Success                 | 70.787                  |
    | _src_nat                |                         |                         |
    +-------------------------+-------------------------+-------------------------+
    | test_public_ip_admin_ac | Success                 | 40.277                  |
    | count                   |                         |                         |
    +-------------------------+-------------------------+-------------------------+
    | test_public_ip_user_acc | Success                 | 10.269                  |
    | ount                    |                         |                         |
    +-------------------------+-------------------------+-------------------------+
    | test_reboot_router      | Success                 | 580.068                 |
    +-------------------------+-------------------------+-------------------------+
    | test_releaseIP          | Success                 | 223.083                 |
    +-------------------------+-------------------------+-------------------------+
    | test_network_rules_acqu | Success                 | 102.652                 |
    | ired_public_ip_1_static |                         |                         |
    | _nat_rule               |                         |                         |
    +-------------------------+-------------------------+-------------------------+
    | test_network_rules_acqu | Success                 | 76.512                  |
    | ired_public_ip_2_nat_ru |                         |                         |
    | le                      |                         |                         |
    +-------------------------+-------------------------+-------------------------+
    | test_network_rules_acqu | Success                 | 86.547                  |
    | ired_public_ip_3_Load_B |                         |                         |
    | alancer_Rule            |                         |                         |
    +-------------------------+-------------------------+-------------------------+
    | test_vm_nic_adapter_vmx | Skipped                 | 0                       |
    | net3                    |                         |                         |
    +-------------------------+-------------------------+-------------------------+
    | test_advZoneVirtualRout | Success                 | 0.022                   |
    | er                      |                         |                         |
    +-------------------------+-------------------------+-------------------------+
    | test_deploy_vm          | Success                 | 0.030                   |
    +-------------------------+-------------------------+-------------------------+
    | test_deploy_vm_multiple | Success                 | 212.751                 |
    +-------------------------+-------------------------+-------------------------+
    | test_01_stop_vm         | Success                 | 30.253                  |
    +-------------------------+-------------------------+-------------------------+
    | test_02_start_vm        | Success                 | 20.209                  |
    +-------------------------+-------------------------+-------------------------+
    | test_03_reboot_vm       | Success                 | 10.160                  |
    +-------------------------+-------------------------+-------------------------+
    | test_06_destroy_vm      | Success                 | 10.141                  |
    +-------------------------+-------------------------+-------------------------+
    | test_07_restore_vm      | Success                 | 0.110                   |
    +-------------------------+-------------------------+-------------------------+
    | test_08_migrate_vm      | Success                 | 65.935                  |
    +-------------------------+-------------------------+-------------------------+
    | test_09_expunge_vm      | Success                 | 605.177                 |
    +-------------------------+-------------------------+-------------------------+
    | test_10_attachAndDetach | Success                 | 111.993                 |
    | _iso                    |                         |                         |
    +-------------------------+-------------------------+-------------------------+
    | test_01_nic             | Success                 | 771.804                 |
    +-------------------------+-------------------------+-------------------------+
    | test_extendPhysicalNetw | exceptions.AttributeErr | 0.027                   |
    | orkVlan                 | or                      |                         |
    +-------------------------+-------------------------+-------------------------+
    | test_01_test_vm_volume_ | marvin.cloudstackExcept | 126.089                 |
    | snapshot                | ion.CloudstackAPIExcept |                         |
    |                         | ion                     |                         |
    +-------------------------+-------------------------+-------------------------+
    | test_01_create_vm_snaps | Success                 | 100.764                 |
    | hots                    |                         |                         |
    +-------------------------+-------------------------+-------------------------+
    | test_02_revert_vm_snaps | Success                 | 211.420                 |
    | hots                    |                         |                         |
    +-------------------------+-------------------------+-------------------------+
    | test_03_delete_vm_snaps | Success                 | 285.208                 |
    | hots                    |                         |                         |
    +-------------------------+-------------------------+-------------------------+
    | test_oobm_background_po | Failure                 | 3016.086                |
    | werstate_sync           |                         |                         |
    +-------------------------+-------------------------+-------------------------+
    | test_oobm_configure_def | Success                 | 0.097                   |
    | ault_driver             |                         |                         |
    +-------------------------+-------------------------+-------------------------+
    | test_oobm_configure_inv | Success                 | 0.081                   |
    | alid_driver             |                         |                         |
    +-------------------------+-------------------------+-------------------------+
    | test_oobm_disable_featu | Success                 | 0.178                   |
    | re_invalid              |                         |                         |
    +-------------------------+-------------------------+-------------------------+
    | test_oobm_disable_featu | Success                 | 0.194                   |
    | re_valid                |                         |                         |
    +-------------------------+-------------------------+-------------------------+
    | test_oobm_enable_featur | Success                 | 0.108                   |
    | e_invalid               |                         |                         |
    +-------------------------+-------------------------+-------------------------+
    | test_oobm_enable_featur | Success                 | 5.178                   |
    | e_valid                 |                         |                         |
    +-------------------------+-------------------------+-------------------------+
    | test_oobm_enabledisable | exceptions.Exception    | 357.263                 |
    | _across_clusterzones    |                         |                         |
    +-------------------------+-------------------------+-------------------------+
    | test_oobm_issue_power_c | exceptions.Exception    | 10.241                  |
    | ycle                    |                         |                         |
    +-------------------------+-------------------------+-------------------------+
    | test_oobm_issue_power_o | exceptions.Exception    | 10.244                  |
    | ff                      |                         |                         |
    +-------------------------+-------------------------+-------------------------+
    | test_oobm_issue_power_o | exceptions.Exception    | 10.262                  |
    | n                       |                         |                         |
    +-------------------------+-------------------------+-------------------------+
    | test_oobm_issue_power_r | exceptions.Exception    | 10.333                  |
    | eset                    |                         |                         |
    +-------------------------+-------------------------+-------------------------+
    | test_oobm_issue_power_s | exceptions.Exception    | 10.311                  |
    | oft                     |                         |                         |
    +-------------------------+-------------------------+-------------------------+
    | test_oobm_issue_power_s | exceptions.Exception    | 10.249                  |
    | tatus                   |                         |                         |
    +-------------------------+-------------------------+-------------------------+
    | test_oobm_multiple_mgmt | Failure                 | 3048.281                |
    | _server_ownership       |                         |                         |
    +-------------------------+-------------------------+-------------------------+
    | test_oobm_zchange_passw | exceptions.Exception    | 10.279                  |
    | ord                     |                         |                         |
    +-------------------------+-------------------------+-------------------------+
    | test_UpdateStorageOverP | Success                 | 0.139                   |
    | rovisioningFactor       |                         |                         |
    +-------------------------+-------------------------+-------------------------+
    | test_01_create_volume   | Success                 | 392.563                 |
    +-------------------------+-------------------------+-------------------------+
    | test_02_attach_volume   | Success                 | 10.775                  |
    +-------------------------+-------------------------+-------------------------+
    | test_03_download_attach | Success                 | 15.335                  |
    | ed_volume               |                         |                         |
    +-------------------------+-------------------------+-------------------------+
    | test_04_delete_attached | Success                 | 10.200                  |
    | _volume                 |                         |                         |
    +-------------------------+-------------------------+-------------------------+
    | test_05_detach_volume   | Success                 | 100.286                 |
    +-------------------------+-------------------------+-------------------------+
    | test_06_download_detach | Success                 | 30.388                  |
    | ed_volume               |                         |                         |
    +-------------------------+-------------------------+-------------------------+
    | test_07_resize_fail     | Success                 | 100.979                 |
    +-------------------------+-------------------------+-------------------------+
    | test_08_resize_volume   | Success                 | 95.902                  |
    +-------------------------+-------------------------+-------------------------+
    | test_09_delete_detached | Success                 | 15.662                  |
    | _volume                 |                         |                         |
    +-------------------------+-------------------------+-------------------------+
    | test_isolate_network_pa | Success                 | 35.914                  |
    | ssword_server           |                         |                         |
    +-------------------------+-------------------------+-------------------------+
    | test_createPortablePubl | Success                 | 15.609                  |
    | icIPAcquire             |                         |                         |
    +-------------------------+-------------------------+-------------------------+
    | test_createPortablePubl | Success                 | 15.315                  |
    | icIPRange               |                         |                         |
    +-------------------------+-------------------------+-------------------------+
    | test_01_create_redundan | Failure                 | 696.990                 |
    | t_VPC_2tiers_4VMs_4IPs_ |                         |                         |
    | 4PF_ACL                 |                         |                         |
    +-------------------------+-------------------------+-------------------------+
    | test_02_redundant_VPC_d | Success                 | 683.697                 |
    | efault_routes           |                         |                         |
    +-------------------------+-------------------------+-------------------------+
    | test_03_create_redundan | Success                 | 692.679                 |
    | t_VPC_1tier_2VMs_2IPs_2 |                         |                         |
    | PF_ACL_reboot_routers   |                         |                         |
    +-------------------------+-------------------------+-------------------------+
    | test_04_rvpc_network_ga | Failure                 | 3890.670                |
    | rbage_collector_nics    |                         |                         |
    +-------------------------+-------------------------+-------------------------+
    | test_05_rvpc_multi_tier | Failure                 | 579.303                 |
    | s                       |                         |                         |
    +-------------------------+-------------------------+-------------------------+
    | test_01_primary_storage | marvin.cloudstackExcept | 0.079                   |
    | _iscsi                  | ion.CloudstackAPIExcept |                         |
    |                         | ion                     |                         |
    +-------------------------+-------------------------+-------------------------+
    | test_01_primary_storage | Success                 | 38.788                  |
    | _nfs                    |                         |                         |
    +-------------------------+-------------------------+-------------------------+
    | test_01_vpc_privategw_a | Failure                 | 86.872                  |
    | cl                      |                         |                         |
    +-------------------------+-------------------------+-------------------------+
    | test_02_vpc_privategw_s | Failure                 | 167.482                 |
    | tatic_routes            |                         |                         |
    +-------------------------+-------------------------+-------------------------+
    | test_03_vpc_privategw_r | Failure                 | 182.436                 |
    | estart_vpc_cleanup      |                         |                         |
    +-------------------------+-------------------------+-------------------------+
    | test_04_rvpc_privategw_ | Failure                 | 358.442                 |
    | static_routes           |                         |                         |
    +-------------------------+-------------------------+-------------------------+
    | test_01_VPC_nics_after_ | Success                 | 583.436                 |
    | destroy                 |                         |                         |
    +-------------------------+-------------------------+-------------------------+
    | test_02_VPC_default_rou | Success                 | 405.551                 |
    | tes                     |                         |                         |
    +-------------------------+-------------------------+-------------------------+
    | test_createRegion       | Success                 | 0.043                   |
    +-------------------------+-------------------------+-------------------------+
    | test_01_reset_vm_on_reb | Success                 | 85.636                  |
    | oot                     |                         |                         |
    +-------------------------+-------------------------+-------------------------+
    | test_01_updatevolumedet | Success                 | 0.081                   |
    | ail                     |                         |                         |
    +-------------------------+-------------------------+-------------------------+
    | test_router_dhcphosts   | Success                 | 107.303                 |
    +-------------------------+-------------------------+-------------------------+
    | test_02_routervm_iptabl | Success                 | 166.938                 |
    | es_policies             |                         |                         |
    +-------------------------+-------------------------+-------------------------+
    | test_01_single_VPC_ipta | Success                 | 330.357                 |
    | bles_policies           |                         |                         |
    +-------------------------+-------------------------+-------------------------+
    | test_01_isolate_network | Success                 | 289.032                 |
    | _FW_PF_default_routes_e |                         |                         |
    | gress_true              |                         |                         |
    +-------------------------+-------------------------+-------------------------+
    | test_02_isolate_network | Success                 | 301.121                 |
    | _FW_PF_default_routes_e |                         |                         |
    | gress_false             |                         |                         |
    +-------------------------+-------------------------+-------------------------+
    | test_01_RVR_Network_FW_ | Failure                 | 486.154                 |
    | PF_SSH_default_routes_e |                         |                         |
    | gress_true              |                         |                         |
    +-------------------------+-------------------------+-------------------------+
    | test_02_RVR_Network_FW_ | Failure                 | 491.002                 |
    | PF_SSH_default_routes_e |                         |                         |
    | gress_false             |                         |                         |
    +-------------------------+-------------------------+-------------------------+
    | test_03_RVR_Network_che | Success                 | 288.226                 |
    | ck_router_state         |                         |                         |
    +-------------------------+-------------------------+-------------------------+
    | test_01_redundant_vpc_s | Failure                 | 448.533                 |
    | ite2site_vpn            |                         |                         |
    +-------------------------+-------------------------+-------------------------+
    | test_01_vpc_remote_acce | exceptions.UnboundLocal | 0.079                   |
    | ss_vpn                  | Error                   |                         |
    +-------------------------+-------------------------+-------------------------+
    | test_01_vpc_site2site_v | Failure                 | 232.181                 |
    | pn                      |                         |                         |
    +-------------------------+-------------------------+-------------------------+
    Trillian env - trillian-pr1638-29-xenserver-65sp1-cs410, Job ID 29


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] cloudstack issue #1620: oobm: simply change password transactional logic

Posted by blueorangutan <gi...@git.apache.org>.
Github user blueorangutan commented on the issue:

    https://github.com/apache/cloudstack/pull/1620
  
    Packaging result: \u2716centos6 \u2716centos7 \u2716debian repo: http://packages.shapeblue.com/cloudstack/pr/1620


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] cloudstack pull request #1620: oobm: simply change password transactional lo...

Posted by rhtyd <gi...@git.apache.org>.
Github user rhtyd closed the pull request at:

    https://github.com/apache/cloudstack/pull/1620


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] cloudstack issue #1620: oobm: simply change password transactional logic

Posted by blueorangutan <gi...@git.apache.org>.
Github user blueorangutan commented on the issue:

    https://github.com/apache/cloudstack/pull/1620
  
    <b>Failed</b> to build one or more CloudStack packages that would have been available at: http://packages.shapeblue.com/cloudstack/custom/github-1620


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] cloudstack issue #1620: oobm: simply change password transactional logic

Posted by rhtyd <gi...@git.apache.org>.
Github user rhtyd commented on the issue:

    https://github.com/apache/cloudstack/pull/1620
  
    Merging based on review and test results above, thanks @blueorangutan.


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] cloudstack issue #1620: oobm: simply change password transactional logic

Posted by blueorangutan <gi...@git.apache.org>.
Github user blueorangutan commented on the issue:

    https://github.com/apache/cloudstack/pull/1620
  
    Packaging result: \u2714centos6 \u2714centos7 \u2714debian repo: http://packages.shapeblue.com/cloudstack/pr/1620


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] cloudstack issue #1620: oobm: simply change password transactional logic

Posted by blueorangutan <gi...@git.apache.org>.
Github user blueorangutan commented on the issue:

    https://github.com/apache/cloudstack/pull/1620
  
    A Trillian-Jenkins job has been kicked to build packages and start testing. I'll keep you posted as I make progress.


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] cloudstack pull request #1620: oobm: simply change password transactional lo...

Posted by rhtyd <gi...@git.apache.org>.
GitHub user rhtyd reopened a pull request:

    https://github.com/apache/cloudstack/pull/1620

    oobm: simply change password transactional logic

    - Simplifies change password transactional logic without using pessmistic locks
    - Adds a re-enter password field in the UI to valid ipmi/oobm password

You can merge this pull request into a Git repository by running:

    $ git pull https://github.com/shapeblue/cloudstack 4.9-oobm-password-fix

Alternatively you can review and apply these changes as the patch at:

    https://github.com/apache/cloudstack/pull/1620.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

    This closes #1620
    
----
commit 8ac3c883cd61fa0e18498aaae4e4f519743295c4
Author: Rohit Yadav <ro...@shapeblue.com>
Date:   2016-07-11T11:36:09Z

    oobm: simply change password transactional logic
    
    - Simplifies change password transactional logic without using pessmistic locks
    - Adds a re-enter password field in the UI to valid ipmi/oobm password
    
    Signed-off-by: Rohit Yadav <ro...@shapeblue.com>

----


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] cloudstack pull request #1620: oobm: simply change password transactional lo...

Posted by rhtyd <gi...@git.apache.org>.
GitHub user rhtyd reopened a pull request:

    https://github.com/apache/cloudstack/pull/1620

    oobm: simply change password transactional logic

    - Simplifies change password transactional logic without using pessmistic locks
    - Adds a re-enter password field in the UI to valid ipmi/oobm password

You can merge this pull request into a Git repository by running:

    $ git pull https://github.com/shapeblue/cloudstack 4.9-oobm-password-fix

Alternatively you can review and apply these changes as the patch at:

    https://github.com/apache/cloudstack/pull/1620.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

    This closes #1620
    
----
commit 8ac3c883cd61fa0e18498aaae4e4f519743295c4
Author: Rohit Yadav <ro...@shapeblue.com>
Date:   2016-07-11T11:36:09Z

    oobm: simply change password transactional logic
    
    - Simplifies change password transactional logic without using pessmistic locks
    - Adds a re-enter password field in the UI to valid ipmi/oobm password
    
    Signed-off-by: Rohit Yadav <ro...@shapeblue.com>

----


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] cloudstack pull request #1620: oobm: simply change password transactional lo...

Posted by asfgit <gi...@git.apache.org>.
Github user asfgit closed the pull request at:

    https://github.com/apache/cloudstack/pull/1620


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] cloudstack pull request #1620: oobm: simply change password transactional lo...

Posted by rhtyd <gi...@git.apache.org>.
GitHub user rhtyd reopened a pull request:

    https://github.com/apache/cloudstack/pull/1620

    oobm: simply change password transactional logic

    - Simplifies change password transactional logic without using pessmistic locks
    - Adds a re-enter password field in the UI to valid ipmi/oobm password

You can merge this pull request into a Git repository by running:

    $ git pull https://github.com/shapeblue/cloudstack 4.9-oobm-password-fix

Alternatively you can review and apply these changes as the patch at:

    https://github.com/apache/cloudstack/pull/1620.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

    This closes #1620
    
----
commit 8ac3c883cd61fa0e18498aaae4e4f519743295c4
Author: Rohit Yadav <ro...@shapeblue.com>
Date:   2016-07-11T11:36:09Z

    oobm: simply change password transactional logic
    
    - Simplifies change password transactional logic without using pessmistic locks
    - Adds a re-enter password field in the UI to valid ipmi/oobm password
    
    Signed-off-by: Rohit Yadav <ro...@shapeblue.com>

----


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] cloudstack issue #1620: oobm: simply change password transactional logic

Posted by blueorangutan <gi...@git.apache.org>.
Github user blueorangutan commented on the issue:

    https://github.com/apache/cloudstack/pull/1620
  
    Trillian test result (trillian-pr1638-29-xenserver-65sp1-cs410): Test | Result | Time
    --- | --- | ---
    test_DeployVmAntiAffinityGroup_in_project | Success | 282.569
    test_create_pvlan_network | Success | 5.224
    test_DeployVmAntiAffinityGroup | Success | 136.402
    test_dedicatePublicIpRange | `marvin.cloudstackException.CloudstackAPIException` | 0.245
    test_deploy_vgpu_enabled_vm | Skipped | 0.004
    test_deploy_vm_from_iso | `exceptions.Exception` | 519.082
    test_01_scale_vm | Success | 5.209
    test_00_deploy_vm_root_resize | Success | 6.418
    test_01_deploy_vm_root_resize | Success | 6.271
    test_02_deploy_vm_root_resize | Success | 6.509
    test_01_router_internal_basic | Success | 0.321
    test_02_router_internal_adv | Success | 0.548
    test_03_restart_network_cleanup | Success | 120.830
    test_04_restart_network_wo_cleanup | Success | 5.404
    test_05_router_basic | Success | 0.032
    test_06_router_advanced | Success | 0.063
    test_07_stop_router | Success | 15.174
    test_08_start_router | Success | 50.420
    test_09_reboot_router | Success | 65.510
    test_deployvm_firstfit | Success | 181.469
    test_deployvm_userconcentrated | Success | 95.964
    test_deployvm_userdispersing | Success | 45.720
    test_deployvm_userdata | Success | 126.146
    test_deployvm_userdata_post | Success | 25.496
    test_01_sys_vm_start | Success | 0.146
    test_02_sys_template_ready | Success | 0.103
    test_01_create_disk_offering | Success | 0.119
    test_02_create_sparse_type_disk_offering | Success | 0.077
    test_04_create_fat_type_disk_offering | Success | 0.071
    test_02_edit_disk_offering | Success | 0.086
    test_03_delete_disk_offering | Success | 0.040
    test_default_role_deletion | Success | 6.490
    test_role_account_acls | Success | 8.130
    test_role_account_acls_multiple_mgmt_servers | Success | 7.846
    test_role_inuse_deletion | Success | 6.309
    test_role_lifecycle_create | Success | 6.329
    test_role_lifecycle_delete | Success | 11.347
    test_role_lifecycle_list | Success | 6.332
    test_role_lifecycle_update | Success | 11.459
    test_role_lifecycle_update_role_inuse | Success | 6.292
    test_rolepermission_lifecycle_concurrent_updates | Success | 6.557
    test_rolepermission_lifecycle_create | Success | 6.329
    test_rolepermission_lifecycle_delete | Success | 6.292
    test_rolepermission_lifecycle_list | Success | 6.408
    test_rolepermission_lifecycle_update | Success | 6.634
    test_01_create_service_offering | Success | 0.087
    test_02_edit_service_offering | Success | 0.078
    test_03_delete_service_offering | Success | 0.042
    test_04_change_offering_small | Success | 126.087
    test_UpdateConfigParamWithScope | Success | 0.145
    ContextSuite context=TestDedicateGuestVlanRange>:setup | `exceptions.AttributeError` | 0.000
    test_01_snapshot_root_disk | Success | 66.466
    test_01_internallb_roundrobin_1VPC_3VM_HTTP_port80 | `exceptions.Exception` | 161.948
    test_01_internallb_roundrobin_1VPC_3VM_HTTP_port80 | `exceptions.Exception` | 172.282
    test_02_internallb_roundrobin_1RVPC_3VM_HTTP_port80 | `exceptions.Exception` | 272.445
    test_02_internallb_roundrobin_1RVPC_3VM_HTTP_port80 | `exceptions.Exception` | 282.629
    test_03_vpc_internallb_haproxy_stats_on_all_interfaces | `exceptions.Exception` | 161.681
    test_03_vpc_internallb_haproxy_stats_on_all_interfaces | `exceptions.Exception` | 166.779
    test_04_rvpc_internallb_haproxy_stats_on_all_interfaces | `exceptions.Exception` | 247.162
    test_04_rvpc_internallb_haproxy_stats_on_all_interfaces | `exceptions.Exception` | 252.263
    test_01_list_sec_storage_vm | Success | 0.092
    test_02_list_cpvm_vm | Success | 0.119
    test_03_ssvm_internals | Success | 3.824
    test_04_cpvm_internals | Success | 0.936
    test_05_stop_ssvm | Success | 138.763
    test_06_stop_cpvm | Success | 196.832
    test_07_reboot_ssvm | Success | 148.905
    test_08_reboot_cpvm | Success | 146.425
    test_09_destroy_ssvm | Success | 198.831
    test_10_destroy_cpvm | Success | 201.686
    test_01_create_iso | Success | 66.416
    test_02_edit_iso | Success | 0.079
    test_03_delete_iso | Success | 95.187
    test_04_extract_Iso | Success | 5.127
    test_05_iso_permissions | Success | 0.056
    test_06_copy_iso | Skipped | 0.000
    test_07_list_default_iso | Success | 0.050
    test_01_list_volumes | Success | 0.025
    test_02_list_templates | Success | 0.040
    test_03_list_snapshots | Success | 0.074
    test_static_role_account_acls | Skipped | 0.020
    test_01_create_lb_rule_src_nat | Success | 208.003
    test_02_create_lb_rule_non_nat | Success | 207.487
    test_assign_and_removal_lb | Success | 148.681
    login_test_saml_user | Success | 21.730
    test_01_create_template | Success | 70.600
    test_CreateTemplateWithDuplicateName | Success | 115.968
    test_02_edit_template | Success | 90.217
    test_03_delete_template | Success | 5.106
    test_04_extract_template | Success | 5.164
    test_05_template_permissions | Success | 0.054
    test_06_copy_template | Skipped | 0.000
    test_07_list_public_templates | Success | 0.031
    test_08_list_system_templates | Success | 0.023
    test_nic_secondaryip_add_remove | Success | 152.613
    test_network_acl | Success | 121.390
    test_delete_account | Success | 248.156
    test_01_port_fwd_on_src_nat | Success | 83.818
    test_02_port_fwd_on_non_src_nat | Success | 70.787
    test_public_ip_admin_account | Success | 40.277
    test_public_ip_user_account | Success | 10.269
    test_reboot_router | Success | 580.068
    test_releaseIP | Success | 223.083
    test_network_rules_acquired_public_ip_1_static_nat_rule | Success | 102.652
    test_network_rules_acquired_public_ip_2_nat_rule | Success | 76.512
    test_network_rules_acquired_public_ip_3_Load_Balancer_Rule | Success | 86.547
    test_vm_nic_adapter_vmxnet3 | Skipped | 0.000
    test_advZoneVirtualRouter | Success | 0.022
    test_deploy_vm | Success | 0.030
    test_deploy_vm_multiple | Success | 212.751
    test_01_stop_vm | Success | 30.253
    test_02_start_vm | Success | 20.209
    test_03_reboot_vm | Success | 10.160
    test_06_destroy_vm | Success | 10.141
    test_07_restore_vm | Success | 0.110
    test_08_migrate_vm | Success | 65.935
    test_09_expunge_vm | Success | 605.177
    test_10_attachAndDetach_iso | Success | 111.993
    test_01_nic | Success | 771.804
    test_extendPhysicalNetworkVlan | `exceptions.AttributeError` | 0.027
    test_01_test_vm_volume_snapshot | `marvin.cloudstackException.CloudstackAPIException` | 126.089
    test_01_create_vm_snapshots | Success | 100.764
    test_02_revert_vm_snapshots | Success | 211.420
    test_03_delete_vm_snapshots | Success | 285.208
    test_oobm_background_powerstate_sync | `Failure` | 3016.086
    test_oobm_configure_default_driver | Success | 0.097
    test_oobm_configure_invalid_driver | Success | 0.081
    test_oobm_disable_feature_invalid | Success | 0.178
    test_oobm_disable_feature_valid | Success | 0.194
    test_oobm_enable_feature_invalid | Success | 0.108
    test_oobm_enable_feature_valid | Success | 5.178
    test_oobm_enabledisable_across_clusterzones | `exceptions.Exception` | 357.263
    test_oobm_issue_power_cycle | `exceptions.Exception` | 10.241
    test_oobm_issue_power_off | `exceptions.Exception` | 10.244
    test_oobm_issue_power_on | `exceptions.Exception` | 10.262
    test_oobm_issue_power_reset | `exceptions.Exception` | 10.333
    test_oobm_issue_power_soft | `exceptions.Exception` | 10.311
    test_oobm_issue_power_status | `exceptions.Exception` | 10.249
    test_oobm_multiple_mgmt_server_ownership | `Failure` | 3048.281
    test_oobm_zchange_password | `exceptions.Exception` | 10.279
    test_UpdateStorageOverProvisioningFactor | Success | 0.139
    test_01_create_volume | Success | 392.563
    test_02_attach_volume | Success | 10.775
    test_03_download_attached_volume | Success | 15.335
    test_04_delete_attached_volume | Success | 10.200
    test_05_detach_volume | Success | 100.286
    test_06_download_detached_volume | Success | 30.388
    test_07_resize_fail | Success | 100.979
    test_08_resize_volume | Success | 95.902
    test_09_delete_detached_volume | Success | 15.662
    test_isolate_network_password_server | Success | 35.914
    test_createPortablePublicIPAcquire | Success | 15.609
    test_createPortablePublicIPRange | Success | 15.315
    test_01_create_redundant_VPC_2tiers_4VMs_4IPs_4PF_ACL | `Failure` | 696.990
    test_02_redundant_VPC_default_routes | Success | 683.697
    test_03_create_redundant_VPC_1tier_2VMs_2IPs_2PF_ACL_reboot_routers | Success | 692.679
    test_04_rvpc_network_garbage_collector_nics | `Failure` | 3890.670
    test_05_rvpc_multi_tiers | `Failure` | 579.303
    test_01_primary_storage_iscsi | `marvin.cloudstackException.CloudstackAPIException` | 0.079
    test_01_primary_storage_nfs | Success | 38.788
    test_01_vpc_privategw_acl | `Failure` | 86.872
    test_02_vpc_privategw_static_routes | `Failure` | 167.482
    test_03_vpc_privategw_restart_vpc_cleanup | `Failure` | 182.436
    test_04_rvpc_privategw_static_routes | `Failure` | 358.442
    test_01_VPC_nics_after_destroy | Success | 583.436
    test_02_VPC_default_routes | Success | 405.551
    test_createRegion | Success | 0.043
    test_01_reset_vm_on_reboot | Success | 85.636
    test_01_updatevolumedetail | Success | 0.081
    test_router_dhcphosts | Success | 107.303
    test_02_routervm_iptables_policies | Success | 166.938
    test_01_single_VPC_iptables_policies | Success | 330.357
    test_01_isolate_network_FW_PF_default_routes_egress_true | Success | 289.032
    test_02_isolate_network_FW_PF_default_routes_egress_false | Success | 301.121
    test_01_RVR_Network_FW_PF_SSH_default_routes_egress_true | `Failure` | 486.154
    test_02_RVR_Network_FW_PF_SSH_default_routes_egress_false | `Failure` | 491.002
    test_03_RVR_Network_check_router_state | Success | 288.226
    test_01_redundant_vpc_site2site_vpn | `Failure` | 448.533
    test_01_vpc_remote_access_vpn | `exceptions.UnboundLocalError` | 0.079
    test_01_vpc_site2site_vpn | `Failure` | 232.181
    Trillian env - trillian-pr1638-29-xenserver-65sp1-cs410, Job ID 29


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] cloudstack issue #1620: oobm: simply change password transactional logic

Posted by rhtyd <gi...@git.apache.org>.
Github user rhtyd commented on the issue:

    https://github.com/apache/cloudstack/pull/1620
  
    @blueorangutan kick


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] cloudstack pull request #1620: oobm: simply change password transactional lo...

Posted by rhtyd <gi...@git.apache.org>.
Github user rhtyd closed the pull request at:

    https://github.com/apache/cloudstack/pull/1620


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] cloudstack issue #1620: oobm: simply change password transactional logic

Posted by murali-reddy <gi...@git.apache.org>.
Github user murali-reddy commented on the issue:

    https://github.com/apache/cloudstack/pull/1620
  
    LGTM on code review, have not tested it.


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] cloudstack pull request #1620: oobm: simply change password transactional lo...

Posted by rhtyd <gi...@git.apache.org>.
Github user rhtyd closed the pull request at:

    https://github.com/apache/cloudstack/pull/1620


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] cloudstack issue #1620: oobm: simply change password transactional logic

Posted by borisstoyanov <gi...@git.apache.org>.
Github user borisstoyanov commented on the issue:

    https://github.com/apache/cloudstack/pull/1620
  
    LGTM


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] cloudstack issue #1620: oobm: simply change password transactional logic

Posted by blueorangutan <gi...@git.apache.org>.
Github user blueorangutan commented on the issue:

    https://github.com/apache/cloudstack/pull/1620
  
    Packaging result: \u2714centos6 \u2714centos7 \u2716debian repo: http://packages.shapeblue.com/cloudstack/pr/1620


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] cloudstack pull request #1620: oobm: simply change password transactional lo...

Posted by rhtyd <gi...@git.apache.org>.
Github user rhtyd closed the pull request at:

    https://github.com/apache/cloudstack/pull/1620


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---