Uploaded image for project: 'Jenkins'
  1. Jenkins
  2. JENKINS-55285

Code Cleanup: Removed exposed compute client in ComputeEngineCloud

XMLWordPrintable

      There are examples throughout the code base of accessing the ComputEngineCloud's compute client through an exposed field. This introduces tight-coupling by introducing an undocumented dependence on the initialization logic/timing within the ComputeEngineCloud. Ideally this field should be encapsulated and current reference sites should be refactored to accessing the compute client in an abstract and uniform manner.

          [JENKINS-55285] Code Cleanup: Removed exposed compute client in ComputeEngineCloud

          Rachel Yen added a comment -

          tracking here:

          https://github.com/jenkinsci/google-compute-engine-plugin/issues/38

          Rachel Yen added a comment - tracking here: https://github.com/jenkinsci/google-compute-engine-plugin/issues/38

            craigbarber Craig Barber
            craigbarber Craig Barber
            Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

              Created:
              Updated:
              Resolved: