Re: Review Request: Add the ability to specify different VIF drivers per traffic type in KVM

[Dave Cahill <dc...@midokura.com>]

Hi,

This patch was submitted 3 days ago, so sending a gentle reminder email as
per ReviewBoard guidelines. :) [1]

If there are any questions or issues, just let me know.

Thanks,
Dave.

[1]
https://cwiki.apache.org/confluence/display/CLOUDSTACK/Review+Board+Guidelines

On Tue, Feb 26, 2013 at 6:49 PM, Dave Cahill <dc...@midokura.com> wrote:

>    This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/9604/
>   Review request for cloudstack and Hugo Trippaers.
> By Dave Cahill.
> Description
>
> Feature spec: https://cwiki.apache.org/confluence/display/CLOUDSTACK/Allow+specification+of+different+VIF+drivers+per+traffic+type+in+KVM
>
> Jira ticket: https://issues.apache.org/jira/browse/CLOUDSTACK-1311
>
> Adds the ability to specify different VIF drivers per traffic type in KVM.
>
>   Testing
>
> Added unit tests to test all variations of this new configuration (no configuration, defaults, override some traffic etc).
>
> Built and deployed, spun up Advanced Isolated network with two VMs, verified internal and external connectivity.
>
>   *Bugs: * CLOUDSTACK-1311
> Diffs
>
>    - plugins/hypervisors/kvm/src/com/cloud/hypervisor/kvm/resource/LibvirtComputingResource.java
>    (99b8723)
>    - plugins/hypervisors/kvm/test/com/cloud/hypervisor/kvm/resource/LibvirtVifDriverTest.java
>    (PRE-CREATION)
>
> View Diff <https://reviews.apache.org/r/9604/diff/>
>