DaanHoogland commented on issue #10163:
URL: https://github.com/apache/cloudstack/issues/10163#issuecomment-2626629814

   > Hi [@DaanHoogland](https://github.com/DaanHoogland) 
[@weizhouapache](https://github.com/weizhouapache) ,
   > 
   > I hope you're doing well.
   > 
   > I have submitted a PR that addresses the issue related to static route 
configuration in the secondary storage VM (NfsSecondaryStorageResource). This 
fix resolves the problem where the route addition fails due to a null storage 
IP, causing errors like ip route add ... via null.
   
   thanks @iishitahere , I have editted #10304 , in the assumption that that is 
the PR you mention. We will get to reviewing it.  I have added the reference to 
this issue in it.
   
   One other initial remark; you have based the fix on main, but the issue is 
reported on 4.20. Do you think you could rebase your solution on the 4.20 
release branch?
   
   > I sincerely apologize for not being active in recent weeks. Due to my 
exams and some unforeseen circumstances, I was unable to contribute as 
consistently as I had hoped. However, I am fully available now to respond to 
feedback and make any changes necessary to improve the PR.
   
   Please don't apologise, some of us have other duties at work, others have 
studies, and again others work completely in their own time. But thanks for 
letting us know anyway.
   


-- 
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]

Reply via email to