Hi, On 09.07.20 11:15, Ilya Dryomov wrote: > On Thu, Jul 9, 2020 at 11:07 AM Lenz Grimmer <lgrimmer@xxxxxxxx> wrote: >> Hi Dan, >> >> On 7/8/20 9:58 PM, Dan Mick wrote: >> >>> I've temporarily removed the requirement the above check, as it's >>> failing often on unrelated pull requests. Once it gets back healthy we >>> can reenable it (it still runs, it just won't cause the pull request to >>> be blocked). >> Thanks for the information and sorry if this test was causing issues. >> Have these failures been reported on the tracker? It would be really >> helpful if we could address these failures to make the tests more stable >> and keep them enabled. >> >> The night tests that perform the API tests on master have been passing, >> so we should take a closer look at why they're failing on these pull >> requests: >> >> https://jenkins.ceph.com/view/all/job/ceph-api-nightly-master-backend/ >> >> In general, please reach out to the dashboard developers if you're >> facing issues with the test, we're glad to help to get them resolved. >> >> Please keep in mind that while it's currently called "Dashboard API", >> the intention is to establish the dashboard backend as the official Ceph >> REST API (as a successor of the "restful" module), to be used for >> managing and monitoring all aspects of Ceph (with or without the >> Dashboard UI). We'll rename the test on the pull requests to "Ceph API" >> to make this more clear. >> >> See https://pad.ceph.com/p/rest-api-stability-requirements and >> https://pad.ceph.com/p/dashboard-api-test-improvements for some >> background information and https://tracker.ceph.com/issues/40907 for an >> overview of all ongoing work around the Ceph API. > Another problem with the dashboard API check in its current form > is wasted lab resources [1]. Perhaps someone from the dashboard > team could assist David there. > > [1] https://lists.ceph.io/hyperkitty/list/dev@xxxxxxx/thread/KMV3HZE6IPEGWY5WP4XXPEKZ225RB3XH/ I'd be +1 for the approach David mentioned earlier [2] and will try to help if there's anything I can support David with [2] https://github.com/ceph/ceph-build/compare/wip-combine-db-prs Laura > > Thanks, > > Ilya > _______________________________________________ > Dev mailing list -- dev@xxxxxxx > To unsubscribe send an email to dev-leave@xxxxxxx >
Attachment:
signature.asc
Description: OpenPGP digital signature
_______________________________________________ Dev mailing list -- dev@xxxxxxx To unsubscribe send an email to dev-leave@xxxxxxx
