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

Remove udp-services ConfigMap from k8gb helm chart templates #354

Merged
merged 1 commit into from
Mar 11, 2021

Conversation

somaritane
Copy link
Contributor

@somaritane somaritane commented Mar 6, 2021

  • Removed NGINX Ingress Controller udp-services ConfigMap from k8gb chart templates.
  • Updated k8gb documentation and example values files in order to reflect removal.
  • Created documentation explaining DNS traffic configuration
  • Amended general and local setup documentation

This PR closes #352 and #323

@somaritane somaritane force-pushed the remove-udp-services-from-helm branch from efeaa2c to 47e2f9c Compare March 6, 2021 10:14
@ytsarev
Copy link
Member

ytsarev commented Mar 6, 2021

Aslo this PR will close https://github.com/AbsaOSS/k8gb/issues/323

@somaritane somaritane force-pushed the remove-udp-services-from-helm branch from 47e2f9c to 9fb3e33 Compare March 10, 2021 21:26
@somaritane somaritane linked an issue Mar 10, 2021 that may be closed by this pull request
@somaritane somaritane requested a review from ytsarev March 10, 2021 21:29
Copy link
Member

@ytsarev ytsarev left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

One typo detected and one suggestion provided.
Otherwise, looks really great.

@somaritane somaritane force-pushed the remove-udp-services-from-helm branch from 9fb3e33 to 724da62 Compare March 10, 2021 22:06
@somaritane somaritane requested a review from ytsarev March 10, 2021 22:08
@somaritane somaritane changed the title Remove upd-services ConfigMap from k8gb helm chart templates Remove udp-services ConfigMap from k8gb helm chart templates Mar 10, 2021
ytsarev
ytsarev previously approved these changes Mar 10, 2021
Copy link
Member

@ytsarev ytsarev left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

Copy link
Contributor

@donovanmuller donovanmuller left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Some suggestions on grammar mainly

@somaritane somaritane force-pushed the remove-udp-services-from-helm branch from 0a1d5fe to 0bd6ca7 Compare March 11, 2021 07:09
@somaritane
Copy link
Contributor Author

Some suggestions on grammar mainly

@donovanmuller , thanks, updated!

@somaritane somaritane requested a review from k0da March 11, 2021 10:13
k0da
k0da previously approved these changes Mar 11, 2021
- Removed NGINX Ingress Controller `udp-services` ConfigMap from k8gb chart templates.
- Updated k8gb documentation and example values files in order to reflect removal.
- Created documentation explaining DNS traffic configuration
- Amended general and local setup documentation
@somaritane somaritane force-pushed the remove-udp-services-from-helm branch from 4ff41c5 to 578bc79 Compare March 11, 2021 10:28
@somaritane somaritane requested a review from k0da March 11, 2021 10:30
@somaritane somaritane merged commit 5dfde26 into master Mar 11, 2021
@somaritane somaritane deleted the remove-udp-services-from-helm branch March 11, 2021 10:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
4 participants