Skip to content
This repository has been archived by the owner on Nov 9, 2020. It is now read-only.

Document volume plugin options and allow config file options for photon driver. #889

Merged
merged 1 commit into from
Feb 9, 2017

Conversation

govint
Copy link
Contributor

@govint govint commented Feb 3, 2017

  1. Updated documentation of supported config file options.
  2. Modified main.go to use the config file for photon driver options.

Verified error handling with command line options for photon driver, will be testing config file based options as well.

## Configuring the Docker Volume Plugin
The docker volume plugin is designed to load run time options and values from a json configuration file (default /etc/docker-volume-vsphere.conf) on the host. The user can also provide a configuration file, via the "--config" option, specifying the full path of the file. The file contains the values for run time options used by the plugin. Options that are currently recognized include the below set. Options provided on the command line will override those in the configuration file.

# Options for the photon volume driver
Copy link
Contributor

Choose a reason for hiding this comment

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

This is not enough, either package a default config that can be renamed to /etc/docker-volume-vsphere.conf or include the sample JSON here at a minimum.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Added a sample configuration.

The docker volume plugin is designed to load run time options and values from a json configuration file (default /etc/docker-volume-vsphere.conf) on the host. The user can also provide a configuration file, via the "--config" option, specifying the full path of the file. The file contains the values for run time options used by the plugin. Options that are currently recognized include the below set. Options provided on the command line will override those in the configuration file.

# Options for the photon volume driver
Target - URL at which to contact the Photon Controller
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 API Endpoint? May be add "for e.g. http://...."

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Added format for target URL.


# Options for the photon volume driver
Target - URL at which to contact the Photon Controller
Project - project ID in Photon to which the docker host belongs
Copy link
Contributor

Choose a reason for hiding this comment

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

Can we add commands or link to Photon documentation on how to to get "Project" and "Host" details. Also add sample value?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Added description of the project and host IDs and link to photon CLI.

"MaxLogSizeMb": 100,
"LogPath": "/var/log/docker-volume-vsphere.log",
"LogLevel": "info"
"Target" : "http://<photon_controller_ip>:<target port>"
Copy link
Contributor

Choose a reason for hiding this comment

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

Add comment "Following options only apply to Photon Platform".

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Done.

@govint govint merged commit 8ad1c6d into master Feb 9, 2017
@govint govint deleted the fix-docs branch February 9, 2017 11:10
# for free to subscribe to this conversation on GitHub. Already have an account? #.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants