[CLOUDSTACK-5251] No Error message is displayed when nonexistent NFS secondary storage is added to CS.

• Type: Bug
• Status: Open
• Priority: Major
• Resolution: Unresolved
• Affects Version/s: 4.3.0
• Fix Version/s: Future
• Component/s: Storage Controller
• Security Level: Public (Anyone can view this level - this is the default.)
• Assignee: Disheng Su
• Reporter: manasaveloori
• Created: 25/Nov/13 08:53
• Updated: 23/Jun/14 08:50

Description

Steps:
1. Deploy a CS .
2. After the UI comes up,try to add the another NFS secondary storage which is not existing in the path added.

Observation:

Secondary storage got added in the UI .
Observing the below Debug messages and not ERROR messages in log:
2013-11-25 07:47:28,352 DEBUG [c.c.a.ApiServlet] (catalina-exec-18:ctx-a89b587d) ===START=== 10.252.192.19 – GET command=addImageStore&response=json&sessionkey=pDnp1RLy0kkw5icS9lA1JS34SR8%3D&name=test5&provider=NFS&zoneid=79124a51-3510-4bf7-b17c-4b8e4818b515&url=nfs%3A%2F%2F10.147.28.7%2Fexport%2Fhome%2Fmanasa%2Fdummy&_=1385364241867
2013-11-25 07:47:28,367 INFO [o.a.c.s.d.l.CloudStackImageStoreLifeCycleImpl] (catalina-exec-18:ctx-a89b587d ctx-8dbd09ee) Trying to add a new data store at nfs://10.147.28.7/export/home/manasa/dummy to data center 1
2013-11-25 07:47:28,480 DEBUG [c.c.a.ApiServlet] (catalina-exec-18:ctx-a89b587d ctx-8dbd09ee) ===END=== 10.252.192.19 – GET command=addImageStore&response=json&sessionkey=pDnp1RLy0kkw5icS9lA1JS34SR8%3D&name=test5&provider=NFS&zoneid=79124a51-3510-4bf7-b17c-4b8e4818b515&url=nfs%3A%2F%2F10.147.28.7%2Fexport%2Fhome%2Fmanasa%2Fdummy&_=1385364241867
2013-11-25 07:47:33,608 DEBUG [c.c.a.m.AgentManagerImpl] (AgentManager-Handler-2:null) SeqA 5-325: Processing Seq 5-325: { Cmd , MgmtId: -1, via: 5, Ver: v1, Flags: 11, [{"com.cloud.agent.api.ConsoleProxyLoadReportCommand":{"_proxyVmId":7,"_loadInfo":"

{\n \"connections\": []\n}

","wait":0}}] }
2013-11-25 07:47:33,618 DEBUG [c.c.a.m.AgentManagerImpl] (AgentManager-Handler-2:null) SeqA 5-325: Sending Seq 5-325: { Ans: , MgmtId: 6758231703598, via: 5, Ver: v1, Flags: 100010, [{"com.cloud.agent.api.AgentControlAnswer":{"result":true,"wait":0}}] }
2013-11-25 07:47:38,911 DEBUG [c.c.s.StatsCollector] (StatsCollector-3:ctx-ca134799) StorageCollector is running...
2013-11-25 07:47:38,988 DEBUG [c.c.a.t.Request] (StatsCollector-3:ctx-ca134799) Seq 4-628555880: Received: { Ans: , MgmtId: 6758231703598, via: 4, Ver: v1, Flags: 10,

{ GetStorageStatsAnswer } }
2013-11-25 07:48:39,942 DEBUG [c.c.a.t.Request] (StatsCollector-3:ctx-0b21c1ed) Seq 4-628555883: Received: { Ans: , MgmtId: 6758231703598, via: 4, Ver: v1, Flags: 10, { GetStorageStatsAnswer }

}
2013-11-25 07:48:40,038 DEBUG [c.c.a.t.Request] (AgentManager-Handler-15:null) Seq 4-628555884: Processing: { Ans: , MgmtId: 6758231703598, via: 4, Ver: v1, Flags: 10, [{"com.cloud.agent.api.Answer":{"result":false,"details":"com.cloud.utils.exception.CloudRuntimeException: GetRootDir for nfs://10.147.28.7/export/home/manasa/sectest failed due to com.cloud.utils.exception.CloudRuntimeException: Unable to mount 10.147.28.7:/export/home/manasa/sectest at /mnt/SecStorage/5885435a-769d-360d-87fa-63f1a925e11b due to mount.nfs: mounting 10.147.28.7:/export/home/manasa/sectest failed, reason given by server: No such file or directory\n\tat org.apache.cloudstack.storage.resource.NfsSecondaryStorageResource.getRootDir(NfsSecondaryStorageResource.java:1939)\n\tat org.apache.cloudstack.storage.resource.NfsSecondaryStorageResource.execute(NfsSecondaryStorageResource.java:1692)\n\tat org.apache.cloudstack.storage.resource.NfsSecondaryStorageResource.executeRequest(NfsSecondaryStorageResource.java:216)\n\tat com.cloud.storage.resource.PremiumSecondaryStorageResource.defaultAction(PremiumSecondaryStorageResource.java:63)\n\tat com.cloud.storage.resource.PremiumSecondaryStorageResource.executeRequest(PremiumSecondaryStorageResource.java:59)\n\tat com.cloud.agent.Agent.processRequest(Agent.java:498)\n\tat com.cloud.agent.Agent$AgentRequestHandler.doTask(Agent.java:806)\n\tat com.cloud.utils.nio.Task.run(Task.java:83)\n\tat java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1146)\n\tat java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:615)\n\tat java.lang.Thread.run(Thread.java:679)\n","wait":0}}] }
2013-11-25 07:48:40,039 DEBUG [c.c.a.t.Request] (StatsCollector-3:ctx-0b21c1ed) Seq 4-628555884: Received: { Ans: , MgmtId: 6758231703598, via: 4, Ver: v1, Flags: 10,

{ Answer } }
2013-11-25 07:48:40,114 DEBUG [c.c.a.t.Request] (AgentManager-Handler-1:null) Seq 4-628555885: Processing: { Ans: , MgmtId: 6758231703598, via: 4, Ver: v1, Flags: 10, [{"com.cloud.agent.api.Answer":{"result":false,"details":"com.cloud.utils.exception.CloudRuntimeException: GetRootDir for nfs://10.147.28.7/export/home/manasa/dummy failed due to com.cloud.utils.exception.CloudRuntimeException: Unable to mount 10.147.28.7:/export/home/manasa/dummy at /mnt/SecStorage/832cd2fc-86a1-3321-a8d1-3c5ffdd5cd95 due to mount.nfs: mounting 10.147.28.7:/export/home/manasa/dummy failed, reason given by server: No such file or directory\n\tat org.apache.cloudstack.storage.resource.NfsSecondaryStorageResource.getRootDir(NfsSecondaryStorageResource.java:1939)\n\tat org.apache.cloudstack.storage.resource.NfsSecondaryStorageResource.execute(NfsSecondaryStorageResource.java:1692)\n\tat org.apache.cloudstack.storage.resource.NfsSecondaryStorageResource.executeRequest(NfsSecondaryStorageResource.java:216)\n\tat com.cloud.storage.resource.PremiumSecondaryStorageResource.defaultAction(PremiumSecondaryStorageResource.java:63)\n\tat com.cloud.storage.resource.PremiumSecondaryStorageResource.executeRequest(PremiumSecondaryStorageResource.java:59)\n\tat com.cloud.agent.Agent.processRequest(Agent.java:498)\n\tat com.cloud.agent.Agent$AgentRequestHandler.doTask(Agent.java:806)\n\tat com.cloud.utils.nio.Task.run(Task.java:83)\n\tat java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1146)\n\tat java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:615)\n\tat java.lang.Thread.run(Thread.java:679)\n","wait":0}}] }
2013-11-25 07:48:40,115 DEBUG [c.c.a.t.Request] (StatsCollector-3:ctx-0b21c1ed) Seq 4-628555885: Received: { Ans: , MgmtId: 6758231703598, via: 4, Ver: v1, Flags: 10, { Answer }

}
2013-11-25 07:48:40,123 DEBUG [c.c.a.m.DirectAgentAttache] (DirectAgent-109:ctx-4308981a) Seq 1-723779901: Executing request
2013-11-25 07:48:40,271 DEBUG [c.c.a.m.AgentManagerImpl] (AgentManager-Handler-2:null) Ping from 4
2013-11-25 07:48:40,885 DEBUG [c.c.a.m.DirectAgentAttache] (DirectAgent-109:ctx-4308981a) Seq 1-723779901: Response Received:

Expected :user should not be allowed to add invalid secondary storage and also there should be proper ERROR messages in Log.

Attaching the MS logs.