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

chore(docs): instruction guide on installing WordPress using OpenEBS NFS Volumes #105

Merged
merged 3 commits into from
Sep 13, 2021

Conversation

mynktl
Copy link
Contributor

@mynktl mynktl commented Sep 13, 2021

Why is this PR required? What issue does it fix?:

What this PR does?:
This PR adds a document on how to install WordPress application using OpenEBS NFS Volumes.

Does this PR require any upgrade changes?:

If the changes in this PR are manually verified, list down the scenarios covered::

Any additional information for your reviewer? :
Mention if this PR is part of any design or a continuation of previous PRs

Checklist:

  • Fixes #
  • PR Title follows the convention of <type>(<scope>): <subject>
  • Has the change log section been updated?
  • Commit has unit tests
  • Commit has integration tests
  • (Optional) Does this PR change require updating NFS-Provisioner Chart? If yes, mention the Helm Chart PR #
  • (Optional) Are upgrade changes included in this PR? If not, mention the issue/PR to track:
  • (Optional) If documentation changes are required, which issue on https://github.com/openebs/openebs-docs is used to track them:

@mynktl mynktl requested review from kmova and mittachaitu September 13, 2021 08:17
@codecov-commenter
Copy link

codecov-commenter commented Sep 13, 2021

Codecov Report

Merging #105 (f9498f0) into develop (e5b789c) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff            @@
##           develop     #105   +/-   ##
========================================
  Coverage    46.97%   46.97%           
========================================
  Files           29       29           
  Lines         2378     2378           
========================================
  Hits          1117     1117           
  Misses        1182     1182           
  Partials        79       79           

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update e5b789c...f9498f0. Read the comment docs.

Copy link
Contributor

@mittachaitu mittachaitu left a comment

Choose a reason for hiding this comment

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

LGTM

--set volumePermissions.enabled=true \
--set autoscaling.enabled=true \
--set autoscaling.minReplicas=1 \
--set autoscaling.maxReplicas=6 \
Copy link
Contributor

Choose a reason for hiding this comment

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

nit: As per scale command below it can be 1

Suggested change
--set autoscaling.maxReplicas=6 \
--set autoscaling.maxReplicas=1 \

Copy link
Collaborator

Choose a reason for hiding this comment

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

Better to keep maxReplicas's as 2 here and then 3 in the scaled command below.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

ok

Copy link
Contributor Author

Choose a reason for hiding this comment

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

removed scale command as autoscaling is enabled in helm install command

@mittachaitu mittachaitu merged commit 23d4176 into openebs-archive:develop Sep 13, 2021
@mynktl mynktl deleted the example-wordpress branch September 13, 2021 14:46
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.

4 participants