Merge lp:~jelmer/launchpad/55288-publisher-unknown-distroseries into lp:launchpad
Proposed by
Jelmer Vernooij
Status: | Merged |
---|---|
Approved by: | Jelmer Vernooij |
Approved revision: | no longer in the source branch. |
Merged at revision: | 11323 |
Proposed branch: | lp:~jelmer/launchpad/55288-publisher-unknown-distroseries |
Merge into: | lp:launchpad |
Diff against target: |
236 lines (+67/-28) 4 files modified
lib/lp/archivepublisher/config.py (+19/-14) lib/lp/archivepublisher/ftparchive.py (+11/-3) lib/lp/archivepublisher/tests/test_config.py (+19/-9) lib/lp/archivepublisher/tests/test_ftparchive.py (+18/-2) |
To merge this branch: | bzr merge lp:~jelmer/launchpad/55288-publisher-unknown-distroseries |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Leonard Richardson (community) | code | Approve | |
Review via email: mp+32232@code.launchpad.net |
Commit message
Warn about distroseries without lucilleconfig rather than failing the entire publisher run.
Description of the change
This makes the publisher more robust against distroseries that have not been initialized for soyuz yet.
Rather than having the publisher configuration raise a LucilleConfigError when a distroseries doesn't have a matching entry in the lucille configuration, it will now write a warning to the log file.
To post a comment you must log in.