Skip to content

Commit

Permalink
tests are failing
Browse files Browse the repository at this point in the history
  • Loading branch information
corymhall committed Sep 11, 2024
1 parent af34fd0 commit 3ef191a
Show file tree
Hide file tree
Showing 2 changed files with 5 additions and 5 deletions.
3 changes: 2 additions & 1 deletion Makefile
Original file line number Diff line number Diff line change
Expand Up @@ -181,7 +181,8 @@ test_nodejs:: provider install_nodejs_sdk

test_nodejs_upgrade:: PATH := $(WORKING_DIR)/bin:$(PATH)
test_nodejs_upgrade:: provider install_nodejs_sdk
cd provider && go test -tags=nodejs -v -json -count=1 -cover -timeout 3h -parallel ${TESTPARALLELISM} . 2>&1 | tee /tmp/gotest.log | gotestfmt
# cd provider && go test -tags=nodejs -v -json -count=1 -cover -timeout 3h -parallel ${TESTPARALLELISM} . 2>&1 | tee /tmp/gotest.log | gotestfmt
cd provider && go test -tags=nodejs -v -count=1 -cover -timeout 3h -parallel ${TESTPARALLELISM} .

test_python:: install_provider test_build
cd examples && go test -tags=python -v -json -count=1 -cover -timeout 3h -parallel ${TESTPARALLELISM} . 2>&1 | tee /tmp/gotest.log | gotestfmt
Expand Down
7 changes: 3 additions & 4 deletions nodejs/eks/cmd/provider/cni.ts
Original file line number Diff line number Diff line change
Expand Up @@ -252,13 +252,12 @@ function applyVpcCniYaml(args: VpcCniInputs): string {
// Check to ensure that a compatible kubectl is installed, as we'll need it in order to deploy k8s resources below.
assertCompatibleKubectlVersionExists();

const kubeconfig = pulumi.output(args.kubeconfig).apply(config => typeof config === "string" ? config : JSON.stringify(config));
const kubeconfig: string =
typeof args.kubeconfig === "string" ? args.kubeconfig : JSON.stringify(args.kubeconfig);

// Dump the kubeconfig to a file.
const tmpKubeconfig = tmp.fileSync();
kubeconfig.apply(config => {
fs.writeFileSync(tmpKubeconfig.fd, config);
})
fs.writeFileSync(tmpKubeconfig.fd, kubeconfig);

// Compute the required CNI YAML and dump it to a file.
const tmpYaml = tmp.fileSync();
Expand Down

0 comments on commit 3ef191a

Please sign in to comment.