If the instance has the name:
EIP (INSTANCE_NAME)

If the instance doesn't has the name:
EIP (INSTNACE_ID)

CommentFileSizeAuthor
#2 3046730-2.patch2.02 KBXiaohua Guan
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Xiaohua Guan created an issue. See original summary.

Xiaohua Guan’s picture

Xiaohua Guan CreditAttribution: Xiaohua Guan commented
FileSize
3046730-2.patch2.02 KB
Xiaohua Guan’s picture

Xiaohua Guan CreditAttribution: Xiaohua Guan commented
Status: Active » Needs review
yas’s picture

yas
🇯🇵 he/him
Primary language Japanese
Location California 🇺🇸
CreditAttribution: yas at DOCOMO Innovations, Inc. commented

@xiaohua-guan

Thank you for making the patch. I confirmed it and looks good to me.

@baldwinlouie
@masami

What do you think?

baldwinlouie’s picture

baldwinlouie CreditAttribution: baldwinlouie commented

@all, this looks good to me.

yas’s picture

yas
🇯🇵 he/him
Primary language Japanese
Location California 🇺🇸
CreditAttribution: yas at DOCOMO Innovations, Inc. commented

@baldwinlouie

Thank you for your review. Then, I'll merge the patch to both 8.x-1.x and 8.x-2.x branch and close this issue as Fixed.

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
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

  • yas committed 5cda33b on 8.x-1.x authored by Xiaohua Guan 
    Issue #3046730 by Xiaohua Guan, yas, baldwinlouie: Identify the EIP name...

  • yas committed 37b9805 on 8.x-2.x authored by Xiaohua Guan 
    Issue #3046730 by Xiaohua Guan, yas, baldwinlouie: Identify the EIP name...

Status: Fixed » Closed (fixed)

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