Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]

Groups > linux.debian.bugs.dist > #1230506 > unrolled thread

Bug#1094356: freeradius-utils radlast and wtmpdb last incompatible

Back to article view | Back to linux.debian.bugs.dist

Contents

  Bug#1094356: freeradius-utils radlast and wtmpdb last incompatible John Chittum <john.chittum@canonical.com> - 2025-01-27 14:40 +0100
    Bug#1094356: freeradius-utils radlast and wtmpdb last incompatible Andrew Bower <andrew@bower.uk> - 2025-02-06 21:40 +0100
      Bug#1094356: freeradius-utils radlast and wtmpdb last incompatible John Chittum <john.chittum@canonical.com> - 2025-02-07 13:30 +0100
        Bug#1094356: freeradius-utils radlast and wtmpdb last incompatible John Chittum <john.chittum@canonical.com> - 2025-02-20 20:10 +0100

#1230506 — Bug#1094356: freeradius-utils radlast and wtmpdb last incompatible

[Multipart message — attachments visible in raw view] — view raw

Package: freeradius-utils
Source: freeradius
Version: 3.2.6+dfsg-3

freeradius:src 3.2.6+dfsg-3 contains the following control change:

  [ Arnaud Rebillout ]
  * freeradius-utils: add wtmpdb needed for radlast

`wtmpdb:last` only operates with `wtpmdb` implementation, reading an sqlite
database with a `wtpm` table. freeradius writes a `utmp` struct file.
running `radlast` results in:

$ radlast
wtmpdb_read_all: SQL error: no such table: wtmp

the old `last` was removed from util-linux for not being 2038 compliant.
`glibc` has made the `utmp` seconds uint, so it's safe for additional time.
`wtmpdb` has not implemented a method for reading the old file, nor
migrating data. see GH issue

https://github.com/thkukuk/wtmpdb/issues/14

filed a bug in Ubuntu to revert the control change, but Ubuntu is similarly
without a `last` command in devel:

https://bugs.launchpad.net/ubuntu/+source/freeradius/+bug/2096611

The bug contains some discussion about possibly splitting the package more
or even removing the functionality entirely.

libc6=2.40-6

-- 

John Chittum

Engineering Manager, Ubuntu Engineering, Server

Email: john.chittum@canonical.com

Location: Maine, USA

[toc] | [next] | [standalone]

#1232039

Control: block -1 by 1086559

Hi John,

On Mon, Jan 27, 2025 at 08:28:02AM -0500, John Chittum wrote:
>    the old `last` was removed from util-linux for not being 2038 compliant.
>    `glibc` has made the `utmp` seconds uint, so it's safe for additional
>    time. `wtmpdb` has not implemented a method for reading the old file, nor
>    migrating data. see GH issue
>    https://github.com/thkukuk/wtmpdb/issues/14

As you note, the wtmpdb upstream hasn't prioritised providing an importer for
utmp-formatted wtmp logs, doesn't expect to, and even if they did, I don't know
if it would be in a form convenient for the purposes of this test.

One option could be to restore the tools in an optional package specifically
for reading legacy files rather than live system administration - e.g. my
merge request on src:sysvinit[1], although I note you have a simpler solution
in mind for freeradius!

[1] https://salsa.debian.org/debian/sysvinit/-/merge_requests/14

[toc] | [prev] | [next] | [standalone]

#1232122

[Multipart message — attachments visible in raw view] — view raw

On Thu, Feb 6, 2025 at 3:33 PM Andrew Bower <andrew@bower.uk> wrote:

> Control: block -1 by 1086559
>
> Hi John,
>
> On Mon, Jan 27, 2025 at 08:28:02AM -0500, John Chittum wrote:
> >    the old `last` was removed from util-linux for not being 2038
> compliant.
> >    `glibc` has made the `utmp` seconds uint, so it's safe for additional
> >    time. `wtmpdb` has not implemented a method for reading the old file,
> nor
> >    migrating data. see GH issue
> >    https://github.com/thkukuk/wtmpdb/issues/14
>
> As you note, the wtmpdb upstream hasn't prioritised providing an importer
> for
> utmp-formatted wtmp logs, doesn't expect to, and even if they did, I don't
> know
> if it would be in a form convenient for the purposes of this test.
>
> One option could be to restore the tools in an optional package
> specifically
> for reading legacy files rather than live system administration - e.g. my
> merge request on src:sysvinit[1], although I note you have a simpler
> solution
> in mind for freeradius!
>

freeradius upstream has pushed a change to make building and including
`radlast` as optional

