Closed rabin-io closed 1 year ago
Hi @rabin-io. Thanks for your PR.
I'm waiting for a kubevirt member to verify that this patch is reasonable to test. If it is, they should reply with /ok-to-test
on its own line. Until that is done, I will not automatically test new commits in this PR, but the usual testing commands by org members will still work. Regular contributors should join the org to skip this step.
Once the patch is verified, the new status will be reflected by the ok-to-test
label.
I understand the commands that are listed here.
thanks something is weird, it doesnt run tests / git actions
/ok-to-test
right thanks, but git action still not running
right thanks, but git action still not running
It looks like the GH workflow has to be approved to run - I can see it here in the UI - I can try and kick it off if ye want?
yes please, thanks, but we might want to add Rabin to kubevirt group so it wont happen ?
we might want to bump soon kubevirtci, as 1.25 is too old https://github.com/kubevirt/kubesecondarydns/pull/62
@rabin-io maybe worth to run please a few times the e2e to see its not flaky ? there was a failure one time according emails
/test pull-kubesecondarydns-e2e-k8s
lets run few times to see if some flake exists (not sure related to this PR if so)
/test pull-kubesecondarydns-e2e-k8s
/test pull-kubesecondarydns-e2e-k8s
we are good to go, anyhow it was a cluster-up that fails one time, not tests related
\approve Thanks!
/approve
[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: AlonaKaplan
The full list of commands accepted by this bot can be found here.
The pull request process is described here
What this PR does / why we need it:
There was a missing
@
at the beginning of the NS record, which caused the line to be invalid, and we didn't get any alerts about it on zone re/load.This also broke recourse query for the NS, as the query will be passed to the internal DNS, but the record is missing.
Special notes for your reviewer:
When asking the parent (forcing no recursive query) DNS for the NS, we get a response,
But when the server redirects us to our internal DNS, we get not respond.
I tested this by manually patching the zone file to include the missing
@
.