Bubble up memory errors when pushing/updating sidecars

[#175822881]

Co-authored-by: Weyman Fung <weymanf@vmware.com>
Co-authored-by: Belinda Liu <bliu@pivotal.io>

Author: weymanf

app/jobs/space_apply_manifest_action_job.rb

@@ -23,7 +23,10 @@ def perform
                ManifestRouteUpdate::InvalidRoute,
                Route::InvalidOrganizationRelation,
                AppApplyManifest::Error,
-               AppApplyManifest::ServiceBindingError => e
+               AppApplyManifest::ServiceBindingError,
+               SidecarCreate::InvalidSidecar,
+               SidecarUpdate::InvalidSidecar,
+               ProcessScale::SidecarMemoryLessThanProcessMemory => e
 
           app_name = AppModel.find(guid: app_guid)&.name
           error_message = app_name ? "For application '#{app_name}': #{e.message}" : e.message

spec/unit/jobs/space_apply_manifest_action_job_spec.rb

@@ -54,7 +54,11 @@ module Jobs
         ProcessScale::InvalidProcess,
         ProcessUpdate::InvalidProcess,
         Route::InvalidOrganizationRelation,
+        AppApplyManifest::Error,
         AppApplyManifest::ServiceBindingError,
+        SidecarCreate::InvalidSidecar,
+        SidecarUpdate::InvalidSidecar,
+        ProcessScale::SidecarMemoryLessThanProcessMemory,
       ].each do |klass|
         it "wraps a #{klass} in an ApiError" do
           allow(apply_manifest_action).to receive(:apply).