Bug 1913460

Summary:	rebase dlm in 8.4
Product:	Red Hat Enterprise Linux 8	Reporter:	David Teigland <teigland>
Component:	dlm	Assignee:	David Teigland <teigland>
Status:	CLOSED ERRATA	QA Contact:	cluster-qe <cluster-qe>
Severity:	unspecified	Docs Contact:
Priority:	high
Version:	8.3	CC:	aahringo, cluster-maint, cmarthal
Target Milestone:	rc	Keywords:	Rebase, Triaged
Target Release:	8.4	Flags:	pm-rhel: mirror+
Hardware:	Unspecified
OS:	Unspecified
Whiteboard:
Fixed In Version:	dlm-4.1.0-1.el8	Doc Type:	If docs needed, set a value
Doc Text:		Story Points:	---
Clone Of:		Environment:
Last Closed:	2021-05-18 15:09:00 UTC	Type:	Bug
Regression:	---	Mount Type:	---
Documentation:	---	CRM:
Verified Versions:		Category:	---
oVirt Team:	---	RHEL 7.3 requirements from Atomic Host:
Cloudforms Team:	---	Target Upstream Version:
Embargoed:

Description David Teigland 2021-01-06 20:19:47 UTC

Description of problem:

Rebase the dlm package in 8.4.  It has not been updated in a year and a half so there are problems with applying new patches which are needed.  Changes are:

This is bug 1866905:
2e893b981b19 dlm_controld: use new quorum api to detect missed failures

These prevent errors when there's a flood of uevents:
a9b6e5beb8c2 dlm_controld: improve netlink ENOBUFS error handling
5d011abcc9f3 dlm_controld: set SO_RCVBUF for netlink socket

These are related to allowing port and skb mark values to be set in dlm.conf, and improve the parsing of config values.  These do nothing on their own and will require corresponding kernel patches eventually to use them.
66e781741ed4 dlm_controld: get notice about failed config parse
2da7a4daaee0 dlm_controld: fix may be used uninitialized
edcad817785f dlm_controld: fix may be used uninitialized
9c67c66ac0e0 dlm_controld: fix -Wstringop-truncation warnings
a95c9298d822 dlm_controld: add default value handling for unsigned int
6a6cefc993e7 dlm_controld: don't abort node configuration
b0222892f0b8 dlm_controld: add support for per nodeid configuration
d0f231a392fc dlm_controld: set listen skb mark setting
85b9536adad6 dlm_controld: add support for unsigned int values

This allows dlm to handle very large ringids from corosync:
dcc945a9d40b dlm_controld: use uint64_t for cluster ringid

This handles invalid name option:
07e321ab2ec4 dlm_tool: fix ls name handling



Version-Release number of selected component (if applicable):


How reproducible:


Steps to Reproduce:
1.
2.
3.

Actual results:


Expected results:


Additional info:

Comment 1 David Teigland 2021-01-12 20:40:31 UTC

scratch build of a rebased dlm-4.1.0 
https://brewweb.engineering.redhat.com/brew/taskinfo?taskID=34225144

If any issues come up I'm planning to push the rebase off until 8.5, and would likely leave dlm unchanged in 8.4.

Comment 2 Nate Straz 2021-01-13 21:01:03 UTC

I ran the scratch build through some regression tests and nothing broken, adding qa_ack+.

Comment 3 David Teigland 2021-01-13 21:28:15 UTC

official build https://brewweb.engineering.redhat.com/brew/taskinfo?taskID=34255789

Comment 8 Nate Straz 2021-02-24 04:42:53 UTC

Found no regressions in testing after rebase.

Comment 10 errata-xmlrpc 2021-05-18 15:09:00 UTC

Since the problem described in this bug report should be
resolved in a recent advisory, it has been closed with a
resolution of ERRATA.

For information on the advisory (dlm bug fix and enhancement update), and where to find the updated
files, follow the link below.

If the solution does not work for you, open a new bug report.

https://access.redhat.com/errata/RHBA-2021:1710