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

[FEATURE REQUEST] Restart CoreDNS pods conditionally #2397

Closed
9 of 12 tasks
to-bar opened this issue Jun 25, 2021 · 1 comment
Closed
9 of 12 tasks

[FEATURE REQUEST] Restart CoreDNS pods conditionally #2397

to-bar opened this issue Jun 25, 2021 · 1 comment

Comments

@to-bar
Copy link
Contributor

to-bar commented Jun 25, 2021

Is your feature request related to a problem? Please describe.
There is no need to restart CoreDNS on each epicli run unless cluster hosts have been changed. This is follow-up task to changes done in PR #2394.

Describe the solution you'd like
Restart CoreDNS only when needed.

Describe alternatives you've considered
Test newer version of CoreDNS to check whether it solves issue #2345.

Additional context
Code already prepared by @to-bar.


DoD checklist

  • Changelog updated
  • COMPONENTS.md updated / doesn't need to be updated
  • Feature has automated tests
  • Automated tests passed (QA pipelines)
    • apply
    • upgrade
  • Idempotency tested
  • Documentation added / updated / doesn't need to be updated
  • All conversations in PR resolved
  • Solution meets requirements and is done according to design doc
  • Usage compliant with license
  • Backport tasks created / doesn't need to be backported
@przemyslavic
Copy link
Collaborator

✔️ Verified pgpool status and connection to PostgreSQL nodes after scaling up PostgreSQL component from 1 to 2 nodes.
CoreDNS pods are restarted conditionally, depending on changes to /etc/hosts and CoreDNS patching.

@mkyc mkyc closed this as completed Jul 20, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants