Skip to content
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

remove unnecessary script part from resources/ext/ezbake.conf #4011

Open
Endesapt opened this issue Oct 12, 2024 · 0 comments
Open

remove unnecessary script part from resources/ext/ezbake.conf #4011

Endesapt opened this issue Oct 12, 2024 · 0 comments

Comments

@Endesapt
Copy link

Endesapt commented Oct 12, 2024

I followed instructions in Installing PuppetDB from source for my ubi8 image and install.sh script was giving errors. I started investigating, and it showed that this part of script
(maybe_dead=''
pid="$(pgrep -f "puppetdb.* -m puppetlabs.puppetdb.main")"
if test $? -eq 0; then
kill "$pid" 2>/dev/null
for i in {1..75}; do # Wait up to ~15s
if kill -0 "$pid" 2>/dev/null; then
maybe_dead=1
break
fi
sleep 0.2
done
if test -z "$maybe_dead"; then
echo 'Unable to kill puppetdb server cleanly; sending KILL' 1>&2
kill -9 "$pid"
fi
/sbin/service puppetdb start > /dev/null 2>&1
fi)
is what cause errors.

As i see from the comments above this part
ppetdb changed its main namespace in 6.3 and the current rpm
upgrade strategy fails to restart the server correctly because
t ends up using the templated stop from the new package which
refers to the wrong namespace for the older pdb that's still
running. Kill the old pdb if found and start the new one.
These changes can be removed in PuppetDB version 8.

And this part of script can be easily deleted without any consequencies

Expected Behavior

after compiling puppetdb-*/pkg folder with lein with-profile ezbake ezbake stage ./install.sh file should work without errors

Steps to Reproduce

Steps to reproduce the behavior:

  1. On RHEL8 or ubi8 try folllowing install from source guide
  2. ./install.sh will give something like

Environment

  • Version [tag 8.7.0]
  • Platform [ubi8]

Additional Context

Add any other context about the problem here.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant