DSEGOG-315 MongoDB 5.0 to 7.0 #118
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #118 +/- ##
==========================================
- Coverage 94.99% 94.91% -0.09%
==========================================
Files 50 50
Lines 2837 2871 +34
Branches 297 300 +3
==========================================
+ Hits 2695 2725 +30
- Misses 105 108 +3
- Partials 37 38 +1 ☔ View full report in Codecov by Sentry. |
There was a problem hiding this comment.
Will come back to this and test the commands work for me on my dev environment.
It doesn't matter because it's symlinked but if these commands are for a Rocky8 machine it will use dnf rather than yum right? Not a request to change just trying to remind myself which way round it is more than anything...
There was a problem hiding this comment.
Mostly just a bunch of commands that need sudo to run. Main point is that this won't work without the changes from #120 (I assume).
docs/mongodb.md
Outdated
|
|
||
| # Paste the following into /etc/yum.repos.d/mongodb-org-7.0.repo | ||
| # Edit the /8/ to /9/ if you're using Rocky 9 | ||
| [mongodb-org-7.0] | ||
| name=MongoDB Repository | ||
| baseurl=https://repo.mongodb.org/yum/redhat/8/mongodb-org/7.0/x86_64/ | ||
| gpgcheck=1 | ||
| enabled=1 | ||
| gpgkey=https://pgp.mongodb.com/server-7.0.asc |
There was a problem hiding this comment.
Since this isn't a bash command I think it would be better to make it a separate code block, e.g (formatting isn't great because of the nested backticks but hopefully it makes sense:
| # Paste the following into /etc/yum.repos.d/mongodb-org-7.0.repo | |
| # Edit the /8/ to /9/ if you're using Rocky 9 | |
| [mongodb-org-7.0] | |
| name=MongoDB Repository | |
| baseurl=https://repo.mongodb.org/yum/redhat/8/mongodb-org/7.0/x86_64/ | |
| gpgcheck=1 | |
| enabled=1 | |
| gpgkey=https://pgp.mongodb.com/server-7.0.asc | |
| Paste the following into /etc/yum.repos.d/mongodb-org-7.0.repo (edit the /8/ to /9/ if you're using Rocky 9): | |
| ```ini | |
| [mongodb-org-7.0] | |
| name=MongoDB Repository | |
| baseurl=https://repo.mongodb.org/yum/redhat/8/mongodb-org/7.0/x86_64/ | |
| gpgcheck=1 | |
| enabled=1 | |
| gpgkey=https://pgp.mongodb.com/server-7.0.asc |
There was a problem hiding this comment.
I've committed all the suggested changes you proposed, thanks for making them. I had to play around with the formatting on this suggestion and settled on the changes in f82776c. Can you take a look and make sure you're happy with how it looks please?
| yum install mongodb-org | ||
| # Start MongoDB, enter the shell and check the "Using MongoDB:" line lists a version 7.x.x | ||
| systemctl start mongod | ||
| mongosh |
There was a problem hiding this comment.
The ingest won't work because we need mongosh but we're still using mongo until #120 is merged. I'd consider this blocked until that's merged.
There was a problem hiding this comment.
Good point, let's leave this unresolved so I remember to not merge this until the other PR is merged.
Co-authored-by: patrick-austin <61705287+patrick-austin@users.noreply.github.com>
docs/mongodb.md
Outdated
| [mongodb-org-7.0] | ||
| name=MongoDB Repository | ||
| baseurl=https://repo.mongodb.org/yum/redhat/8/mongodb-org/7.0/x86_64/ | ||
| gpgcheck=1 | ||
| enabled=1 | ||
| gpgkey=https://pgp.mongodb.com/server-7.0.asc | ||
|
|
||
| ``` | ||
| ``` |
There was a problem hiding this comment.
| ``` | |
| ```bash |
Really minor, we can use the bash language support for this block like the first one (practically I think the only difference is it will gray out the comments). Appreciate this was probably missing from my suggested change because it struggles with nesting back ticks inside the suggestion backtick box.
There was a problem hiding this comment.
Good spot, I like to use them too but forgot about it in that case. Committed your suggestion
Co-authored-by: patrick-austin <61705287+patrick-austin@users.noreply.github.com>
This is a small PR which helps to move MongoDB 5.0 to 7.0. This changes the GitHub Actions to install MongoDB 7.0 and there's also a guide that allows developers to make this change on their development environments. This change is happening because Database Services are now hosting a database using MongoDB 7.0, so we should move to that version as developers for consistency.