On 2023/11/08 20:12:53 +0100, Alexander Klimov <a...@kli.mov> wrote:
> > [...]
> > Are you running -stable by chance?  I'm wondering whether this change
> Yes, I apply all package updates ASAP btw.

I've replicated the issue on -STABLE too.  So it was introduced at least
with the 2.10 update, if not earlier.  Usually I don't stop/restart
opensearch-dashboards, but from now on I'll pay extra attention when
updating.

I've backported the fix to -stable, so in the next days
opensearch-dashboards-2.10.0p0 should be available with the fixed rc
script.

Thanks!

> [...]
> > --- pkg/opensearch_dashboards.rc    3 Mar 2023 18:36:04 -0000       1.2
> > +++ pkg/opensearch_dashboards.rc    8 Nov 2023 08:07:15 -0000
> Too bad packages may not change /etc and all the "service files" are located 
> there!

I'm not sure I'm understanding.  pkg has 'magic' to install these files
in /etc/rc.d, so this is actually fixing
/etc/rc.d/opensearch_dashboards.rc

(see the @rcscript entry in pkg/PLIST and pkg_create(1) if you're curious
to see what happens behind the scenes.)

> > @@ -6,7 +6,7 @@ daemon_execdir="${TRUEPREFIX}/opensearch
> >   
> >   . /etc/rc.d/rc.subr
> >   
> > -pexp="${TRUEPREFIX}/bin/node 
> > ${TRUEPREFIX}/opensearch-dashboards/bin/../src/cli/dist${daemon_flags:+ 
> > ${daemon_flags}}"
> > +pexp="${TRUEPREFIX}/bin/node 
> > ${TRUEPREFIX}/opensearch-dashboards/src/cli/dist${daemon_flags:+ 
> > ${daemon_flags}}"
> Indeed it helped changing mine to
> /usr/local/bin/node /usr/local/opensearch-dashboards/src/cli/dist
> as per ps aux.
> 
> Thanks for the hint!

Yes, in the meantime you can just edit the file by hand.  I'm not sure
if pkg_check or pkg_add -u will complain later though.

> >   rc_bg=YES
> >   rc_reload=NO
> >   
> > 


Reply via email to