Madhu Rajanna
90cc5a4fd0
rbd: replace klog with util logger in rbd_journal.go
...
replace klog with util logger in rbd_journal.go
Signed-off-by: Madhu Rajanna <madhupr007@gmail.com>
2020-08-19 14:38:12 +00:00
Madhu Rajanna
9512aa6206
rbd: replace klog with util logger in rbd_attach.go
...
replace klog with util logger in rbd_attach.go
Signed-off-by: Madhu Rajanna <madhupr007@gmail.com>
2020-08-19 14:38:12 +00:00
Madhu Rajanna
e768c0dfc0
rbd: replace klog with util logger in nodeserver.go
...
replace klog with util logger in nodeserver.go
Signed-off-by: Madhu Rajanna <madhupr007@gmail.com>
2020-08-19 14:38:12 +00:00
Madhu Rajanna
883485b082
rbd: replace klog with util logger in driver.go
...
replace klog with util logger in driver.go
Signed-off-by: Madhu Rajanna <madhupr007@gmail.com>
2020-08-19 14:38:12 +00:00
Madhu Rajanna
c4f30c23c1
rbd: replace klog with util logger in clone.go
...
replace klog with util logger in clone.go
Signed-off-by: Madhu Rajanna <madhupr007@gmail.com>
2020-08-19 14:38:12 +00:00
Madhu Rajanna
d8ca56fbe9
rbd: replace klog with util in controllerserver.go
...
replace klog with util in controllerserver.go
Signed-off-by: Madhu Rajanna <madhupr007@gmail.com>
2020-08-19 14:38:12 +00:00
Humble Chirammal
1f5b84745f
rbd: Bail out from nodeexpansion if its block mode pvc
...
At CSI spec < 1.2.0, there was no volumecapability in the
expand request. However its available from v1.2+ which allows
us to declare the node operations based on the volume mode.
Signed-off-by: Humble Chirammal <hchiramm@redhat.com>
2020-08-19 12:34:20 +00:00
Madhu Rajanna
787ba69089
cephfs: fix lock release in volumeDelete
...
currently the lock is not released which is
taken on the request name. this is causing issues
when the subvolume is requested for delete.
Signed-off-by: Madhu Rajanna <madhupr007@gmail.com>
2020-08-18 06:12:43 +00:00
Madhu Rajanna
31c7c95fdb
cephfs: check error output contains Error: ENOENT
...
execCommandErr returns both error and stderror
message. checking strings.HasPrefix is not helpful
as the stderr will be the first string. its good
to do string comparison and find out that error
is volume not found error.
Signed-off-by: Madhu Rajanna <madhupr007@gmail.com>
2020-08-18 06:12:43 +00:00
Madhu Rajanna
f566b0fc2d
cephfs: replace Warningf with WarningLog in unmountVolume
...
replace Errorf with ErrorLog in unmountVolume
for context based logging.
Signed-off-by: Madhu Rajanna <madhupr007@gmail.com>
2020-08-13 09:44:12 +00:00
Madhu Rajanna
3fe3d25dda
cephfs: replace Errorf with ErrorLogMsg in loadAvailableMounters
...
replace Errorf with ErrorLog in loadAvailableMounters
for context based logging.
Signed-off-by: Madhu Rajanna <madhupr007@gmail.com>
2020-08-13 09:44:12 +00:00
Madhu Rajanna
fbe090065d
cephfs: replace Errorf with ErrorLog in purgeVolume
...
replace Errorf with ErrorLog in purgeVolume
for context based logging
Signed-off-by: Madhu Rajanna <madhupr007@gmail.com>
2020-08-13 09:44:12 +00:00
Madhu Rajanna
5a1652e5e8
cephfs: replace Errorf with ErrorLog in resizeVolume
...
replace Errorf with ErrorLog in resizeVolume
for context based logging.
Signed-off-by: Madhu Rajanna <madhupr007@gmail.com>
2020-08-13 09:44:12 +00:00
Madhu Rajanna
a882c1194b
cephfs: replace Errorf with ErrorLog in createVolume
...
replace Errorf with ErrorLog in createVolume
for context based logging.
Signed-off-by: Madhu Rajanna <madhupr007@gmail.com>
2020-08-13 09:44:12 +00:00
Madhu Rajanna
f3d9b8933b
cephfs: replace Errorf with ErrorLog in getSubVolumeInfo
...
replace Errorf with ErrorLog in getSubVolumeInfo
for context based logging.
Signed-off-by: Madhu Rajanna <madhupr007@gmail.com>
2020-08-13 09:44:12 +00:00
Madhu Rajanna
2b7aade289
cephfs: replace Errorf with ErrorLog in getVolumeRootPathCeph
...
replace Errorf with ErrorLog in getVolumeRootPathCeph
for context based logging.
Signed-off-by: Madhu Rajanna <madhupr007@gmail.com>
2020-08-13 09:44:12 +00:00
Madhu Rajanna
d1ac832266
cephfs: replace Errorf with ErrorLog in genSnapFromOptions
...
replace Errorf with ErrorLog in genSnapFromOptions
for context based logging.
Signed-off-by: Madhu Rajanna <madhupr007@gmail.com>
2020-08-13 09:44:12 +00:00
Madhu Rajanna
e83a51c1e1
cephfs: replace Errorf with ErrorLog in parseTime
...
replace Errorf with ErrorLog in parseTime
for context based logging.
Signed-off-by: Madhu Rajanna <madhupr007@gmail.com>
2020-08-13 09:44:12 +00:00
Madhu Rajanna
e7a043271d
cephfs: replace Errorf with ErrorLog in cloneSnapshot
...
replace Errorf with ErrorLog in cloneSnapshot
for context based logging.
Signed-off-by: Madhu Rajanna <madhupr007@gmail.com>
2020-08-13 09:44:12 +00:00
Madhu Rajanna
841a3713f5
cephfs: replace Errorf with ErrorLog in unprotectSnapshot
...
replace Errorf with ErrorLog in unprotectSnapshot
for context based logging.
Signed-off-by: Madhu Rajanna <madhupr007@gmail.com>
2020-08-13 09:44:12 +00:00
Madhu Rajanna
91811c18c5
cephfs: replace Errorf with ErrorLog in protectSnapshot
...
replace Errorf with ErrorLog in protectSnapshot
for context based logging.
Signed-off-by: Madhu Rajanna <madhupr007@gmail.com>
2020-08-13 09:44:12 +00:00
Madhu Rajanna
72dfe6a265
cephfs: replace Errorf with ErrorLog in getSnapshotInfo
...
replace Errorf with ErrorLog in getSnapshotInfo
for context based logging.
Signed-off-by: Madhu Rajanna <madhupr007@gmail.com>
2020-08-13 09:44:12 +00:00
Madhu Rajanna
7e8cc41aca
cephfs: replace Errorf with ErrorLog in deleteSnapshot
...
replace Errorf with ErrorLog in deleteSnapshot
for context based logging.
Signed-off-by: Madhu Rajanna <madhupr007@gmail.com>
2020-08-13 09:44:12 +00:00
Madhu Rajanna
7f00fa66f4
cephfs: replace Errorf with ErrorLog in createSnapshot
...
replace Errorf with ErrorLog in createSnapshot
for context based logging.
Signed-off-by: Madhu Rajanna <madhupr007@gmail.com>
2020-08-13 09:44:12 +00:00
Madhu Rajanna
57454cf379
cephfs: replace Errorf with ErrorLog in nodeUnstageVolume
...
replace Errorf with ErrorLog in nodeUnstageVolume
for context based logging.
Signed-off-by: Madhu Rajanna <madhupr007@gmail.com>
2020-08-13 09:44:12 +00:00
Madhu Rajanna
0a5581b7a0
cephfs: replace Errorf with ErrorLog in NodeUnPublishVolume
...
cephfs: replace Errorf with ErrorLog in NodeUnPublishVolume
for context based logging.
Signed-off-by: Madhu Rajanna <madhupr007@gmail.com>
2020-08-13 09:44:12 +00:00
Madhu Rajanna
79dbc41d14
cephfs: replace Errorf with ErrorLog in NodePublishVolume
...
replace Errorf with ErrorLog in NodePublishVolume
for context based logging.
Signed-off-by: Madhu Rajanna <madhupr007@gmail.com>
2020-08-13 09:44:12 +00:00
Madhu Rajanna
48a313c6cc
cephfs: replace Errorf with ErrorLog in mount
...
replace Errorf with ErrorLog in mount for
context based logging.
Signed-off-by: Madhu Rajanna <madhupr007@gmail.com>
2020-08-13 09:44:12 +00:00
Madhu Rajanna
032901b8d0
cephfs: replace Errorf with ErrorLog in nodeStageVolume
...
replace Errorf with ErrorLog in nodeStageVolume
for context based logging.
Signed-off-by: Madhu Rajanna <madhupr007@gmail.com>
2020-08-13 09:44:12 +00:00
Madhu Rajanna
c57189382f
cephfs: replace Errorf with ErrorLog in checkSnapExists
...
replace Errorf with ErrorLog in checkSnapExists
for context based logging.
Signed-off-by: Madhu Rajanna <madhupr007@gmail.com>
2020-08-13 09:44:12 +00:00
Madhu Rajanna
98e97baa60
cephfs: replace Errorf with ErrorLog in checkVolExists
...
replace Errorf with ErrorLog in checkVolExists
for context based logging.
Signed-off-by: Madhu Rajanna <madhupr007@gmail.com>
2020-08-13 09:44:12 +00:00
Madhu Rajanna
abd548e420
cephfs: replace warning with WarningLogMsg in Run
...
replace warning with WarningLogMsg in Run for
context based logging.
Signed-off-by: Madhu Rajanna <madhupr007@gmail.com>
2020-08-13 09:44:12 +00:00
Madhu Rajanna
38273f879f
cephfs: replace fatal with util.FatalLog in Run
...
replace fatal with util.FatalLog in Run for
context based logging.
Signed-off-by: Madhu Rajanna <madhupr007@gmail.com>
2020-08-13 09:44:12 +00:00
Madhu Rajanna
eeb0859f99
util: rename FatalLog to FatalLogMsg for parity
...
rename FatalLog to FatalLogMsg to keep functions
in parity functions ends with Msg if its only message
logging.
Signed-off-by: Madhu Rajanna <madhupr007@gmail.com>
2020-08-13 09:44:12 +00:00
Madhu Rajanna
71a6dc3ba8
cephfs: replace Errorf with util.ErrorLog in getCloneInfo
...
replace Errorf with util.ErrorLog in getCloneInfo
for context based logging.
Signed-off-by: Madhu Rajanna <madhupr007@gmail.com>
2020-08-13 09:44:12 +00:00
Madhu Rajanna
ae6b77f105
cephfs: add comment for exported CloneStatus struct
...
added comment for exported structure.
Signed-off-by: Madhu Rajanna <madhupr007@gmail.com>
2020-08-13 09:44:12 +00:00
Madhu Rajanna
18d65ba3f5
cephfs: replace Errorf with ErrorLog in createCloneFromSnapshot
...
replace Errorf with ErrorLog in createCloneFromSnapshot
for context based logging.
Signed-off-by: Madhu Rajanna <madhupr007@gmail.com>
2020-08-13 09:44:12 +00:00
Madhu Rajanna
3fa512617b
cephfs: replace Errorf with util.ErrorLog in cleanupClone
...
replace Errorf with util.ErrorLog in cleanupCloneFromSubvolumeSnapshot
for context based logging.
Signed-off-by: Madhu Rajanna <madhupr007@gmail.com>
2020-08-13 09:44:12 +00:00
Madhu Rajanna
e1fb708d48
cephfs: replace Errorf with util.ErrorLog in createCloneFromSubvolume
...
replace Errorf with util.ErrorLog in createCloneFromSubvolume
for context based logging.
Signed-off-by: Madhu Rajanna <madhupr007@gmail.com>
2020-08-13 09:44:12 +00:00
Madhu Rajanna
20dbbe54b0
cephfs: replace klog.Warningf with util.WarningLog in deleteSnapshot
...
replace klog.Warningf with util.WarningLog in deleteSnapshot
for context based logging.
Signed-off-by: Madhu Rajanna <madhupr007@gmail.com>
2020-08-13 09:44:12 +00:00
Madhu Rajanna
7eefadf1eb
cephfs: replace klog.Warning with util.WarningLog in DeleteVolume
...
replace klog.Warning with util.WarningLog in DeleteVolume
for context based logging.
Signed-off-by: Madhu Rajanna <madhupr007@gmail.com>
2020-08-13 09:44:12 +00:00
Madhu Rajanna
e3c3bbf0e3
cephfs: replace klog.Warning with util.WarningLog in CreateVolume
...
replace klog.Warning with util.WarningLog in CreateVolume
for context based logging.
Signed-off-by: Madhu Rajanna <madhupr007@gmail.com>
2020-08-13 09:44:12 +00:00
Madhu Rajanna
3d12456493
util: add WarningLog to log with context
...
added WarningLog to log with context
Signed-off-by: Madhu Rajanna <madhupr007@gmail.com>
2020-08-13 09:44:12 +00:00
Madhu Rajanna
b223924044
util: rename WarningLog to WarningLogMsg
...
renamed WarningLog function to WarningLogMsg
to log with message.
Signed-off-by: Madhu Rajanna <madhupr007@gmail.com>
2020-08-13 09:44:12 +00:00
Madhu Rajanna
f45ad230a2
cephfs: replace klog.Error with util.ErrorLog in deleteSnapshot
...
replace klog.Error with util.ErrorLog in deleteSnapshot
for context based logging.
Signed-off-by: Madhu Rajanna <madhupr007@gmail.com>
2020-08-13 09:44:12 +00:00
Madhu Rajanna
52f14cf9fe
cephfs: replace klog.Error with util.ErrorLog in validateSnapshotReq
...
replace klog.Error with util.ErrorLog in validateSnapshotReq
for context based logging.
Signed-off-by: Madhu Rajanna <madhupr007@gmail.com>
2020-08-13 09:44:12 +00:00
Madhu Rajanna
d101cbc888
cephfs: replace klog.Error with util.ErrorLog in doSnapshot
...
replace klog.Error with util.ErrorLog in doSnapshot
for context based logging.
Signed-off-by: Madhu Rajanna <madhupr007@gmail.com>
2020-08-13 09:44:12 +00:00
Madhu Rajanna
5802265b28
cephfs: replace klog.Error with util.ErrorLog in createSnapshot
...
replace klog.Error with util.ErrorLog in createSnapshot
for context based logging.
Signed-off-by: Madhu Rajanna <madhupr007@gmail.com>
2020-08-13 09:44:12 +00:00
Madhu Rajanna
81b52b311d
cephfs: replace klog.Error with util.ErrorLog in ControllerExpand
...
replace klog.Error with util.ErrorLog in ControllerExpand
for context based logging.
Signed-off-by: Madhu Rajanna <madhupr007@gmail.com>
2020-08-13 09:44:12 +00:00
Madhu Rajanna
7ed5dfe81a
cephfs: replace klog.Error with util.ErrorLog in DeleteVolume
...
replaced klog.Error with util.ErrorLog in DeleteVolume
for context based logging.
Signed-off-by: Madhu Rajanna <madhupr007@gmail.com>
2020-08-13 09:44:12 +00:00
Madhu Rajanna
cc600dab76
cephfs: replace klog.Error with util.ErrorLog in CreateVolume
...
replaced klog.Error with util.ErrorLog
in createVolume function.
Signed-off-by: Madhu Rajanna <madhupr007@gmail.com>
2020-08-13 09:44:12 +00:00
Madhu Rajanna
3c45ec32db
cephfs: replace klog.Error in createBackingVolume
...
replace klog.Error with util.ErrorLog in
createBackingVolume function.
Signed-off-by: Madhu Rajanna <madhupr007@gmail.com>
2020-08-13 09:44:12 +00:00
Madhu Rajanna
8c32045c4e
util: use ErrorLog for context based logging
...
use ErrorLog for context based logging for
more detailed logging.
Signed-off-by: Madhu Rajanna <madhupr007@gmail.com>
2020-08-13 09:44:12 +00:00
Madhu Rajanna
7c24f4d597
util: add ErrorLog for log with context
...
added a new function ErrorLog to log with
context.
Signed-off-by: Madhu Rajanna <madhupr007@gmail.com>
2020-08-13 09:44:12 +00:00
Madhu Rajanna
9229e84a77
util: rename ErrorLog to ErrorLogMsg to parity
...
as we have 2 functions for logging. one for logging
with message and another one is for logging with
context. renamed ErrorLog to ErrorLogMsg to log
with messages.
Signed-off-by: Madhu Rajanna <madhupr007@gmail.com>
2020-08-13 09:44:12 +00:00
Madhu Rajanna
d4186a6880
rbd: discard flattening error if the image doesnt have parent
...
if the image is created without flattening image-feature
the image will get few image-features by default, deep-flatten
is one of them. if the image doesnot have any parent
the rbd image flattening will fail, This commit discards
error message if the image doesnt have any parent.
Signed-off-by: Madhu Rajanna <madhupr007@gmail.com>
2020-08-13 08:37:53 +00:00
Madhu Rajanna
8a581ec6c0
rbd: fix build issue for rbd
...
merging of https://github.com/ceph/ceph-csi/pull/1035
broken the cephcsi building. This commits fixes
the build issue.
Signed-off-by: Madhu Rajanna <madhupr007@gmail.com>
2020-08-12 19:53:48 +05:30
Mehdy Khoshnoody
3081eabba5
doc: Add rados namespace to option to csi-configs
...
A minimal documentation on how to use a rados namespace with rbd.
Signed-off-by: Mehdy Khoshnoody <mehdy.khoshnoody@gmail.com>
2020-08-12 16:22:58 +05:30
Mehdy Khoshnoody
fc5eadf106
rbd: Add rados namespace support for rbd
...
Make sure to operate within the namespace if any given
when dealing with rbd images and snapshots and their journals.
Signed-off-by: Mehdy Khoshnoody <mehdy.khoshnoody@gmail.com>
2020-08-12 16:22:58 +05:30
Madhu Rajanna
bfde065f92
util: move getMonsAndClusterID to util
...
as we had duplicate functions in both cephfs
and rbd this commit moves the function to util.
Signed-off-by: Madhu Rajanna <madhupr007@gmail.com>
2020-08-11 10:02:53 +00:00
Madhu Rajanna
18f4e9d519
util: move csiconfigfile to util
...
as we have csiconfigfile in both cephfs
and rbd moving the configfile path to
util folder.
Signed-off-by: Madhu Rajanna <madhupr007@gmail.com>
2020-08-11 10:02:53 +00:00
Niels de Vos
b864afd6d7
util: use local ErrorLog() for Create/RemoveObject() instead of klog
...
Signed-off-by: Niels de Vos <ndevos@redhat.com>
2020-08-11 08:11:37 +00:00
Niels de Vos
eaea18df0f
util: use local FatalLog() for NewK8sClient() instead of klog
...
Signed-off-by: Niels de Vos <ndevos@redhat.com>
2020-08-11 08:11:37 +00:00
Niels de Vos
9d4020d255
util: use local ErrorLog() for CheckKernelSupport() instead of klog
...
Signed-off-by: Niels de Vos <ndevos@redhat.com>
2020-08-11 08:11:37 +00:00
Niels de Vos
75ae16cfab
util: use local DefaultLog() for util/topology instead of klog
...
Signed-off-by: Niels de Vos <ndevos@redhat.com>
2020-08-11 08:11:37 +00:00
Niels de Vos
bf02227207
util: use local ErrorLog() for OperationLock.release() instead of klog
...
Signed-off-by: Niels de Vos <ndevos@redhat.com>
2020-08-11 08:11:37 +00:00
Niels de Vos
fb56a7595a
util: use local ErrorLog() for GetCryptoPassphrase() instead of klog
...
Signed-off-by: Niels de Vos <ndevos@redhat.com>
2020-08-11 08:11:37 +00:00
Niels de Vos
61924adf57
util: use FatalLog() in util/httpserver.go
...
Hide the use of klog by using our own FatalLog() function.
Signed-off-by: Niels de Vos <ndevos@redhat.com>
2020-08-11 08:11:37 +00:00
Niels de Vos
ab033f85df
util: add FatalLog(), ErrorLog() and WarningLog() functions
...
These functions will always log the message, irrespectively of the
log-level that has been configured.
Signed-off-by: Niels de Vos <ndevos@redhat.com>
2020-08-11 08:11:37 +00:00
Niels de Vos
44863a9d29
cleanup: move log functions to util/log.go
...
There is a util/log.go file, so the log functions in util/util.go can be
moved there.
Signed-off-by: Niels de Vos <ndevos@redhat.com>
2020-08-11 08:11:37 +00:00
Madhu Rajanna
b86d329d1e
rbd: add stagingpath to error message
...
if the stagingpath is not present inside
the container send the staging path in error
message.
Signed-off-by: Madhu Rajanna <madhupr007@gmail.com>
2020-08-10 08:51:23 +00:00
Humble Chirammal
7d2bba1375
cephfs: introduce parsetime() to parse createdAt field in snap return
...
Signed-off-by: Humble Chirammal <hchiramm@redhat.com>
2020-08-08 00:11:35 +00:00
Humble Chirammal
c3400bfb97
cephfs: add snapshot create and delete functionalilies
...
It also add helper routines like parsetime,doSnapshot..etc
Signed-off-by: Humble Chirammal <hchiramm@redhat.com>
2020-08-08 00:11:35 +00:00
Humble Chirammal
20c90ddfc4
cephfs: getVolumeRootPathCeph and purgeVolume use new error strings
...
Signed-off-by: Humble Chirammal <hchiramm@redhat.com>
2020-08-08 00:11:35 +00:00
Humble Chirammal
1f4839cb5e
cephfs: add subvolume struct and getSubvolumeInfo calls
...
Signed-off-by: Humble Chirammal <hchiramm@redhat.com>
2020-08-08 00:11:35 +00:00
Humble Chirammal
31e9288f8f
cephfs: remove inValidCommmand and errNotFoundString
...
Signed-off-by: Humble Chirammal <hchiramm@redhat.com>
2020-08-08 00:11:35 +00:00
Humble Chirammal
5bceb590fd
cephfs: add snap reserve/unreserve and snap exist functionalities
...
Signed-off-by: Humble Chirammal <hchiramm@redhat.com>
2020-08-08 00:11:35 +00:00
Humble Chirammal
38d005e4e6
cephfs: validate create volume request
...
Also add functionality to generate snap from request
and to get mon and clusterID from the request
Signed-off-by: Humble Chirammal <hchiramm@redhat.com>
2020-08-08 00:11:35 +00:00
Humble Chirammal
9c000add29
cephfs: Change checkVolExist for snapshot and clone workflow
...
Signed-off-by: Humble Chirammal <hchiramm@redhat.com>
2020-08-08 00:11:35 +00:00
Humble Chirammal
c773097f85
cephfs: add snapshot and clone helper functions
...
Signed-off-by: Humble Chirammal <hchiramm@redhat.com>
2020-08-08 00:11:35 +00:00
Humble Chirammal
d1fe12b4f0
cephfs: change createvolume for snapshot/clone workflow
...
Signed-off-by: Humble Chirammal <hchiramm@redhat.com>
2020-08-08 00:11:35 +00:00
Humble Chirammal
c31f349c6f
cephfs: use delete operation lock in delete volume
...
Signed-off-by: Humble Chirammal <hchiramm@redhat.com>
2020-08-08 00:11:35 +00:00
Humble Chirammal
f9257ed1cb
cephfs: add snapshot create/delete capabilities in controllerserver
...
Signed-off-by: Humble Chirammal <hchiramm@redhat.com>
2020-08-08 00:11:35 +00:00
Humble Chirammal
5a71949dcd
cephfs: introduce newSnapshotOptionsFromID to generate volOpt and sid
...
From provided CSI volume ID this populate volumeOptions and snapshot
identifier after connecting to the snapJournal.
Signed-off-by: Humble Chirammal <hchiramm@redhat.com>
2020-08-08 00:11:35 +00:00
Humble Chirammal
a4f2c5617c
cephfs: retrieve NewCSISnapshotJournal with namespace set
...
Signed-off-by: Humble Chirammal <hchiramm@redhat.com>
2020-08-08 00:11:35 +00:00
Humble Chirammal
1746b4e6ee
cephfs: add various error strings for command validation
...
Compared to previous version of the error strings, this change
depend on error strings like ENOENT, EEXIST, EINVAL..etc
The format of the error strings change in different cluster versions
and the error code return should not change. This also add extra
error strings for snapshot and clone operation outputs
Signed-off-by: Humble Chirammal <hchiramm@redhat.com>
2020-08-08 00:11:35 +00:00
Humble Chirammal
6dcf03a4bc
cephfs: add checkContentSource() to validate the data source
...
checkContentSource() validate the data source in the request
and then populate volumeOptions or snapshotshot identifier in
case of snapshot source. If the data source is volume, then
parentVolumeOption and pvID are populated.
Signed-off-by: Humble Chirammal <hchiramm@redhat.com>
2020-08-08 00:11:35 +00:00
Humble Chirammal
2bb0bbde7f
cephfs: change createBackingVolume to accomodate clone operations
...
adjust createBackingVolume() to create a subvolume from snapshot
or existing subvolume by taking restore or clone operation locks
Signed-off-by: Humble Chirammal <hchiramm@redhat.com>
2020-08-08 00:11:35 +00:00
Humble Chirammal
c968380134
cephfs: add snapshotlock and operationlock to controllerserver
...
Signed-off-by: Humble Chirammal <hchiramm@redhat.com>
2020-08-08 00:11:35 +00:00
Humble Chirammal
66827facc3
cephfs: make use of expand operation lock in ControllerExpandVolume
...
Signed-off-by: Humble Chirammal <hchiramm@redhat.com>
2020-08-08 00:11:35 +00:00
Madhu Rajanna
2458ec6573
rbd: return error if fetching cluster id fails
...
if we are not able to fetch the cluster-ID from
the createSnapshot request and also if we are
not able to get the monitor information from
the cluster-ID return error instead of using
the parent image information.
Signed-off-by: Madhu Rajanna <madhupr007@gmail.com>
2020-08-03 14:25:06 +00:00
Mudit Agarwal
9ed0811422
rbd: implement rbdVolume.resize() with go-ceph
...
Replaced command execution with go-ceph Resize() function.
Volsize is being updated before waiting for resize() to return,
fixed it to get updated only after resize() is successful.
Signed-off-by: Mudit Agarwal <muagarwa@redhat.com>
2020-08-03 10:50:01 +00:00
Niels de Vos
be9e7cf956
rbd: pass context.Context to rbdVolume.resize()
...
While adding the context.Context to the resizeRBDimage() function, it
became a little ugly. So renaming the function to resize() and making it
a method of the rbdVolume type.
Signed-off-by: Niels de Vos <ndevos@redhat.com>
2020-07-24 16:04:13 +00:00
Niels de Vos
36469b87e2
util: make ExecComand return stdout and stderr as string
...
Most consumers of util.ExecCommand() need to convert the returned []byte
format of stdout and/or stderr to string. By having util.ExecCommand()
return strings instead, the code gets a little simpler.
A few commands return JSON that needs to be parsed. These commands will
be replaced by go-ceph implementations later on. For now, convert the
strings back to []byte when needed.
Signed-off-by: Niels de Vos <ndevos@redhat.com>
2020-07-24 16:04:13 +00:00
Niels de Vos
ddac66d76b
util: use context.Context for logging in ExecCommand
...
All calls to util.ExecCommand() now pass the context.Context. In some
cases this is not possible or needed, and util.ExecCommand() will not
log the command.
This should make debugging easier when command executions fail.
Signed-off-by: Niels de Vos <ndevos@redhat.com>
2020-07-24 16:04:13 +00:00
Niels de Vos
bb4f1c7c9d
rbd: use util.ExecCommand() instead of execCommand()
...
Signed-off-by: Niels de Vos <ndevos@redhat.com>
2020-07-24 16:04:13 +00:00
Niels de Vos
457d846241
cephfs: use util.ExecCommand() instead of execCommand()
...
Signed-off-by: Niels de Vos <ndevos@redhat.com>
2020-07-24 16:04:13 +00:00
Niels de Vos
47d5b60af8
rbd: disable reflink while creating XFS filesystems
...
Current versions of the mkfs.xfs binary enable reflink support by
default. This causes problems on systems where the kernel does not
support this feature. When the kernel the feature does not support, but
the filesystem has it enabled, the following error is logged in `dmesg`:
XFS: Superblock has unknown read-only compatible features (0x4) enabled
Introduce a check to see if mkfs.xfs supports the `-m reflink=` option.
In case it does, pass `-m reflink=0` while creating the filesystem.
The check is executed once during the first XFS filesystem creation. The
result of the check is cached until the nodeserver restarts.
Fixes : #966
Signed-off-by: Niels de Vos <ndevos@redhat.com>
2020-07-24 13:37:51 +00:00
Niels de Vos
526da43b6a
rbd: remove unused rbdStatus()
...
Signed-off-by: Niels de Vos <ndevos@redhat.com>
2020-07-24 11:34:48 +00:00
Niels de Vos
7afaac9c66
rbd: implement rbdVolume.isInUse() with go-ceph
...
The new rbdVolume.isInUse() method will replace the rbdStatus()
function. This removes one more rbd command execution in the
DeleteVolume path.
Signed-off-by: Niels de Vos <ndevos@redhat.com>
2020-07-24 11:34:48 +00:00