Use a different package for dependency test#18
Conversation
The `dummy-test-package-crested` package is no more maintained, let's use a different one.
|
Hi, Feel free to cherry-pick this commit into your PR: anyway, upgrade started, but machine was not upgraded. It was still f43 after the upgrade to f44.
|
LecrisUT
left a comment
There was a problem hiding this comment.
LGTM. @psss can you confirm again the merge order of these? This one should be merged first and then teemtee/tmt#4725 should go in?
anyway, upgrade started, but machine was not upgraded. It was still f43 after the upgrade to f44. No idea whats happening during the reboot as its not visible from logs :(
It should be independent to this change though right? For now want to focus on getting teemtee/tmt#4725 in
Yes, whenever 4725 is at the top of the merge queue, this one should be merged, 4725 test jobs restarted to make sure everything works fine with the new required package and merge. |
@psklenar, thanks for the hint. Actually, we've discussed using There @mmacura311 suggested this is low-level functionality not expected to be used by users:
Now I see that
Hmmm, does it seem to be limited to |
|
Well, will merge and go ahead with teemtee/tmt#4725. We can figure out the rest with the subsequent PRs that are coming |
The `dummy-test-package-crested` package is no more maintained, let's use a different one. * Update require to use `tree` instead * Rename `finish` to `cleanup` to fix guest cleanup * Remove the temporary workaround pointing to a specific `ref` Related pull requests: * teemtee/upgrade#18 * teemtee/upgrade#15
The
dummy-test-package-crestedpackage is no more maintained, let's use a different one.Related pull request: