Skip to content

pgwire,debug: expose the HBA configuration via /debug/hba_conf#43730

Merged
craig[bot] merged 1 commit intocockroachdb:masterfrom
knz:2020105-debug-hba
Jan 6, 2020
Merged

pgwire,debug: expose the HBA configuration via /debug/hba_conf#43730
craig[bot] merged 1 commit intocockroachdb:masterfrom
knz:2020105-debug-hba

Conversation

@knz
Copy link
Copy Markdown
Contributor

@knz knz commented Jan 6, 2020

This patch exposes, on each node, the HBA configuration cache as
computed from the cluster setting.

This is used in the authentication tests to ascertain that the parsed
configuration is equivalent to the one provided as input.

Release note: None

@knz knz requested a review from madelynnblue January 6, 2020 04:10
@cockroach-teamcity
Copy link
Copy Markdown
Member

This change is Reviewable

This patch exposes, on each node, the HBA configuration cache as
computed from the cluster setting.

This is used in the authentication tests to ascertain that the parsed
configuration is equivalent to the one provided as input.

Release note: None
@knz knz force-pushed the 2020105-debug-hba branch from 9055666 to ee06658 Compare January 6, 2020 04:49
@knz
Copy link
Copy Markdown
Contributor Author

knz commented Jan 6, 2020

Thanks!

bors r+

craig bot pushed a commit that referenced this pull request Jan 6, 2020
43730:  pgwire,debug: expose the HBA configuration via /debug/hba_conf r=knz a=knz

This patch exposes, on each node, the HBA configuration cache as
computed from the cluster setting.

This is used in the authentication tests to ascertain that the parsed
configuration is equivalent to the one provided as input.

Release note: None

Co-authored-by: Raphael 'kena' Poss <knz@thaumogen.net>
@craig
Copy link
Copy Markdown
Contributor

craig bot commented Jan 6, 2020

Build succeeded

@craig craig bot merged commit ee06658 into cockroachdb:master Jan 6, 2020
@knz knz deleted the 2020105-debug-hba branch January 9, 2020 18:06
@knz
Copy link
Copy Markdown
Contributor Author

knz commented Jan 9, 2020

(used to fix #31113)

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