Github user sureshanaparti commented on the issue:
https://github.com/apache/cloudstack/pull/1878
@rafaelweingartner Thanks for reviewing, will work on the changes suggested.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as
Github user sureshanaparti commented on the issue:
https://github.com/apache/cloudstack/pull/1878
@PaulAngus What was the error shown for duplicate MAC in vCenter. Can you
post the log snippet with the error. These changes were tested in vCenter 5.5
and 6.0, didn't observe any issues
Github user PaulAngus commented on the issue:
https://github.com/apache/cloudstack/pull/1878
@sureshanaparti I see in vCenter any deployment of an RVR with a duplicate
MAC, failing with a vCenter error regarding a duplicate MAC.
---
If your project is set up for it, you can reply to
Github user remibergsma commented on the issue:
https://github.com/apache/cloudstack/pull/1878
@sureshanaparti Thanks for the info!
---
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
Github user sureshanaparti commented on the issue:
https://github.com/apache/cloudstack/pull/1878
@remibergsma Same MAC for RVR has been re-introducted as part of
[CLOUDSTACK-985](https://issues.apache.org/jira/browse/CLOUDSTACK-985). It
confirms that peer NICs of RVRs should have
Github user remibergsma commented on the issue:
https://github.com/apache/cloudstack/pull/1878
@sureshanaparti why do the mac addresses need to be the same on both
routers? We're also executing arpings to update our neighbours. Networking wise
there is no need for them to be the
Github user blueorangutan commented on the issue:
https://github.com/apache/cloudstack/pull/1878
Trillian test result (tid-873)
Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7
Total time taken: 33766 seconds
Marvin logs:
Github user borisstoyanov commented on the issue:
https://github.com/apache/cloudstack/pull/1878
@blueorangutan test
---
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
Github user blueorangutan commented on the issue:
https://github.com/apache/cloudstack/pull/1878
@borisstoyanov a Trillian-Jenkins test job (centos7 mgmt + kvm-centos7) has
been kicked to run smoke tests
---
If your project is set up for it, you can reply to this email and have your
Github user blueorangutan commented on the issue:
https://github.com/apache/cloudstack/pull/1878
Packaging result: âcentos6 âcentos7 âdebian. JID-519
---
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
Github user borisstoyanov commented on the issue:
https://github.com/apache/cloudstack/pull/1878
@blueorangutan package
---
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
Github user sureshanaparti commented on the issue:
https://github.com/apache/cloudstack/pull/1878
@borisstoyanov Can you please run trillian tests for this PR?
---
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
Github user sureshanaparti commented on the issue:
https://github.com/apache/cloudstack/pull/1878
@rafaelweingartner Updated the changes.
---
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
Github user sureshanaparti commented on the issue:
https://github.com/apache/cloudstack/pull/1878
@rafaelweingartner Sure. I'll move these lines to a method and push the
changes.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub
Github user rafaelweingartner commented on the issue:
https://github.com/apache/cloudstack/pull/1878
Hi @sureshanaparti
Would you mind extracting the code at lines 1967-1972 to a specific method?
Then, it allows you to write test cases for the newly created method and proper
15 matches
Mail list logo
