Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

VAULT-12144: add openapi responses for assorted /sys endpoints #18628

Merged
merged 14 commits into from
Mar 28, 2023
Merged

VAULT-12144: add openapi responses for assorted /sys endpoints #18628

merged 14 commits into from
Mar 28, 2023

Conversation

dhuckins
Copy link
Contributor

@dhuckins dhuckins commented Jan 6, 2023 •
edited
Loading

Path Operation response docs notes
/sys/version-history/ LIST link func is covered, but could not find test
/sys/leader READ link no test exists
/sys/ha-status READ link is covered but couldn't find test
/sys/host-info READ link no test exists
/sys/in-flight-req READ link no test exists

@dhuckins
added response struct for version-history
a37f092 
Signed-off-by: Daniel Huckins <dhuckins@users.noreply.github.com>
@dhuckins
add response struct for leader
0ff0c2c 
Signed-off-by: Daniel Huckins <dhuckins@users.noreply.github.com>
@dhuckins
add response struct for ha-status
692e40b 
Signed-off-by: Daniel Huckins <dhuckins@users.noreply.github.com>
@dhuckins
add response struct for host-info
1bb54f6 
Signed-off-by: Daniel Huckins <dhuckins@users.noreply.github.com>
@dhuckins
add response struct for in-flight-req
6cd2ea5 
Signed-off-by: Daniel Huckins <dhuckins@users.noreply.github.com>
@dhuckins dhuckins added this to the 1.13.0-rc1 milestone Jan 6, 2023
@dhuckins
added changelog
e9fd9cf 
Signed-off-by: Daniel Huckins <dhuckins@users.noreply.github.com>
@dhuckins dhuckins marked this pull request as ready for review January 18, 2023 21:55
@dhuckins dhuckins requested a review from a team January 18, 2023 21:55
@dhuckins @averche
Apply suggestions from code review
f0851dc 
Co-authored-by: Anton Averchenkov <84287187+averche@users.noreply.github.com>
@dhuckins dhuckins requested a review from averche February 2, 2023 21:49
@averche averche modified the milestones: 1.13.0-rc1, 1.14 Feb 13, 2023
@dhuckins
make fmt
12dbb58 
Signed-off-by: Daniel Huckins <dhuckins@users.noreply.github.com>
@dhuckins
Merge branch 'main' of github.com:hashicorp/vault into dh/response-st…
d596660 
…ructures-sys-3-status
@dhuckins dhuckins merged commit cf755f4 into main Mar 28, 2023
@dhuckins dhuckins deleted the dh/response-structures-sys-3-status branch March 28, 2023 19:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@averche averche averche approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
1.14
Development

Successfully merging this pull request may close these issues.
2 participants
@dhuckins @averche