DaiZhiwei

Results 10 comments of DaiZhiwei

> @Zhiwei-Dai thanks for the contribution, this is a great feature! i'm seeing some test failures though, could you please take a look? > in addition to those 3 test...

> can you please add this to the PR? ok, of course

@cbodley I have updated the implementation and ceph's s3-tests for sse encryption are all passed.

@cbodley would you mind review the pr again? Thanks

> i've opened #48609 to document this new feature in PendingReleaseNotes Nice work

看你的报错,应该是你的这部分内容是追加到了openssl.cnf的头部,导致qatengine_section段解析的时候,错误得解析到了“HOME”,因此报错。 ``` openssl_conf = openssl_def [openssl_def] engines = engine_section [engine_section] qatengine = qatengine_section [qatengine_section] engine_id = qatengine dynamic_path = /usr/local/ssl/lib/engines-1.1/qatengine.so default_algorithms = ALL ``` 解决办法:将openssl.cnf原文件备份,新建openssl.cnf,然后将上述配置放进去即可。

@alfredodeza, i was wondering if u could review the pr, ths

Ths your question. when follow three 'if' have right 'return' type , the printing info just show a fact. if can't return right type, the printing provides infos to enhance...

Usually, this file `/etc/rpm/macros.dist` is existing for rpm & distribution building. If not existing, maybe it is deleted by mistake. Look this system file in centos7.6: ``` # dist macros....

@HacKanCuBa Well, when I switch to use context manager to open file, exception handling prompts me to catch the `ValueError` of `int(...)`. Nice job.