Hi Neha, On 21/09/2021 00:43, Neha Ojha wrote: > Hi Loïc, > > This makes sense to me. I believe Deepika has a PR out for review to > implement something very similar in > https://github.com/ceph/ceph/pull/43159. Yes! We discussed it off-list and the outcome of this discussion is in a branch[0] that is ready for testing. Cheers [0] https://lab.fedeproxy.eu/ceph/ceph-backport/-/commits/wip-backport-sh > > Thanks, > Neha > > > On Mon, Sep 20, 2021 at 9:07 AM Loïc Dachary <loic@xxxxxxxxxxx> wrote: >> >> Bonjour, >> >> Here is more concrete example that should work in GitHub (it runs in GitLab): >> >> https://lab.fedeproxy.eu/ceph/ceph/-/blob/wip-backports/src/test/run-ceph-backport-tests.sh >> >> if ! git diff --stat ${GITHUB_BASE:=origin/master}..${GITHUB_SHA:=HEAD} | grep -q $SCRIPT ; then >> echo SKIP because $SCRIPT is not modified >> exit 0 >> fi >> >> and it will be triggered from: >> >> https://lab.fedeproxy.eu/ceph/ceph/-/blob/wip-backports/src/test/CMakeLists.txt#L611 >> >> add_ceph_test(run-ceph-backport-tests.sh ${CMAKE_CURRENT_SOURCE_DIR}/run-ceph-backport-tests.sh) >> >> >> Cheers >> >> On 24/08/2021 11:06, Loïc Dachary wrote: >>> Hi Neha, >>> >>> On 18/08/2021 19:52, Neha Ojha wrote: >>>> On Tue, Aug 10, 2021 at 5:52 AM Loïc Dachary <loic@xxxxxxxxxxx> wrote: >>>>> >>>>> Bonjour, >>>>> >>>>> The ceph-backport.sh is used on a regular basis by developers helping with backports: it proved to be stable and robust over the years. Minimal integration tests were added[0] this week to minimize the risk of a regression when improving it, for instance to help with the ordering of backports[1]. The test environment[2] spawns a redmine instance configured in the same way as https://tracker.ceph.com/ and a Gitea instance that has the same API as GitHub[3] with repositories configured in the same way as https://github.com/ceph/ceph/. >>>>> >>>>> I think it would make sense to include the tests in the CI[4]. The test itself would do nothing if the PR does not contain any change related to ceph-backport.sh. Does that sound sensible? >>>> >>>> I don't see any issues with it given that the tests would only run for >>>> changes touching ceph-backport.sh. I am not very clear on how the >>>> tests will be integrated with the CI, can you please elaborate? >>> >>> It would be something like: >>> >>> new file src/script/test-ceph-backport.sh >>> @@ -0,0 +1,8 @@ >>> +#!/bin/bash >>> + >>> +if git $current_pr stats | grep ceph-backport.sh ; then >>> + git clone https://lab.fedeproxy.eu/ceph/ceph-backport >>> + cd ceph-backport >>> + tests/setup.sh # launches a redmine and a gitea in docker containers >>> + tests/test-ceph-backport.sh >>> +fi >>> modified src/test/CMakeLists.txt >>> @@ -591,6 +591,8 @@ add_ceph_test(run-cli-tests ${CMAKE_CURRENT_SOURCE_DIR}/run-cli-tests) >>> >>> add_ceph_test(smoke.sh ${CMAKE_CURRENT_SOURCE_DIR}/smoke.sh) >>> >>> +add_ceph_test(test-ceph-backport.sh ${CMAKE_CURRENT_SOURCE_DIR}/test-ceph-backport.sh) >>> + >>> find_program(PROMTOOL_EXECUTABLE promtool) >>> if(PROMTOOL_EXECUTABLE) >>> execute_process( >>> >>> Of course it is less than ideal to have a dependency on a third party repository and the following steps would be to gradually move what is in https://lab.fedeproxy.eu/ceph/ceph-backport into the source tree so that it is self contained. The dependencies are python and docker. >>> >>> I hope that answers your question :-) >>> >>> Cheers >>> >>>> >>>> Thanks, >>>> Neha >>>> >>>> >>>>> >>>>> Cheers >>>>> >>>>> [0] https://tracker.ceph.com/issues/51222 >>>>> [1] https://tracker.ceph.com/issues/51224 >>>>> [2] https://lab.fedeproxy.eu/ceph/ceph-backport/ >>>>> [3] https://github.com/go-gitea/gitea/blob/main/CONTRIBUTING.md#api-v1 >>>>> [4] https://tracker.ceph.com/issues/52122 >>>>> -- >>>>> Loïc Dachary, Artisan Logiciel Libre >>>>> >>>>> _______________________________________________ >>>>> Dev mailing list -- dev@xxxxxxx >>>>> To unsubscribe send an email to dev-leave@xxxxxxx >>>> >>> >> >> -- >> Loïc Dachary, Artisan Logiciel Libre >> > -- Loïc Dachary, Artisan Logiciel Libre
Attachment:
OpenPGP_signature
Description: OpenPGP digital signature
_______________________________________________ Dev mailing list -- dev@xxxxxxx To unsubscribe send an email to dev-leave@xxxxxxx