-
Notifications
You must be signed in to change notification settings - Fork 1.5k
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(monitoring): add servicemonitor and port to exporter sidecar #8240
feat(monitoring): add servicemonitor and port to exporter sidecar #8240
Conversation
Signed-off-by: Karsten Siemer <karsten.siemer@sda.se>
As mentioned here, we are considering breaking all the 3rd party bits like Prometheus, |
@cneill it is indeed a tough problem and I generally do agree with your statements. If it is decided to not include third party tools, this chart needs to include a lot of loops to iterate in sidecars, volumes etc. In regards to serviceMonitor, it is a staple in many opensource projects and included in a lot of charts. If it is already in, I am happy to use it and thought many others might be as well. Would the PR be acceptable, if the serviceMonitor is removed? I have no problems with adding third party stuff another way, but I certainly need the port on the sidecar. |
@KarstenSiemer as you stated this is the common way how people are distributing this days their helm chart, I like see it is improvement. |
@DSSever, I'm uncertain if there is a definitive "sweet spot" to be found, as even a term like "production ready" can have different interpretations depending on the observer. To cater to the broadest range of users, the most suitable production ready state for this chart would likely be a minimal configuration that can be expanded upon. Users would have the option to add more sidecars and their corresponding ports, for instance. By excluding any vendor-specific code for AWS, GCP, or Azure, this chart would cater to bare metal users while still allowing expandability for AWS users who may prefer using their own custom sidecars and code. For example, on AWS, one might opt to use IAM instead of a password for connecting to a managed PostgreSQL database. In this approach, a serviceMonitor would not be included in the chart based on the provided definition. The primary focus of this chart would be to deploy DefectDojo efficiently, keeping the scope limited and the codebase more manageable and conducive to contributions. Any additional value that currently resides in the cloud-specific parts of the chart could be extracted and documented separately, providing examples and instructions on how to achieve those specific configurations. |
Closing due to stale |
We are narrowing the scope of acceptable enhancements to DefectDojo in preparation for v3. Learn more here:
https://github.com/DefectDojo/django-DefectDojo/blob/master/readme-docs/CONTRIBUTING.md
Description
Describe the feature / bug fix implemented by this PR.
If this is a new parser, the parser guide may be worth (re)reading.
Test results
Ideally you extend the test suite in
tests/
anddojo/unittests
to cover the changed in this PR.Alternatively, describe what you have and haven't tested.
Documentation
Please update any documentation when needed in the documentation folder)
Checklist
This checklist is for your information.
dev
.dev
.bugfix
branch.Extra information
Please clear everything below when submitting your pull request, it's here purely for your information.
Moderators: Labels currently accepted for PRs:
Contributors: Git Tips
Rebase on dev branch
If the dev branch has changed since you started working on it, please rebase your work after the current dev.
On your working branch
mybranch
:In case of conflict:
When everything's fine on your local branch, force push to your
myOrigin
remote:To cancel everything:
Squashing commits
pick
byfixup
on the commits you want squashed outpick
byreword
on the first commit if you want to change the commit messageForce push to your
myOrigin
remote: