- Issue created by @yas
- last update
over 1 year ago 366 pass - @yas opened merge request.
- Status changed to Needs review
over 1 year ago 12:32am 25 April 2023 - πΊπΈUnited States yas California πΊπΈ
@baldwinlouie, @xiaohua-guan, @masami, @sekinet
Can you please review the patch? Thanks!
- last update
over 1 year ago 366 pass - πΊπΈUnited States baldwinlouie
@yas, Thank you for the patch. It looks good to me.
- π―π΅Japan xiaohua guan Tokyo, Japan
@yas
Thanks for the patch. It looks good to me.
- Status changed to RTBC
over 1 year ago 8:05am 26 April 2023 - π―π΅Japan Masami
The patch looks good to me, so I change the status to RTBC.
- πΊπΈUnited States yas California πΊπΈ
@sekinet @baldwinlouie @xiaohua-guan @masami
Thank you for your review. I'll merge the patch to
4.x
,5.x
and6.x
, and close this issue as Fixed. - last update
over 1 year ago 366 pass - last update
over 1 year ago 366 pass - Status changed to Fixed
over 1 year ago 9:27am 26 April 2023 - Status changed to Needs work
over 1 year ago 4:45am 8 May 2023 - πΊπΈUnited States yas California πΊπΈ
We need to fix the following error:
Error: Using $this when not in object context in Drupal\aws_cloud\Plugin\Block\LowUtilizationInstancesBlock::Drupal\aws_cloud\Plugin\Block\{closure}() (line 217 of /var/www/cloud_orchestrator/docroot/modules/contrib/cloud/modules/cloud_service_providers/aws_cloud/src/Plugin/Block/LowUtilizationInstancesBlock.php) #0 [internal function]: Drupal\aws_cloud\Plugin\Block\LowUtilizationInstancesBlock::Drupal\aws_cloud\Plugin\Block\{closure}(Object(Drupal\aws_cloud\Entity\Ec2\Instance)) #1 /var/www/cloud_orchestrator/docroot/modules/contrib/cloud/modules/cloud_service_providers/aws_cloud/src/Plugin/Block/LowUtilizationInstancesBlock.php(216): array_map(Object(Closure), Array) #2 /var/www/cloud_orchestrator/docroot/modules/contrib/cloud/modules/cloud_service_providers/aws_cloud/src/Plugin/Block/LowUtilizationInstancesBlock.php(191): Drupal\aws_cloud\Plugin\Block\LowUtilizationInstancesBlock->buildInstanceList() #3 /var/www/cloud_orchestrator/docroot/profiles/contrib/cloud_orchestrator/src/Controller/DashboardController.php(263): Drupal\aws_cloud\Plugin\Block\LowUtilizationInstancesBlock->build() #4 /var/www/cloud_orchestrator/docroot/profiles/contrib/cloud_orchestrator/src/Controller/DashboardController.php(165): Drupal\cloud_orchestrator\Controller\DashboardController->getBlock('aws_cloud_low_u...', Array) #5 /var/www/cloud_orchestrator/docroot/profiles/contrib/cloud_orchestrator/src/Controller/DashboardController.php(54): Drupal\cloud_orchestrator\Controller\DashboardController->getAwsBlocks() #6 [internal function]: Drupal\cloud_orchestrator\Controller\DashboardController->getDashboard()
- Status changed to Needs review
over 1 year ago 5:07am 8 May 2023 - last update
over 1 year ago 366 pass - Status changed to RTBC
over 1 year ago 5:31am 8 May 2023 - πΊπΈUnited States yas California πΊπΈ
@all The tests have been passed successfully, so I'll merge the patch to
4.x
,5.x
and6.x
, and close this issue as Fixed. - Status changed to Fixed
over 1 year ago 5:41am 8 May 2023 Automatically closed - issue fixed for 2 weeks with no activity.