Skip to content
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

dal: Add integration tests target on production services #8

Closed
Tracked by #3677 ...
Xuanwo opened this issue Dec 13, 2021 · 3 comments · Fixed by #13
Closed
Tracked by #3677 ...

dal: Add integration tests target on production services #8

Xuanwo opened this issue Dec 13, 2021 · 3 comments · Fixed by #13
Assignees

Comments

@Xuanwo
Copy link
Member

Xuanwo commented Dec 13, 2021

Summary

Add integration tests that target production services.

  • s3
  • fs
  • azblob
@ZhiHanZ
Copy link
Contributor

ZhiHanZ commented Dec 13, 2021

we could add them into test framework as well~

@Xuanwo
Copy link
Member Author

Xuanwo commented Dec 13, 2021

we could add them into test framework as well~

I think there are two parts:

  • Integration tests to make sure dal works correctly.
  • Integration tests for the whole services.

@ZhiHanZ
Copy link
Contributor

ZhiHanZ commented Dec 13, 2021

we could add them into test framework as well~

I think there are two parts:

  • Integration tests to make sure dal works correctly.
  • Integration tests for the whole services.

For the second one, we should add operator into integration test blueprint which was discussed in
databendlabs/databend#1773
and there has something left for platform specific tasks which should also get tested in platform managed kubernetes such as iam role and access token refresh part for DAL access

@Xuanwo Xuanwo mentioned this issue Feb 14, 2022
6 tasks
@Xuanwo Xuanwo transferred this issue from databendlabs/databend Feb 14, 2022
@Xuanwo Xuanwo self-assigned this Feb 14, 2022
This was referenced Feb 14, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants