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 GSLB external plugin #320

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Add GSLB external plugin #320

wants to merge 1 commit into from

Conversation

dmachard
Copy link

@dmachard dmachard commented Feb 9, 2025

"I added the GSLB external plugin, which enhances CoreDNS with global server load balancing (GSLB) capabilities. This allows CoreDNS to function similarly to a BigIP F5 GSLB or similar, for example. For more details, see the description at https://github.com/dmachard/coredns-gslb#readme.

@dmachard
Copy link
Author

@chrisohaver
Hi,
Any chance to have this pull request merged or I need to do something more ?
Thanks for the great CoreDNS
Denis

@SuperQ
Copy link
Contributor

SuperQ commented Feb 22, 2025

It doesn't look like there are any semver releases/tags for this codebase. It's nicer for users if there are releases that they can use to get updates.

@dmachard
Copy link
Author

dmachard commented Feb 22, 2025

It's done,
Thanks for the feedback.

@SuperQ
Copy link
Contributor

SuperQ commented Feb 22, 2025

Looking over the design, I'm not sure I'm a fan of the external yaml configuration. Why not make all parameters part of the Corefile?

@dmachard
Copy link
Author

In fact, I chose to handle GSLB records the same way as the DB zone file (using external files) but in YAML format.

In a real configuration, there can be a large number of records to configure, so I'm not sure it's a good idea to put them directly in the Corefile.

You can find a basic example of how to define a record here:
https://github.com/dmachard/coredns-gslb/blob/main/coredns/gslb_config.example.com.yml

@dmachard
Copy link
Author

What to do next ? No more info exists in https://coredns.io/2017/07/23/add-external-plugins/

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