Comment 26 for bug 2023243

Revision history for this message
Andreas Hasenack (ahasenack) wrote :

Hi, SRU team member here. I read the comments, and considering the problem statement from the bug description:
```
1. drmgr command fails as it doesn't find a slot
2. drmgr throws error while removing the slot
3. ctrl+c closes the terminal opened for any vm
4. ctrl+] doesn't close the terminal. But comes out with an extra 'Enter'
```

I didn't see confirmation in the comments that problems (3) and (4) are also fixed by the proposed two patches. comment #24 states that there are missing devices in /dev, and it's not clear to me if that has relation to the bugs being fixed here, or if they are new bugs. Or even if the powerpc-utils package would be responsible for that (sounds like an issue with some udev rule?).

We can accept the package as is, and the testing with the proposed packages will tell, as the testing must verify that all 4 issues are fixed, but I would like to give a chance to the people involved to clarify this issue first.

It might also well be that the patched package is already better than what we have in focal currently, and the missing devices are indeed a new separate bug, in which case I would echo what Patricia said in comment #25 and ask for a new bug to be filed.

In short, can someone please clarify on whether the two patches are enough and that we can proceed with accepting the package into focal-proposed, or if more investigation is needed?