RHEL Engineering is moving the tracking of its product development work on RHEL 6 through RHEL 9 to Red Hat Jira (issues.redhat.com). If you're a Red Hat customer, please continue to file support cases via the Red Hat customer portal. If you're not, please head to the "RHEL project" in Red Hat Jira and file new tickets here. Individual Bugzilla bugs in the statuses "NEW", "ASSIGNED", and "POST" are being migrated throughout September 2023. Bugs of Red Hat partners with an assigned Engineering Partner Manager (EPM) are migrated in late September as per pre-agreed dates. Bugs against components "kernel", "kernel-rt", and "kpatch" are only migrated if still in "NEW" or "ASSIGNED". If you cannot log in to RH Jira, please consult article #7032570. That failing, please send an e-mail to the RH Jira admins at rh-issues@redhat.com to troubleshoot your issue as a user management inquiry. The email creates a ServiceNow ticket with Red Hat. Individual Bugzilla bugs that are migrated will be moved to status "CLOSED", resolution "MIGRATED", and set with "MigratedToJIRA" in "Keywords". The link to the successor Jira issue will be found under "Links", have a little "two-footprint" icon next to it, and direct you to the "RHEL project" in Red Hat Jira (issue links are of type "https://issues.redhat.com/browse/RHEL-XXXX", where "X" is a digit). This same link will be available in a blue banner at the top of the page informing you that that bug has been migrated.
Bug 1933768 - cloud-utils-growpart: Wrong parsing of sfdisk output in french locale
Summary: cloud-utils-growpart: Wrong parsing of sfdisk output in french locale
Keywords:
Status: CLOSED ERRATA
Alias: None
Product: Red Hat Enterprise Linux 8
Classification: Red Hat
Component: cloud-utils-growpart
Version: CentOS Stream
Hardware: Unspecified
OS: Unspecified
unspecified
unspecified
Target Milestone: rc
: ---
Assignee: Eduardo Otubo
QA Contact: xiachen
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2021-03-01 16:39 UTC by Nicolas Chauvet (kwizart)
Modified: 2023-03-14 14:25 UTC (History)
11 users (show)

Fixed In Version: cloud-utils-growpart-0.31-2.el8
Doc Type: If docs needed, set a value
Doc Text:
Clone Of:
Environment:
Last Closed: 2021-11-09 18:43:52 UTC
Type: Bug
Target Upstream Version:
Embargoed:

Attachments (Terms of Use)


Links
System ID Private Priority Status Summary Last Updated
Red Hat Product Errata RHBA-2021:4264 0 None None None 2021-11-09 18:43:54 UTC

Description Nicolas Chauvet (kwizart) 2021-03-01 16:39:06 UTC 
Description of problem:
As reported in upstream cloud-utils, if growpart is used when the locale is set, the disk parsing is wrong leading to failure to extend the partition.

Version-Release number of selected component (if applicable):
cloud-utils-growpart-0.31-1

How reproducible:
always

Steps to Reproduce:
1. extend any pre-existing VM local disk.
2. export LANG=fr_FR-utf8 (or in cloud meta-data)
3. run growpart -N /dev/vda 2


Actual results:
./growpart -N /dev/vda 2
./growpart: ligne 322: 22548578304 octets, : erreur de syntaxe : opérateur arithmétique non valable (le symbole erroné est «  octets, »)

(an attempt to divide by a variable containing some text).

Expected results:
CHANGE: partition=2 start=2099200 old: size=39843840 end=41943040 new: size=41940959 end=44040159
# === old sfdisk -d ===
label: dos
label-id: 0x2ea6aa65
device: /dev/vda
unit: sectors

/dev/vda1 : start= 2048, size= 2097152, type=83, bootable
/dev/vda2 : start= 2099200, size= 39843840, type=8e
# === new sfdisk -d ===
label: dos
label-id: 0x2ea6aa65
device: /dev/vda
unit: sectors

/dev/vda1 : start= 2048, size= 2097152, type=83, bootable
/dev/vda2 : start= 2099200, size= 41940959, type=8e

Additional info:

