Skip to content

Conversation

@nitin-maharana
Copy link
Contributor

In Basic zone, while deploying a VM it fails with an unexpected exception.
This is reproducible only when java asserts are enabled.

Copy link
Contributor

@jburwell jburwell left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@nitin-maharana always love a small fix to a big problem :)

This fix would be a very useful for LTS users. Could you change the base branch to 4.9?

Finally, does an existing Marvin test case verify this fix? If not, could you please add one?

@nitin-maharana nitin-maharana changed the base branch from master to 4.9 December 6, 2016 08:09
@nitin-maharana
Copy link
Contributor Author

@jburwell : Switched the base branch to 4.9

@nitin-maharana
Copy link
Contributor Author

ping @sateesh-chodapuneedi @karuturi

@koushik-das
Copy link
Contributor

Code changes LGTM
@nitin-maharana Can you check on the travis failure?

@nitin-maharana
Copy link
Contributor Author

@koushik-das : No failure now.

@karuturi
Copy link
Member

karuturi commented Mar 6, 2017

code LGTM.
@nitin-maharana Please initiate internal CI run on @cloudmonger.

@nitin-maharana
Copy link
Contributor Author

@karuturi : Currently running. Will post once completes.

@cloudmonger
Copy link

ACS CI BVT Run

Sumarry:
Build Number 432
Hypervisor xenserver
NetworkType Advanced
Passed=105
Failed=0
Skipped=7

Link to logs Folder (search by build_no): https://www.dropbox.com/sh/yj3wnzbceo9uef2/AAB6u-Iap-xztdm6jHX9SjPja?dl=0

Failed tests:

Skipped tests:
test_01_test_vm_volume_snapshot
test_vm_nic_adapter_vmxnet3
test_static_role_account_acls
test_11_ss_nfs_version_on_ssvm
test_nested_virtualization_vmware
test_3d_gpu_support
test_deploy_vgpu_enabled_vm

Passed test suits:
test_deploy_vm_with_userdata.py
test_affinity_groups_projects.py
test_portable_publicip.py
test_over_provisioning.py
test_global_settings.py
test_scale_vm.py
test_service_offerings.py
test_routers_iptables_default_policy.py
test_loadbalance.py
test_routers.py
test_reset_vm_on_reboot.py
test_deploy_vms_with_varied_deploymentplanners.py
test_network.py
test_router_dns.py
test_non_contigiousvlan.py
test_login.py
test_deploy_vm_iso.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_vm_life_cycle.py
test_routers_network_ops.py
test_disk_offerings.py

@nitin-maharana
Copy link
Contributor Author

tag:mergeready

@karuturi karuturi added this to the 4.10.0.0 milestone Apr 27, 2017
@koushik-das koushik-das modified the milestones: 4.11, 4.10.0.0 Apr 27, 2017
@karuturi karuturi merged commit 8854d4c into apache:master May 2, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants