Skip to content

Commit

Permalink
Close embercsi#76: Allow enabling/disabling probe
Browse files Browse the repository at this point in the history 
Seems like there are a number of sidecars that don't properly support
probing, and expect probes to take less than 1 second, which isn't
usually the case.

We are now disabling the probe by default unless explicitly enabled on
the X_CSI_EMBER_CONFIG variable using the enable_probe key.
  • Loading branch information
@Akrog
Akrog committed Jan 29, 2019
1 parent 3bf78b8 commit dcc76e7
Show file tree
Hide file tree
Showing 4 changed files with 20 additions and 18 deletions.
28 changes: 14 additions & 14 deletions README.md
View file
Original file line number Diff line number Diff line change
Expand Up @@ -104,20 +104,20 @@ For RBD we'll also need a specific package:

The CSI driver is configured via environmental variables, any value that doesn't have a default is a required value.

| Name | Role | Description | Default | Example |
| -------------------------- | ---------- | ------------------------------------------------------------- | ------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------ | --------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------- |
| `CSI_ENDPOINT` | all | IP and port to bind the service | [::]:50051 | 192.168.1.22:50050 |
| `CSI_MODE` | all | Role the service should perform: controller, node, all | all | controller |
| `X_CSI_SPEC_VERSION` | all | CSI Spec version to run. Supported v0.2 and v1.0 | v0.2.0 | 0.2.0
| `X_CSI_STORAGE_NW_IP` | node | IP address in the Node used to connect to the storage | IP resolved from Node's fqdn | 192.168.1.22 |
| `X_CSI_NODE_ID` | node | ID used by this node to identify itself to the controller | Node's fqdn | csi_test_node |
| `X_CSI_PERSISTENCE_CONFIG` | all | Configuration of the `cinderlib` metadata persistence plugin. | {"storage": "crd", "namespace": "default"} | {"storage": "db", "connection": "mysql+pymysql://root:stackdb@192.168.1.1/cinder?charset=utf8"} |
| `X_CSI_EMBER_CONFIG` | all | Global `cinderlib` configuration | {'project_id': 'io.ember-csi', 'user_id': 'io.ember-csi', 'root_helper': 'sudo', 'request_multipath': false, 'plugin_name': 'io.ember-csi', 'file_locks_path': '/var/lib/ember-csi/locks'} | {"project_id":"k8s project","user_id":"csi driver","root_helper":"sudo",'plugin_name': 'io.ember-csi.external-ceph'} |
| `X_CSI_BACKEND_CONFIG` | controller | Driver configuration | | {"name": "rbd", "driver": "RBD", "rbd_user": "cinder", "rbd_pool": "volumes", "rbd_ceph_conf": "/etc/ceph/ceph.conf", "rbd_keyring_conf": "/etc/ceph/ceph.client.cinder.keyring"} |
| `X_CSI_DEFAULT_MOUNT_FS` | node | Default mount filesystem when missing in publish calls | ext4 | btrfs |
| `X_CSI_SYSTEM_FILES` | all | All required storage driver-specific files archived in tar, tar.gz or tar.bz2 format| | /path/to/etc-ceph.tar.gz |
| `X_CSI_DEBUG_MODE` | all | Debug mode (rpdb, pdb) to use. Disabled by default. | | rpdb |
| `X_CSI_ABORT_DUPLICATES` | all | If we want to abort or queue (default) duplicated requests. | false | true |
| Name | Role | Description | Default | Example |
| -------------------------- | ---------- | ------------------------------------------------------------- | ------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------- | --------------------------------------------------------------------------------------------------------------------------------------------------------------------------------- |
| `CSI_ENDPOINT` | all | IP and port to bind the service | [::]:50051 | 192.168.1.22:50050 |
| `CSI_MODE` | all | Role the service should perform: controller, node, all | all | controller |
| `X_CSI_SPEC_VERSION` | all | CSI Spec version to run. Supported v0.2 and v1.0 | v0.2.0 | 0.2.0 |
| `X_CSI_STORAGE_NW_IP` | node | IP address in the Node used to connect to the storage | IP resolved from Node's fqdn | 192.168.1.22 |
| `X_CSI_NODE_ID` | node | ID used by this node to identify itself to the controller | Node's fqdn | csi_test_node |
| `X_CSI_PERSISTENCE_CONFIG` | all | Configuration of the `cinderlib` metadata persistence plugin. | {"storage": "crd", "namespace": "default"} | {"storage": "db", "connection": "mysql+pymysql://root:stackdb@192.168.1.1/cinder?charset=utf8"} |
| `X_CSI_EMBER_CONFIG` | all | Global `Ember` and `cinderlib` configuration | {"project_id": "io.ember-csi", "user_id": "io.ember-csi", "root_helper": "sudo", "request_multipath": false, "plugin_name": "io.ember-csi", "file_locks_path": "/var/lib/ember-csi/locks", "name": "io.ember-csi", "grpc_workers": 30, "enable_probe": false} | {"project_id":"k8s project","user_id":"csi driver","root_helper":"sudo","plugin_name": "io.ember-csi.external-ceph"} |
| `X_CSI_BACKEND_CONFIG` | controller | Driver configuration | | {"name": "rbd", "driver": "RBD", "rbd_user": "cinder", "rbd_pool": "volumes", "rbd_ceph_conf": "/etc/ceph/ceph.conf", "rbd_keyring_conf": "/etc/ceph/ceph.client.cinder.keyring"} |
| `X_CSI_DEFAULT_MOUNT_FS` | node | Default mount filesystem when missing in publish calls | ext4 | btrfs |
| `X_CSI_SYSTEM_FILES` | all | All required storage driver-specific files archived in tar, tar.gz or tar.bz2 format| | /path/to/etc-ceph.tar.gz |
| `X_CSI_DEBUG_MODE` | all | Debug mode (rpdb, pdb) to use. Disabled by default. | | rpdb |
| `X_CSI_ABORT_DUPLICATES` | all | If we want to abort or queue (default) duplicated requests. | false | true |

