Skip to content

Commit

Permalink
results: add --arch filter
Browse files Browse the repository at this point in the history
Adds --arch to results options and  parameter filter_architecture to api params in dashboard_fetch_summary, dashboard_fetch_builds,
dashboard_fetch_boots, and dashboard_fetch_tests.

Signed-off-by: Camila Alvarez <cam.alvarez.i@gmail.com>
  • Loading branch information
CamilaAlvarez committed Mar 4, 2025
1 parent 780f9da commit eddc48c
Showing 1 changed file with 24 additions and 12 deletions.
36 changes: 24 additions & 12 deletions kcidev/subcommands/results.py
Original file line number Diff line number Diff line change
Expand Up @@ -55,47 +55,51 @@ def dashboard_api_fetch(endpoint, params, max_retries=3):
raise click.Abort()


def dashboard_fetch_summary(origin, giturl, branch, commit):
def dashboard_fetch_summary(origin, giturl, branch, commit, arch):
endpoint = f"tree/{commit}/summary"
params = {
"origin": origin,
"git_url": giturl,
"git_branch": branch,
}

if arch is not None:
params["filter_architecture"] = arch
return dashboard_api_fetch(endpoint, params)


def dashboard_fetch_builds(origin, giturl, branch, commit):
def dashboard_fetch_builds(origin, giturl, branch, commit, arch):
endpoint = f"tree/{commit}/builds"
params = {
"origin": origin,
"git_url": giturl,
"git_branch": branch,
}

if arch is not None:
params["filter_architecture"] = arch
return dashboard_api_fetch(endpoint, params)


def dashboard_fetch_boots(origin, giturl, branch, commit):
def dashboard_fetch_boots(origin, giturl, branch, commit, arch):
endpoint = f"tree/{commit}/boots"
params = {
"origin": origin,
"git_url": giturl,
"git_branch": branch,
}

if arch is not None:
params["filter_architecture"] = arch
return dashboard_api_fetch(endpoint, params)


def dashboard_fetch_tests(origin, giturl, branch, commit):
def dashboard_fetch_tests(origin, giturl, branch, commit, arch):
endpoint = f"tree/{commit}/tests"
params = {
"origin": origin,
"git_url": giturl,
"git_branch": branch,
}

if arch is not None:
params["filter_architecture"] = arch
return dashboard_api_fetch(endpoint, params)


Expand Down Expand Up @@ -439,6 +443,10 @@ def common_options(func):
is_flag=True,
help="Select latest results available",
)
@click.option(
"--arch",
help="Filter by arch"
)
@wraps(func)
def wrapper(*args, **kwargs):
return func(*args, **kwargs)
Expand Down Expand Up @@ -488,12 +496,13 @@ def summary(
branch,
commit,
latest,
arch
):
"""Display a summary of results."""
giturl, branch, commit = set_giturl_branch_commit(
origin, giturl, branch, commit, latest, git_folder
)
data = dashboard_fetch_summary(origin, giturl, branch, commit)
data = dashboard_fetch_summary(origin, giturl, branch, commit, arch)
cmd_summary(data)


Expand Down Expand Up @@ -521,6 +530,7 @@ def builds(
branch,
commit,
latest,
arch,
download_logs,
status,
filter,
Expand All @@ -529,7 +539,7 @@ def builds(
giturl, branch, commit = set_giturl_branch_commit(
origin, giturl, branch, commit, latest, git_folder
)
data = dashboard_fetch_builds(origin, giturl, branch, commit)
data = dashboard_fetch_builds(origin, giturl, branch, commit, arch)
cmd_builds(data, commit, download_logs, status)


Expand All @@ -545,6 +555,7 @@ def boots(
branch,
commit,
latest,
arch,
download_logs,
status,
filter,
Expand All @@ -553,7 +564,7 @@ def boots(
giturl, branch, commit = set_giturl_branch_commit(
origin, giturl, branch, commit, latest, git_folder
)
data = dashboard_fetch_boots(origin, giturl, branch, commit)
data = dashboard_fetch_boots(origin, giturl, branch, commit, arch)
cmd_tests(data["boots"], commit, download_logs, status, filter)


Expand All @@ -569,6 +580,7 @@ def tests(
branch,
commit,
latest,
arch,
download_logs,
status,
filter,
Expand All @@ -577,7 +589,7 @@ def tests(
giturl, branch, commit = set_giturl_branch_commit(
origin, giturl, branch, commit, latest, git_folder
)
data = dashboard_fetch_tests(origin, giturl, branch, commit)
data = dashboard_fetch_tests(origin, giturl, branch, commit, arch)
cmd_tests(data["tests"], commit, download_logs, status, filter)


Expand Down

0 comments on commit eddc48c

Please sign in to comment.