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

src/teuthology_api/suite: Modify how we handle Error and Success runs #51

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

kamoltat
Copy link
Member

@kamoltat kamoltat commented Mar 13, 2024

As per As per: ceph/pulpito-ng#23
The changes includes:

  1. make suite route return
    {"run": run_details, "logs": logs, "job_count": job_count}

  2. Improve how we handle Exception by utilizing Queue from
    python multiprocessing library.

  3. Set the timeout for the process to be 180 seconds, if teuthology
    doesn't respond back within that time, then we return a Process Timeout

  4. Add .env to gitignore

Contribution Guidelines

To sign and test your commits, please refer to Contibution guidelines.

Checklist

@kamoltat kamoltat requested review from VallariAg and zmc March 13, 2024 21:57
@kamoltat kamoltat force-pushed the wip-ksirivad-suite-change branch 2 times, most recently from bd56531 to 9f0b855 Compare May 9, 2024 05:37
@kamoltat kamoltat self-assigned this May 9, 2024
@kamoltat kamoltat added the enhancement Improving existing feature or logic label May 9, 2024
The changes includes:

1. make suite route return
{"run": run_details, "logs": logs, "job_count": job_count}

2. Improve how we handle Exception by utilizing Queue from
python multiprocessing library.

3. Set the timeout for the process to be 180 seconds, if teuthology
doesn't respond back within that time, then we return a Process Timeout

Signed-off-by: Kamoltat Sirivadhna <ksirivad@redhat.com>
Signed-off-by: Kamoltat Sirivadhna <ksirivad@redhat.com>
Added job counts to unittest.

Signed-off-by: Kamoltat Sirivadhna <ksirivad@redhat.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement Improving existing feature or logic
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant