Skip to content

Commit d18bcf1

Browse files
northdpoleptzianos
authored andcommitted
nit: add logging to the metadata writer step of the image-getter
1 parent fb7988e commit d18bcf1

File tree

1 file changed

+14
-3
lines changed
  • components/targets/image-get/metadata-writer/cmd

1 file changed

+14
-3
lines changed

components/targets/image-get/metadata-writer/cmd/main.go

Lines changed: 14 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -3,6 +3,7 @@ package main
33
import (
44
"context"
55
"log"
6+
"log/slog"
67
"os"
78
"path"
89
"strings"
@@ -59,12 +60,16 @@ func NewConf(envLoader env.Loader) (*Conf, error) {
5960
}, nil
6061
}
6162

62-
func WriteTargetMetadata(conf *Conf) error {
63+
func WriteTargetMetadata(ctx context.Context, conf *Conf) error {
64+
logger := component.LoggerFromContext(ctx)
65+
6366
purl, err := packageUrlFromImage(conf.Image)
6467
if err != nil {
6568
return errors.Errorf("could not get package url from image: %w", err)
6669
}
6770

71+
logger.Info("package url from image", slog.String("purl", purl.ToString()))
72+
6873
dataSource := &ocsffindinginfo.DataSource{
6974
TargetType: ocsffindinginfo.DataSource_TARGET_TYPE_CONTAINER_IMAGE,
7075
OciPackageMetadata: &ocsffindinginfo.DataSource_OCIPackageMetadata{
@@ -148,6 +153,8 @@ func main() {
148153
}
149154

150155
func Main(ctx context.Context) error {
156+
logger := component.LoggerFromContext(ctx)
157+
151158
conf, err := NewConf(nil)
152159
if err != nil {
153160
return errors.Errorf("could not create new configuration: %w", err)
@@ -167,7 +174,7 @@ func Main(ctx context.Context) error {
167174
); err != nil {
168175
return errors.Errorf("could not run target: %w", err)
169176
}
170-
177+
logger.Info("target finished successfully")
171178
return nil
172179
}
173180

@@ -182,13 +189,17 @@ func NewTarget(conf *Conf) (*imageMetadataWriterTarget, error) {
182189
}
183190

184191
func (t *imageMetadataWriterTarget) Prepare(ctx context.Context) error {
192+
logger := component.LoggerFromContext(ctx)
193+
185194
if t.conf == nil {
186195
return errors.New("conf cannot be nil")
187196
}
188197

189-
if err := WriteTargetMetadata(t.conf); err != nil {
198+
if err := WriteTargetMetadata(ctx, t.conf); err != nil {
190199
return errors.Errorf("could not write target metadata: %w", err)
191200
}
201+
logger.Info("target metadata written to ", slog.String("metadata_loc", t.conf.TargetMetadataPath))
202+
logger.Debug("Prepare method completed successfully for target")
192203

193204
return nil
194205
}

0 commit comments

Comments
 (0)