-
Notifications
You must be signed in to change notification settings - Fork 116
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
[PULP-235] Bump minor version #6138
base: main
Are you sure you want to change the base?
Conversation
e2bf6c7
to
ce25df5
Compare
1c5e4f8
to
a2bff51
Compare
a2bff51
to
8aab825
Compare
def test_emmiting_unblocked_task_telemetry(dispatch_task, pulpcore_bindings, pulp_settings): | ||
# Checking online workers ready to get a task | ||
workers_online = pulpcore_bindings.WorkersApi.list(online="true").count | ||
|
||
# We need to generate long running tasks to block the workers from executing other tasks | ||
resident_task_hrefs = [ | ||
dispatch_task("pulpcore.app.tasks.test.sleep", args=(30,)) | ||
for worker in range(workers_online) | ||
] | ||
|
||
# Then we dispatch a quick unblockable task just to keep it waiting in the queue | ||
task_href = dispatch_task("pulpcore.app.tasks.test.sleep", args=(0,)) | ||
|
||
task = pulpcore_bindings.TasksApi.read(task_href) | ||
assert task.state == "waiting" | ||
|
||
[ | ||
pulpcore_bindings.TasksApi.tasks_cancel(task_href, {"state": "canceled"}) | ||
for task_href in resident_task_hrefs | ||
] | ||
|
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.
Reference for future me: Just in case you are wondering why this test get's removed. I has had no guts anymore since we stopped testing for the actual telemetry to arrive somewhere. This is just an empty shell eating CPU cycles.
8aab825
to
1bc3611
Compare
s3 tests are failing for (damn it pip!):
after
and
when installing the bindings over the application packages. |
5fd49b4
to
1b9cf36
Compare
1b9cf36
to
4af672e
Compare
So let's see if adjusting the dateutil requirements in our bindings builds solve the last issues. |
4af672e
to
3c0f335
Compare
db4272d
to
4c27f78
Compare
473c421
to
c291528
Compare
c291528
to
b448501
Compare
Automated changes by create-pull-request GitHub action