Re: [PATCH 0/3] Add support for UUIDv7
Hi Tim, On Tue, Apr 23, 2024 at 09:03:32PM +0200, Tim Düsterhus wrote: > Hi > > On 4/19/24 21:07, Willy Tarreau wrote: > > it! I'll have a look on Monday, I'm really done for this week, need to > > Monday is gone. So here's a friendly reminder :-) Yeah and I'm sorry, my week started with two day-long meetings, I met my keyboard first around 6pm :-/ I'm doing my best for tomorrow morning, I promis I'm not forgetting you (nor David who also sent something). Thanks, Willy
Re: [PATCH 0/3] Add support for UUIDv7
Hi On 4/19/24 21:07, Willy Tarreau wrote: it! I'll have a look on Monday, I'm really done for this week, need to Monday is gone. So here's a friendly reminder :-) Best regards Tim Düsterhus
Re: [PATCH 0/3] Add support for UUIDv7
Hi Tim! On Fri, Apr 19, 2024 at 09:01:24PM +0200, Tim Duesterhus wrote: > Willy, > > as requested in the thread "[ANNOUNCE] haproxy-3.0-dev7": > > > Regarding UUIDs, though, I've recently come across UUIDv7 which I found > > particularly interesting, and that I think would be nice to implement > > in the uuid() sample fetch function before 3.0 is released. > > No reg-tests added, as those doesn't allow meaningfully testing that the > UUIDv7 is actually a UUIDv7. I have manually checked the output against > https://uuid7.com/. Oh, many thanks for this, it was still on my todo list and I sense that the review of your patches will take me less effort than implementing it! I'll have a look on Monday, I'm really done for this week, need to have some sleep. Thanks and have a nice week-end! Willy
[PATCH 0/3] Add support for UUIDv7
Willy, as requested in the thread "[ANNOUNCE] haproxy-3.0-dev7": > Regarding UUIDs, though, I've recently come across UUIDv7 which I found > particularly interesting, and that I think would be nice to implement > in the uuid() sample fetch function before 3.0 is released. No reg-tests added, as those doesn't allow meaningfully testing that the UUIDv7 is actually a UUIDv7. I have manually checked the output against https://uuid7.com/. Best regards Tim Duesterhus (3): MINOR: tools: Rename `ha_generate_uuid` to `ha_generate_uuid_v4` MINOR: Add `ha_generate_uuid_v7` MINOR: Add support for UUIDv7 to the `uuid` sample fetch addons/ot/src/scope.c | 2 +- doc/configuration.txt | 3 ++- include/haproxy/tools.h | 3 ++- src/flt_spoe.c | 2 +- src/sample.c| 42 + src/tools.c | 29 ++-- 6 files changed, 63 insertions(+), 18 deletions(-) -- 2.43.2
