cloudstack-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Sheng Yang" <sh...@yasker.org>
Subject Re: Review Request 12810: DnsMasqConfigurator need to be rewrite in bash script
Date Tue, 23 Jul 2013 20:51:35 GMT

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/12810/#review23713
-----------------------------------------------------------



patches/systemvm/debian/config/root/dnsmasq.sh
<https://reviews.apache.org/r/12810/#comment47591>

    dns2 can be null.



patches/systemvm/debian/config/root/dnsmasq.sh
<https://reviews.apache.org/r/12810/#comment47592>

    dhcp-option=tag.* is not in the log



patches/systemvm/debian/config/root/dnsmasq.sh
<https://reviews.apache.org/r/12810/#comment47594>

    Backup the failure configures would also help to investigate in the future.



patches/systemvm/debian/config/root/dnsmasq.sh
<https://reviews.apache.org/r/12810/#comment47593>

    No, don't override the original result. It failed, then we need to fail loudly. The overriding
here would cover the fact that it's failed to apply the new config.


It's in much better shape than previous edition. Please do more testing, e.g. VPC, multiple
guest network for user VM. It's likely we can get it in tomorrow.

Thanks!

- Sheng Yang


On July 23, 2013, 1:07 p.m., bharat kumar wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/12810/
> -----------------------------------------------------------
> 
> (Updated July 23, 2013, 1:07 p.m.)
> 
> 
> Review request for cloudstack, Alena Prokharchyk and Sheng Yang.
> 
> 
> Bugs: CLOUDSTACK-3694
> 
> 
> Repository: cloudstack-git
> 
> 
> Description
> -------
> 
> https://issues.apache.org/jira/browse/CLOUDSTACK-3694
> 
> wrote the dnsmasq config in bash instead of creating the config file in java and overwriting.

> 
> 
> Diffs
> -----
> 
>   core/src/com/cloud/agent/api/routing/DnsMasqConfigCommand.java 521ad70 
>   core/src/com/cloud/agent/resource/virtualnetwork/VirtualRoutingResource.java 0b26220

>   core/src/com/cloud/network/DnsMasqConfigurator.java 3fc61df 
>   patches/systemvm/debian/config/etc/init.d/cloud-early-config c04ff90 
>   patches/systemvm/debian/config/root/dnsmasq.sh b70e2d3 
>   plugins/hypervisors/vmware/src/com/cloud/hypervisor/vmware/resource/VmwareResource.java
c7f487e 
>   plugins/hypervisors/xen/src/com/cloud/hypervisor/xen/resource/CitrixResourceBase.java
f80d4b6 
>   scripts/vm/hypervisor/xenserver/vmops f8c0253 
>   server/src/com/cloud/network/router/VirtualNetworkApplianceManagerImpl.java 6c0f7a1

> 
> Diff: https://reviews.apache.org/r/12810/diff/
> 
> 
> Testing
> -------
> 
> Tested on old master using xenserver.
>  Could not test on the latest one as it is broken.
> 
> 
> Thanks,
> 
> bharat kumar
> 
>


Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message