account for an s3 folder when listing objects

[briandowns]

Proposed Changes

Additional logic needed to be added to account for the use of folders within AWS S3 buckets in the "list snapshots" routine.

Types of Changes

Verification

Run the command below and verify that what is listed in the ConfigMap is what's in the AWS folder.

kubectl get cm -n kube-system k3s-etcd-snapshots -o yaml 

Linked Issues

• s3 snapshots do not show in the rancher 2.6 UI #3811
• s3 snapshots do not show in the rancher 2.6 UI rancher/rke2#1551

User-Facing Change

Further Comments

[codecov-commenter]

Codecov Report

Merging #3807 (4f99f23) into master (bc96ffb) will decrease coverage by 0.01%.
The diff coverage is 0.00%.

❗ Current head 4f99f23 differs from pull request most recent head 798a1e4. Consider uploading reports for the commit 798a1e4 to get more accurate results

@@            Coverage Diff             @@
##           master    #3807      +/-   ##
==========================================
- Coverage   11.55%   11.54%   -0.02%     
==========================================
  Files         136      136              
  Lines        8815     8823       +8     
==========================================
  Hits         1019     1019              
- Misses       7575     7583       +8     
  Partials      221      221              

Flag	Coverage Δ
inttests	0.62% <0.00%> (-0.01%)	⬇️
unittests	11.26% <0.00%> (-0.02%)	⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files	Coverage Δ
pkg/etcd/etcd.go	16.51% <0.00%> (-0.21%)	⬇️

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update bc96ffb...798a1e4. Read the comment docs.

[brandond]

Does this currently find everything properly if there's not a prefix specified? I guess it just non-recursively lists everything in the top-level directory (no prefix)?

[briandowns]

Does this currently find everything properly if there's not a prefix specified? I guess it just non-recursively lists everything in the top-level directory (no prefix)?

Correct. Only looks at the top level (bucket level).