iishitahere commented on issue #10007: URL: https://github.com/apache/cloudstack/issues/10007#issuecomment-2506652864
> Hello, @luganofer and @iishitahere > > From what I remember of the code, the throttling is applied differently if the NIC of the VM is the default one or not. > > https://github.com/apache/cloudstack/blob/db2e89a3a2c9e8df2d24ed3e6aeae280bf22f0ba/server/src/main/java/com/cloud/network/NetworkModelImpl.java#L1037 > > For the default NIC, the network throttling is applied based on the compute offering network bandwidth of the VM. If there is no limit set in the compute offering, the value of the configuration `vm.network.throttling.rate` will be used. > > For additional NICs, the network throttling is applied based on the network offering bandwidth. If there is no limit set in the network offering, then the value of the configuration `network.throttling.rate` will be used. > > To be fair, this behavior is kind of confusing, and maybe a discussion should be done before any actual implementation. > > What do you guys think? cc @DaanHoogland @JoaoJandre @GutoVeronezi @weizhouapache Hello @BryanMLima , @luganofer Thank you for pointing out the behavior of network throttling for default and additional NICs. I agree that the current approach can be somewhat confusing. I’d like to explore this issue further to clarify and potentially improve the implementation. Here’s how I plan to proceed: Analyze the existing logic in NetworkModelImpl.java to ensure a comprehensive understanding of the throttling rules for both NIC types. Propose a consistent or more intuitive mechanism for throttling, balancing functionality and ease of configuration. Incorporate input from the team to ensure alignment with the broader system design. Could you assign this issue to me so I can take the lead on the discussion and possible implementation? I’m excited to contribute! Looking forward to your thoughts. Best regards, Ishita -- 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]
