I forgot to mention that all Ceph commits need to be signed off. This is done with the "-s" option to "git commit". You can fix this in this commit with:
git commit -s --amend
And then save the commit message unchanged. Then force push ("git push -f") the branch again.

Signed the commit and pushed again

there's a qa/workunits/rgw/run-s3tests.sh script that you can run from your ceph build directory. it will clone the https://github.com/ceph/s3-tests repo and run the subset of test cases that we expect to pass

you can add a test case for this fix to https://github.com/ceph/s3-tests/blob/master/s3tests_boto3/functional/test_s3.py

@cbodley in teuthology should @AliMasarweh just be running just the rgw:verify suite with a revised s3-tests for this fix?

@cbodley in teuthology should @AliMasarweh just be running just the rgw:verify suite with a revised s3-tests for this fix?

the full rgw suite against revised s3-tests please

@AliMasarweh can you please share the teuthology results?

and is there an associated s3test case that needs to merge also?

this is the teuthology results
http://pulpito.front.sepia.ceph.com/alimasa-2023-04-04_09:48:28-rgw-wip-ali-masa-multipart-populate-etag-distro-default-smithi/
I haven't touched any of the s3test cases, maybe @dang can tell if we need to add a test to verify we populate the etag