Comment 3 for bug 45270

Daniel Silverstone wrote:
> The configuration is per distribution and per distrorelease. We should
> not expect it to be in the launchpad.conf

Gina already does it, however I agree that gina works in a more
controlled environment than the distro/distrorelease.

> By all means make it an XML configuration blob which its in the column
> in the DB.

XML & config represent the same issue, I don't think people would be
able to edit/fine-tune it in place.

> Alternatively we can just use proper columns instead of a nasty text
> field.

Could be, but anyway we don't need that bunch of attributes in the DB.

See an example of IDistribution.lucilleconfig

[publishing]
pendingremovalduration=5
root=/var/tmp/archive
archiveroot=/var/tmp/archive/ubuntutest
poolroot=/var/tmp/archive/ubuntutest/pool
distsroot=/var/tmp/archive/ubuntutest/dists
overrideroot=/var/tmp/archive/ubuntutest-overrides
cacheroot=/var/tmp/archive/ubuntutest-cache
miscroot=/var/tmp/archive/ubuntutest-misc

The only attribute we need in IDistribution is the
pendingremovalduration, which could also defaults to a zope config value.

'root' sounds like a machine dependendent value to me and should fit in
the zope config as well, local_archive_path.

Maybe we still need to handle specific publication paths fo each
distribuiton (mirroring and permission issues), so, I'd not mind to have
 an IDistribution.archive_path, which would be joined with
local_archive_path to build former 'root'.

And then a set of path.join s for poolroot, distroot, overrideroot,
cacheroot and miscroot, which could also be provided as properties in
IDistribution. I can't see any use-case for customized cache/misc/pool.

And the IDistroRelease.lucilleconfig:

[publishing]
components = main restricted universe multiverse

Isn't it the same as IDistroRelease.components() ?

[]
--
Celso Providelo <email address hidden>
Canonical Ltd - http://www.canonical.com