Skip to content

Merge main into staging-0.9.0 - #7015

Open
martinflorian-da wants to merge 51 commits into
staging-0.9.0from
martinflorian-da/cni-2106-sync-0.9.0
Open

Merge main into staging-0.9.0#7015
martinflorian-da wants to merge 51 commits into
staging-0.9.0from
martinflorian-da/cni-2106-sync-0.9.0

Conversation

@martinflorian-da

Copy link
Copy Markdown
Contributor

Wanted for the the follow-up PR to #6957 (see https://github.com/DACH-NY/canton-network-internal/issues/2106)

Pull Request Checklist

Cluster Testing

  • If a cluster test is required, comment /cluster_test on this PR to request it, and ping someone with access to the DA-internal system to approve it.
  • If an upgrade test is required, comment /upgrade_test on this PR to request it, and ping someone with access to the DA-internal system to approve it.
  • If a hard-migration test is required (from the latest release), comment /hdm_test on this PR to request it, and ping someone with access to the DA-internal system to approve it.
  • If a logical synchronizer upgrade test is required (from canton-3.5), comment /lsu_test on this PR to request it, and ping someone with access to the DA-internal system to approve it.

PR Guidelines

  • Include any change that might be observable by our partners or affect their deployment in the release notes.
  • Specify fixed issues with Fixes #n, and mention issues worked on using #n
  • Include a screenshot for frontend-related PRs - see README or use your favorite screenshot tool

Merge Guidelines

  • Make the git commit message look sensible when squash-merging on GitHub (most likely: just copy your PR description).

kajalshah-da and others added 30 commits August 21, 2026 18:24
* Partition overflow problem solved per issue #6919 [ci]

Signed-off-by: Kajal <kajal.shah@digitalasset.com>
Signed-off-by: Robert Autenrieth <robert.autenrieth@digitalasset.com>
[static]

