You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@cloudstack.apache.org by GitBox <gi...@apache.org> on 2020/06/02 07:23:20 UTC

[GitHub] [cloudstack] Spaceman1984 commented on a change in pull request #4004: Fixed null pointer and deployment issue on Xenserver with L2 Guest network with configDrive

Spaceman1984 commented on a change in pull request #4004:
URL: https://github.com/apache/cloudstack/pull/4004#discussion_r433671286



##########
File path: server/src/main/java/com/cloud/network/element/ConfigDriveNetworkElement.java
##########
@@ -478,7 +476,7 @@ private boolean deleteConfigDriveIso(final VirtualMachine vm) throws ResourceUna
         return true;
     }
 
-    private void addConfigDriveDisk(final VirtualMachineProfile profile, final DataStore dataStore) throws ResourceUnavailableException {
+    public static void addConfigDriveDisk(final VirtualMachineProfile profile, final DataStore dataStore) throws ResourceUnavailableException {

Review comment:
       Must be a leftover artifact from long days of experimentation, will remove.




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

For queries about this service, please contact Infrastructure at:
users@infra.apache.org