Bug 1311234 - backup-volfile-servers queried on every remount of gluster storage domain
Summary: backup-volfile-servers queried on every remount of gluster storage domain
Keywords:
Status: CLOSED WONTFIX
Alias: None
Product: vdsm
Classification: oVirt
Component: Gluster
Version: 4.17.18
Hardware: Unspecified
OS: Unspecified
medium
medium
Target Milestone: ---
: ---
Assignee: Ala Hino
QA Contact: SATHEESARAN
URL:
Whiteboard:
: 1311235 (view as bug list)
Depends On:
Blocks: Gluster-HC-3
TreeView+ depends on / blocked
 
Reported: 2016-02-23 16:45 UTC by Sahina Bose
Modified: 2022-06-27 12:32 UTC (History)
3 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2017-09-12 12:23:53 UTC
oVirt Team: Storage
Embargoed:
sbonazzo: ovirt-4.2-


Attachments (Terms of Use)


Links
System ID Private Priority Status Summary Last Updated
Red Hat Issue Tracker RHV-46716 0 None None None 2022-06-27 12:32:21 UTC

Description Sahina Bose 2016-02-23 16:45:31 UTC
Description of problem:
When gluster volume is used as storage domain and mount path given as server1:/vol1, the volfile is fetched from server1 to introspect the possible  alternative servers (say, server2, server3). These servers are then passed as mount option "backup-volfile-servers=server2:server3".

However since the backup-volfile-servers option is not persisted, each time vdsm tries to mount the storage domain, the primary server (server1) is queried and activation of storage domain fails if primary server is down - even though the other servers in cluster are available.


The alternative backup servers should be persisted in DB the first time, and used in subsequent mount calls

Comment 1 Allon Mureinik 2016-02-24 08:07:42 UTC
Doing this will make VDSM oblivious to changes on the gluster side. I don't think it's a good idea.

Comment 2 Allon Mureinik 2016-02-24 08:08:35 UTC
*** Bug 1311235 has been marked as a duplicate of this bug. ***

Comment 3 Yaniv Kaul 2016-03-21 14:28:36 UTC
(In reply to Allon Mureinik from comment #1)
> Doing this will make VDSM oblivious to changes on the gluster side. I don't
> think it's a good idea.

Sahina - CLOSE-WONTFIX?

Comment 4 Sahina Bose 2016-03-22 06:13:52 UTC
If we close this, we are essentially removing the support for using backup-volfile-servers. 
I think whenever gluster volfile changes are detected, the DB should be updated either automatically, or via user initiated action.

Comment 5 Allon Mureinik 2016-03-22 14:20:45 UTC
(In reply to Sahina Bose from comment #4)
> If we close this, we are essentially removing the support for using
> backup-volfile-servers. 
> I think whenever gluster volfile changes are detected, the DB should be
> updated either automatically, or via user initiated action.

This is the the same situation we had before integrating backup-volfile-servers, so there's no urgency here.

This should be separated into two issues:
- In HC, VDSM shouldn't detect anything anyway - the engine also manages the volume, so it should send the proper configuration on each connectStorageServer request.
- In a non-HC environment this would mean new verbs for the engine to query the gluster topology, store it somehow, and send it on each connectStorageServer request. Looks like a lot of work for a very small benefit, if any. Definitely not 4.0 material, at least not with the given timeframe and capacity.

Comment 6 Sahina Bose 2017-01-09 16:20:18 UTC
Defering this as https://bugzilla.redhat.com/show_bug.cgi?id=1177782 has a way of persisting backup-volfile-server info in engine, if storage domain is linked to gluster volume

Comment 7 Allon Mureinik 2017-09-12 12:23:53 UTC
(In reply to Sahina Bose from comment #6)
> Defering this as https://bugzilla.redhat.com/show_bug.cgi?id=1177782 has a
> way of persisting backup-volfile-server info in engine, if storage domain is
> linked to gluster volume
This seems good enough to me, closing.


Note You need to log in before you can comment on or make changes to this bug.