[GitHub] [cloudstack] andrijapanicsb commented on issue #3186: Add possibility to set KVM MTU size for all NIC

2020-01-03 Thread GitBox
andrijapanicsb commented on issue #3186: Add possibility to set KVM MTU size 
for all NIC
URL: https://github.com/apache/cloudstack/pull/3186#issuecomment-570591658
 
 
   @svenvogel do you think you could work on this, or should we move it to the 
4.15 milestone?


This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] [cloudstack] andrijapanicsb commented on issue #3186: Add possibility to set KVM MTU size for all NIC

2019-12-17 Thread GitBox
andrijapanicsb commented on issue #3186: Add possibility to set KVM MTU size 
for all NIC
URL: https://github.com/apache/cloudstack/pull/3186#issuecomment-566563338
 
 
   I agree with @GabrielBrascher, I've seen some nasty issues with MTU not 
being in place correctly.
   
   I do want an opinion from @rhtyd as well on this one.
   
   That being said, I do need to back out from this PR, due to lack of time 
(instead of silently ignoring it) as it would mean considerable testing effort 
(I can't help much with code obviously).


This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] [cloudstack] andrijapanicsb commented on issue #3186: Add possibility to set KVM MTU size for all NIC

2019-11-18 Thread GitBox
andrijapanicsb commented on issue #3186: Add possibility to set KVM MTU size 
for all NIC
URL: https://github.com/apache/cloudstack/pull/3186#issuecomment-555100288
 
 
   > @andrijapanicsb yes we tested it on a new 4.13 install. seems to work fine.
   
   I was speaking about upgrades, where the config key (older style of code) 
will not be executed during the upgrades and the setting will be missing 
completely.
   


This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] [cloudstack] andrijapanicsb commented on issue #3186: Add possibility to set KVM MTU size for all NIC

2019-11-12 Thread GitBox
andrijapanicsb commented on issue #3186: Add possibility to set KVM MTU size 
for all NIC
URL: https://github.com/apache/cloudstack/pull/3186#issuecomment-553137251
 
 
   @svenvogel I kindly suggest that you guys install clean 4.13 version and 
then upgrade to packages built from this PR, to see if the config key defined 
global setting will be added or not during upgrades. I've seen cases where it 
is NOT added during upgrades


This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services