Skip to content

Commit 3addfe3

Browse files
committed
bib: replace internal/{buildconfig,container,source} with images
This commit removes the internal/{buildconfig,container,source} packages and replaces them with the (idential but renamed) versions in the images library. This needs osbuild/images#1655
1 parent c187d2d commit 3addfe3

File tree

13 files changed

+23
-1385
lines changed

13 files changed

+23
-1385
lines changed

bib/cmd/bootc-image-builder/image.go

Lines changed: 7 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -12,6 +12,7 @@ import (
1212
"strings"
1313

1414
"github.com/osbuild/images/pkg/arch"
15+
"github.com/osbuild/images/pkg/bib/osinfo"
1516
"github.com/osbuild/images/pkg/blueprint"
1617
"github.com/osbuild/images/pkg/container"
1718
"github.com/osbuild/images/pkg/customizations/anaconda"
@@ -28,10 +29,8 @@ import (
2829
"github.com/osbuild/images/pkg/runner"
2930
"github.com/sirupsen/logrus"
3031

31-
"github.com/osbuild/bootc-image-builder/bib/internal/buildconfig"
3232
"github.com/osbuild/bootc-image-builder/bib/internal/distrodef"
3333
"github.com/osbuild/bootc-image-builder/bib/internal/imagetypes"
34-
"github.com/osbuild/bootc-image-builder/bib/internal/source"
3534
)
3635

3736
// TODO: Auto-detect this from container image metadata
@@ -45,7 +44,7 @@ type ManifestConfig struct {
4544
ImageTypes imagetypes.ImageTypes
4645

4746
// Build config
48-
Config *buildconfig.BuildConfig
47+
Config *blueprint.Blueprint
4948

5049
// CPU architecture of the image
5150
Architecture arch.Arch
@@ -58,8 +57,8 @@ type ManifestConfig struct {
5857
DistroDefPaths []string
5958

6059
// Extracted information about the source container image
61-
SourceInfo *source.Info
62-
BuildSourceInfo *source.Info
60+
SourceInfo *osinfo.Info
61+
BuildSourceInfo *osinfo.Info
6362

6463
// RootFSType specifies the filesystem type for the root partition
6564
RootFSType string
@@ -460,7 +459,7 @@ func manifestForDiskImage(c *ManifestConfig, rng *rand.Rand) (*manifest.Manifest
460459
return &mf, nil
461460
}
462461

463-
func labelForISO(os *source.OSRelease, arch *arch.Arch) string {
462+
func labelForISO(os *osinfo.OSRelease, arch *arch.Arch) string {
464463
switch os.ID {
465464
case "fedora":
466465
return fmt.Sprintf("Fedora-S-dvd-%s-%s", arch, os.VersionID)
@@ -478,7 +477,7 @@ func labelForISO(os *source.OSRelease, arch *arch.Arch) string {
478477
}
479478
}
480479

481-
func needsRHELLoraxTemplates(si source.OSRelease) bool {
480+
func needsRHELLoraxTemplates(si osinfo.OSRelease) bool {
482481
return si.ID == "rhel" || slices.Contains(si.IDLike, "rhel") || si.VersionID == "eln"
483482
}
484483

@@ -607,7 +606,7 @@ func manifestForISO(c *ManifestConfig, rng *rand.Rand) (*manifest.Manifest, erro
607606
return &mf, err
608607
}
609608

610-
func getDistroAndRunner(osRelease source.OSRelease) (manifest.Distro, runner.Runner, error) {
609+
func getDistroAndRunner(osRelease osinfo.OSRelease) (manifest.Distro, runner.Runner, error) {
611610
switch osRelease.ID {
612611
case "fedora":
613612
version, err := strconv.ParseUint(osRelease.VersionID, 10, 64)

bib/cmd/bootc-image-builder/image_test.go

Lines changed: 4 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -15,8 +15,7 @@ import (
1515
"github.com/osbuild/images/pkg/runner"
1616

1717
bib "github.com/osbuild/bootc-image-builder/bib/cmd/bootc-image-builder"
18-
"github.com/osbuild/bootc-image-builder/bib/internal/buildconfig"
19-
"github.com/osbuild/bootc-image-builder/bib/internal/source"
18+
"github.com/osbuild/images/pkg/bib/osinfo"
2019
)
2120

2221
func TestGetDistroAndRunner(t *testing.T) {
@@ -46,7 +45,7 @@ func TestGetDistroAndRunner(t *testing.T) {
4645

4746
for _, c := range cases {
4847
t.Run(fmt.Sprintf("%s-%s", c.id, c.versionID), func(t *testing.T) {
49-
osRelease := source.OSRelease{
48+
osRelease := osinfo.OSRelease{
5049
ID: c.id,
5150
VersionID: c.versionID,
5251
}
@@ -685,7 +684,7 @@ func TestGenPartitionTableDiskCustomizationSizes(t *testing.T) {
685684
func TestManifestFilecustomizationsSad(t *testing.T) {
686685
config := getBaseConfig()
687686
config.ImageTypes = []string{"qcow2"}
688-
config.Config = &buildconfig.BuildConfig{
687+
config.Config = &blueprint.Blueprint{
689688
Customizations: &blueprint.Customizations{
690689
Files: []blueprint.FileCustomization{
691690
{
@@ -703,7 +702,7 @@ func TestManifestFilecustomizationsSad(t *testing.T) {
703702
func TestManifestDirCustomizationsSad(t *testing.T) {
704703
config := getBaseConfig()
705704
config.ImageTypes = []string{"qcow2"}
706-
config.Config = &buildconfig.BuildConfig{
705+
config.Config = &blueprint.Blueprint{
707706
Customizations: &blueprint.Customizations{
708707
Directories: []blueprint.DirectoryCustomization{
709708
{

bib/cmd/bootc-image-builder/main.go

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -19,6 +19,7 @@ import (
1919
"golang.org/x/exp/slices"
2020

2121
"github.com/osbuild/images/pkg/arch"
22+
"github.com/osbuild/images/pkg/bib/blueprintload"
2223
"github.com/osbuild/images/pkg/cloud"
2324
"github.com/osbuild/images/pkg/cloud/awscloud"
2425
"github.com/osbuild/images/pkg/container"
@@ -28,10 +29,9 @@ import (
2829
"github.com/osbuild/images/pkg/osbuild"
2930
"github.com/osbuild/images/pkg/rpmmd"
3031

31-
"github.com/osbuild/bootc-image-builder/bib/internal/buildconfig"
32-
podman_container "github.com/osbuild/bootc-image-builder/bib/internal/container"
3332
"github.com/osbuild/bootc-image-builder/bib/internal/imagetypes"
34-
"github.com/osbuild/bootc-image-builder/bib/internal/source"
33+
podman_container "github.com/osbuild/images/pkg/bib/container"
34+
"github.com/osbuild/images/pkg/bib/osinfo"
3535

3636
"github.com/osbuild/image-builder-cli/pkg/progress"
3737
"github.com/osbuild/image-builder-cli/pkg/setup"
@@ -246,7 +246,7 @@ func manifestFromCobra(cmd *cobra.Command, args []string, pbar progress.Progress
246246
return nil, nil, fmt.Errorf("cannot detect build types %v: %w", imgTypes, err)
247247
}
248248

249-
config, err := buildconfig.ReadWithFallback(userConfigFile)
249+
config, err := blueprintload.LoadWithFallback(userConfigFile)
250250
if err != nil {
251251
return nil, nil, fmt.Errorf("cannot read config: %w", err)
252252
}
@@ -286,7 +286,7 @@ func manifestFromCobra(cmd *cobra.Command, args []string, pbar progress.Progress
286286
}
287287

288288
// Gather some data from the containers distro
289-
sourceinfo, err := source.LoadInfo(container.Root())
289+
sourceinfo, err := osinfo.Load(container.Root())
290290
if err != nil {
291291
return nil, nil, err
292292
}
@@ -310,7 +310,7 @@ func manifestFromCobra(cmd *cobra.Command, args []string, pbar progress.Progress
310310
startedBuildContainer = true
311311

312312
// Gather some data from the containers distro
313-
buildSourceinfo, err = source.LoadInfo(buildContainer.Root())
313+
buildSourceinfo, err = osinfo.Load(buildContainer.Root())
314314
if err != nil {
315315
return nil, nil, err
316316
}

bib/cmd/bootc-image-builder/main_test.go

Lines changed: 6 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -15,16 +15,15 @@ import (
1515
"github.com/stretchr/testify/require"
1616

1717
"github.com/osbuild/images/pkg/arch"
18+
"github.com/osbuild/images/pkg/bib/osinfo"
1819
"github.com/osbuild/images/pkg/blueprint"
1920
"github.com/osbuild/images/pkg/container"
2021
"github.com/osbuild/images/pkg/dnfjson"
2122
"github.com/osbuild/images/pkg/manifest"
2223
"github.com/osbuild/images/pkg/rpmmd"
2324

2425
main "github.com/osbuild/bootc-image-builder/bib/cmd/bootc-image-builder"
25-
"github.com/osbuild/bootc-image-builder/bib/internal/buildconfig"
2626
"github.com/osbuild/bootc-image-builder/bib/internal/imagetypes"
27-
"github.com/osbuild/bootc-image-builder/bib/internal/source"
2827
)
2928

3029
func TestCanChownInPathHappy(t *testing.T) {
@@ -75,8 +74,8 @@ func getBaseConfig() *main.ManifestConfig {
7574
return &main.ManifestConfig{
7675
Architecture: arch.ARCH_X86_64,
7776
Imgref: "testempty",
78-
SourceInfo: &source.Info{
79-
OSRelease: source.OSRelease{
77+
SourceInfo: &osinfo.Info{
78+
OSRelease: osinfo.OSRelease{
8079
ID: "fedora",
8180
VersionID: "40",
8281
Name: "Fedora Linux",
@@ -100,7 +99,7 @@ func getUserConfig() *main.ManifestConfig {
10099
return &main.ManifestConfig{
101100
Architecture: arch.ARCH_X86_64,
102101
Imgref: "testuser",
103-
Config: &buildconfig.BuildConfig{
102+
Config: &blueprint.Blueprint{
104103
Customizations: &blueprint.Customizations{
105104
User: []blueprint.UserCustomization{
106105
{
@@ -111,8 +110,8 @@ func getUserConfig() *main.ManifestConfig {
111110
},
112111
},
113112
},
114-
SourceInfo: &source.Info{
115-
OSRelease: source.OSRelease{
113+
SourceInfo: &osinfo.Info{
114+
OSRelease: osinfo.OSRelease{
116115
ID: "fedora",
117116
VersionID: "40",
118117
Name: "Fedora Linux",

bib/internal/buildconfig/config.go

Lines changed: 0 additions & 145 deletions
This file was deleted.

0 commit comments

Comments
 (0)