Closed barnumbirr closed 3 years ago
Just tested locally after downgrading to Python 3.7 and upgrading octodns
to v0.9.11
(not strictly necessary), works like a charm.
π Thank you @barnumbirr !
I appreciate you providing these details and demonstrating downgrading python eliminated the problem.
I'd like to work out the version changes in #25. Let's keep this open for now, so it doesn't get lost.
Hey @solvaholic,
thanks for looking into this. Just my two cents:
octodns
? Pulling from the projects master branch might be a bit optimistic, we could however always pull the latest version released to PyPi.Hi @barnumbirr π
I took both your suggestions, to upgrade octodns and downgrade Python. Thanks again for sorting this out.
Please test with the changes in main
and let me know how it goes. You can run locally or use the @main
version of the action:
uses: solvaholic/octodns-sync@main
Hi @solvaholic,
both this issue and the one in #23 have been fixed in main, nicely done. π
Thanks again for looking into these and thanks for creating octodns-sync
.
Hello,
What I did
I tried to add DKIM records to a domain using the OVH provider.
What I expected to happen
DKIM record is added successfully
What happened instead
Additional info
This error could be blamed on several factors: 1)
octodns-sync
is pinned to "old" octodns version, runningv0.9.10
even thoughv0.9.11
was released about a month ago. 2) Usage of Python 3.9 even though octodns wasn't being tested on that specific version up until a few days ago.Regarding
1)
, running the latest version of octodns wouldn't have helped in this case. A PR to fix the issue was merged two days ago but no new release was created. This PR also introduced CI/CD for Python 3.9(see 2))
. In this particular instance, the easiest fix would be to downgradeoctodns-sync
's Python version to 3.7. It would however require a discussion on how to handle versioning of upstream in the future,Cheers.