-
Notifications
You must be signed in to change notification settings - Fork 164
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
Optimize drone.star file #8794
Comments
After discussing with @ScharfViktor I've added an other todo item to change the destination of the notifications |
Dev team might miss the ocis builds status so should we move to |
This is to reduce the noise. What about putting the normal builds into the ocis channel and only the nightly into |
sounds good to me |
Current CI takes less than 30 minutes to complete. Likewise, each pipeline completes within 15 minutes or less. Of all the localApiTests, |
Let's close the issue as done. |
Let's make all functions that return step or pipeline return list ( |
better to split the |
@PrajwolAmatya Can we close this one? |
With the changes, the CI completes inside 30 mins. But, regular inspection would be needed for the CI run time. As all the tasks in this issue has been covered, we can close this issue. |
Describe the bug
There are several areas where optimization can be applied to the drone. While I've highlighted some points, please continue to explore additional areas where optimization is needed during the process.
IMO opinion we need to split shareng tests in 2 parts as you can see it crossing 15 min and still there are ongoing PR which adds tests on shreng
- one way to split is separate tests of sharing space using root endpoint in separate folder
@saw-jan any idea about this or we don't need to split ❓
chat notification only trigger on failure. trigger chat notification both for passed and failures on nightly [tests-only][full-ci] notify chat also on nightly success #8832
check
stable and master
branch drone.star file (make it similar as much as possible)Check naming [tests-only][full-ci]rename e2e test suite #8867
send chat notifications of normal build to
infinitescale
and nightly tobuilds
channel [tests-only][full-ci]send nightly notification to builds channel #8866do not run web smoke test pipelines in nightly and full-ci PRs [tests-only] [full-ci] remove web smoke tests from nightly and full ci #8929
check if web stable has runner or not. If possible backport runner implementation PR on web stable [tests-only][full-ci] backport e2e runner implementation #8934
make return type similar to maintain standard (some returns
dict
data type where some returnslist
) [tests-only][full-ci] make function's return type in drone file similar #9039split
apiSharingNg
test suite to tow or more parts [tests-only][full-ci] divide shareNg test suite into multiple parts #9158The text was updated successfully, but these errors were encountered: