[MDEV-26613] Start using Debug::pkgProblemResolver with apt to make CI logs more verbose Created: 2021-09-15 Updated: 2021-10-12 Resolved: 2021-10-12 |
|
| Status: | Closed |
| Project: | MariaDB Server |
| Component/s: | N/A |
| Fix Version/s: | N/A |
| Type: | Task | Priority: | Minor |
| Reporter: | Otto Kekäläinen | Assignee: | Daniel Bartholomew |
| Resolution: | Fixed | Votes: | 0 |
| Labels: | None | ||
| Description |
|
I have in the past year used successfully several times apt debugging output, and I propose you would consider replacing all apt-get install .. commands in the buildbot code with apt-get install -o Debug::pkgProblemResolver=1 .... Works for upgrade and dist-upgrade too. In https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=977137#62 you can see an example of what kind of debug output Debug::pkgProblemResolver produces. Let me know when you have this in place and I can help test it. |
| Comments |
| Comment by Elena Stepanova [ 2021-09-15 ] | |||
|
I wouldn't rush into using it everywhere. I tried it recently while digging into yet another weird issue, and – coincidentally or not – I could never reproduce the issue with debugging enabled. It could of course be just bad luck, but given how often debugging interferes with timing and race conditions, it's not at all impossible that it did affect the outcome. When there is a danger of changing the behavior, installation tests should stick to the commands which we suggest to users. We could however try to use it after a problem occurs, but it will require a bit of extra scripting to catch exceptions and do the cleanup, since dpkg/apt-get aren't good at rolling back failed attempts. | |||
| Comment by Otto Kekäläinen [ 2021-09-17 ] | |||
|
I have never seem this cause any problems:
The issue you reference was probably not regarding apt but the debugging mode of some other software. For apt enabling this more verbose mode is perfectly safe to enable in buildbot. It can just as easily be reverted in buildbot if it would cause some issues. If you are afraid of enabling it everywhere, could you at least start by adding it to the line
so we get
and I could debug e.g. http://buildbot.askmonty.org/buildbot/builders/kvm-deb-bionic-amd64/builds/8339/steps/minor-upgrade-deps/logs/stdio ? Thanks | |||
| Comment by Elena Stepanova [ 2021-09-18 ] | |||
|
I didn't say it caused a problem. I said that it appeared to make a problem go away, which is more dangerous for testing than causing a problem, as it can lead to undetectable false negatives. | |||
| Comment by Otto Kekäläinen [ 2021-09-27 ] | |||
|
dbart Can we please start using this? If not everywhere, then at least as suggested above? | |||
| Comment by Daniel Bartholomew [ 2021-09-27 ] | |||
|
otto I've added `-o Debug::pkgProblemResolver=1` to the script that runs the minor-upgrade-deps tests, so you will now see the additional debugging output for that test | |||
| Comment by Daniel Bartholomew [ 2021-09-28 ] | |||
|
Here's the log of a rebuild of the build referenced in a previous comment with the new option in use: http://buildbot.askmonty.org/buildbot/builders/kvm-deb-bionic-amd64/builds/8408/steps/minor-upgrade-deps/logs/stdio | |||
| Comment by Otto Kekäläinen [ 2021-10-12 ] | |||
|
The verbose output was applied on at least one apt upgrade stage and that is enough for now, so closing issue. |