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

Add missing linux installation pages #672

Merged
merged 12 commits into from
Apr 18, 2024
Merged

Conversation

antejavor
Copy link
Contributor

@antejavor antejavor commented Apr 9, 2024

Description

Add installation process for CentOS
Add installation process for Fedora
Add installation process for Rocky
Add installation process for Redhat
Add installation process for Amazon Linux

Closes #667
Closes #668
Closes #669

  • Check all content with Grammarly
  • Perform a self-review of my code
  • Make corresponding changes to the rest of the documentation (consult with the DX team)
  • The build passes locally
  • My changes generate no new warnings or errors
  • Add a corresponding label
  • If release-related, add a product and version label
  • If release-related, add release note on product PR

Copy link

vercel bot commented Apr 9, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
documentation ✅ Ready (Inspect) Visit Preview 💬 Add feedback Apr 18, 2024 2:07pm

@antejavor antejavor changed the title Init the CentoOS page. Add missing linux installation pages. Apr 9, 2024
@antejavor antejavor changed the title Add missing linux installation pages. Add missing linux installation pages Apr 9, 2024
@antejavor antejavor added the status: ready PR is ready for review label Apr 10, 2024
@antejavor antejavor self-assigned this Apr 10, 2024
@antejavor antejavor marked this pull request as ready for review April 10, 2024 13:53
pages/getting-started.mdx Outdated Show resolved Hide resolved
pages/getting-started/install-memgraph.mdx Outdated Show resolved Hide resolved
pages/getting-started/install-memgraph/amazon-linux.mdx Outdated Show resolved Hide resolved
Comment on lines +86 to +116
When you start a Memgraph instance, you may see the following list of notes in
your terminal:

```console
You are running Memgraph v1.4.0-community

NOTE: Please install networkx to be able to use graph_analyzer module. Using Python:
3.8.2 (default, Jul 16 2020, 14:00:26)
[GCC 9.3.0]

NOTE: Please install networkx to be able to use Memgraph NetworkX wrappers. Using Python:
3.8.2 (default, Jul 16 2020, 14:00:26)
[GCC 9.3.0]

NOTE: Please install networkx, numpy, scipy to be able to use proxied NetworkX algorithms. E.g., CALL nxalg.pagerank(...).
Using Python:
3.8.2 (default, Jul 16 2020, 14:00:26)
[GCC 9.3.0]

NOTE: Please install networkx to be able to use wcc module.
Using Python:
3.8.2 (default, Jul 16 2020, 14:00:26)
[GCC 9.3.0]
```

If you wish to work with built-in NetworkX modules in Memgraph, you need to
install the following Python libraries:
* [NumPy](https://numpy.org/)
* [SciPy](https://www.scipy.org/)
* [NetworkX](https://networkx.org/)

Copy link
Contributor

Choose a reason for hiding this comment

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

Is this still true? I don't have Linux, so just checking here. I guess it's a bit different in Docker

Copy link
Contributor Author

@antejavor antejavor Apr 16, 2024

Choose a reason for hiding this comment

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

Yes, this is true. Still present issue.

Copy link
Contributor

@kgolubic kgolubic left a comment

Choose a reason for hiding this comment

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

In new docs you have section that looks like:

## Installation guide

  <Steps>

  {<h3>Download Memgraph RPM Package</h3>}

Downloaded the latest Memgraph RPM Package from the [Memgraph Download
Hub](https://memgraph.com/download/).

You can also use [direct download
links](/getting-started/install-memgraph/direct-download-links) to get the
latest Memgraph packages. 


To me it makes no sense to have a section that talks about generic case (Download Memgraph RPM Package) and mentions Downloaded the latest Memgraph RPM Package from the [Memgraph Download Hub](https://memgraph.com/download/). when the page talks about specific Linux distribution.

And this part is mentioned, e.g. "For example for Amazon Linux 2:".

I think that it would make sense to remove this generic part about RPM and talk aonly about each distro in the individual documentation, e.g.

## Installation guide

  <Steps>

  {<h3>Download Memgraph for Amazon Linux</h3>}

Downloaded the latest Memgraph Package for Amazon Linux from the [Memgraph Download
Hub](https://memgraph.com/download/).

You can also use [direct download
links](/getting-started/install-memgraph/direct-download-links) to get the
latest Memgraph packages. 


@antejavor antejavor mentioned this pull request Apr 16, 2024
31 tasks
antejavor and others added 2 commits April 16, 2024 13:52
Co-authored-by: Katarina Supe <61758502+katarinasupe@users.noreply.github.com>
@antejavor
Copy link
Contributor Author

I have updated the PR.

Copy link
Contributor

@kgolubic kgolubic left a comment

Choose a reason for hiding this comment

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

Everything looks OK to me.

@antejavor antejavor merged commit c1eeafc into BP-deployment Apr 18, 2024
2 checks passed
@antejavor antejavor deleted the linux-installation-pages branch April 18, 2024 14:08
kgolubic added a commit that referenced this pull request May 15, 2024
* Updates to native install process  (#661)

Update the OS install commands and OS information.

* Refactor the configuration guide  (#691)

Refactoring of the configuration page.

* Add missing linux installation pages (#672)

Add Linux installation pages. 
---------

Co-authored-by: Katarina Supe <61758502+katarinasupe@users.noreply.github.com>
Co-authored-by: Kruno Golubic <46486712+kgolubic@users.noreply.github.com>

* Add context for config via file  (#707)

Add context for configuration files. 

Co-authored-by: Katarina Supe <61758502+katarinasupe@users.noreply.github.com>

---------

Co-authored-by: Katarina Supe <61758502+katarinasupe@users.noreply.github.com>

* Update config settings for Docker Compose (#709)

* Update config settings for Docker Compose

* update content

* Update pages/getting-started/install-memgraph/docker-compose.mdx

Co-authored-by: Katarina Supe <61758502+katarinasupe@users.noreply.github.com>

---------

Co-authored-by: Katarina Supe <61758502+katarinasupe@users.noreply.github.com>

* Update docker config

* Add db management placeholder

* update conf settings for docker compose

* docker management init (#716)

* docker management init

* update

* code fixes

* update

* Fix vercel build error

* Change naming

---------

Co-authored-by: kgolubic <kgolubic@gmail.com>

* Add new structure

* update structure

* Update data durability

* Update logs

* grammarly

* Update auth

* update db management

* update config

* update backup

* update clustering

* update query metadata

* Update monitoring

* update enterprise

* Fix typo

* Update image paths

* Add redirects

* Update links

* update

* update docker

* update HA

* update ssl

* update volumes

* update

* update replication

* Add Linux deployment page (#751)

Add linux page. 

Co-authored-by: Katarina Supe <61758502+katarinasupe@users.noreply.github.com>

---------

Co-authored-by: Katarina Supe <61758502+katarinasupe@users.noreply.github.com>

* update

* update restore on linux

* Fix build errors

---------

Co-authored-by: Katarina Supe <61758502+katarinasupe@users.noreply.github.com>
Co-authored-by: Kruno Golubic <46486712+kgolubic@users.noreply.github.com>
Co-authored-by: Matea Pesic <80577904+matea16@users.noreply.github.com>
Co-authored-by: katarinasupe <supe.katarina@gmail.com>
Co-authored-by: matea16 <mateapesic@hotmail.com>
Co-authored-by: kgolubic <kgolubic@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
status: ready PR is ready for review
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants