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 lip-computing/dataverse to prep.rst #8965

Merged
merged 1 commit into from
Sep 8, 2022
Merged

Conversation

zbenta
Copy link
Contributor

@zbenta zbenta commented Sep 8, 2022

Added LIP's repo with resources for an advanced installation as proposed by Philip Durbin.

What this PR does / why we need it:
Added some new installation method to the advanced installation documentation.

Which issue(s) this PR closes:
The unavalabilty of a recipe for an advanced installation with all the resources needed to deploy a full functioning dataverse service with a fault tolerant architecture

Closes #8963

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:

Added LIP's repo with resources for an advanced installation as proposed by Philip Durbin.
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.

https://dataverse-guide--8965.org.readthedocs.build/en/8965/installation/prep.html#advanced-installation looks good!

I added #8963 to the "closes" line in the PR description (so that the issue get closed when we merge the PR).

Heads up to @donsizemore @poikilotherm @4tikhonov and @lwo that @zbenta agreed that for a future PR, we might want to give each installation option more room on the page than just a link. A paragraph perhaps. Or maybe even move to a different page. For now we just wanted to get the link in there.

@pdurbin pdurbin changed the title Update prep.rst add lip-computing/dataverse to prep.rst Sep 8, 2022
@4tikhonov
Copy link
Contributor

Hi @pdurbin, I can suggest to put them in the order when the advanced installation was created. So @zbenta should place it as last one in the list.

@zbenta
Copy link
Contributor Author

zbenta commented Sep 8, 2022

Hi @pdurbin, I can suggest to put them in the order when the advanced installation was created. So @zbenta should place it as last one in the list.

I was going to do so @4tikhonov , but @pdurbin Suggested I added it on the second position since the puppet version was very old, but I can change it if you guys prefer.

@4tikhonov
Copy link
Contributor

@zbenta, right, so we also need indication if the module is up-to-date and maintained, and who is maintainer. @pdurbin ?

@pdurbin
Copy link
Member

pdurbin commented Sep 8, 2022

I like the current order, which puts the older, less-maintained options at the bottom.

@4tikhonov certainly we could get more organized, list maintainers, etc. Good ideas for phase 2! 😄 If someone wants to go ahead and create an issue for future improvements, please go ahead (and drop a link in here please). We can pick up the conversation there.

This is just a one line change so I'm going to merge it. Thanks, all! 🚀


Update: Now that the PR is merged, you can see the change at http://preview.guides.gdcc.io/en/develop/installation/prep.html

@pdurbin pdurbin merged commit 0c16ba2 into IQSS:develop Sep 8, 2022
@pdurbin pdurbin added this to the 5.12 milestone Sep 8, 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.

add Dataverse Fault Tolerant Architecture link under Advanced Installation
3 participants