autopkgtest does not skip debian/tests/control stanzas with unknown fields

Bug #1434543 reported by Scott Moser
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
autopkgtest (Ubuntu)
Fix Released
Medium
Martin Pitt

Bug Description

doc/README.package-tests.rst says:
 | Control fields
 | --------------
 | The fields which may appear in debian/tests/control stanzas are:
...
 | Any unknown fields will cause the whole stanza to be skipped.

It wasn't clear how I could comment out a test. But the above seemed like one solution. So I tried to effectively stop a test from running until bug 1422681 in systemd/debian/control/tests by adding
  Disabled-until-bug-fixed: http://pad.lv/1422681
to the stanza of the test I wanted to skip.

That didn't seem to work though, the test still ran. See ubuntu test run at [2].

Its possible that I just misunderstood the doc. but it would be nice to clarify that, and if there *is* a way to add comments (and thus comment out the tes), then that'd be nice to document too.

--
[1] https://launchpadlibrarian.net/200652701/systemd_219-4ubuntu6_219-4ubuntu7.diff.gz
[2] https://jenkins.qa.ubuntu.com/job/vivid-adt-systemd/125/ARCH=amd64,label=adt/consoleFull

ProblemType: Bug
DistroRelease: Ubuntu 15.04
Package: autopkgtest 3.12
ProcVersionSignature: User Name 3.19.0-9.9-generic 3.19.1
Uname: Linux 3.19.0-9-generic x86_64
ApportVersion: 2.16.2-0ubuntu3
Architecture: amd64
Date: Fri Mar 20 12:50:11 2015
PackageArchitecture: all
ProcEnviron:
 TERM=screen
 PATH=(custom, no user)
 XDG_RUNTIME_DIR=<set>
 LANG=en_US.UTF-8
 SHELL=/bin/bash
SourcePackage: autopkgtest
UpgradeStatus: No upgrade log present (probably fresh install)

Revision history for this message
Scott Moser (smoser) wrote :
Revision history for this message
Martin Pitt (pitti) wrote :

For the record, you can just comment out stanzas with "#" as usual. I'll document this and also fix the "unknown fields will cause the whole stanza to be skipped" behaviour.

Changed in autopkgtest (Ubuntu):
importance: Undecided → Medium
status: New → Triaged
Martin Pitt (pitti)
Changed in autopkgtest (Ubuntu):
assignee: nobody → Martin Pitt (pitti)
Revision history for this message
Martin Pitt (pitti) wrote :
Changed in autopkgtest (Ubuntu):
status: Triaged → Fix Committed
Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package autopkgtest - 3.12.1

---------------
autopkgtest (3.12.1) unstable; urgency=medium

  [ Didier Roche ]
  * setup-commands/ubuntu-touch-session: Use "upstart" instead of "init".
    (LP: #1422681)

  [ Martin Pitt ]
  * adt-build-lxc: When removing the old root file system, remove btrfs
    subvolumes in it.
  * README.package-tests.rst: Document that '#' can be used for comments.
    (Part of LP #1434543)
  * Ignore debian/tests/control stanzas with unknown fields, as documented.
    (LP: #1434543)
  * adt-buildvm-ubuntu-cloud: Drop workaround for LP #1428495, it got fixed in
    cloud-init a while ago.
  * adt-buildvm-ubuntu-cloud: Fix cloud-init's prerm hacking to apply after
    dist-upgrades.

 -- Martin Pitt <email address hidden> Tue, 07 Apr 2015 07:26:27 +0200

Changed in autopkgtest (Ubuntu):
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.