diff --git a/docs/services/general-info/new-service-checklist.md b/docs/services/general-info/new-service-checklist.md index 2640ca8c1..eb72f9ad4 100644 --- a/docs/services/general-info/new-service-checklist.md +++ b/docs/services/general-info/new-service-checklist.md @@ -24,13 +24,18 @@ Use this checklist with copy/paste in your PR - right from the beginning. It ren bad `` good `` - [ ] If new CLI commands are introduced, those commands must be described in the README.md. + - Commands are added to `ocis/pkg/command` - [ ] If new global envvars are introduced, the name must start with `OCIS_`. - [ ] Add the service to the makefile in the ocis repo root. -- [ ] Make the service startable for binary and individual startup: - - For single binary add service to `ocis/pkg/runtime` - - For individual startup add service to `ocis/pkg/commands` +- [ ] Service startup: + - add it to `ocis/pkg/command/services.go` + - Include for automatic startup: + - add it to `ocis/pkg/runtime/service/service.go` + - Exclude from automatic startup: + - add it to the `populate optional services` block in `ocis/pkg/runtime/service/service.go` - Add the service config to `ocis-pkg/config/defaultconfig.go` - [ ] If the service is using service accounts, add it to `ocis/pkg/init/init.go` +- [ ] Check that the service properly responds to `ocis health` and has `/healthz` and `/readyz` endpoints - [ ] Add the service to `.drone.star` to enable CI. - [ ] Inform doc team in an *early stage* to review the readme AND the environment variables created. - The description must reflect the behaviour AND usually has a positive code quality impact. @@ -45,4 +50,5 @@ Use this checklist with copy/paste in your PR - right from the beginning. It ren - [ ] Add metrics to the code to enable monitoring. See the proxy service for implementation details. - Plus add documentation about monitoring in the README.md file - [ ] When the service requires translations that have to be covered by the service and are not sourced by web, see the [add translation]({{< ref "./add-translations.md" >}}) documentation for more details. +- [ ] If the service requires a `cache` or `store`, check existing services for implementation and add a documentation in the README.md ```