Problem/Motivation

When deleting a multiple cloud service providers that include K8s and AWS, there is an 500 error when trying to delete K8s cloud service provider.

Drupal\Core\Entity\EntityStorageException: Cannot load cloud service provider plugin: <em class="placeholder">k8s</em> (CloudConfig::$cloudContext) in Drupal\Core\Entity\Sql\SqlContentEntityStorage->delete() (line 794 of /var/www/cloud_orchestrator/docroot/core/lib/Drupal/Core/Entity/Sql/SqlContentEntityStorage.php).
CommentFileSizeAuthor
#6 41-2.patch1.45 KBbaldwinlouie
#4 41.patch1.07 KBbaldwinlouie
#3 41.diff.txt1.07 KBbaldwinlouie

Issue fork cloud-3186140

Command icon Show commands

Start within a Git clone of the project using the version control instructions.

Or, if you do not have SSH keys set up on git.drupalcode.org:

About issue forks

Comments

baldwinlouie created an issue. See original summary.

baldwinlouie opened merge request !41

baldwinlouie’s picture

baldwinlouie commented
Status: Active » Needs review
StatusFileSize
new 41.diff.txt1.07 KB

Attaching patch for unit testing.

baldwinlouie’s picture

baldwinlouie commented
StatusFileSize
new 41.patch1.07 KB

Status: Needs review » Needs work

The last submitted patch, 4: 41.patch, failed testing. View results

baldwinlouie’s picture

baldwinlouie commented
Status: Needs work » Needs review
StatusFileSize
new 41-2.patch1.45 KB

Updating patch file to fix unit testing.

yas’s picture

yas
🇯🇵 he/him
Primary language Japanese
Location California 🇺🇸
commented
Status: Needs review » Reviewed & tested by the community

@baldwinlouie

Thank you for fixing the issue. It looks good to me. I'll merge the patch to 8.x-2.x and 3.x and close this issue as Fixed.

  • yas committed b91884b on 8.x-2.x authored by baldwinlouie 
    Issue #3186140 by baldwinlouie, yas: Fix an Internal Server Error when a...

  • yas committed 29d4157 on 3.x authored by baldwinlouie 
    Issue #3186140 by baldwinlouie, yas: Fix an Internal Server Error when a...
yas’s picture

yas
🇯🇵 he/him
Primary language Japanese
Location California 🇺🇸
commented
Status: Reviewed & tested by the community » Fixed

yas closed merge request !41

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.