workers-sdk icon indicating copy to clipboard operation
workers-sdk copied to clipboard

CC-5749 Drop deprecated containers observability.logging field

Open ichernetsky-cf opened this issue 1 month ago • 4 comments

Fixes CC-5749.

This is to remove a field that was deprecated many months ago and not returned by server anymore.


  • Tests
    • [ ] Tests included
    • [x] Tests not necessary because: Some tests that use the field in question are actually dropped too.
  • Public documentation
    • [ ] Cloudflare docs PR(s):
    • [x] Documentation not necessary because: This field is not used in the documentation.
  • Wrangler V3 Backport
    • [ ] Wrangler PR:
    • [x] Not necessary because: Containers are not supported in V3.

ichernetsky-cf avatar Nov 26 '25 17:11 ichernetsky-cf

🦋 Changeset detected

Latest commit: a53762000d33c5be1e75d9ac3b2593efcdbec38e

The changes in this PR will be included in the next version bump.

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

changeset-bot[bot] avatar Nov 26 '25 17:11 changeset-bot[bot]

create-cloudflare

npm i https://pkg.pr.new/create-cloudflare@11437
@cloudflare/kv-asset-handler

npm i https://pkg.pr.new/@cloudflare/kv-asset-handler@11437
miniflare

npm i https://pkg.pr.new/miniflare@11437
@cloudflare/pages-shared

npm i https://pkg.pr.new/@cloudflare/pages-shared@11437
@cloudflare/unenv-preset

npm i https://pkg.pr.new/@cloudflare/unenv-preset@11437
@cloudflare/vite-plugin

npm i https://pkg.pr.new/@cloudflare/vite-plugin@11437
@cloudflare/vitest-pool-workers

npm i https://pkg.pr.new/@cloudflare/vitest-pool-workers@11437
@cloudflare/workers-editor-shared

npm i https://pkg.pr.new/@cloudflare/workers-editor-shared@11437
@cloudflare/workers-utils

npm i https://pkg.pr.new/@cloudflare/workers-utils@11437
wrangler

npm i https://pkg.pr.new/wrangler@11437

commit: b0d6116

pkg-pr-new[bot] avatar Nov 26 '25 17:11 pkg-pr-new[bot]

@edmundhung I have re-run the Windows CI job a dozen times to no avail. Its failures do not seem to be related to this change. Should I keep re-running it? I can at least try rebasing my PR at more time

ichernetsky-cf avatar Dec 02 '25 16:12 ichernetsky-cf

@ichernetsky-cf Let's rebase it once more. We have merged some fixes to make tests less flaky yesterday and also this morning. Sorry for the trouble!

cc: @jamesopstad

edmundhung avatar Dec 02 '25 16:12 edmundhung