[PATCH v2 compass-ci 1/2] container/web-backend: add api for group jobs stats count
Signed-off-by: Li Yuanchao <lyc163mail@163.com> --- container/web-backend/web-backend | 25 +++++++++++++++++++++++++ 1 file changed, 25 insertions(+) diff --git a/container/web-backend/web-backend b/container/web-backend/web-backend index 33a3fae..2971719 100755 --- a/container/web-backend/web-backend +++ b/container/web-backend/web-backend @@ -146,3 +146,28 @@ end get '/get_repo_statistics' do new_refs_statistics(params) end + +# GET /get_jobs_summary?suite=iperf&dimension=my_name +# must: +# - query_conditions +# - suite / group_id / ... +# - dimension +# - group_id / my_email / my_name +# +# Response like: +# - { +# "kezhiming" : { +# "all" : $all, +# "nr_pass" : $nr_pass, +# "nr_fail" : $nr_fali +# }, +# "chenqun" : { +# "all" : $all, +# "nr_pass" : $nr_pass, +# "nr_fail" : $nr_fali +# }, +# ... +# } +get '/get_jobs_summary' do + group_jobs_stats(params) +end -- 2.23.0
On Wed, Mar 17, 2021 at 08:17:01PM +0800, Li Yuanchao wrote:
Signed-off-by: Li Yuanchao <lyc163mail@163.com> --- container/web-backend/web-backend | 25 +++++++++++++++++++++++++ 1 file changed, 25 insertions(+)
diff --git a/container/web-backend/web-backend b/container/web-backend/web-backend index 33a3fae..2971719 100755 --- a/container/web-backend/web-backend +++ b/container/web-backend/web-backend @@ -146,3 +146,28 @@ end get '/get_repo_statistics' do new_refs_statistics(params) end + +# GET /get_jobs_summary?suite=iperf&dimension=my_name +# must: +# - query_conditions +# - suite / group_id / ... +# - dimension
dimension => GROUP_BY Here used upper case, hopefully to distinguish it from normal filter conditions.
+# - group_id / my_email / my_name +# +# Response like: +# - { +# "kezhiming" : { +# "all" : $all,
all => nr_all to be consistent
+# "nr_pass" : $nr_pass, +# "nr_fail" : $nr_fali
nr_fali => nr_fail Can you foresee more fields in future? To deserve the general "/get_jobs_summary" API name. Thanks, Fengguang
+# }, +# "chenqun" : { +# "all" : $all, +# "nr_pass" : $nr_pass, +# "nr_fail" : $nr_fali +# }, +# ... +# } +get '/get_jobs_summary' do + group_jobs_stats(params) +end -- 2.23.0
On Fri, Mar 19, 2021 at 09:00:27AM +0800, Wu Fengguang wrote:
On Wed, Mar 17, 2021 at 08:17:01PM +0800, Li Yuanchao wrote:
Signed-off-by: Li Yuanchao <lyc163mail@163.com> --- container/web-backend/web-backend | 25 +++++++++++++++++++++++++ 1 file changed, 25 insertions(+)
diff --git a/container/web-backend/web-backend b/container/web-backend/web-backend index 33a3fae..2971719 100755 --- a/container/web-backend/web-backend +++ b/container/web-backend/web-backend @@ -146,3 +146,28 @@ end get '/get_repo_statistics' do new_refs_statistics(params) end + +# GET /get_jobs_summary?suite=iperf&dimension=my_name +# must: +# - query_conditions +# - suite / group_id / ... +# - dimension
dimension => GROUP_BY
Here used upper case, hopefully to distinguish it from normal filter conditions.
ok
+# - group_id / my_email / my_name +# +# Response like: +# - { +# "kezhiming" : { +# "all" : $all,
all => nr_all
to be consistent
ok
+# "nr_pass" : $nr_pass, +# "nr_fail" : $nr_fali
nr_fali => nr_fail
Can you foresee more fields in future? To deserve the general "/get_jobs_summary" API name.
ok Thanks, Yuanchao
participants (2)
-
Li Yuanchao -
Wu Fengguang