[release-line-3.3] Update 2025-05-06.23

Reference commit: 4f9bd42580

Author: canton-machine

UNRELEASED.md

@@ -1949,8 +1949,7 @@ object CantonConfig {
         .foldLeft(c) { case (subConfig, (key, obj)) =>
           subConfig.withValue(key, goVal(key, obj))
         }
-    go(config)
-      .resolve()
+    go(config.resolve()) // Resolve the config _before_ redacting confidential information
       .root()
       .get("canton")
       .render(CantonConfig.defaultConfigRenderer)

community/app-base/src/main/scala/com/digitalasset/canton/config/CantonConfig.scala

@@ -1,4 +1,4 @@
-sdk-version: 3.3.0-snapshot.20250415.13756.0.vafc5c867
+sdk-version: 3.3.0-snapshot.20250502.13767.0.v2fc6c7e2
 build-options: 
 - --target=2.dev
 name: CantonLfDev

community/app/src/test/daml/CantonLfDev/daml.yaml

@@ -1,4 +1,4 @@
-sdk-version: 3.3.0-snapshot.20250415.13756.0.vafc5c867
+sdk-version: 3.3.0-snapshot.20250502.13767.0.v2fc6c7e2
 build-options: 
 - --target=2.1
 - --enable-interfaces=yes

community/app/src/test/daml/CantonLfV21/daml.yaml

@@ -1,4 +1,4 @@
-sdk-version: 3.3.0-snapshot.20250415.13756.0.vafc5c867
+sdk-version: 3.3.0-snapshot.20250502.13767.0.v2fc6c7e2
 build-options: 
 - --target=2.1
 name: CantonTests

community/app/src/test/daml/CantonTest/daml.yaml

@@ -1,4 +1,4 @@
-sdk-version: 3.3.0-snapshot.20250415.13756.0.vafc5c867
+sdk-version: 3.3.0-snapshot.20250502.13767.0.v2fc6c7e2
 build-options: 
 - --target=2.dev
 name: CantonTestsDev

community/app/src/test/daml/CantonTestDev/daml.yaml

@@ -95,7 +95,7 @@ trait SequencerOnboardingTombstoneTest
     participant1.ledger_api.javaapi.commands.submit_async(
       Seq(participant1.id.adminParty),
       cycle,
-      commandId = "commandId",
+      commandId = "long-running-tx-id",
     )
 
     // Make sure that the participant's request has reached the sequencer
@@ -146,7 +146,7 @@ trait SequencerOnboardingTombstoneTest
       loggerFactory.assertLogsUnorderedOptional(
         {
 
-          clue("participant1 connects to sequencer2") {
+          clue("participant1 connects to sequencer2 the first time") {
             participant1.synchronizers.reconnect_all(ignoreFailures = false)
           }
 
@@ -248,7 +248,7 @@ trait SequencerOnboardingTombstoneTest
         sequencer2.sequencerConnection.withAlias(SequencerAlias.tryCreate("seq2x")),
       )
 
-      clue("participant1 connects to sequencer2") {
+      clue("participant1 connects to sequencer2 the second time") {
         participant1.synchronizers.reconnect_all(ignoreFailures = false)
       }
 

community/app/src/test/scala/com/digitalasset/canton/integration/tests/bftsynchronizer/SequencerOnboardingTombstoneTest.scala

@@ -8,7 +8,11 @@ import com.digitalasset.canton.config
 import com.digitalasset.canton.config.DbConfig
 import com.digitalasset.canton.console.CommandFailure
 import com.digitalasset.canton.console.commands.SynchronizerChoice
-import com.digitalasset.canton.integration.plugins.{UseCommunityReferenceBlockSequencer, UseH2}
+import com.digitalasset.canton.integration.plugins.{
+  UseCommunityReferenceBlockSequencer,
+  UseH2,
+  UsePostgres,
+}
 import com.digitalasset.canton.integration.{
   CommunityIntegrationTest,
   ConfigTransforms,
@@ -112,7 +116,7 @@ class SimpleFunctionalNodesTestH2 extends SimpleFunctionalNodesTest {
   registerPlugin(new UseCommunityReferenceBlockSequencer[DbConfig.H2](loggerFactory))
 }
 
-//class SimpleFunctionalNodesTestPostgres extends SimpleFunctionalNodesTest {
-//  registerPlugin(new UsePostgres(loggerFactory))
-//  registerPlugin(new UseReferenceBlockSequencer[DbConfig.Postgres](loggerFactory))
-//}
+class SimpleFunctionalNodesTestPostgres extends SimpleFunctionalNodesTest {
+  registerPlugin(new UsePostgres(loggerFactory))
+  registerPlugin(new UseCommunityReferenceBlockSequencer[DbConfig.Postgres](loggerFactory))
+}

community/app/src/test/scala/com/digitalasset/canton/integration/tests/bftsynchronizer/SimpleFunctionalNodesTest.scala

@@ -119,13 +119,13 @@ trait SequencerPruningIntegrationTest extends CommunityIntegrationTest with Shar
   }
 
   protected val pruningRegexWithTrafficPurchase =
-    """Removed at least ([1-9]\d*) events, at least (\d+) payloads, at least ([1-9]\d*) counter checkpoints"""
+    """Removed at least ([1-9]\d*) events, at least (\d+) payloads"""
 
   protected val pruningRegex =
-    """Removed at least ([1-9]\d*) events, at least (\d+) payloads, at least ([1-9]\d*) counter checkpoints"""
+    """Removed at least ([1-9]\d*) events, at least (\d+) payloads"""
 
   protected val pruningNothing =
-    """Removed at least 0 events, at least 0 payloads, at least 0 counter checkpoints"""
+    """Removed at least 0 events, at least 0 payloads"""
 
   "prune only removes events up the point where all enabled clients have acknowledgements" in {
     implicit env =>

community/app/src/test/scala/com/digitalasset/canton/integration/tests/pruning/SequencerPruningIntegrationTest.scala

@@ -37,8 +37,8 @@ import com.digitalasset.canton.sequencing.protocol.{
 import com.digitalasset.canton.sequencing.traffic.TrafficReceipt
 import com.digitalasset.canton.store.SequencedEventStore.{
   LatestUpto,
-  OrdinarySequencedEvent,
   PossiblyIgnoredSequencedEvent,
+  SequencedEventWithTraceContext,
 }
 import com.digitalasset.canton.synchronizer.sequencer.errors.SequencerError.InvalidAcknowledgementTimestamp
 import com.digitalasset.canton.topology.MediatorGroup.MediatorGroupIndex
@@ -231,7 +231,7 @@ trait IgnoreSequencedEventsIntegrationTest extends CommunityIntegrationTest with
         participant1.health.ping(participant1)
       }
 
-      // TODO(#11834): Ignoring future events is incompatible with the counter based event ignoring/unignoring APIs,
+      // TODO(#25162): Ignoring future events is incompatible with the counter based event ignoring/unignoring APIs,
       //  because the future timestamp are unknown unlike the counters. Need to consider and implement
       //  a new timestamp-based API for the use case of ignoring future events, should it still be necessary.
       "insert an empty ignored event, therefore ignore the next ping and then successfully ping again" ignore {
@@ -305,8 +305,7 @@ trait IgnoreSequencedEventsIntegrationTest extends CommunityIntegrationTest with
         // Choose DeliverError as type of tampered event, because we don't expect DeliverErrors to be stored
         // as part of the previous tests.
         val tamperedEvent = DeliverError.create(
-          lastStoredEvent.counter,
-          None, // TODO(#11834): Make sure that ignored sequenced events works with previous timestamps
+          None,
           lastStoredEvent.timestamp,
           daId,
           MessageId.tryCreate("schnitzel"),
@@ -315,7 +314,7 @@ trait IgnoreSequencedEventsIntegrationTest extends CommunityIntegrationTest with
           Option.empty[TrafficReceipt],
         )
         val tracedSignedTamperedEvent =
-          OrdinarySequencedEvent(lastEvent.copy(content = tamperedEvent))(traceContext)
+          SequencedEventWithTraceContext(lastEvent.copy(content = tamperedEvent))(traceContext)
 
         // Replace last event by the tamperedEvent
         val p1Node = participant1.underlying.value
@@ -504,7 +503,7 @@ trait IgnoreSequencedEventsIntegrationTest extends CommunityIntegrationTest with
         participant1.repair.ignore_events(
           daId,
           lastRequestSequencerCounter,
-          // TODO(#11834): This ignores the future event, which is incompatible with previous timestamps.
+          // TODO(#25162): This ignores the future event, which is incompatible with previous timestamps.
           //  The test work probably because the result message is ignored without prior confirmation request.
           //  Need to check if that is good enough and if we don't need to extend event ignoring API
           //  to support ignoring "future" timestamps.