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.

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

Fixes #18482: Move health check API in /system/ #3333

Conversation

ElaadF
Copy link
Member

@ElaadF ElaadF commented Nov 4, 2020

@ElaadF ElaadF requested a review from fanf November 4, 2020 17:50
@ElaadF ElaadF force-pushed the arch_18482/move_health_check_api_in_system branch from 70648b1 to 6045c4b Compare November 4, 2020 18:04
@ElaadF
Copy link
Member Author

ElaadF commented Nov 4, 2020

Commit modified

@ElaadF ElaadF requested a review from amousset November 4, 2020 18:05
@@ -153,6 +151,8 @@ paths:
$ref: paths/system/archives-restore.yml
"/system/archives/{archiveKind}/zip/{commitId}":
$ref: paths/system/archives-zip.yml
"/system/healthcheck":
$ref: paths/system/healthcheck.yml
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You also need to remove the "Health check" tag from the openapi.src.yml and healthcheck.yml files

@ElaadF ElaadF force-pushed the arch_18482/move_health_check_api_in_system branch from 6045c4b to 680d54e Compare November 5, 2020 09:18
@ElaadF
Copy link
Member Author

ElaadF commented Nov 5, 2020

Commit modified

@ElaadF ElaadF requested a review from amousset November 5, 2020 09:18
@Normation-Quality-Assistant
Copy link
Contributor

OK, merging this PR

@Normation-Quality-Assistant Normation-Quality-Assistant merged commit 3967128 into Normation:branches/rudder/6.2 Nov 7, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
4 participants