Github user asfgit closed the pull request at:
https://github.com/apache/cloudstack/pull/1045
---
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
Github user resmo commented on the pull request:
https://github.com/apache/cloudstack/pull/1045#issuecomment-158350734
@remibergsma @bhaisaab, no, not yet, created a patch for master before
release of 4.6 #1042 but I was told it is not necessary, script is not used,
was replaced by a
Github user bhaisaab commented on the pull request:
https://github.com/apache/cloudstack/pull/1045#issuecomment-158349810
LGTM, @remibergsma did we have the fix on 4.6/master already?
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHu
Github user resmo commented on the pull request:
https://github.com/apache/cloudstack/pull/1045#issuecomment-158030442
Our test case on ACS 4.5.1:
# Before fix
## vanilla router deployed:
~~~
# related script
ae693b5ef2d9b1fed1307f6b38213823 /opt/cloud/bin/edit
GitHub user resmo opened a pull request:
https://github.com/apache/cloudstack/pull/1045
[4.5] CLOUDSTACK-9042: VR: DHCP: fix missing DHCP host enties
in /etc/dhcpentries.txt we had:
02:00:1e:07:01:53,set:10_102_92_5,10.102.92.5,songlog-1,infinite
02:00:0b:a2:00:3d,set:10
