-
Notifications
You must be signed in to change notification settings - Fork 2k
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
(proxy-maint) net-dns/coredns: bump v1.6.9, drop old #14325
Conversation
Pull Request assignmentSubmitter: @podshumok net-dns/coredns: Linked bugsNo bugs to link found. If your pull request references any of the Gentoo bug reports, please add appropriate GLEP 66 tags to the commit message and request reassignment. If you do not receive any reply to this pull request, please open or link a bug to attract the attention of maintainers. In order to force reassignment and/or bug reference scan, please append Docs: Code of Conduct ● Copyright policy (expl.) ● Devmanual ● GitHub PRs ● Proxy-maint guide |
Pull request CI reportReport generated at: 2020-04-24 10:29 UTC There are existing issues already. Please look into the report to make sure none of them affect the packages in question: |
Signed-off-by: Konstantin Podshumok <kpp.live+signed@gmail.com> Closes: gentoo#14325
Pull request CI reportReport generated at: 2020-04-24 19:25 UTC There are existing issues already. Please look into the report to make sure none of them affect the packages in question: |
net-dns/coredns/coredns-1.6.9.ebuild
Outdated
|
||
LICENSE="Apache-2.0 MIT BSD ISC MPL-2.0 BSD-2" | ||
SLOT="0" | ||
IUSE="" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can be removed.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
done
net-dns/coredns/coredns-1.6.9.ebuild
Outdated
|
||
src_install() { | ||
dobin ${PN} | ||
dodoc README.md |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
einstalldocs
, just so it may catch something else useful.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
done
net-dns/coredns/coredns-1.6.9.ebuild
Outdated
for suite in request core coremain plugin; do | ||
pushd "${suite}" || die "no tests in ${S}/${suite}" | ||
go test -race ./... || die "tests for ${S}/${suite} failed" | ||
popd |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
|| die
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
done
net-dns/coredns/coredns-1.6.9.ebuild
Outdated
# mod files don't bring anything useful for reader of that ebuild | ||
# but they are required by go modules | ||
# SRC_URI will be defined after them | ||
EGO_SUM+=( |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Could you move this list and the SRC_URI
part above among with the other list? Never seen any ebuild split like this so it feels weird.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ok
if [[ -n "${COREDNS_HEALTHCHECK_URI}" ]]; then | ||
healthcheck_delay=20 | ||
healthcheck_timer=30 | ||
|
||
healthcheck() { | ||
command -v wget || return 0 | ||
wget -Oq- "${COREDNS_HEALTHCHECK_URI}" | ||
} | ||
fi |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Does this really make a difference? Wouldn't you always want it to be 127.0.0.1:8092/health
?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It is not enabled by default by the upstream, port can also be different, as well as bind address (for example, I use 127.1.1.1)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Oh and also split adding an ebuild and removing an ebuild to separate commits. So this should have 2 commits instead of one.
Closes: gentoo#14325 Signed-off-by: Konstantin Podshumok <kpp.live+signed@gmail.com>
Signed-off-by: Konstantin Podshumok <kpp.live+signed@gmail.com>
done |
Pull request CI reportReport generated at: 2020-04-28 19:46 UTC There are existing issues already. Please look into the report to make sure none of them affect the packages in question: |
Signed-off-by: Konstantin Podshumok <kpp.live+signed@gmail.com> Closes: #14325 Signed-off-by: Joonas Niilola <juippis@gentoo.org>
No description provided.