The only role that has been tested at the moment is the default one, where Controller and Node servicer are executed in the same service (`CSI_MODE=all`), and other modes are expected to have issues at the moment.

Expand Down
2 changes: 1 addition & 1 deletion ember_csi/base.py
View file
Original file line number Diff line number Diff line change
Expand Up @@ -162,7 +162,7 @@ def GetPluginCapabilities(self, request, context):
def Probe(self, request, context):
# Proving may take a couple of seconds, and attacher sidecar prior to
# v0.4 will fail due to small timeout.
if self.csi_version < '1.0.0':
if not config.ENABLE_PROBE:
return self.PROBE_RESP

try:
Expand Down
6 changes: 4 additions & 2 deletions ember_csi/config.py
View file
Original file line number Diff line number Diff line change
Expand Up @@ -68,8 +68,8 @@ def _get_system_fs_types():
defaults.PERSISTENCE_CFG)
EMBER_CONFIG = _load_json_config('X_CSI_EMBER_CONFIG',
defaults.EMBER_CFG)
# REQUEST_MULTIPATH, WORKERS, and PLUGIN_NAME are set from EMBER_CONFIG on
# _set_defaults_ember_cfg
# REQUEST_MULTIPATH, WORKERS, PLUGIN_NAME, ENABLE_PROBE are set from
# EMBER_CONFIG on _set_defaults_ember_cfg
BACKEND_CONFIG = _load_json_config('X_CSI_BACKEND_CONFIG')
NODE_ID = os.environ.get('X_CSI_NODE_ID') or socket.getfqdn()
DEFAULT_MOUNT_FS = os.environ.get('X_CSI_DEFAULT_MOUNT_FS', defaults.MOUNT_FS)
Expand Down Expand Up @@ -167,6 +167,7 @@ def _set_defaults_ember_cfg():
global REQUEST_MULTIPATH
global WORKERS
global PLUGIN_NAME
global ENABLE_PROBE

# First set defaults for missing keys
for key, value in defaults.EMBER_CFG.items():
Expand All @@ -184,6 +185,7 @@ def _set_defaults_ember_cfg():
REQUEST_MULTIPATH = EMBER_CONFIG.pop('request_multipath')
WORKERS = EMBER_CONFIG.pop('grpc_workers')
PLUGIN_NAME = EMBER_CONFIG.pop('plugin_name')
ENABLE_PROBE = EMBER_CONFIG.pop('enable_probe')


def _create_default_dirs_files():
Expand Down
2 changes: 1 addition & 1 deletion ember_csi/defaults.py
View file
Original file line number Diff line number Diff line change
Expand Up @@ -27,7 +27,7 @@
'root_helper': ROOT_HELPER,
'request_multipath': REQUEST_MULTIPATH,
'file_locks_path': LOCKS_DIR, 'state_path': STATE_PATH,
'grpc_workers': WORKERS}
'enable_probe': False, 'grpc_workers': WORKERS}

LOG_LEVELS = ('amqp=WARN,amqplib=WARN,boto=WARN,qpid=WARN,sqlalchemy=WARN,'
'suds=WARN,oslo.messaging=WARN,oslo_messaging=WARN,iso8601=WARN,'
Expand Down

0 comments on commit dcc76e7

Please sign in to comment.