bradh352 commented on issue #12574: URL: https://github.com/apache/cloudstack/issues/12574#issuecomment-3853093933
> [@bradh352](https://github.com/bradh352) , seen the pr and looks good to me (pending testing) > > 1. I do see a slight functional problem (in hindsight) akin to your reply to [@chunkyen](https://github.com/chunkyen) ; Do we really want all networks to have the same proxy timeout? I could imagine different applications requiring different settings. That said, this is certainly a good enhancement Right, this is sort of taking a sledgehammer to the situation. I could see people wanting to set this to an hour globally though for everything. 50s seems really short for just about all scenarios to me. > 3. did you test the upgrade scenario? it seems not exiting, but it might .> If by upgrade scenario, you mean the migration from pre-feature to post-feature. It appears to have just worked for me. It showed up in the UI with the default value. I'm pretty sure using the newer ConfigKey framework handled that behind the scenes. > I don’t see these remarks as a blocker to your PR atm. -- 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. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
