Problem/Motivation

  • Manage Cloud Orchestrator deployment (Update site url)

Issue fork cloud-3293244

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
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Xiaohua Guan created an issue. See original summary.

Xiaohua Guan opened merge request !1144

Xiaohua Guan’s picture

Xiaohua Guan CreditAttribution: Xiaohua Guan commented
Status: Active » Needs review

@yas

Please review the code. Thanks.

yas’s picture

yas
🇯🇵 he/him
Primary language Japanese
Location California 🇺🇸
CreditAttribution: yas at DOCOMO Innovations, Inc. commented
Issue summary: View changes
Status: Needs review » Needs work

@xiaohua-guan

Thank you for the patch. Could you please modify the drush_queue_run.sh script? (Simply add the module name into the variable MODULES)

Hint: Currently, the following is the output of drush queue-list for instance:

# drush queue-list
 ----------------------------------------------------------- ------- --------------------------------- 
  Queue                                                       Items   Class                            
 ----------------------------------------------------------- ------- --------------------------------- 
  aws_cloud_update_resources_queue:aws_ap_northeast_1         8       Drupal\Core\Queue\DatabaseQueue  
  aws_cloud_update_resources_queue:aws_us_east_1              7       Drupal\Core\Queue\DatabaseQueue  
  aws_cloud_update_resources_queue:aws_us_west_2              14      Drupal\Core\Queue\DatabaseQueue  
  aws_cloud_update_iam_permissions_queue:aws_ap_northeast_1   1       Drupal\Core\Queue\DatabaseQueue  
  aws_cloud_update_iam_permissions_queue:aws_us_east_1        1       Drupal\Core\Queue\DatabaseQueue  
  aws_cloud_update_iam_permissions_queue:aws_us_west_2        1       Drupal\Core\Queue\DatabaseQueue  
  cloud_cluster_sync_resources_queue                          0       Drupal\Core\Queue\DatabaseQueue  
  k8s_update_resources_queue:cloudopt_k8s_1                   30      Drupal\Core\Queue\DatabaseQueue  
  k8s_update_resources_queue:cloudopt_k8s_4                   0       Drupal\Core\Queue\DatabaseQueue  
  k8s_update_resources_queue:cloudopt_k8s_5                   15      Drupal\Core\Queue\DatabaseQueue  
 ----------------------------------------------------------- ------- ---------------------------------
Xiaohua Guan’s picture

Xiaohua Guan CreditAttribution: Xiaohua Guan commented
Status: Needs work » Needs review

@yas

I just added queue for cloud_formation which belongs to the module aws_cloud, so it is unnecessary to modify the shell script.

yas’s picture

yas
🇯🇵 he/him
Primary language Japanese
Location California 🇺🇸
CreditAttribution: yas at DOCOMO Innovations, Inc. commented
Status: Needs review » Reviewed & tested by the community

@xiaohua-guan

Thank you for your message. I understood. In that sense, I found we need to add cloud_cluster module name to the script. That can be another issue to add, though.

yas’s picture

yas
🇯🇵 he/him
Primary language Japanese
Location California 🇺🇸
CreditAttribution: yas at DOCOMO Innovations, Inc. commented
Title: Manage Cloud Orchestrator deployment (Update site url) » Manage Cloud Orchestrator deployment (Update site URL)

  • yas committed a84b9f8 on 5.x authored by Xiaohua Guan 
    Issue #3293244 by Xiaohua Guan, yas: Manage Cloud Orchestrator...

  • yas committed dbf43a3 on 4.x authored by Xiaohua Guan 
    Issue #3293244 by Xiaohua Guan, yas: Manage Cloud Orchestrator...
yas’s picture

yas
🇯🇵 he/him
Primary language Japanese
Location California 🇺🇸
CreditAttribution: yas at DOCOMO Innovations, Inc. commented
Status: Reviewed & tested by the community » Fixed
anoopsingh92’s picture

anoopsingh92
Location Rajasthan, India
CreditAttribution: anoopsingh92 at Valuebound for Valuebound commented
Assigned: Xiaohua Guan » Unassigned

Hi @Xiaohua Guan,

Your change works finely for me.

Xiaohua Guan’s picture

Xiaohua Guan CreditAttribution: Xiaohua Guan commented
Assigned: Unassigned » Xiaohua Guan

Status: Fixed » Closed (fixed)

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