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? 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 _______________________________________________ Dev mailing list -- dev@xxxxxxx To unsubscribe send an email to dev-leave@xxxxxxx
