-
Notifications
You must be signed in to change notification settings - Fork 2.2k
[Feature][Connector-V2] add Aliyun SLS connector #3733 #7348
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
|
This Pr For Sls(Aliyun Log Service) Source Connector |
|
Thanks Please update:
|
|
1、2、4、5 already done,next , add e2e testcase |
|
I have successfully run the test case locally, but need a personal account to run it |
Commit the code and disable the test case, then provide a screenshot of the successful test. |
|
cc @Hisoka-X A screenshot of the successful test and Is it as indicated in the screenshot to disable the test case |
|
Please fix ci. |
Fixed |
|
done, Do we need to run ci again ? |
Hisoka-X
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM if ci passes
|
Thanks @XenosK ! |

Purpose of this pull request
Does this PR introduce any user-facing change?
How was this patch tested?
Check list
New License Guide
release-note.