Skip to content

secrets/database: Fix test for autoMTLS#10204

Closed
briankassouf wants to merge 1 commit into
masterfrom
fix-autotls-test
Closed

secrets/database: Fix test for autoMTLS#10204
briankassouf wants to merge 1 commit into
masterfrom
fix-autotls-test

Conversation

@briankassouf
Copy link
Copy Markdown
Contributor

With this test change the old SHA with AutoMTLS passes. I think we should re-enable that

@briankassouf briankassouf changed the title Fix test for autoMTLS secrets/database: Fix test for autoMTLS Oct 21, 2020
@pcman312
Copy link
Copy Markdown
Contributor

This looks good when I test it locally, however this PR isn't updated against master which has removed the AutoMTLS option (originally anticipating we wouldn't get it in). There are a number of other things in flight that need to get merged first, but this goes on the top of the list once those are completed (hopefully today/tomorrow).

@calvn
Copy link
Copy Markdown
Contributor

calvn commented Oct 26, 2020

@briankassouf @pcman312 I believe this can be closed now.

@pcman312
Copy link
Copy Markdown
Contributor

@calvn I agree - this change was incorporated into #10220

@calvn calvn closed this Oct 26, 2020
@briankassouf briankassouf deleted the fix-autotls-test branch October 27, 2020 16:38
tsaarni pushed a commit to Nordix/vault that referenced this pull request Oct 23, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants