Adding context.Context support for logging for use in the Terraform p…

…rovider

.github/workflows/build.yml

@@ -1,9 +1,14 @@
 name: Build
 on:
   push:
+    branches:
+      - main
   pull_request:
   schedule:
     - cron: '0 17 * * 2'
+permissions:
+  contents: write
+  pull-requests: write
 jobs:
   fmt:
     name: gofmt
@@ -48,7 +53,7 @@ jobs:
       - name: Run golangci-lint
         uses: golangci/golangci-lint-action@v2
         with:
-          version: v1.32.2
+          version: v1.45.2
           args: --timeout=5m -E asciicheck -E bodyclose -E dupl -E errorlint -E exportloopref -E funlen
   codeql:
     name: CodeQL

README.md

@@ -104,6 +104,8 @@ type Logger interface {
 	Infof(format string, args ...interface{})
 	Warningf(format string, args ...interface{})
 	Errorf(format string, args ...interface{})
+
+    WithContext(ctx context.Context) Logger
 }
 ```
 
@@ -129,6 +131,10 @@ func (k klogLogger) Warningf(format string, args ...interface{}) {
 func (k klogLogger) Errorf(format string, args ...interface{}) {
 	klog.Errorf(format, args...)
 }
+
+func (k klogLogger) WithContext(_ context.Context) Logger {
+    return k
+}
 ```
 
 You can then create a new logger copy like this:

go.mod

@@ -1,3 +1,3 @@
-module github.com/ovirt/go-ovirt-client-log/v2
+module github.com/ovirt/go-ovirt-client-log/v3
 
 go 1.16

go.sum

@@ -1,4 +0,0 @@
-github.com/go-logr/logr v0.4.0 h1:K7/B1jt6fIBQVd4Owv2MqGQClcgf0R266+7C/QjRcLc=
-github.com/go-logr/logr v0.4.0/go.mod h1:z6/tIYblkpsD+a4lm/fGIIU9mZ+XfAiaFtq7xTgseGU=
-k8s.io/klog/v2 v2.10.0 h1:R2HDMDJsHVTHA2n4RjwbeYXdOcBymXdX/JRb1v0VGhE=
-k8s.io/klog/v2 v2.10.0/go.mod h1:hy9LJ/NvuK+iVyP4Ehqva4HxZG/oXyIS3n3Jmire4Ec=

go_logger.go

@@ -1,6 +1,7 @@
 package ovirtclientlog
 
 import (
+	"context"
 	"fmt"
 	"log"
 )
@@ -24,6 +25,10 @@ type goLogger struct {
 	logger *log.Logger
 }
 
+func (g *goLogger) WithContext(_ context.Context) Logger {
+	return g
+}
+
 func (g *goLogger) write(format string, args ...interface{}) {
 	if g.logger == nil {
 		log.Printf(fmt.Sprintf("%s\n", format), args...)

go_logger_test.go

@@ -6,7 +6,7 @@ import (
 	"strings"
 	"testing"
 
-	"github.com/ovirt/go-ovirt-client-log/v2"
+	"github.com/ovirt/go-ovirt-client-log/v3"
 )
 
 func TestGoLogger(t *testing.T) {

logger.go

@@ -1,9 +1,15 @@
 package ovirtclientlog
 
+import "context"
+
 // Logger provides pluggable logging for oVirt client libraries.
 type Logger interface {
 	Debugf(format string, args ...interface{})
 	Infof(format string, args ...interface{})
 	Warningf(format string, args ...interface{})
 	Errorf(format string, args ...interface{})
+
+	// WithContext returns a logger that adheres to a specific context. This is useful when the backend logger needs
+	// access to the context the current operation is running under.
+	WithContext(ctx context.Context) Logger
 }

noop_logger.go

@@ -1,5 +1,7 @@
 package ovirtclientlog
 
+import "context"
+
 // NewNOOPLogger returns a logger that does nothing.
 func NewNOOPLogger() Logger {
 	return &noopLogger{}
@@ -8,6 +10,10 @@ func NewNOOPLogger() Logger {
 type noopLogger struct {
 }
 
+func (n noopLogger) WithContext(_ context.Context) Logger {
+	return n
+}
+
 func (n noopLogger) Debugf(_ string, _ ...interface{}) {
 
 }

test_logger.go

@@ -1,6 +1,7 @@
 package ovirtclientlog
 
 import (
+	"context"
 	"testing"
 )
 
@@ -15,6 +16,10 @@ type testLogger struct {
 	t *testing.T
 }
 
+func (t *testLogger) WithContext(_ context.Context) Logger {
+	return t
+}
+
 func (t *testLogger) Debugf(format string, args ...interface{}) {
 	t.t.Helper()
 	t.t.Logf(format, args...)