-
Notifications
You must be signed in to change notification settings - Fork 1
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
base: develop
Are you sure you want to change the base?
Conversation
c34c684
to
08b9cc8
Compare
f988d13
to
eabee2c
Compare
85d1b93
to
2250bdc
Compare
.github/workflows/deploy.py
Outdated
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() |
There was a problem hiding this comment.
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)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Done ✅
There was a problem hiding this comment.
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>
There was a problem hiding this comment.
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])
.
It CAN BE MERGED ONLY after https://github.com/neonlabsorg/tracer-api/pull/163 will be merged |
4d2c110
to
6a0350c
Compare
6a0350c
to
3e81817
Compare
3e81817
to
34bc5ac
Compare
34bc5ac
to
0b2fee8
Compare
No description provided.