Skip to content

[Tech Debt] Remove dead code: OracleVMManager #266

@sonikro

Description

@sonikro

Problem

packages/providers/src/cloud-providers/oracle/OracleVMManager.ts is a 266-line implementation that is entirely unused in production. It is imported in ServerManagerFactory.ts but its usage is commented out:

// In ServerManagerFactory.ts
import { AWSServerManager, OracleVMManager } from "../cloud-providers";

// ...
case CloudProvider.ORACLE:
    return new OCIServerManager({ ... });
    // return OracleVMManager.create({ ... });  <-- commented out

OCIServerManager is the active Oracle implementation. OracleVMManager is dead code that adds confusion, maintenance burden, and import noise.

Fix

  1. Delete packages/providers/src/cloud-providers/oracle/OracleVMManager.ts
  2. Delete packages/providers/src/cloud-providers/oracle/OracleVMManager.test.ts
  3. Remove the OracleVMManager import from ServerManagerFactory.ts
  4. Remove the commented-out return OracleVMManager.create(...) block
  5. Clean up packages/providers/src/cloud-providers/oracle/index.ts barrel export

Metadata

Metadata

Labels

tech-debtTechnical debt items

Projects

No projects

Milestone

No milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions