Repository navigation
fix(ci): refresh native input coverage for Dev releases - #129
Merged
Merged
Conversation
Review all 30 native inputs including project-qualified bucket naming, run external-source coverage without Go test caching, and allow 45 minutes for Dev staging and cold/warm Runner verification. Preserve individual step deadlines and release qualification gates. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
lweberru
marked this pull request as ready for review
October 21, 2026 12:20
lweberru
requested review from
BackInBash,
Breeless,
JoshSTACKIT,
LembDo and
dweezl
as code owners
October 21, 2026 12:20
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
The Dev release after #128 stopped before deployment because
TestCompleteRootVariableInventorystill expected 29 inputs although the reviewedbucket_namingaddition makes 30. Update that coverage check and explicitly verify the new input's type, validated optional status andproject-iddefault.Run the Go tests with
-count=1in PR and release validation so the coverage of external Accelerator source files is checked freshly. The previous green PR check explicitly reported(cached)instead of rerunning this assertion. Increase only the Dev deployment job's aggregate timeout from 30 to 45 minutes: the earlier deployment exhausted the overall deadline while its Runner verification was still running. Individual step deadlines, required checks, qualification rules and environment protections remain intact. Document both details in English and German.Validation: uncached Go adapter tests; all three catalogue/inventory freshness checks; 34 release/image/configuration guards in each of
qualifiedanddev-skip-assistant-qualification; source-bound Assistant catalogue freshness; actionlint for both changed workflows; clean diff check. No native infrastructure inputs or Assistant facts changed.Release evidence: the qualified attempt failed solely on one Model Serving timeout (117/118 cases passed; continuous journey 13/13). Its failure is retained. The explicit Dev exception attempt then exposed the stale 29-input coverage assertion. Both stopped before deployment. After review and merge, the already requested Dev release can resume through the explicit exception with the Assistant enabled and
runner_pool_size=1; no customer Apply is part of this work.Final CI passed on
4beb8c39b2034731f74f03c37e3a07d02ada61be: Validate Configurator. Unchanged native formatting/lint/validation suites are scope-skipped; all applicable checks succeeded. Ready for independent review.