Summary: | sys-block/tgt-1.0.88 does not respect tgtd_opts | ||
---|---|---|---|
Product: | Gentoo Linux | Reporter: | ixuz <xman00> |
Component: | Current packages | Assignee: | Matthew Thode ( prometheanfire ) <prometheanfire> |
Status: | RESOLVED FIXED | ||
Severity: | normal | CC: | ago, cluster, esigra, forza |
Priority: | Normal | ||
Version: | unspecified | ||
Hardware: | All | ||
OS: | Linux | ||
Whiteboard: | |||
Package list: | Runtime testing required: | --- |
Description
ixuz
2023-10-03 20:55:41 UTC
Please don't CC manually - a bunch of the people you CCed aren't even involved with tgt. Thanks for the report. I think we can fix this like so: # Default configuration fike : "${tgtd_conf:=/etc/tgt/targets.conf}" : "${tgtd_opts:=" "}" pidfile="/var/run/${RC_SVCNAME}.pid" command="/usr/sbin/tgtd" command_args_background="--pid-file ${pidfile} ${tgtd_opts}" extra_commands="forcedstop" extra_started_commands="forcedreload reload" @ixuz, I'm curious what extra options you are using? (In reply to Forza from comment #3) > @ixuz, I'm curious what extra options you are using? I need IP address/port binding with "--iscsi portal=". I wonder where/when /etc/conf.d/tgtd is parsed for tgtd_opts ? Your addition to my change works also, thank you. Thanks for your update. OpenRC always sources the /etc/conf.d/${RC_SVCNAME} file, so it is not required to include it manually. If you create a service /etc/init.d/foo, the /etc/conf.d/foo will be sourced automatically. That make sense. Thanks for clarification. I guess you already have seen my comment under your pull request at the Github upstream repo!? Maybe you could do the pull request which the author/maintainer requested? Would appreciate it. Before doing a new PR on Github, is there anything else we ought to change? We can opt to use the upstream version of this init script, instead of keeping a Gentoo custom version. In Alpine Linux we have two init scripts. https://git.alpinelinux.org/aports/tree/testing/scsi-tgt?h=master I don't know. For me it works as expected with this little change. Didn't even know that there is a difference between upstreams OpenRC script and this one from Gentoo. (In reply to ixuz from comment #8) > I don't know. For me it works as expected with this little change. > > Didn't even know that there is a difference between upstreams OpenRC script > and this one from Gentoo. Have a look and see if you are OK with the change. https://github.com/fujita/tgt/pull/60 LGTM, thank you. The fix has been merged upstream. (In reply to Forza from comment #11) > The fix has been merged upstream. Make a PR to pull it in? (In reply to Sam James from comment #12) > (In reply to Forza from comment #11) > > The fix has been merged upstream. > > Make a PR to pull it in? https://github.com/gentoo/gentoo/pull/33368 The bug has been closed via the following commit(s): https://gitweb.gentoo.org/repo/gentoo.git/commit/?id=db5fe2a0dd4dd1f6bb1761110922b645451a4cf8 commit db5fe2a0dd4dd1f6bb1761110922b645451a4cf8 Author: Forza <68693597+Forza-tng@users.noreply.github.com> AuthorDate: 2023-10-17 06:40:13 +0000 Commit: Sam James <sam@gentoo.org> CommitDate: 2023-10-17 09:17:56 +0000 sys-block/tgt: Update tgtd.initd-new Add missing `${tgtd_opts}` which was missing in previous commit. Closes: https://bugs.gentoo.org/915140 Signed-off-by: Forza <68693597+Forza-tng@users.noreply.github.com> Closes: https://github.com/gentoo/gentoo/pull/33368 Signed-off-by: Sam James <sam@gentoo.org> sys-block/tgt/files/tgtd.initd-new | 2 +- sys-block/tgt/{tgt-1.0.87-r1.ebuild => tgt-1.0.87-r2.ebuild} | 0 sys-block/tgt/{tgt-1.0.88.ebuild => tgt-1.0.88-r1.ebuild} | 0 3 files changed, 1 insertion(+), 1 deletion(-) Forza, now that these are upstream, could we switch the ebuild (in the next release) to use the upstream versions and not use the ones from files/? (In reply to ixuz from comment #8) > I don't know. For me it works as expected with this little change. > > Didn't even know that there is a difference between upstreams OpenRC script > and this one from Gentoo. Forza only added it upstream recently :) (In reply to Sam James from comment #15) > Forza, now that these are upstream, could we switch the ebuild (in the next > release) to use the upstream versions and not use the ones from files/? I don't see why not. But maybe @prometheanfire should should decide this as I am not a Gentoo maintainer (yet :). |