https://github.com/FreeRADIUS/freeradius-server/releases/tag/release_3_2_7

relevant commit:

https://github.com/FreeRADIUS/freeradius-server/commit/fece06e2f4984a1f4c227ba9ef3edf4a8ad5e2ee

if Debian wishes to move to 3.2.7 before freeze, we could remove the
`wtpmdb` dependency.
If you believe `radlast` is important, package splitting makes sense, but
we'll also need a `last` command
provided, and I'm unsure what to suggest as a replacement.




>
> [1] https://salsa.debian.org/debian/sysvinit/-/merge_requests/14
>


-- 
[image: Canonical-20th-anniversary]

John Chittum

Engineering Manager, Ubuntu Engineering, Server

Email: john.chittum@canonical.com

Location: Maine, USA

canonical.com

ubuntu.com

[toc] | [prev] | [next] | [standalone]

#1234276

[Multipart message — attachments visible in raw view] — view raw

On Fri, Feb 7, 2025 at 7:21 AM John Chittum <john.chittum@canonical.com>
wrote:

>
>
> On Thu, Feb 6, 2025 at 3:33 PM Andrew Bower <andrew@bower.uk> wrote:
>
>> Control: block -1 by 1086559
>>
>> Hi John,
>>
>> On Mon, Jan 27, 2025 at 08:28:02AM -0500, John Chittum wrote:
>> >    the old `last` was removed from util-linux for not being 2038
>> compliant.
>> >    `glibc` has made the `utmp` seconds uint, so it's safe for additional
>> >    time. `wtmpdb` has not implemented a method for reading the old
>> file, nor
>> >    migrating data. see GH issue
>> >    https://github.com/thkukuk/wtmpdb/issues/14
>>
>> As you note, the wtmpdb upstream hasn't prioritised providing an importer
>> for
>> utmp-formatted wtmp logs, doesn't expect to, and even if they did, I
>> don't know
>> if it would be in a form convenient for the purposes of this test.
>>
>> One option could be to restore the tools in an optional package
>> specifically
>> for reading legacy files rather than live system administration - e.g. my
>> merge request on src:sysvinit[1], although I note you have a simpler
>> solution
>> in mind for freeradius!
>>
>
I see this merge is still open, so no `last` command in Debian sid still.
And thank you for the
discussion in util-linux[0]


>
> freeradius upstream has pushed a change to make building and including
> `radlast` as optional
>
> https://github.com/FreeRADIUS/freeradius-server/releases/tag/release_3_2_7
>
> relevant commit:
>
>
> https://github.com/FreeRADIUS/freeradius-server/commit/fece06e2f4984a1f4c227ba9ef3edf4a8ad5e2ee
>
> if Debian wishes to move to 3.2.7 before freeze, we could remove the
> `wtpmdb` dependency.
> If you believe `radlast` is important, package splitting makes sense, but
> we'll also need a `last` command
> provided, and I'm unsure what to suggest as a replacement.
>
>
I see that 3.2.7 got merged, however a choice was made to keep `radlast`
without a `last` command being available.

>   FreeRADIUS will still write the file using the old format, but there is
>  currently no implementation in Debian which can read it.
>
 > For now we install the radlast wrapper script, but it will not work
 > until the last implementation in Debian can read those old-formatted
 > files. See Bug#1094356 and Bug#1095490 for details.
>
 > -- Bernhard Schmidt <berni@debian.org>  Mon, 10 Feb 2025 23:01:18 +0100

as of 3 weeks ago, upstream has removed a bunch of tools[1]

I know it's late in the cycle, and I don't know when the next FreeRADIUS
release will happen. short-term, it may
be worth dropping usr/bin/radlast from d/freeradius-utils.install . This
has already happened upstream
in the linked commit. On the Ubuntu side, with FF for 25.04 Plucky Puffin
happening right now, we are likely
to take this approach, rather than shipping a utility which will return
`not found`

[0] https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=1095490

[1]
https://github.com/FreeRADIUS/freeradius-server/commit/b0f4123c84a0aeaa6fc393fd5e6fdaa0e0a86eaf

>
>
>
>>
>> [1] https://salsa.debian.org/debian/sysvinit/-/merge_requests/14
>>
>
>
> --
>
> John Chittum
>
> Engineering Manager, Ubuntu Engineering, Server
>
> Email: john.chittum@canonical.com
>
>
>
>
>

-- 

John Chittum

Engineering Manager, Ubuntu Engineering, Server

Email: john.chittum@canonical.com

[toc] | [prev] | [standalone]

Back to top | Article view | linux.debian.bugs.dist

csiph-web