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

Support kind: Service as resource #346

Closed
wants to merge 3 commits into from

Conversation

daniel-ciaglia
Copy link

With this PR the replicator supports annotated kind: Service which will be replicated to another namespace as type: ExternalName. This feature allows to cover 2 use cases

  1. common DNS domain for services or a very lightweight service mesh
  2. migration of services into own namespaces while keeping their known DNS names

Please note:

  • metadata.annotations are not replicated by default as on a kind: service they usually drive load-balancer operators. You can explicitly set replicator.v1.mittwald.de/strip-annotations: "false" to keep them.
  • there is only the replicator.v1.mittwald.de/replicate-to option implemented
  • pre-existing target kind: Service will happily be patched/overwritten ;-)

Thanks for the very clean code and ease of extensibility which comes with it. Much appreciated 🙏

* Implement `kind: services`
* adjust Dockerfiles to match go-releaser
* add more tests and documentation
@daniel-ciaglia
Copy link
Author

Solves #347

@daniel-ciaglia daniel-ciaglia marked this pull request as ready for review August 26, 2024 09:21
* introduce generic synced map, use in generic_iterator (mittwald#348)
* Bump go to 1.22 (mittwald#343)

Co-authored-by: JorgeQ <[email protected]>
Co-authored-by: Martin Helmich <[email protected]>

* Bump chartVersion and appVersion to 'v2.10.2'

---------

Co-authored-by: Stephan Aßmus <[email protected]>
Co-authored-by: JorgeAndresQuintero <[email protected]>
Co-authored-by: JorgeQ <[email protected]>
Co-authored-by: Martin Helmich <[email protected]>
Co-authored-by: Mittwald Machine <[email protected]>
@daniel-ciaglia
Copy link
Author

I am sorry, I messed up my fork by squash merging upstream. Will close this PR and create a clean, new one.

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.

1 participant