iishitahere commented on PR #10060: URL: https://github.com/apache/cloudstack/pull/10060#issuecomment-2573001436
> @iishitahere I see you've multiple PRs which are trying to solve the issue - #10007 The behaviour, though confusing is already mentioned in the code/documentation, > > https://github.com/apache/cloudstack/blob/db2e89a3a2c9e8df2d24ed3e6aeae280bf22f0ba/server/src/main/java/com/cloud/network/NetworkModelImpl.java#L1045-L1051 > > If you're looking to simplify it maybe first you need to discuss your approach to prevent re-works. You may discuss with others on the issue itself or start a thread at dev mailing list. > > As I understand you're interested in contributing to the project and still getting started, I would suggest you to look into this guide to get started with setup and development, https://github.com/shapeblue/hackerbook/tree/main Maybe also start with a more simpler issue which may not require an actual hypervisor to develop/test and simulator can be used. I see one here, #10103. It has proper reproduction steps and which you can do on a simulator-based env. You've already added your interest on #10122. You can deploy your simulator dev environment for it and it may require changing some .vue file and adding/updating key in en.json translation file. Hi @shwstppr, I sincerely apologize for not discussing my approach before submitting the multiple PRs related to issue #10007. I understand the importance of having a clear and coordinated plan, and I realize this could have led to unnecessary rework. I will make sure to avoid this in the future and will discuss my approach before starting any new work. Thank you for the helpful suggestion and for pointing me to the documentation and setup guide. I’ll make sure to review everything more thoroughly moving forward. I appreciate your support and feedback, and I’ll consider your advice to start with simpler issues like #10103. Thanks again! Best regards, Ishita Jaiswal -- 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]
