Skip to content

#8619 installation prerequisites documentation fixes#8620

Merged
kcondon merged 3 commits intoIQSS:developfrom
j-n-c:8619-install-prerequisites-doc-fixes
May 2, 2022
Merged

#8619 installation prerequisites documentation fixes#8620
kcondon merged 3 commits intoIQSS:developfrom
j-n-c:8619-install-prerequisites-doc-fixes

Conversation

@j-n-c
Copy link
Contributor

@j-n-c j-n-c commented Apr 19, 2022

What this PR does / why we need it:

  • Downloading Country GeoIP databases from Maxmind now requires having an account
  • sudo yum makecache fast does not work on RHEL8

Which issue(s) this PR closes:

Closes #8619

Special notes for your reviewer:

Suggestions on how to test this:

Does this PR introduce a user interface change? If mockups are available, please link/include them here:

Is there a release notes update needed for this change?:

Additional documentation:

Copy link
Member

@pdurbin pdurbin left a comment

Choose a reason for hiding this comment

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

There are merge conflicts, probably because PR #8391 tried to address some of this MaxMind/Geolite stuff (#6629).

@j-n-c if you're happy with the MaxMind/Geolite changes above, perhaps you could limit this pull request to just the "makecache" changes. Whatever makes sense. Thanks!

@j-n-c
Copy link
Contributor Author

j-n-c commented Apr 22, 2022

@pdurbin limited the changes to the PostgreSQL install with EL8 derivative
Found some typos in #6629. Left fix proposal to @qqmyers in comments section

Copy link
Contributor

@scolapasta scolapasta left a comment

Choose a reason for hiding this comment

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

checked with @donsizemore and got the 👍

Copy link
Member

@pdurbin pdurbin left a comment

Choose a reason for hiding this comment

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

The MaxMind/Geolite changes were removed. Since @donsizemore is happy with the makecache stuff (I'm not familiar with it), I'm approving this.

@kcondon kcondon self-assigned this May 2, 2022
@kcondon kcondon merged commit 50f093b into IQSS:develop May 2, 2022
@pdurbin pdurbin added this to the 5.11 milestone May 4, 2022
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

Successfully merging this pull request may close these issues.

Documentation fixes: installation/prerequisites.html

4 participants