CommentFileSizeAuthor
#5 3015916-5.patch5.47 KBxiaohua guan
#2 3015916_20181126.patch4.79 KBxiaohua guan

Comments

Xiaohua Guan created an issue. See original summary.

xiaohua guan’s picture

xiaohua guan commented
StatusFileSize
new 3015916_20181126.patch4.79 KB
xiaohua guan’s picture

xiaohua guan commented
Status: Active » Needs review
xiaohua guan’s picture

xiaohua guan commented

It seems that the reason is the property security group of instance entity wasn't updated when the instance was updated by cron job.

xiaohua guan’s picture

xiaohua guan commented
StatusFileSize
new 3015916-5.patch5.47 KB
xiaohua guan’s picture

xiaohua guan commented

I did the following modifications.

1. Add updating security group when updating instances.
2. Add a test case to test updating instances.

xiaohua guan’s picture

xiaohua guan commented

@yas

Please review the patch file.

yas’s picture

yas
🇯🇵 he/him
Primary language Japanese
Location California 🇺🇸
commented

@xiaohua-guan Thank you for providing the patch. It looks good to me.

@baldwinlouie How do you think?

baldwinlouie’s picture

baldwinlouie commented
Status: Needs review » Reviewed & tested by the community

Looks good!

  • yas committed 573c0de on 8.x-1.x authored by Xiaohua Guan 
    Issue #3015916 by Xiaohua Guan, yas, baldwinlouie: Can't show all...
yas’s picture

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

@xiaohua-guan
@baldwinlouie

Thank you for your contribution. I merged the patch.

Status: Fixed » Closed (fixed)

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