Signed-off-by: Nicu Reut <nicu.reut@digitalasset.com>
Signed-off-by: Puneet Bharti <puneet.bharti@finoa.io>
Co-authored-by: Puneet Bharti <puneet.bharti@finoa.io>
…eployment on prod clusters (#6908)

Signed-off-by: Mateusz Błażejewski <mateusz.blazejewski@digitalasset.com>
Signed-off-by: Vladislav Lapin <v.lapin@scalablesolutions.io>
Co-authored-by: Vladislav Lapin <v.lapin@scalablesolutions.io>
…t-yet-ingested (#6694)

Signed-off-by: Tim Emiola <adetokunbo@emio.la>
Fixes #4265

This removes `RestartDsoDelegateBasedAutomationTrigger` and updates `SvDsoAutomationService` to construct and start the `DsoDelegateBasedAutomationService`.

Signed-off-by: Matt Dziuban <mrdziuban@gmail.com>
I thought about this a bit too late / hit merge on #6935 a bit too quickly...

[force] (is fine because that log message doesn't exist anymore)

Signed-off-by: Martin Florian <martin.florian@digitalasset.com>
…tionId` (#6744)

* Replace all `(Long, CantonTimestamp)` tuples with `TimestampWithMigrationId`.
* Fix regex to ignore sbt GC warning.

Signed-off-by: Matt Dziuban <mrdziuban@gmail.com>
…#6949)

[static]

Signed-off-by: Mateusz Błażejewski <mateusz.blazejewski@digitalasset.com>
[ci]

Signed-off-by: moritz.kiefer@digitalasset.com <moritz.kiefer@purelyfunctional.org>
Fixes #6640

---------

Signed-off-by: krzysztofczyz-da <krzysztof.czyz@digitalasset.com>
[ci]

Signed-off-by: moritz.kiefer@digitalasset.com <moritz.kiefer@purelyfunctional.org>
…6955)

Signed-off-by: Puneet Bharti <puneet.bharti@finoa.io>
Signed-off-by: Itai Segall <itai.segall@digitalasset.com>
* Revert "add rate limits by CIDRs (#6660)"

This reverts commit 5b48337.

* [static] run

Signed-off-by: Oriol Muñoz <oriol.munoz@digitalasset.com>

---------

Signed-off-by: Oriol Muñoz <oriol.munoz@digitalasset.com>
Signed-off-by: Itai Segall <itai.segall@digitalasset.com>
Co-authored-by: Stephen Compall <stephen.compall@digitalasset.com>
Signed-off-by: Itai Segall <itai.segall@digitalasset.com>
Co-authored-by: Amanda L Martin <hythloda@gmail.com>
Co-authored-by: Stephen Compall <stephen.compall@digitalasset.com>
Signed-off-by: Ilya Silvestrov <tellary@gmail.com>
include target host in legend and add error table with flexible filter

[static]

Signed-off-by: Nicu Reut <nicu.reut@digitalasset.com>
…ers (#6964)

We don't keep track of usage for the actual attribute values so the overall reported usage is actually the global usage

[static]

Signed-off-by: Nicu Reut <nicu.reut@digitalasset.com>
…utting-down canton (#6958)

---------

Signed-off-by: Oriol Muñoz <oriol.munoz@digitalasset.com>
See DACH-NY/canton-network-internal#6371

Regular alerts for now, plan is to promote at least some of them to sev-2 and/or sev-1 after some testing.

- Discarded requests / replays <- also adds this as panels on the Global Synchronizer Utilization dashboard
- Failed confirmation requests
- TPS drop
* Expose sv app endpoints to the minimum users

Also extend open api so the other SVs can easily apply the same restriction

[static]

Signed-off-by: Nicu Reut <nicu.reut@digitalasset.com>
[ci]

fixes #6970

Signed-off-by: moritz.kiefer@digitalasset.com <moritz.kiefer@purelyfunctional.org>
Co-authored-by: Moritz Kiefer <moritz.kiefer@purelyfunctional.org>
Signed-off-by: Robert Autenrieth <robert.autenrieth@digitalasset.com>
puneetfinoa and others added 21 commits August 26, 2026 12:52
Signed-off-by: Puneet Bharti <puneet.bharti@finoa.io>
Fixes #2336

Since #2336 was filed, #4113 deprecated `scan_txlog.py` and removed the code that ran it, but #5922 added a max tap amount of 100,000,000, so the existing test code failed with a "requirement ... was not met" error. This updates the test code to instead tap the max amount.

Signed-off-by: Matt Dziuban <mrdziuban@gmail.com>
Signed-off-by: Kajal <kajal.shah@digitalasset.com>
…6978)

---------

Signed-off-by: Oriol Muñoz <oriol.munoz@digitalasset.com>
* add the configs

Signed-off-by: Itai Segall <itai.segall@digitalasset.com>

* [ci] done

Signed-off-by: Itai Segall <itai.segall@digitalasset.com>

* [ci] fmt

Signed-off-by: Itai Segall <itai.segall@digitalasset.com>

* [ci] cleanup

Signed-off-by: Itai Segall <itai.segall@digitalasset.com>

* [ci] fix the test

Signed-off-by: Itai Segall <itai.segall@digitalasset.com>

* [ci]

Signed-off-by: Itai Segall <itai.segall@digitalasset.com>

---------

Signed-off-by: Itai Segall <itai.segall@digitalasset.com>
That warning doesn't exist in Canton anymore, see https://github.com/DACH-NY/canton/pull/30310/changes#diff-81c61bdb4b62b057717883fd710a5f662ee23d6769f03115d2db110dfda5829aL127

[ci]

Fixes #2706

Signed-off-by: Martin Florian <martin.florian@digitalasset.com>
---------

Signed-off-by: Oriol Muñoz <oriol.munoz@digitalasset.com>
)

And also the dashboard panel, and add a new panel that shows duplicate confirmation requests.

Follow-up to #6969

[static]

Signed-off-by: Martin Florian <martin.florian@digitalasset.com>
[ci]

Signed-off-by: Nicu Reut <nicu.reut@digitalasset.com>
[ci]

Signed-off-by: moritz.kiefer@digitalasset.com <moritz.kiefer@purelyfunctional.org>
Co-authored-by: moritz.kiefer@digitalasset.com <moritz.kiefer@purelyfunctional.org>
---------

Signed-off-by: Oriol Muñoz <oriol.munoz@digitalasset.com>
[static]

Signed-off-by: Nicu Reut <nicu.reut@digitalasset.com>
Signed-off-by: Oriol Muñoz <oriol.munoz@digitalasset.com>
* add port 80 to helm VirtualServices

* direct GKE gateway to port 80 on istio

* also port 80 for cn-release-bundles and docs routes

* Cloud Armor got more restrictive about condition regexes

* fix sequencer gRPC ingress by forcing HTTP/2 in istio gateway chart

- found with docker-compose validator, doing this

    SEQ_HOST=sequencer-0.sv-2.$GCP_CLUSTER_HOSTNAME
    curl -sv --http2 \
        -H 'content-type: application/grpc' \
        -H 'te: trailers' \
        --data-binary $'\x00\x00\x00\x00\x00' \
        "https://${SEQ_HOST}/grpc.health.v1.Health/Check" \
        -o /dev/null

yields

    < HTTP/2 200
    < content-type: application/grpc
    < grpc-status: 14
    < grpc-message: upstream connect error or disconnect/reset before headers. reset reason: protocol error
    < server: istio-envoy
    < x-envoy-upstream-service-time: 3
    < via: 1.1 google

whereas with GKE L7 off we get

    < HTTP/2 200
    < content-type: application/grpc
    < grpc-status: 13
    < grpc-message: Half-closed without a request
    < x-envoy-upstream-service-time: 4
    < date: Mon, 24 Aug 2026 22:17:07 GMT
    < server: istio-envoy

---------

Assisted-by: Copilot:gpt-5.3-codex
Signed-off-by: Stephen Compall <stephen.compall@digitalasset.com>
Signed-off-by: Itai Segall <itai.segall@digitalasset.com>
Signed-off-by: Itai Segall <itai.segall@digitalasset.com>
Signed-off-by: Puneet Bharti <puneet.bharti@finoa.io>
* [splice] bump resources for deployment tests

Signed-off-by: Itai Segall <itai.segall@digitalasset.com>

* [static]

Signed-off-by: Itai Segall <itai.segall@digitalasset.com>

---------

Signed-off-by: Itai Segall <itai.segall@digitalasset.com>
Signed-off-by: Itai Segall <itai.segall@digitalasset.com>
...and use the scan version of that endpoint for SV onboarding.

Main part of DACH-NY/canton-network-internal#2106

Signed-off-by: Martin Florian <martin.florian@digitalasset.com>
Wanted for the the follow-up PR to #6957 (see DACH-NY/canton-network-internal#2106)

[ci]

Signed-off-by: Martin Florian <martin.florian@digitalasset.com>
@github-actions

Copy link
Copy Markdown

Deploy upgrade pipeline triggered for Commit b134f89fcc20155ecbe64c36fd3658c5d8c825ed in , please contact a Contributor to approve it in CircleCI: https://app.circleci.com/pipelines/github/DACH-NY/canton-network-internal/80187

@canton-network-da

Copy link
Copy Markdown
Contributor

[backport] Reminder

Please consider backporting to the following branches:

  • main
  • release-line-0.7.5
  • release-line-0.7.4
  • release-line-0.7.3
  • release-line-0.7.1

▶️ Please check the boxes for branches that you wish to backport to and backport PRs will
automatically be created when you merge this PR.

And your PR is currently against base branch: staging-0.9.0.

Note: Any PR comment containing [backport] will be considered for auto-backporting upon merge,
you can always add those manually for PRs that did not get these reminders. You can also edit
this comment manually and add more branches that this should be backported to.

@martinflorian-da
martinflorian-da marked this pull request as ready for review August 28, 2026 14:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.