Skip to content

Fix resource config generator to correctly generate nested fields - #6366

Queued
andrewnester wants to merge 4 commits into
mainfrom
fix/app-perm-drift
Queued

Fix resource config generator to correctly generate nested fields#6366
andrewnester wants to merge 4 commits into
mainfrom
fix/app-perm-drift

Conversation

@andrewnester

@andrewnester andrewnester commented Aug 24, 2026

Copy link
Copy Markdown
Contributor

Changes

Fix resource config generator to correctly generate nested fields
Fixes #6342

Why

These fields were missed from generation and now correctly counted as input/output only

Tests

Added an acceptance test

Comment thread bundle/direct/dresources/resources.yml Outdated
@eng-dev-ecosystem-bot

eng-dev-ecosystem-bot commented Aug 24, 2026

Copy link
Copy Markdown
Collaborator

Integration test report

Commit: c23ff8f

Run: 32761623180

Env 💚​RECOVERED 🙈​SKIP ✅​pass 🙈​skip Time
💚​ aws linux 1 4 274 1168 10:01
💚​ aws windows 1 4 276 1166 5:47
💚​ azure linux 1 4 273 1168 9:38
💚​ azure windows 1 4 275 1166 6:18
💚​ gcp linux 1 4 274 1168 8:19
💚​ gcp windows 1 4 276 1166 7:19
Test Name aws linux aws windows azure linux azure windows gcp linux gcp windows
💚​ TestAccept 💚​R 💚​R 💚​R 💚​R 💚​R 💚​R
🙈​ TestAccept/bundle/invariant/no_drift 🙈​S 🙈​S 🙈​S 🙈​S 🙈​S 🙈​S
🙈​ TestAccept/bundle/resources/vector_search_endpoints/drift/recreated_same_name 🙈​S 🙈​S 🙈​S 🙈​S 🙈​S 🙈​S
🙈​ TestAccept/bundle/resources/vector_search_indexes/recreate/embedding_dimension 🙈​S 🙈​S 🙈​S 🙈​S 🙈​S 🙈​S
🙈​ TestAccept/ssh/connection 🙈​S 🙈​S 🙈​S 🙈​S 🙈​S 🙈​S
Top 5 slowest tests (at least 2 minutes):
duration env testname
3:10 aws windows TestAccept
3:08 gcp windows TestAccept
2:59 azure windows TestAccept
2:20 azure linux TestImportDir
2:16 aws windows TestFilerWorkspaceFilesExtensionsReadDir

@janniklasrose janniklasrose left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@andrewnester andrewnester changed the title Fix permanent drift on securable_kind in apps Fix resource config generator to correctly generate nested fields Aug 24, 2026
@andrewnester
andrewnester enabled auto-merge August 24, 2026 15:21
@andrewnester
andrewnester added this pull request to the merge queue Aug 24, 2026
Any commits made after this event will not be merged.
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.

bundle plan reports a permanent, unconvergeable diff on apps.*.resources[].uc_securable.securable_kind (direct engine)

3 participants