PR for git.centos.org in progress...
Comment 1 Nicolas Chauvet (kwizart) 2021-03-01 17:16:02 UTC 
Upstream bug: https://bugs.launchpad.net/cloud-utils/+bug/1860479
Pagure MR for git.centos.org: https://git.centos.org/rpms/cloud-utils-growpart/pull-request/1
Comment 2 xiachen 2021-03-02 10:23:26 UTC 
Reproduced the issue with cloud-utils-growpart-0.31-1.el8.noarch on RHEL8.4 in french locale (LANG=fr_FR.UTF-8)
set qa_ack+

According to the code changes https://github.com/canonical/cloud-utils/pull/28, 
I modified the source code of growpart and verified it, result is PASS.
so once the MR is accepted by upstream, we can merge it to downstream.
Comment 3 Nicolas Chauvet (kwizart) 2021-03-19 16:44:21 UTC 
(In reply to xiachen from comment #2)
Hello xiachen,


Just a notification that the PR have just been accepted upstream.

Thanks for your work on this issue.
Have a nice day.
Comment 4 xiachen 2021-03-22 13:40:59 UTC 
Great! thanks for your notification.
I think, Eduardo will do the backport to downstream according to his plan.


(In reply to Nicolas Chauvet (kwizart) from comment #3)
> (In reply to xiachen from comment #2)
> Hello xiachen,
> 
> 
> Just a notification that the PR have just been accepted upstream.
> 
> Thanks for your work on this issue.
> Have a nice day.
Comment 5 Eduardo Otubo 2021-03-29 11:16:16 UTC 
(In reply to xiachen from comment #4)
> Great! thanks for your notification.
> I think, Eduardo will do the backport to downstream according to his plan.
> 
> 
> (In reply to Nicolas Chauvet (kwizart) from comment #3)
> > (In reply to xiachen from comment #2)
> > Hello xiachen,
> > 
> > 
> > Just a notification that the PR have just been accepted upstream.
> > 
> > Thanks for your work on this issue.
> > Have a nice day.

I'm only waiting for the gitlab branches to be created.
I'll handle this bug as soon as we can.
Comment 6 Eduardo Otubo 2021-03-29 11:47:19 UTC 
(In reply to Eduardo Otubo from comment #5)
> (In reply to xiachen from comment #4)
> > Great! thanks for your notification.
> > I think, Eduardo will do the backport to downstream according to his plan.
> > 
> > 
> > (In reply to Nicolas Chauvet (kwizart) from comment #3)
> > > (In reply to xiachen from comment #2)
> > > Hello xiachen,
> > > 
> > > 
> > > Just a notification that the PR have just been accepted upstream.
> > > 
> > > Thanks for your work on this issue.
> > > Have a nice day.
> 
> I'm only waiting for the gitlab branches to be created.
> I'll handle this bug as soon as we can.

Well, that was quick.
MR opened: https://gitlab.com/redhat/rhel/src/cloud-utils/-/merge_requests/1
Comment 10 Nicolas Chauvet (kwizart) 2021-05-07 22:12:29 UTC 
Hello, any update on this fix ? I haven't seen it land in either c8s or c9s.

Thanks for any updates...
Comment 19 Rick Barry 2021-06-04 12:32:21 UTC 
Clearing the needinfo's.
Comment 20 Nicolas Chauvet (kwizart) 2021-06-04 12:55:30 UTC 
Thanks for the update on this. Can you please advertises the next steps ?
I don't see cloud-utils-growpart-0.31-2.el8 on git.centos.org yet...

Also I wonder how about the EL9 branch ?

Thanks in advances.
Comment 30 Carl George 🤠 2021-06-11 22:36:26 UTC 
cloud-utils-growpart-0.31-2.el8 has been exported to git.c.o [0] and built in koji [1].  It should be included in the next CentOS Stream 8 compose.

[0] https://git.centos.org/rpms/cloud-utils-growpart/c/d4e673aff4f622b5d834f3c21b14e6fb7fcc5d56?branch=c8s
[1] https://koji.mbox.centos.org/koji/buildinfo?buildID=18024
Comment 34 errata-xmlrpc 2021-11-09 18:43:52 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 (cloud-utils-growpart 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:4264
Note You need to log in before you can comment on or make changes to this bug.