Uploaded image for project: 'CloudStack'
  1. CloudStack
  2. CLOUDSTACK-9314

Remove unused code from XenServerStorageProcessor and change methods access level

    XMLWordPrintableJSON

    Details

    • Type: Improvement
    • Status: Reopened
    • Priority: Trivial
    • Resolution: Unresolved
    • Affects Version/s: 4.9.0
    • Fix Version/s: None
    • Component/s: XenServer
    • Security Level: Public (Anyone can view this level - this is the default.)
    • Labels:
      None

      Description

      Remove unused code from "com.cloud.hypervisor.xenserver.resource.XenServerStorageProcessor" class and modify some methods access level.

      Remove unused methods:

      com.cloud.hypervisor.xenserver.resource.XenServerStorageProcessor.getSRByNameLabel(Connection, String)
      com.cloud.hypervisor.xenserver.resource.XenServerStorageProcessor.getNfsSR(Connection, StorageFilerTO)
      com.cloud.hypervisor.xenserver.resource.XenServerStorageProcessor.directDownloadHttpTemplate(CopyCommand, DecodedDataObject, DecodedDataObject)
      com.cloud.hypervisor.xenserver.resource.XenServerStorageProcessor.execute(AttachPrimaryDataStoreCmd)

      change access level:

      from public to private: com.cloud.hypervisor.xenserver.resource.XenServerStorageProcessor.swiftUpload(Connection, SwiftTO, String, String, String, Boolean, int)
      from public to protected: com.cloud.hypervisor.xenserver.resource.XenServerStorageProcessor.swiftBackupSnapshot(Connection, SwiftTO, String, String, String, Boolean, int)
      from protected to private: com.cloud.hypervisor.xenserver.resource.XenServerStorageProcessor.getSnapshotSize(Connection, String, String, Boolean, int)
      from protected to private: com.cloud.hypervisor.xenserver.resource.XenServerStorageProcessor.backupSnapshot(Connection, String, String, String, String, String, String, Boolean, int)

        Attachments

          Issue Links

            Activity

              People

              • Assignee:
                Unassigned
                Reporter:
                brascher Gabriel Beims Bräscher
              • Votes:
                0 Vote for this issue
                Watchers:
                3 Start watching this issue

                Dates

                • Created:
                  Updated: