Hello, lgnasi:
It is happy to receive your email.Accoding your describe, I check my code,
but I don’t know how to fix.
The purpose of our code is to achieve two things:
1. Add aliyun object storage function(public cloud and private cloud).
Aliyun is a subsidiary of Chinese company Alibaba, it provides cloud service. A
lot of Chinese people use it. The SDK of aliyun object storage is compatible
with AWS S3, except for the signature authentication. Therefore, our developers
are imitating jclouds' support for aws s3 and adding relevant code of aliyun
object storage cloud.
2. Add aws s3 AWS S3 China(cn-north-1) support.
On our storage device, we've tested and passed this code. However, because our
developers do not understand the structure of jclouds code, the code we write
does not meet your requirements. And we also didn't find a way to do unit
testing, so we didn't write unit test code.
Do you have any Suggestions for this situation?
发件人: Ignasi Barrera [mailto:[email protected]]
发送时间: 2018年12月17日 10:50
收件人: jclouds/jclouds <[email protected]>
抄送: Bruce Xing(邢云阳) <[email protected]>; Author <[email protected]>
主题: Re: [jclouds/jclouds] support aliyun oss and chinese region of aws s3 on
jclouds 1.9.1 (#1263)
It is very unlikely that we release another 1.9 version, so I'd suggest you
open the pull request to the master branch, then we can cherry-pick it to 2.1.x.
This said, there are things that are fundaamentally wrong in this pull request
such as hardcoding endpoint logic in jclouds-core (RestAnnotationProcessor for
example) that need to be changed and redesigned in order for the PR to be
mergeable.
I'll be happy to give advice and guidance if you drop an email to the jclouds
dev@ list describing the use case and the fix that needs to be done. Then we'll
be able to point you to the proper way of getting it done.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
<https://github.com/jclouds/jclouds/pull/1263#issuecomment-447706776> , or mute
the thread
<https://github.com/notifications/unsubscribe-auth/AriYf8MWQLEjuqIbg8-hNSnWj-rNeNu1ks5u5wZBgaJpZM4ZOnCa>
.
<https://github.com/notifications/beacon/AriYfwOOyPGI_OrBRf1vB4-6ZAUqDeTRks5u5wZBgaJpZM4ZOnCa.gif>
--
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds/pull/1263#issuecomment-447735054
