You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@cloudstack.apache.org by pavanb018 <gi...@git.apache.org> on 2016/01/08 11:05:41 UTC
[GitHub] cloudstack pull request: Automation for CLOUDSTACK-9217
GitHub user pavanb018 opened a pull request:
https://github.com/apache/cloudstack/pull/1322
Automation for CLOUDSTACK-9217
This script automates the below functionality:
CloudStack should block migration to a Storage pool which is in maintenance mode.
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/pavanb018/cloudstack cs9217
Alternatively you can review and apply these changes as the patch at:
https://github.com/apache/cloudstack/pull/1322.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 #1322
----
commit 0c014cd6540bbfab3c5bba4e18fd7f3e0f655c60
Author: pavanb018 <pa...@gmail.com>
Date: 2016-01-08T09:51:34Z
Automation for CLOUDSTACK-9217
----
---
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: Automation for CLOUDSTACK-9217 script to ...
Posted by DaanHoogland <gi...@git.apache.org>.
Github user DaanHoogland commented on the pull request:
https://github.com/apache/cloudstack/pull/1322#issuecomment-171350897
I have no time to look in this further right now. maybe you recognise the problem right away.
---
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: Automation for CLOUDSTACK-9217
Posted by DaanHoogland <gi...@git.apache.org>.
Github user DaanHoogland commented on the pull request:
https://github.com/apache/cloudstack/pull/1322#issuecomment-170353002
@pavanb018 I like this code. Can you point me to the (minimal) marvin definition of the environment needed to run this?
---
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: Automation for CLOUDSTACK-9217 script to ...
Posted by bvbharatk <gi...@git.apache.org>.
Github user bvbharatk commented on the pull request:
https://github.com/apache/cloudstack/pull/1322#issuecomment-222125996
### ACS CI BVT Run
**Sumarry:**
Build Number 64
Hypervisor xenserver
NetworkType Advanced
Passed=70
Failed=3
Skipped=3
_Link to logs Folder (search by build_no):_ https://www.dropbox.com/sh/yj3wnzbceo9uef2/AAB6u-Iap-xztdm6jHX9SjPja?dl=0
**Failed tests:**
* test_scale_vm.py
* ContextSuite context=TestScaleVm>:setup Failing since 2 runs
* test_deploy_vm_iso.py
* test_deploy_vm_from_iso Failing since 6 runs
* test_vm_life_cycle.py
* test_10_attachAndDetach_iso Failing since 2 runs
**Skipped tests:**
test_vm_nic_adapter_vmxnet3
test_static_role_account_acls
test_deploy_vgpu_enabled_vm
**Passed test suits:**
test_deploy_vm_with_userdata.py
test_affinity_groups_projects.py
test_portable_publicip.py
test_vpc_vpn.py
test_over_provisioning.py
test_global_settings.py
test_service_offerings.py
test_routers_iptables_default_policy.py
test_routers.py
test_reset_vm_on_reboot.py
test_snapshots.py
test_deploy_vms_with_varied_deploymentplanners.py
test_list_ids_parameter.py
test_public_ip_range.py
test_multipleips_per_nic.py
test_regions.py
test_affinity_groups.py
test_network_acl.py
test_pvlan.py
test_volumes.py
test_nic.py
test_deploy_vm_root_resize.py
test_resource_detail.py
test_secondary_storage.py
test_disk_offerings.py
---
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: Automation for CLOUDSTACK-9217 script to ...
Posted by DaanHoogland <gi...@git.apache.org>.
Github user DaanHoogland commented on the pull request:
https://github.com/apache/cloudstack/pull/1322#issuecomment-171349179
@pavanb018 no luck
[1322.results.txt](https://github.com/apache/cloudstack/files/89168/1322.results.txt)
---
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: Automation for CLOUDSTACK-9217 script to ...
Posted by pavanb018 <gi...@git.apache.org>.
Github user pavanb018 commented on the pull request:
https://github.com/apache/cloudstack/pull/1322#issuecomment-171534991
From the results it looks like , suitable pools for migration for the volume returned no pools...
I too didn't look at the file in detail. If you have setup you can quickly verify from UI if trying to migrate root disk lists the other pool in the setup. If that's the case then issue might be with script else it might be with the setup
---
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: Automation for CLOUDSTACK-9217 script to ...
Posted by pavanb018 <gi...@git.apache.org>.
Github user pavanb018 commented on the pull request:
https://github.com/apache/cloudstack/pull/1322#issuecomment-171271657
@DaanHoogland You will need two primary storage pools to test this , no advanced setup is needed to test the script. The script will skip if there are less than two storage pools in the environment.
---
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: Automation for CLOUDSTACK-9217
Posted by pavanb018 <gi...@git.apache.org>.
Github user pavanb018 commented on the pull request:
https://github.com/apache/cloudstack/pull/1322#issuecomment-169954673
Test Results:
==========
Trying to migrate a volume to a pool in maintenance mode should fail ... === TestName: test_02_migrate_volume_to_maintenance_pool | Status : SUCCESS ===
ok
----------------------------------------------------------------------
Ran 1 test in 173.482s
OK
---
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: Automation for CLOUDSTACK-9217 script to ...
Posted by bvbharatk <gi...@git.apache.org>.
Github user bvbharatk commented on the pull request:
https://github.com/apache/cloudstack/pull/1322#issuecomment-201706492
_Link to logs Folder (search by build_no):_ https://www.dropbox.com/sh/yj3wnzbceo9uef2/AAB6u-Iap-xztdm6jHX9SjPja?dl=0
### ACS CI BVT Run
**Sumarry:**
Build Number 130
Hypervisor xenserver
NetworkType Advanced
Passed=101
Failed=4
Skipped=5
**Failed tests:**
* <nose.suite
* ContextSuite context=TestVpcRemoteAccessVpn>:setup Failing since 2 runs
* ContextSuite context=TestResetVmOnReboot>:setup Failing since 2 runs
* integration.smoke.test_volumes.TestCreateVolume
* test_01_create_volume Failed
* test_06_download_detached_volume Failing since 2 runs
**Skipped tests:**
test_vm_nic_adapter_vmxnet3
test_deploy_vgpu_enabled_vm
ContextSuite context=TestCreateTemplate>:setup
test_06_copy_template
test_06_copy_iso
**Passed test suits:**
integration.smoke.test_deploy_vm_with_userdata.TestDeployVmWithUserData
integration.smoke.test_affinity_groups_projects.TestDeployVmWithAffinityGroup
integration.smoke.test_portable_publicip.TestPortablePublicIPAcquire
integration.smoke.test_over_provisioning.TestUpdateOverProvision
integration.smoke.test_global_settings.TestUpdateConfigWithScope
integration.smoke.test_guest_vlan_range.TestDedicateGuestVlanRange
integration.smoke.test_scale_vm.TestScaleVm
integration.smoke.test_service_offerings.TestCreateServiceOffering
integration.smoke.test_loadbalance.TestLoadBalance
integration.smoke.test_routers.TestRouterServices
integration.smoke.test_snapshots.TestSnapshotRootDisk
integration.smoke.test_deploy_vms_with_varied_deploymentplanners.TestDeployVmWithVariedPlanners
integration.smoke.test_network.TestDeleteAccount
integration.smoke.test_non_contigiousvlan.TestUpdatePhysicalNetwork
integration.smoke.test_deploy_vm_iso.TestDeployVMFromISO
integration.smoke.test_public_ip_range.TestDedicatePublicIPRange
integration.smoke.test_multipleips_per_nic.TestDeployVM
integration.smoke.test_regions.TestRegions
integration.smoke.test_affinity_groups.TestDeployVmWithAffinityGroup
integration.smoke.test_network_acl.TestNetworkACL
integration.smoke.test_pvlan.TestPVLAN
integration.smoke.test_ssvm.TestSSVMs
integration.smoke.test_nic.TestNic
integration.smoke.test_deploy_vm_root_resize.TestDeployVM
integration.smoke.test_resource_detail.TestResourceDetail
integration.smoke.test_secondary_storage.TestSecStorageServices
integration.smoke.test_vm_life_cycle.TestDeployVM
integration.smoke.test_disk_offerings.TestCreateDiskOffering
---
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: Automation for CLOUDSTACK-9217
Posted by DaanHoogland <gi...@git.apache.org>.
Github user DaanHoogland commented on the pull request:
https://github.com/apache/cloudstack/pull/1322#issuecomment-170353332
Also I would like to see the commit and PR have a different name like:
```CLOUDSTACK-9217 test block volume migration to a pool in maintenance mode```
---
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: Automation for CLOUDSTACK-9217 script to ...
Posted by pavanb018 <gi...@git.apache.org>.
Github user pavanb018 commented on the pull request:
https://github.com/apache/cloudstack/pull/1322#issuecomment-171298675
The above configuration is good. No need to have two clusters
---
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: Automation for CLOUDSTACK-9217 script to ...
Posted by DaanHoogland <gi...@git.apache.org>.
Github user DaanHoogland commented on the pull request:
https://github.com/apache/cloudstack/pull/1322#issuecomment-171290722
@pavanb018 , running the test with a definition like below or should I have two clusters as well?
```
"clusters": [
{
"clustername": "MCCT-KVM-1",
"hypervisor": "KVM",
"hosts": [
{
"username": "root",
"url": "http://kvm1",
"password": "password"
},
{
"username": "root",
"url": "http://kvm2",
"password": "password"
}
],
"clustertype": "CloudManaged",
"primaryStorages": [
{
"url": "nfs://192.168.22.1:/data/storage/primary/MCCT-KVM-1",
"name": "MCCT-KVM-1-primary"
},
{
"url": "nfs://192.168.22.1:/data/storage/primary/MCCT-KVM-2",
"name": "MCCT-KVM-2-primary"
}
]
}
```
---
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.
---