Skip to content

enh: Always use the chat method#48

Merged
marcelklehr merged 14 commits intomainfrom
enh/always-use-chat
Feb 18, 2025
Merged

enh: Always use the chat method#48
marcelklehr merged 14 commits intomainfrom
enh/always-use-chat

Conversation

@marcelklehr
Copy link
Member

No description provided.

Signed-off-by: Marcel Klehr <mklehr@gmx.net>
Signed-off-by: Marcel Klehr <mklehr@gmx.net>
Signed-off-by: Marcel Klehr <mklehr@gmx.net>
Signed-off-by: Marcel Klehr <mklehr@gmx.net>
Signed-off-by: Marcel Klehr <mklehr@gmx.net>
Copy link
Contributor

@kyteinsky kyteinsky left a comment

Choose a reason for hiding this comment

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

all looks good other than these small issues.

@marcelklehr
Copy link
Member Author

Any idea why the tests are failing?

@kyteinsky
Copy link
Contributor

It looks like the test waits for 5 mins (25*24 secs) before failing at STATUS_SCHEDULED state.
Maybe we can increase the timeout and also add some logs if and when the task was picked up.

@marcelklehr
Copy link
Member Author

Shouldn't 5 mins be enough?

Signed-off-by: Marcel Klehr <mklehr@gmx.net>
@kyteinsky
Copy link
Contributor

it should but just for testing, and it might be the case that it wasn't picked up even

Signed-off-by: Marcel Klehr <mklehr@gmx.net>
@marcelklehr
Copy link
Member Author

I think it's not picked up, if it was picked up, status would be RUNNING.

@marcelklehr
Copy link
Member Author

Trying with 10mins now

Signed-off-by: Marcel Klehr <mklehr@gmx.net>
Signed-off-by: Marcel Klehr <mklehr@gmx.net>
Signed-off-by: Marcel Klehr <mklehr@gmx.net>
Signed-off-by: Marcel Klehr <mklehr@gmx.net>
+ reload constructively every 5 minutes

Signed-off-by: Marcel Klehr <mklehr@gmx.net>
Signed-off-by: Marcel Klehr <mklehr@gmx.net>
Signed-off-by: Marcel Klehr <mklehr@gmx.net>
@marcelklehr marcelklehr merged commit 39b374f into main Feb 18, 2025
5 checks passed
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.

3 participants