-
Notifications
You must be signed in to change notification settings - Fork 1.3k
CLOUDSTACK-9583: VR: In CsDhcp.py preseed both hostaname and localhost to resolve to 127.0.0.1 #1757
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
…t to resolve to 127.0.0.1 The VR executes a ip route flush command as part of configurations. This command performs a DNS lookup on the VR hostname. Since the VR does not have a DNS entry, the ip command would wait 5 seconds before timing out and executing the flush operation. This fix adds the VR hostname to /etc/hosts mapped to 127.0.0.1 to answer the DNS lookup – reducing the execution time.
|
@blueorangutan package |
|
@murali-reddy a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress. |
|
Packaging result: ✔centos6 ✔centos7 ✔debian. JID-160 |
|
@blueorangutan test |
|
@murali-reddy a Trillian-Jenkins test job (centos7 mgmt + kvm-centos7) has been kicked to run smoke tests |
jburwell
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM for code review
|
@blueorangutan test matrix |
|
@jburwell a Trillian-Jenkins matrix job (centos6 mgmt + xs65sp1, centos7 mgmt + vmware55u3, centos7 mgmt + kvmcentos7) has been kicked to run smoke tests |
|
Trillian test result (tid-338)
|
|
Trillian test result (tid-339)
|
|
Trillian test result (tid-337)
|
jburwell
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM for code review
|
@rhtyd @swill @abhinandanprateek @karuturi could you review the test results/perform further testing for a second LGTM? I would like to get this PR into 4.8.2.0/4.9.1.0/4.10.0.0 as it greatly improves the performance of VR configuration. |
|
@blueorangutan package |
|
@rhtyd a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress. |
|
Packaging result: ✔centos6 ✔centos7 ✔debian. JID-172 |
|
LGTM, merging this based on code review and test results. Failures in VPC/RVR are known intermittent issues. |
CLOUDSTACK-9583: VR: In CsDhcp.py preseed both hostaname and localhost to resolve to 127.0.0.1 The VR executes a ip route flush command as part of configurations. This command performs a DNS lookup on the VR hostname. Since the VR does not have a DNS entry, the ip command would wait 5 seconds before timing out and executing the flush operation. This fix adds the VR hostname to /etc/hosts mapped to 127.0.0.1 to answer the DNS lookup reducing the execution time. * pr/1757: CLOUDSTACK-9583: VR: In CsDhcp.py preseed both hostaname and localhost to resolve to 127.0.0.1 Signed-off-by: Rohit Yadav <rohit.yadav@shapeblue.com>
borisstoyanov
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, tested the change 127.0.0.1 is added to /etc/hosts.
…clouds' Corrigir alocação de VMs com múltiplos _clusters_ Closes apache#1757 See merge request scclouds/scclouds!721
The VR executes a ip route flush command as part of configurations. This command performs a
DNS lookup on the VR hostname. Since the VR does not have a DNS entry, the ip command would
wait 5 seconds before timing out and executing the flush operation. This fix adds the VR
hostname to /etc/hosts mapped to 127.0.0.1 to answer the DNS lookup – reducing the
execution time.