cloudstack-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From koushik-das <...@git.apache.org>
Subject [GitHub] cloudstack pull request: CLOUDSTACK-8485: listAPIs are taking too ...
Date Mon, 02 Nov 2015 09:39:44 GMT
GitHub user koushik-das opened a pull request:

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

    CLOUDSTACK-8485: listAPIs are taking too long to return results

    - Removed regex. based search/replace of sensitive data on API response introduced as
part of commit b0c6d4734724358df97b6fa4d8c5beb0f447745e
    - Added new response serializer to skip sensitive data from getting logged based on annotation
present in resposne object fields
    - Added annotation (@LogLevel(Log4jLevel.Off)) to sensitive response object fields
    
    Ran the following tests on simulator:
    
    test_vm_life_cycle.py
    
    Test advanced zone virtual router ... === TestName: test_advZoneVirtualRouter | Status
: SUCCESS ===
    ok
    Test Deploy Virtual Machine ... === TestName: test_deploy_vm | Status : SUCCESS ===
    ok
    Test Multiple Deploy Virtual Machine ... === TestName: test_deploy_vm_multiple | Status
: SUCCESS ===
    ok
    Test Stop Virtual Machine ... === TestName: test_01_stop_vm | Status : SUCCESS ===
    ok
    Test Start Virtual Machine ... === TestName: test_02_start_vm | Status : SUCCESS ===
    ok
    Test Reboot Virtual Machine ... === TestName: test_03_reboot_vm | Status : SUCCESS ===
    ok
    Test destroy Virtual Machine ... === TestName: test_06_destroy_vm | Status : SUCCESS ===
    ok
    Test recover Virtual Machine ... === TestName: test_07_restore_vm | Status : SUCCESS ===
    ok
    Test migrate VM ... === TestName: test_08_migrate_vm | Status : SUCCESS ===
    ok
    Test destroy(expunge) Virtual Machine ... === TestName: test_09_expunge_vm | Status :
SUCCESS ===
    ok
    
    ----------------------------------------------------------------------
    Ran 10 tests in 306.429s
    
    OK
    
    test_volumes.py
    
    Download a Volume attached to a VM ... === TestName: test_03_download_attached_volume
| Status : SUCCESS ===
    ok
    Delete a Volume attached to a VM ... === TestName: test_04_delete_attached_volume | Status
: SUCCESS ===
    ok
    Detach a Volume attached to a VM ... === TestName: test_05_detach_volume | Status : SUCCESS
===
    ok
    Delete a Volume unattached to an VM ... === TestName: test_09_delete_detached_volume |
Status : SUCCESS ===
    ok
    
    ----------------------------------------------------------------------
    Ran 4 tests in 184.132s
    
    OK
    
    test_network.py
    
    Test for delete account ... === TestName: test_delete_account | Status : SUCCESS ===
    ok
    Test for Associate/Disassociate public IP address for admin account ... === TestName:
test_public_ip_admin_account | Status : SUCCESS ===
    ok
    Test for Associate/Disassociate public IP address for user account ... === TestName: test_public_ip_user_account
| Status : SUCCESS ===
    ok
    Test for release public IP address ... === TestName: test_releaseIP | Status : SUCCESS
===
    ok
    
    ----------------------------------------------------------------------
    Ran 4 tests in 783.726s
    
    OK
    
    test_routers.py
    
    Test router internal advanced zone ... SKIP: Marvin configuration has no host credentials
                           to check router services
    Test restart network ... === TestName: test_03_restart_network_cleanup | Status : SUCCESS
===
    ok
    Test router basic setup ... === TestName: test_05_router_basic | Status : SUCCESS ===
    ok
    Test router advanced setup ... === TestName: test_06_router_advanced | Status : SUCCESS
===
    ok
    Test stop router ... === TestName: test_07_stop_router | Status : SUCCESS ===
    ok
    Test start router ... === TestName: test_08_start_router | Status : SUCCESS ===
    ok
    Test reboot router ... === TestName: test_09_reboot_router | Status : SUCCESS ===
    ok
    
    ----------------------------------------------------------------------
    Ran 7 tests in 42.958s
    
    OK (SKIP=1)
    
    test_global_settings.py
    
    test update configuration setting at zone level scope ... === TestName: test_UpdateConfigParamWithScope
| Status : SUCCESS ===
    ok
    
    ----------------------------------------------------------------------
    Ran 1 test in 0.127s
    
    OK
    
    test_resource_detail.py
    
    Test volume detail ... === TestName: test_01_updatevolumedetail | Status : SUCCESS ===
    ok
    
    ----------------------------------------------------------------------
    Ran 1 test in 11.492s
    
    OK


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

    $ git pull https://github.com/koushik-das/cloudstack CLOUDSTACK-8485

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

    https://github.com/apache/cloudstack/pull/1021.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 #1021
    
----
commit a32c91f290f3d7300c4a0d40455f28cf99f4e769
Author: Koushik Das <koushik@apache.org>
Date:   2015-11-02T09:29:00Z

    CLOUDSTACK-8485: listAPIs are taking too long to return results
    - Removed regex. based search/replace of sensitive data on API response introduced as
part of commit b0c6d4734724358df97b6fa4d8c5beb0f447745e
    - Added new response serializer to skip sensitive data from getting logged based on annotation
present in resposne object fields
    - Added annotation (@LogLevel(Log4jLevel.Off)) to sensitive response object fields

----


---
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.
---

Mime
View raw message