internal: FIXME notes for container package set

Tracing the package set and repository journey, I found that the
container package defined on the image type is never really used.

Added notes to fix later.
This commit is contained in:
Achilleas Koutsou 2023-02-22 16:10:56 +01:00 committed by Tomáš Hozza
parent d2a5c9aafa
commit f11589367e
3 changed files with 3 additions and 2 deletions

View file

@ -38,7 +38,7 @@ var (
osPkgsKey: edgeCommitPackageSet,
containerPkgsKey: func(t *imageType) rpmmd.PackageSet {
return rpmmd.PackageSet{
Include: []string{"nginx"},
Include: []string{"nginx"}, // FIXME: this has no effect
}
},
},

View file

@ -22,7 +22,7 @@ type OSTreeContainer struct {
OSTreeRef string
OSTreeParent *ostree.CommitSpec
OSVersion string
ExtraContainerPackages rpmmd.PackageSet
ExtraContainerPackages rpmmd.PackageSet // FIXME: this is never read
ContainerLanguage string
Filename string
}

View file

@ -60,6 +60,7 @@ func NewOSTreeCommitServer(m *Manifest,
}
func (p *OSTreeCommitServer) getPackageSetChain() []rpmmd.PackageSet {
// FIXME: container package is defined here
packages := []string{"nginx"}
return []rpmmd.PackageSet{
{