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

Fixes #22689: silent rudder-upgrade failure / plugins were not upgraded when upgrading from 7.2.5 to 7.3.1~nightly #2723

Conversation

peckpeck
Copy link
Member

then
echo "INFO: Existing database detected, skipping database creation"
CREATE_DB="false"
fi
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

you are msising a fi

@peckpeck
Copy link
Member Author

PR updated with a new commit

@Normation-Quality-Assistant
Copy link
Contributor

OK, squash merging this PR

…ed when upgrading from 7.2.5 to 7.3.1~nightly
@Normation-Quality-Assistant Normation-Quality-Assistant force-pushed the bug_22689/silent_rudder_upgrade_failure_plugins_were_not_upgraded_when_upgrading_from_7_2_5_to_7_3_1_nightly branch from 08b4fae to 1956312 Compare May 5, 2023 06:42
@Normation-Quality-Assistant Normation-Quality-Assistant merged commit 1956312 into Normation:branches/rudder/7.2 May 5, 2023
CREATE_DB="false"
else
USER=$(grep rudder.jdbc.username /opt/rudder/etc/rudder-web.properties)
CHK_PG_USER=$(su postgres -c "psql -t -c \"select count(1) from pg_user where usename = '${USERNAME}'\"")
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

USERNAME variable is not created, it is USER. And the value of USER variable is wrong the command gives

grep rudder.jdbc.username /opt/rudder/etc/rudder-web.properties
rudder.jdbc.username=rudder

on my rudder server

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

postgresql needs to be started

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