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

feat: probe endpoint /status #13

Closed
wants to merge 1 commit into from

Conversation

dnplkndll
Copy link
Contributor

adding a probe endpoint
bump
gitlab build ci (server)

@sbidoul
Copy link
Member

sbidoul commented Apr 10, 2022

@dnplkndll thanks for the contrib.

Could you do the 0.12.6 Dockerfile and the status endpoints in separate PRs ?
Note the Dockerfiles were modified/fixed in #12.

Regarding CI, I don't have travis credits but I made the build and publish workflow to ghcr.io.

I'll see if I can move the tests to GitHub actions too.

@dnplkndll dnplkndll force-pushed the add_probe_endpoint branch from e852243 to 1b0a3f2 Compare April 10, 2022 16:25
@dnplkndll
Copy link
Contributor Author

fixed up

reasoning is we use the autodevops to build and review/release. I have had to put in a few updates. but the resulting helm chart will now allow an odoo with workers(cron/queue) and then reporting service I was thinking.

https://gitlab.com/gitlab-org/cluster-integration/auto-deploy-image/-/merge_requests/259

I was a little curious on any sizing recommendations?

@sbidoul
Copy link
Member

sbidoul commented Apr 10, 2022

Memory and CPU requirements heavily depend on the actual reports being generated, so you'll need to experiment with your workloads.

@dnplkndll
Copy link
Contributor Author

Memory and CPU requirements heavily depend on the actual reports being generated, so you'll need to experiment with your workloads.

well guess I should do worker hpa feature next!

@jappi00
Copy link

jappi00 commented Aug 25, 2023

Any updates on this? Would be nice for our docker compose setups

@sbidoul sbidoul mentioned this pull request Dec 1, 2024
@sbidoul
Copy link
Member

sbidoul commented Dec 1, 2024

I had lost track for this. Thanks for the contrib. I cherry-picked it and added a test in #20.

@sbidoul sbidoul closed this Dec 1, 2024
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