Skip to content

Commit

Permalink
Merge pull request #3798 from twz123/inttest-cr-logger
Browse files Browse the repository at this point in the history
Set controller-runtime logger in inttest
  • Loading branch information
twz123 authored Dec 11, 2023
2 parents 4274ecb + f4825f0 commit cc177ef
Show file tree
Hide file tree
Showing 2 changed files with 10 additions and 6 deletions.
2 changes: 1 addition & 1 deletion go.mod
Original file line number Diff line number Diff line change
Expand Up @@ -24,6 +24,7 @@ require (
github.com/estesp/manifest-tool/v2 v2.1.3
github.com/evanphx/json-patch v5.6.0+incompatible
github.com/fsnotify/fsnotify v1.7.0
github.com/go-logr/logr v1.2.4
github.com/go-openapi/jsonpointer v0.20.0
github.com/go-playground/validator/v10 v10.16.0
github.com/google/go-cmp v0.6.0
Expand Down Expand Up @@ -145,7 +146,6 @@ require (
github.com/ghodss/yaml v1.0.0 // indirect
github.com/go-errors/errors v1.4.2 // indirect
github.com/go-gorp/gorp/v3 v3.1.0 // indirect
github.com/go-logr/logr v1.2.4 // indirect
github.com/go-logr/stdr v1.2.2 // indirect
github.com/go-openapi/jsonreference v0.20.2 // indirect
github.com/go-openapi/swag v0.22.4 // indirect
Expand Down
14 changes: 9 additions & 5 deletions inttest/addons/addons_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -25,24 +25,28 @@ import (
"testing"
"time"

"github.com/stretchr/testify/suite"

"github.com/k0sproject/k0s/internal/pkg/templatewriter"
"github.com/k0sproject/k0s/inttest/common"
"github.com/k0sproject/k0s/pkg/apis/helm/v1beta1"
v1 "k8s.io/apimachinery/pkg/apis/meta/v1"
metav1 "k8s.io/apimachinery/pkg/apis/meta/v1"
"k8s.io/apimachinery/pkg/util/wait"
k8s "k8s.io/client-go/kubernetes"
"k8s.io/client-go/kubernetes/scheme"
"sigs.k8s.io/controller-runtime/pkg/client"
crlog "sigs.k8s.io/controller-runtime/pkg/log"
"sigs.k8s.io/yaml"

"github.com/go-logr/logr/testr"
"github.com/stretchr/testify/suite"
)

type AddonsSuite struct {
common.BootlooseSuite
}

func (as *AddonsSuite) TestHelmBasedAddons() {
crlog.SetLogger(testr.New(as.T()))

addonName := "test-addon"
ociAddonName := "oci-addon"
fileAddonName := "tgz-addon"
Expand Down Expand Up @@ -100,7 +104,7 @@ func (as *AddonsSuite) deleteRelease(chart *v1beta1.Chart) {
as.Require().NoError(err)
as.Require().NoError(wait.PollUntilContextCancel(ctx, 1*time.Second, true, func(pollCtx context.Context) (done bool, err error) {
as.T().Logf("Expecting have no secrets left for release %s/%s", chart.Namespace, chart.Name)
items, err := k8sclient.CoreV1().Secrets("default").List(pollCtx, v1.ListOptions{
items, err := k8sclient.CoreV1().Secrets("default").List(pollCtx, metav1.ListOptions{
LabelSelector: fmt.Sprintf("name=%s", chart.Name),
})
if err != nil {
Expand Down Expand Up @@ -189,7 +193,7 @@ func (as *AddonsSuite) checkCustomValues(releaseName string) error {
}
return wait.PollUntilContextCancel(ctx, 1*time.Second, true, func(pollCtx context.Context) (done bool, err error) {
serverDeployment := fmt.Sprintf("%s-echo-server", releaseName)
d, err := kc.AppsV1().Deployments("default").Get(pollCtx, serverDeployment, v1.GetOptions{})
d, err := kc.AppsV1().Deployments("default").Get(pollCtx, serverDeployment, metav1.GetOptions{})
if err != nil {
if ctxErr := context.Cause(ctx); ctxErr != nil {
return false, errors.Join(err, ctxErr)
Expand Down

0 comments on commit cc177ef

Please sign in to comment.