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

Update developer docs to work on Fedora 40+ #10385

Merged
merged 1 commit into from
Nov 25, 2024

Conversation

ekohl
Copy link
Member

@ekohl ekohl commented Nov 21, 2024

On Fedora 40 bundler installs by default to vendor/bundle and this ignores the directory. While installing it was also noticed that postgresql-server is the real server package and that the server needs to be running. For simplicity it's configured to always start on boot.

cc @ColeHiggins2

On Fedora 40 bundler installs by default to vendor/bundle and this
ignores the directory. While installing it was also noticed that
postgresql-server is the real server package and that the server needs
to be running. For simplicity it's configured to always start on boot.
@ekohl
Copy link
Member Author

ekohl commented Nov 21, 2024

We also noticed that ovirt-engine-sdk failed to compile so we disabled that group. That probably will be an issue on EL 10 as well.

Comment on lines -58 to 61
sudo dnf install postgresql
sudo dnf install postgresql-server
sudo postgresql-setup --initdb
sudo systemctl enable --now postgresql
sudo -u postgres createuser --createdb $USER
Copy link
Contributor

Choose a reason for hiding this comment

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

Thinking out loud:
Since we're considering a move to containers, should we suggest something container-based instead? Something like

podman run -e POSTGRES_USER=$USER -e POSTGRES_PASSWORD=postgres -e POSTGRES_DB=foreman -p 5432:5432 -d --name foreman-db postgres:16-alpine
sudo dnf install postgresql # For pg_dump

Copy link
Member Author

Choose a reason for hiding this comment

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

Today none of this document mentions containers so I'd like to separate those changes.

Copy link
Contributor

@adamruzicka adamruzicka left a comment

Choose a reason for hiding this comment

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

Nothing to add then

@ekohl ekohl merged commit 538b5a1 into theforeman:develop Nov 25, 2024
51 of 52 checks passed
@ekohl ekohl deleted the better-developer-docs branch November 25, 2024 14:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants