chore(pulumi-2.22.3): final minor cleanup from code review

Minor bookkeeping items from the milestone-wide code review:

- Add inline comment on s3-replication QoS policy documenting the par5-only
  scope choice (mirror on pa7 if dual-sided throttling is required).
- Document schema.json.functions = 41 vs prov.DataSources = 40 rationale:
  the extra entry is pulumi:providers:flashblade/terraformConfig, a
  provider-level function injected by the bridge, not a data source.
- Add CONVENTIONS.md exception for bridge tests: TestProviderInfo_* is
  the approved pattern for tests under pulumi/provider/ (not TestUnit_).

Skipped items (with rationale):
- M1 go directives: pulumi/sdk/v3 v3.231.0 requires go >= 1.25.8, so
  pulumi/provider/go.mod correctly uses 1.25.8 (not aspirational — it is
  the minimum enforced by the toolchain). pulumi/sdk/go/go.mod uses 1.22
  (bridge generator default, preserved across regeneration by Makefile
  bak/restore). Root go.mod stays at 1.25.0 (TF provider, no Pulumi SDK
  dep). No alignment needed.
- go.work: go work sync triggers deep dep updates across all example
  go.mod files — too invasive for a minor cleanup item. IDE-only benefit
  does not justify the churn.

Author: SoulKyu

CONVENTIONS.md

@@ -128,6 +128,8 @@ Reference: `internal/provider/server_resource.go` (v0→v1→v2).
 
 `TestUnit_<Resource>_<Operation>[_<Variant>]` — e.g. `TestUnit_Target_Get_Found`, `TestUnit_TargetResource_Lifecycle`, `TestUnit_TargetDataSource_Basic`.
 
+**Exception — bridge tests:** tests under `pulumi/provider/` use the `TestProviderInfo_*` pattern because they test the Pulumi bridge ProviderInfo configuration, not individual TF resource/data source logic. This naming is intentional and approved.
+
 ### Client tests
 
 `httptest.NewServer()`, mock `/api/login` for `x-auth-token`, use `newTestClient(t, srv)`. One test per CRUD + edge cases.

pulumi/examples/s3-replication-go/main.go

@@ -655,6 +655,8 @@ func main() {
 
 		// ------------------------------------------------------------------
 		// Step 14: QoS policy (optional)
+		// QoS policy is managed centrally on the par5 side for this example.
+		// Mirror on pa7 if dual-sided throttling is required.
 		// ------------------------------------------------------------------
 		if qos != nil {
 			_, err := flashblade.NewQosPolicy(ctx, "this", &flashblade.QosPolicyArgs{

pulumi/examples/s3-replication-py/__main__.py

@@ -520,6 +520,8 @@
 
 # ---------------------------------------------------------------------------
 # Step 14: QoS policy (optional)
+# QoS policy is managed centrally on the par5 side for this example.
+# Mirror on pa7 if dual-sided throttling is required.
 # ---------------------------------------------------------------------------
 if qos:
     flashblade.QosPolicy("this",

pulumi/provider/resources_test.go

@@ -14,6 +14,10 @@ import (
 
 // Expected counts. Matches TF provider registrations (54 resources, 40 data sources).
 // Update when TF provider resource set changes.
+//
+// Note: schema.json contains 41 entries under "functions" — the extra entry is
+// "pulumi:providers:flashblade/terraformConfig", a provider-level function
+// injected by the bridge, not a data source. prov.DataSources = 40 is correct.
 const (
 	expectedResources   = 54
 	expectedDataSources = 40