Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

NDEV-3008 trigger tracer ci tests #14

Open
wants to merge 2 commits into
base: develop
Choose a base branch
from

Conversation

andrei-neon
Copy link
Contributor

No description provided.

@andrei-neon andrei-neon changed the title trigger tracer ci tests NDEV-3008 trigger tracer ci tests May 15, 2024
@andrei-neon andrei-neon force-pushed the ndev_3008_trigger_tracer_tests branch 2 times, most recently from c34c684 to 08b9cc8 Compare May 20, 2024 15:47
@andrei-neon andrei-neon force-pushed the ndev_3008_trigger_tracer_tests branch from f988d13 to eabee2c Compare May 22, 2024 08:00
@andrei-neon andrei-neon force-pushed the ndev_3008_trigger_tracer_tests branch from 85d1b93 to 2250bdc Compare May 22, 2024 14:47
@andrei-neon andrei-neon marked this pull request as ready for review May 23, 2024 13:54
Get all runs for Tracer CI from particular date.
"""
payload = {"created": created_time}
resp = session.get("https://api.github.com/repos/neonlabsorg/tracer-api/actions/runs", params=payload).json()
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

you can add here branch name at least ( to minimalize situation when someone run the build at the same time)

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Done ✅

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

There is the parameter branch for the actions/runs request
Like /actions/runs?branch=<branch_name>

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Makes sense, I'll add filtering for request in this case and remove filtering for response as it's done now
len([run for run in resp["workflow_runs"] if run["head_branch"] == ref]).

@kristinaNikolaevaa
Copy link
Contributor

kristinaNikolaevaa commented May 23, 2024

It CAN BE MERGED ONLY after https://github.com/neonlabsorg/tracer-api/pull/163 will be merged

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants