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.

Sign up for GitHub

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

Jump to bottom

Remove references to postgresql-evr #3167

Draft
wants to merge 5 commits into
base: master
Choose a base branch
from
+47 −136
Draft

Remove references to postgresql-evr #3167

Show file tree
Hide file tree
Changes from 4 commits
Commits
Show all changes
5 commits
Select commit Hold shift + click to select a range
3a439ca
Remove references to postgresql-evr
ianballou Jul 25, 2024
80d52a6
Remove references to Foreman repositories for external DBs
ianballou Jul 31, 2024
1149379
Add release note for new remote DB possibilities
ianballou Aug 5, 2024
fd58d72
Remove excess parts of external DB host setup
ianballou Aug 5, 2024
987cc45
Remove prepping a host for ext DBs guide + introduce evr permissions …
ianballou Aug 7, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
4 changes: 2 additions & 2 deletions guides/common/modules/proc_installing-postgresql.adoc
View file
Open in desktop
ekohl marked this conversation as resolved.
Show resolved Hide resolved
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,7 @@
= Installing PostgreSQL

You can install only the same version of PostgreSQL that is installed with the `{foreman-installer}` tool during an internal database installation.
{Project} supports PostgreSQL version 12.
{Project} supports PostgreSQL version 13.

include::snip_firewalld.adoc[]

Expand All @@ -12,7 +12,7 @@ include::snip_firewalld.adoc[]
+
----
ifdef::katello,satellite,orcharhino[]
# dnf install postgresql-server postgresql-evr postgresql-contrib
# dnf install postgresql-server postgresql-contrib
endif::[]
ifndef::katello,satellite,orcharhino[]
# dnf install postgresql-server
Expand Down
49 changes: 4 additions & 45 deletions guides/common/modules/proc_preparing-a-host-for-external-databases.adoc
View file
Open in desktop
ekohl marked this conversation as resolved.
Show resolved Hide resolved
Copy link
Member

Choose a reason for hiding this comment

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

I also think you can drop lines 6-9 because now a base RHEL subscription gives you access to the right repositories. Though with service level agreement I'm a bit uncertain.

Do we still need procedure? After removing the specific Satellite repositories, isn't it sufficient to say you need a RHEL 8 or 9 system with an attached subscription? Essentially, drop the whole chapter Preparing a host for external databases and make them prerequisites in the Installing PostgreSQL procedure

@asteflova @apinnick what do you think?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I've ripped out so much of this page that I think I could just include this info as prerequisites.

Copy link
Contributor

Choose a reason for hiding this comment

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

Sounds good to me from the docs perspective. Looking at the procedure steps, it looks like you could save a lot of space by shrinking it into a prerequisite bullet without losing much value. More importantly (to me), anything that's a prerequisite should be labeled as "prerequisite", not "preparing" or anything like that. We are supposed to apply the same wording and terminology consistently (this helps make the content predictable).

Original file line number Diff line number Diff line change
Expand Up @@ -10,13 +10,16 @@ endif::[]

.Prerequisites
* The prepared host must meet {Project}'s {InstallingServerDocURL}storage-requirements_{project-context}[Storage Requirements].
ifndef::satellite[]
* Ensure the host has base operating system repositories enabled.
endif::[]
ifdef::satellite[]
* You must attach a {Project} subscription to your server.
For more information about subscription, see {InstallingServerDocURL}attaching-infrastructure-subscription_{project-context}[Attaching the {Project} Infrastructure Subscription] in _{InstallingServerDocTitle}_.
endif::[]

ifdef::satellite[]
.Procedure
ifndef::orcharhino[]
Select the operating system and version you are installing external database on:

* xref:#repositories-externaldb-el-9[{EL} 9]
Expand All @@ -25,9 +28,6 @@ Select the operating system and version you are installing external database on:
[id="repositories-externaldb-el-9"]
== {EL} 9

:distribution-major-version: 9

ifdef::satellite[]
. Disable all repositories:
+
[options="nowrap"]
Expand All @@ -39,31 +39,15 @@ ifdef::satellite[]
[options="nowrap" subs="+quotes,attributes"]
----
# subscription-manager repos \
--enable={RepoRHEL9ServerSatelliteServerProjectVersion} \
--enable={RepoRHEL9ServerSatelliteMaintenanceProjectVersion} \
--enable={RepoRHEL9BaseOS} \
--enable={RepoRHEL9AppStream}
----
endif::[]

ifdef::katello[]
. Install the `katello-repos-latest.rpm` package
+
[options="nowrap" subs="+quotes,attributes"]
----
# {client-package-install-el8} https://yum.theforeman.org/releases/{ProjectVersion}/el{distribution-major-version}/x86_64/foreman-release.rpm \
https://yum.theforeman.org/katello/{KatelloVersion}/katello/el{distribution-major-version}/x86_64/katello-repos-latest.rpm
----
endif::[]

include::snip_verification-enabled-repolist.adoc[]

[id="repositories-externaldb-el-8"]
== {EL} 8

:distribution-major-version: 8

ifdef::satellite[]
. Disable all repositories:
+
[options="nowrap"]
Expand All @@ -75,34 +59,9 @@ ifdef::satellite[]
[options="nowrap" subs="+quotes,attributes"]
----
# subscription-manager repos \
--enable={RepoRHEL8ServerSatelliteServerProjectVersion} \
--enable={RepoRHEL8ServerSatelliteMaintenanceProjectVersion} \
--enable={RepoRHEL8BaseOS} \
--enable={RepoRHEL8AppStream}
----
endif::[]

ifdef::katello[]
. Install the `katello-repos-latest.rpm` package
+
[options="nowrap" subs="+quotes,attributes"]
----
# {client-package-install-el8} https://yum.theforeman.org/releases/{ProjectVersion}/el{distribution-major-version}/x86_64/foreman-release.rpm \
https://yum.theforeman.org/katello/{KatelloVersion}/katello/el{distribution-major-version}/x86_64/katello-repos-latest.rpm
----
endif::[]

. Enable the following module:
+
[options="nowrap" subs="+quotes,attributes"]
----
# dnf module enable {dnf-module}
----
+
[NOTE]
====
include::snip_dnf-module-enable-note.adoc[]
====

include::snip_verification-enabled-repolist.adoc[]
endif::[]
Expand Down
6 changes: 5 additions & 1 deletion guides/doc-Release_Notes/topics/katello.adoc
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -4,7 +4,11 @@
[id="katello-headline-features"]
== Headline Features

There are no highlights with Katello {KatelloVersion}.
=== postgresql-evr extension no longer required

Installation of the Katello database on remote systems where root access is not available is now possible.
Only a basic PostgreSQL installation is required, which should enable installs on systems like Amazon RDS or Azure Database for PostgreSQL.
Copy link
Member

Choose a reason for hiding this comment

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

This is out of scope, but we have a guide https://docs.theforeman.org/nightly/Deploying_Project_on_AWS/index-katello.html and using Amazon RDS would be a good thing to document, after it's been verified.

The contents of postgresql-evr have been moved into Katello itself as a migration, which enables this new capability.

[id="katello-upgrade-warnings"]
== Upgrade Warnings
Expand Down