search

zabbix-community / helm-zabbix

Helm chart for Zabbix
https://artifacthub.io/packages/helm/zabbix-community/zabbix
Apache License 2.0
79 stars 45 forks source link

Update statefulset-postgresql.yaml #39

Closed alezkv closed 11 months ago

alezkv commented 1 year ago

What this PR does / why we need it:

Which issue this PR fixes

(optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close that issue when PR gets merged)

Special notes for your reviewer:

Checklist

[Place an '[x]' (no spaces) in all applicable fields. Please remove unrelated fields.]

aeciopires commented 11 months ago

Hi @alezkv!

I'm closing this Pull Request because I included your changes in this PR: https://github.com/zabbix-community/helm-zabbix/pull/25

Thanks for your contributition.

alezkv commented 11 months ago

@aeciopires Great way how to not grow projects contributes. Just saying.

aeciopires commented 11 months ago

Hi @alezkv,

Sorry if I gave the wrong impression. My PR has been waiting since January/2023 and it contains a big change. I have included your changes in my PR with the intention of speeding up solutions and have included your name in the release notes as a thank you for your work. I have not removed your credits.

See this message https://github.com/zabbix-community/helm-zabbix/releases/tag/zabbix-4.0.0

Imagine the time and work you would have if you had to understand my merge, resolve code conflicts and only then be able to be reviewed again...

You have the right to be ungry/sad, but I did not intend to harm your contribution or the evolution of the project.

You are welcome to open new Pull Requests and issues.

Have a nice week

alezkv commented 11 months ago

@aeciopires Thanks Aécio for explanation. I personally have no expectation or strong feeling about that. And cleanly understand the intent for that action.

Thanks again for your time and effort.