Skip to content

Commit

Permalink
Updating the doc string for TestVault_WebhookCerts
Browse files Browse the repository at this point in the history
  • Loading branch information
jmurret committed May 21, 2022
1 parent 905ea5f commit 11db258
Showing 1 changed file with 4 additions and 7 deletions.
11 changes: 4 additions & 7 deletions acceptance/tests/vault/vault_webhook_certs_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -18,6 +18,10 @@ import (
// It then configures Consul to use vault as the backend and checks that it works
// by turning off web cert manager and configuring controller and connect injector
// to receive ca bundles and tls certs from Vault PKI engine.
// This test is modeled after TestVault() with the addition of configuring it
// to turn off web cert manager and have controller get tls and ca certs
// from Vault PKI Engine.

func TestVault_WebhookCerts(t *testing.T) {
cfg := suite.Config()
ctx := suite.Environment().DefaultContext(t)
Expand Down Expand Up @@ -170,13 +174,6 @@ func TestVault_WebhookCerts(t *testing.T) {
PolicyNames: serverPKIConfig.PolicyName,
})

// pathForConnectInjectWebookCerts :=
// vault.ConfigurePKICertificatesForConnectInjectWebhook(t, vaultClient,
// consulReleaseName, ns, "dc1", "1h")
// pathForControllerWebookCerts :=
// vault.ConfigurePKICertificatesForControllerWebhook(t, vaultClient,
// consulReleaseName, ns, "dc1", "1h")

vaultCASecret := vault.CASecretName(vaultReleaseName)

consulHelmValues := map[string]string{
Expand Down

0 comments on commit 11db258

Please sign in to comment.