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

Update manual_install_guide_hpe_3par_plg_open_kub #451

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

ajrich2
Copy link

@ajrich2 ajrich2 commented Jan 16, 2019

Update manual_install_guide_hpe_3par_plugin_with_openshift_kubernetes.md (Minor changes for 3.0, etcd, and port typo.)

Update manual_install_guide_hpe_3par_plugin_with_openshift_kubernetes.md (Minor changes for 3.0, etcd,  and port typo.)
@@ -70,16 +70,16 @@ $ export HostIP="<Master node IP>"
>**NOTE:** This etcd instance is separate from the **etcd** deployed by Kubernetes/OpenShift and is **required** for managing the **HPE 3PAR Volume Plug-in for Docker**. We need to modify the default ports (**2379, 4001, 2380**) of the **new etcd** instance to prevent conflicts. This allows two instances of **etcd** to safely run in the environment.`

```yaml
sudo docker run -d -v /usr/share/ca-certificates/:/etc/ssl/certs -p 40010:40010 \
-p 23800:23800 -p 23790:23790 \
sudo docker run -d --restart unless-stopped -v /usr/share/ca-certificates/:/etc/ssl/certs -p 4001:4001 \
Copy link
Collaborator

Choose a reason for hiding this comment

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

Please do not change the ports in the md file as we are using different ports to avoid conflicts with k8s internal etcd.

polling_interval 10
max_fds 8192
}

devices
{
devices{
Copy link
Collaborator

Choose a reason for hiding this comment

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

Please put "device {" also on the same line, the way it is for defaults and devices.

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.

2 participants