action #107311
closedcoordination #103962: [saga][epic] Easy multi-machine handling: MM-tests as first-class citizens
coordination #15850: [epic] Improve displaying job dependencies
The dependency tree of `openqa-clone-job --clone-children` is broken
Description
Observation¶
Then cloning a job with children using openqa-clone-job --clone-children ... the resulting dependency tree is not identical to the original tree.
Steps to reproduce¶
openqa-clone-job --host http://duck-norris.qam.suse.de --clone-children --skip-download https://openqa.suse.de/tests/8215448 HDD_1=publiccloud_tools_0027.qcow2- Compare the dependency tree of the original job vs the cloned job.
Problem¶
- Dependency tree is not identical
Suggestion¶
- Fix the issue
Workaround¶
- None identified
Files
Updated by ph03nix almost 3 years ago
- File Screenshot 2022-02-23 at 09-27-55 openQA sle-15-SP3-AZURE-BYOS-Updates-x86_64-Build20220223-1-publiccloud_download_testrepo[...].png added
- File Screenshot 2022-02-23 at 09-27-46 phoenix_-openQA server sle-15-SP3-AZURE-BYOS-Updates-x86_64-Build20220223-1-publiccloud_d[...].png added
Updated by ph03nix almost 3 years ago
- File deleted (
Screenshot 2022-02-23 at 09-27-46 phoenix_-openQA server sle-15-SP3-AZURE-BYOS-Updates-x86_64-Build20220223-1-publiccloud_d[...].png)
Updated by ph03nix almost 3 years ago
- File deleted (
Screenshot 2022-02-23 at 09-27-55 openQA sle-15-SP3-AZURE-BYOS-Updates-x86_64-Build20220223-1-publiccloud_download_testrepo[...].png)
Updated by ph03nix almost 3 years ago
- File Original.png Original.png added
- File Broken.png Broken.png added
Screenshot of the original dependency tree:
Screenshot of the resulting dependency tree:
Updated by okurz almost 3 years ago
- Related to action #69976: Show dependency graph for cloned jobs added
Updated by okurz almost 3 years ago
- Assignee set to mkittler
- Priority changed from Normal to High
- Target version set to Ready
Sounds like a recent regression. @mkittler please look into this.
Updated by mkittler almost 3 years ago
I can reproduce the problem locally. However, I'd like to note that I only changed displaying the tree. So https://github.com/os-autoinst/openQA/pull/4507 shouldn't affect cloning. It only changes the functions used to make the JSON data for the graph rendering library, not the cluster computation. Maybe it is caused by https://github.com/os-autoinst/openQA/pull/4498 as it also changes the cluster computation or the problem was even introduced quite a while ago when changing some logic within the clone script.
Updated by ph03nix almost 3 years ago
This issue is already present for a long time*, this is not something that has been introduced recently.
- out of my head - more than a year, I also remember other colleagues to complain about this issue
Updated by mkittler almost 3 years ago
- Status changed from New to In Progress
This issue is already present for a long time*, this is not something that has been introduced recently.
I can only concur. I've just cloned the dependency tree again and fixed it afterwards via a manual SQL query. This allows me to clone the "good" graph from my local instance to my local instance. So I can control the openQA version on both ends. The problem was reproducible on 633b6df5b which was before my recent dependency changes.
I will of course nevertheless try to fix it. As far as I can see (via some debug printing), openQA's API returns correct cluster information. So the bug is entirely within the clone script.
Updated by mkittler almost 3 years ago
Updated by openqa_review almost 3 years ago
- Due date set to 2022-03-10
Setting due date based on mean cycle time of SUSE QE Tools
Updated by mkittler almost 3 years ago
- Related to deleted (action #69976: Show dependency graph for cloned jobs)
Updated by mkittler almost 3 years ago
- Status changed from In Progress to Feedback
Updated by mkittler almost 3 years ago
- Status changed from Feedback to Resolved
The PR has been merged and is deployed on OSD. (The deployment on OSD doesn't really matter as the fix is specific to the local openqa-clone-job script.)