* Re: unbound: add logging service
2021-07-16 0:38 [PR PATCH] unbound: add logging service AluminumTank
@ 2021-07-17 2:59 ` ericonr
2021-07-17 11:51 ` leahneukirchen
` (11 subsequent siblings)
12 siblings, 0 replies; 14+ messages in thread
From: ericonr @ 2021-07-17 2:59 UTC (permalink / raw)
To: ml
[-- Attachment #1: Type: text/plain, Size: 161 bytes --]
New comment by ericonr on void-packages repository
https://github.com/void-linux/void-packages/pull/31992#issuecomment-881805974
Comment:
@leahneukirchen LGTM
^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: unbound: add logging service
2021-07-16 0:38 [PR PATCH] unbound: add logging service AluminumTank
2021-07-17 2:59 ` ericonr
@ 2021-07-17 11:51 ` leahneukirchen
2021-07-17 11:52 ` leahneukirchen
` (10 subsequent siblings)
12 siblings, 0 replies; 14+ messages in thread
From: leahneukirchen @ 2021-07-17 11:51 UTC (permalink / raw)
To: ml
[-- Attachment #1: Type: text/plain, Size: 164 bytes --]
New comment by leahneukirchen on void-packages repository
https://github.com/void-linux/void-packages/pull/31992#issuecomment-881886174
Comment:
no `-t unbound`?
^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: unbound: add logging service
2021-07-16 0:38 [PR PATCH] unbound: add logging service AluminumTank
2021-07-17 2:59 ` ericonr
2021-07-17 11:51 ` leahneukirchen
@ 2021-07-17 11:52 ` leahneukirchen
2021-07-17 11:53 ` leahneukirchen
` (9 subsequent siblings)
12 siblings, 0 replies; 14+ messages in thread
From: leahneukirchen @ 2021-07-17 11:52 UTC (permalink / raw)
To: ml
[-- Attachment #1: Type: text/plain, Size: 177 bytes --]
New comment by leahneukirchen on void-packages repository
https://github.com/void-linux/void-packages/pull/31992#issuecomment-881886235
Comment:
ah it autodetects that, fine.
^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: unbound: add logging service
2021-07-16 0:38 [PR PATCH] unbound: add logging service AluminumTank
` (2 preceding siblings ...)
2021-07-17 11:52 ` leahneukirchen
@ 2021-07-17 11:53 ` leahneukirchen
2021-07-17 13:54 ` AluminumTank
` (8 subsequent siblings)
12 siblings, 0 replies; 14+ messages in thread
From: leahneukirchen @ 2021-07-17 11:53 UTC (permalink / raw)
To: ml
[-- Attachment #1: Type: text/plain, Size: 183 bytes --]
New comment by leahneukirchen on void-packages repository
https://github.com/void-linux/void-packages/pull/31992#issuecomment-881886386
Comment:
But `log/supervise` is missing, no?
^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: unbound: add logging service
2021-07-16 0:38 [PR PATCH] unbound: add logging service AluminumTank
` (3 preceding siblings ...)
2021-07-17 11:53 ` leahneukirchen
@ 2021-07-17 13:54 ` AluminumTank
2021-07-17 13:56 ` AluminumTank
` (7 subsequent siblings)
12 siblings, 0 replies; 14+ messages in thread
From: AluminumTank @ 2021-07-17 13:54 UTC (permalink / raw)
To: ml
[-- Attachment #1: Type: text/plain, Size: 253 bytes --]
New comment by AluminumTank on void-packages repository
https://github.com/void-linux/void-packages/pull/31992#issuecomment-881903069
Comment:
That's automatically created by `sv` at runtime. I don't think it's supposed to be included in the package.
^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: unbound: add logging service
2021-07-16 0:38 [PR PATCH] unbound: add logging service AluminumTank
` (4 preceding siblings ...)
2021-07-17 13:54 ` AluminumTank
@ 2021-07-17 13:56 ` AluminumTank
2021-07-17 19:14 ` Duncaen
` (6 subsequent siblings)
12 siblings, 0 replies; 14+ messages in thread
From: AluminumTank @ 2021-07-17 13:56 UTC (permalink / raw)
To: ml
[-- Attachment #1: Type: text/plain, Size: 355 bytes --]
New comment by AluminumTank on void-packages repository
https://github.com/void-linux/void-packages/pull/31992#issuecomment-881903376
Comment:
If you're trying to install and run this on your machine with unbound already installed and service enabled, you'll have to manually enable the log service, or just unlink and relink to `/var/service/unbound`
^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: unbound: add logging service
2021-07-16 0:38 [PR PATCH] unbound: add logging service AluminumTank
` (5 preceding siblings ...)
2021-07-17 13:56 ` AluminumTank
@ 2021-07-17 19:14 ` Duncaen
2021-07-17 19:15 ` Duncaen
` (5 subsequent siblings)
12 siblings, 0 replies; 14+ messages in thread
From: Duncaen @ 2021-07-17 19:14 UTC (permalink / raw)
To: ml
[-- Attachment #1: Type: text/plain, Size: 567 bytes --]
New comment by Duncaen on void-packages repository
https://github.com/void-linux/void-packages/pull/31992#issuecomment-881946914
Comment:
> That's automatically created by sv at runtime. I don't think it's supposed to be included in the package.
No `runsv` creates the directory, not the symlink. It would just create a directory in the service directory if the symlink is missing.
`vsv` in xbps-src should create both supervise symlinks, but for some reason it does not do in the CI run.
https://github.com/void-linux/void-packages/runs/3082099002#step:9:66
^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: unbound: add logging service
2021-07-16 0:38 [PR PATCH] unbound: add logging service AluminumTank
` (6 preceding siblings ...)
2021-07-17 19:14 ` Duncaen
@ 2021-07-17 19:15 ` Duncaen
2021-07-20 18:59 ` ericonr
` (4 subsequent siblings)
12 siblings, 0 replies; 14+ messages in thread
From: Duncaen @ 2021-07-17 19:15 UTC (permalink / raw)
To: ml
[-- Attachment #1: Type: text/plain, Size: 567 bytes --]
New comment by Duncaen on void-packages repository
https://github.com/void-linux/void-packages/pull/31992#issuecomment-881946914
Comment:
> That's automatically created by sv at runtime. I don't think it's supposed to be included in the package.
No `runsv` creates the directory, not the symlink. It would just create a directory in the service directory if the symlink is missing.
`vsv` in xbps-src should create both supervise symlinks, but for some reason it does not do in the CI run.
https://github.com/void-linux/void-packages/runs/3082099002#step:9:67
^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: unbound: add logging service
2021-07-16 0:38 [PR PATCH] unbound: add logging service AluminumTank
` (7 preceding siblings ...)
2021-07-17 19:15 ` Duncaen
@ 2021-07-20 18:59 ` ericonr
2021-07-20 19:15 ` AluminumTank
` (3 subsequent siblings)
12 siblings, 0 replies; 14+ messages in thread
From: ericonr @ 2021-07-20 18:59 UTC (permalink / raw)
To: ml
[-- Attachment #1: Type: text/plain, Size: 257 bytes --]
New comment by ericonr on void-packages repository
https://github.com/void-linux/void-packages/pull/31992#issuecomment-883621997
Comment:
So, is this actually necessary? I get unbound information just fine in `svlogtail` output without a `log` service...
^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: unbound: add logging service
2021-07-16 0:38 [PR PATCH] unbound: add logging service AluminumTank
` (8 preceding siblings ...)
2021-07-20 18:59 ` ericonr
@ 2021-07-20 19:15 ` AluminumTank
2021-07-20 19:15 ` [PR PATCH] [Closed]: " AluminumTank
` (2 subsequent siblings)
12 siblings, 0 replies; 14+ messages in thread
From: AluminumTank @ 2021-07-20 19:15 UTC (permalink / raw)
To: ml
[-- Attachment #1: Type: text/plain, Size: 685 bytes --]
New comment by AluminumTank on void-packages repository
https://github.com/void-linux/void-packages/pull/31992#issuecomment-883632182
Comment:
Huh, you're right. I think I got confused when I was trying to enable logging within unbound because I copied a config from somewhere which didn't have the `use-syslog: true` directive. I'll close then, thanks for the tip.
As an aside, the issue about `log/supervise` missing may be related to `log/run` being a symlink to `/usr/bin/vlogger` since I checked several other services in void-packages using this symlink (`bluez`, dhcp`, `caddy`) and none of them have `log/supervise` either. Not sure if this is intended behavior or not...
^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: [PR PATCH] [Closed]: unbound: add logging service
2021-07-16 0:38 [PR PATCH] unbound: add logging service AluminumTank
` (9 preceding siblings ...)
2021-07-20 19:15 ` AluminumTank
@ 2021-07-20 19:15 ` AluminumTank
2021-07-20 19:16 ` AluminumTank
2021-07-20 19:27 ` ericonr
12 siblings, 0 replies; 14+ messages in thread
From: AluminumTank @ 2021-07-20 19:15 UTC (permalink / raw)
To: ml
[-- Attachment #1: Type: text/plain, Size: 1439 bytes --]
There's a closed pull request on the void-packages repository
unbound: add logging service
https://github.com/void-linux/void-packages/pull/31992
Description:
<!-- Mark items with [x] where applicable -->
#### General
- [ ] This is a new package and it conforms to the [quality requirements](https://github.com/void-linux/void-packages/blob/master/Manual.md#quality-requirements)
#### Have the results of the proposed changes been tested?
- [x] I use the packages affected by the proposed changes on a regular basis and confirm this PR works for me
- [ ] I generally don't use the affected packages but briefly tested this PR
<!--
If GitHub CI cannot be used to validate the build result (for example, if the
build is likely to take several hours), make sure to
[skip CI](https://github.com/void-linux/void-packages/blob/master/CONTRIBUTING.md#continuous-integration).
When skipping CI, uncomment and fill out the following section.
Note: for builds that are likely to complete in less than 2 hours, it is not
acceptable to skip CI.
-->
<!--
#### Does it build and run successfully?
(Please choose at least one native build and, if supported, at least one cross build. More are better.)
- [ ] I built this PR locally for my native architecture, (ARCH-LIBC)
- [ ] I built this PR locally for these architectures (if supported. mark crossbuilds):
- [ ] aarch64-musl
- [ ] armv7l
- [ ] armv6l-musl
-->
^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: unbound: add logging service
2021-07-16 0:38 [PR PATCH] unbound: add logging service AluminumTank
` (10 preceding siblings ...)
2021-07-20 19:15 ` [PR PATCH] [Closed]: " AluminumTank
@ 2021-07-20 19:16 ` AluminumTank
2021-07-20 19:27 ` ericonr
12 siblings, 0 replies; 14+ messages in thread
From: AluminumTank @ 2021-07-20 19:16 UTC (permalink / raw)
To: ml
[-- Attachment #1: Type: text/plain, Size: 686 bytes --]
New comment by AluminumTank on void-packages repository
https://github.com/void-linux/void-packages/pull/31992#issuecomment-883632182
Comment:
Huh, you're right. I think I got confused when I was trying to enable logging within unbound because I copied a config from somewhere which didn't have the `use-syslog: true` directive. I'll close then, thanks for the tip.
As an aside, the issue about `log/supervise` missing may be related to `log/run` being a symlink to `/usr/bin/vlogger` since I checked several other services in void-packages using this symlink (`bluez`, `dhcp`, `caddy`) and none of them have `log/supervise` either. Not sure if this is intended behavior or not...
^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: unbound: add logging service
2021-07-16 0:38 [PR PATCH] unbound: add logging service AluminumTank
` (11 preceding siblings ...)
2021-07-20 19:16 ` AluminumTank
@ 2021-07-20 19:27 ` ericonr
12 siblings, 0 replies; 14+ messages in thread
From: ericonr @ 2021-07-20 19:27 UTC (permalink / raw)
To: ml
[-- Attachment #1: Type: text/plain, Size: 271 bytes --]
New comment by ericonr on void-packages repository
https://github.com/void-linux/void-packages/pull/31992#issuecomment-883638589
Comment:
> Not sure if this is intended behavior or not...
I don't think so. Can you open an issue so we can remember to try and fix it?
^ permalink raw reply [flat|nested] 14+ messages in thread