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

Add the ability to add extra containers to the Pihole pod #260

Merged
merged 7 commits into from
Jun 28, 2023
Merged

Add the ability to add extra containers to the Pihole pod #260

merged 7 commits into from
Jun 28, 2023

Conversation

kenlasko
Copy link
Contributor

I have a container that I use to run a process to monitor the Pihole logs for specific DNS queries. The container uses a shared volume created on /var/log/pihole. I currently cannot use this Helm chart because there is no way to add my own container.

This pull request simply adds the ability to add any number of containers via values.yaml.

It passes LINT and I was able to successfully run my fork of the Helm chart to get my secondary container running within the Pihole pod.

This is my first ever contribution to any Github project. Please be gentle, but feedback is appreciated. :)

@MoJo2600
Copy link
Owner

Sorry, I merged another PR before yours. Thank you for your contribution, it looks good to me.

@kenlasko
Copy link
Contributor Author

No worries. I bumped the version up another notch. Hope I'm doing this right.

@MoJo2600
Copy link
Owner

Ha, now I screwed up the merge by adding trailing whitespaces :) Don't ask me if you doning it right when I can't do it right in the first place ;)

@MoJo2600 MoJo2600 merged commit 3841501 into MoJo2600:master Jun 28, 2023
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.

2 participants