• Add a delete function for each YAML file to export / import the YAML files on an S3 bucket

screenshot-20191021a.png
screenshot-20191021b.png
screenshot-20191021c.png
screenshot-20191021d.png
screenshot-20191021e.png
screenshot-20191021f.png
screenshot-20191021g.png
screenshot-20191021h.png
screenshot-20191021i.png
screenshot-20191021j.png
screenshot-20191021k.png
screenshot-20191021l.png

CommentFileSizeAuthor
#5 screenshot-20191021l.png856.89 KByas
#5 screenshot-20191021k.png805.44 KByas
#5 screenshot-20191021j.png572.49 KByas
#5 screenshot-20191021i.png778.85 KByas
#5 screenshot-20191021h.png924.98 KByas
#5 screenshot-20191021g.png759.51 KByas
#5 screenshot-20191021f.png717.76 KByas
#5 screenshot-20191021e.png805.44 KByas
#5 screenshot-20191021d.png523.26 KByas
#5 screenshot-20191021c.png778.85 KByas
#5 screenshot-20191021b.png681.77 KByas
#5 screenshot-20191021a.png818.64 KByas
#5 3088716-5-interdiff.txt770 bytesyas
#5 3088716-5.patch6.79 KByas
#4 3088716-4.patch6.32 KBXiaohua Guan
#2 3088716-2.patch6.28 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
3088716-2.patch6.28 KB
Xiaohua Guan’s picture

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

@yas

Please review the patch file. Thanks.

Xiaohua Guan’s picture

Xiaohua Guan CreditAttribution: Xiaohua Guan commented
FileSize
3088716-4.patch6.32 KB
yas’s picture

yas
🇯🇵 he/him
Primary language Japanese
Location California 🇺🇸
CreditAttribution: yas at DOCOMO Innovations, Inc. commented
Title: Add annotations to a YAML file of an S3 bucket for CRUD operations » Add annotations to a YAML file of an S3 bucket for a delete operation
Issue summary: View changes
Related issues: +#3086158: Add a software component to check and apply YAML file(s) from Cloud Orchestrator to an S3 Bucket (1), +#3084313: Composer could not parse version constraint / could not load package
FileSize
3088716-5.patch6.79 KB
3088716-5-interdiff.txt770 bytes
screenshot-20191021a.png818.64 KB
screenshot-20191021b.png681.77 KB
screenshot-20191021c.png778.85 KB
screenshot-20191021d.png523.26 KB
screenshot-20191021e.png805.44 KB
screenshot-20191021f.png717.76 KB
screenshot-20191021g.png759.51 KB
screenshot-20191021h.png924.98 KB
screenshot-20191021i.png778.85 KB
screenshot-20191021j.png572.49 KB
screenshot-20191021k.png805.44 KB
screenshot-20191021l.png856.89 KB

@xiaohua-guan

Thank you for adding the feature. I tested the patch and it looks working well. I fixed some coding standard violations so please review it.

Xiaohua Guan’s picture

Xiaohua Guan CreditAttribution: Xiaohua Guan commented

@yas

Thanks for fixing coding standard violations. It's OK to me.

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 providing the patch. I'll merge the patch to 8.x-2.x and close this issue as Fixed

  • yas committed f38b3d2 on 8.x-2.x authored by Xiaohua Guan 
    Issue #3088716 by Xiaohua Guan, yas: Add annotations to a YAML file of...
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’s picture

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

Status: Fixed » Closed (fixed)

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