Skip to content

Comments

Setup github actions for building & releasing#1

Open
andsens wants to merge 4 commits intoproject0:mainfrom
orbit-online:ft-github-actions
Open

Setup github actions for building & releasing#1
andsens wants to merge 4 commits intoproject0:mainfrom
orbit-online:ft-github-actions

Conversation

@andsens
Copy link

@andsens andsens commented Nov 25, 2025

I added some Github actions so that you can release by pushing a tag. main is built automatically as well.
Everything is linted as well. I had to bump the golanci version to get the lint action working.

I don't think it's possible (I tried!) to do matrix builds with goreleaser. So the images are now built by through a normal docker-buildx action, though the binary itself is still managed by gorelease. By running the compilations on the proper architectures the ARM build should run considerably faster, and actually works. When using qemu to do image builds I have in the past seen 20+ min. runtimes, timeouts, and all kinds of other failures.

@project0
Copy link
Owner

Thanks, i appreciate the PR.
Actually i had problems to get the webhook working at all (thus, there is no release yet), i had tremendous issues getting consistent sync to dns due some issues with the labels.
Did you tried it already? Maybe recent version of external-dns fixed something that make it suddenly work correctly.

@andsens
Copy link
Author

andsens commented Nov 26, 2025

Hm, well something is definitely off. It keeps syncing the same registry TXT record over and over again. Is that the issue you're talking about? Otherwise post a repro as a DNSEndpoint CRD and I can quickly test it. Like so:

apiVersion: externaldns.k8s.io/v1alpha1
kind: DNSEndpoint
metadata:
  name: examplednsrecord
spec:
  endpoints:
  - dnsName: test.example.com
    recordTTL: 180
    recordType: A
    targets:
    - 127.0.0.1

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