Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

StartExperimentSFCEdge.java file potential bug or intended feature #19

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

Ten000hours
Copy link

Hi, I think I run across a bug when I run the example java file(StartExperimentSFCEdge) with program arguments LFF 0 example-edge/edge.physical.json example-edge/edge.virtual.json example-edge/ edge.workload_host1.csv edge.workload_host2.csv
It throws the Null pointer exception instead of finishing correctly. The exception info as follows.

java.lang.NullPointerException
at org.cloudbus.cloudsim.sdn.example.StartExperimentSFCEdge.getAllHostList(StartExperimentSFCEdge.java:360)
at org.cloudbus.cloudsim.sdn.example.StartExperimentSFCEdge.startSimulation(StartExperimentSFCEdge.java:340)
at org.cloudbus.cloudsim.sdn.example.StartExperimentSFCEdge.main(StartExperimentSFCEdge.java:316)

and this execption

java.lang.Exception: net : Error - this entity has no PEs. Therefore, can't process any Cloudlets.
at org.cloudbus.cloudsim.Datacenter.(Datacenter.java:104)
at org.cloudbus.cloudsim.sdn.physicalcomponents.SDNDatacenter.(SDNDatacenter.java:61)
at org.cloudbus.cloudsim.sdn.example.StartExperimentSFCEdge.createSDNDatacenter(StartExperimentSFCEdge.java:443)
at org.cloudbus.cloudsim.sdn.example.StartExperimentSFCEdge.createPhysicalTopology(StartExperimentSFCEdge.java:379)
at org.cloudbus.cloudsim.sdn.example.StartExperimentSFCEdge.main(StartExperimentSFCEdge.java:291)

The enviornment:
--jdk1.8.0_181
--cloudsim 4.0 jar
--idea 2019.2 Ultimate

I try to fix this bug by adding function to assert the type of dcName field ("network" or "cloud"). If this is an intended feature, I am sorry for disturbing you guys.

@hetianzhang
Copy link
Contributor

Hi, thank you for your comments. I think you are correct with the bug originally published by the author. However, I had patched the zero PEs bug for NOS_net type in the DataCenter.java from CloudSim.

@magaboomchen
Copy link

Dear author, I meet this problem again in the latest release.
Here are my settings:
cloudsimsdn version: cloudsimsdn-2.0.1-beta
cloudsim jar version: 4.0 (I also create jar file from cloudsim 5.0 source code and change pom.xml)
IDE: Eclipse 4.16
JDK: 1.11
OS: win10

I have tried cloudsim-4.0.jar from release page, jar file created from cloudsim 4.0 and 5.0 source code.

@hetianzhang
Copy link
Contributor

Dear author, I meet this problem again in the latest release.
Here are my settings:
cloudsimsdn version: cloudsimsdn-2.0.1-beta
cloudsim jar version: 4.0 (I also create jar file from cloudsim 5.0 source code and change pom.xml)
IDE: Eclipse 4.16
JDK: 1.11
OS: win10

I have tried cloudsim-4.0.jar from release page, jar file created from cloudsim 4.0 and 5.0 source code.

Hi, I can't reproduce the bug you encountered. To clarify any confusion of the CloudSim jar version, I simply update the installation guild. Please check.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants