andrijapanicsb commented on issue #14585: URL: https://github.com/apache/cloudstack/issues/14585#issuecomment-6707445357
Perhaps just do NOT search anything, as it will also match some old containers etc. (happened with some production use I've seen) - and the optional configuration key inside agent.properties, MUST BE MANDATORY. I don't understand the genius idea of searching the whole file system... Not a good (AI) engineering. Immediate fix - check/fix documentation to explicitly state that the VDDK path is a required setting in the agent.properties (cheap fix) and then, when possible, remove this "engineering art" of doing a search randomly and matching whatever it finds... 🤦♂️ -- 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]
