sureshanaparti commented on a change in pull request #5886:
URL: https://github.com/apache/cloudstack/pull/5886#discussion_r790742891



##########
File path: server/src/main/java/com/cloud/resource/DiscovererBase.java
##########
@@ -51,6 +54,16 @@
     protected ResourceManager _resourceMgr;
     @Inject
     protected DataCenterDao _dcDao;
+    @Inject
+    private ImageStoreDao imageStoreDao;
+
+    protected void setSecondaryStorageNfsVersionToParams(Long zoneId, 
Map<String, Object> params) {
+        ImageStoreVO imageStoreInZone = 
imageStoreDao.findOneByZoneAndProtocol(zoneId, "nfs");

Review comment:
       ok @nvazquez, will there be any issues if the nfs version is set based 
on one image store when multiple image stores are added (with different 
versions).




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