Repository navigation
test: add protected DEV IAM catalog verification - #1854
Closed
Jacky-Pham wants to merge 3 commits into
Closed
Jacky-Pham wants to merge 3 commits into
Jacky-Pham wants to merge 3 commits into
Conversation
|
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.



DEV runtime metadata and local mocks do not establish a successful Cloud SQL IAM login or effective database grants. Add a manual verifier using the existing STRR GitHub → tools Cloud Build route and the actual shared connector for the fixed DEV API/job identities.
PRs run offline checks only. The cloud job requires manual dispatch from protected
bcgov/STRRmain in the existing DEV environment. It checks the WIF/account route and actual worker identity, uses 600-second impersonated credentials, then reads database identity, server/extension versions and catalog privileges in a read-only transaction. Output contains allowlisted metadata/error classes;loginVerifiedand conservativewriteCatalogReadyare separate, andconsumersVerifiedremains false. No business rows, product jobs, SQL writes, grants, deployment commands or arbitrary target/ref inputs are exposed.The validation job has no cloud credentials or OIDC permission. It builds the commit/hash-pinned common helper into a fixed directory, verifies its connector module, tests the wheel and uploads it with a checksum manifest. The cloud job takes only that immutable same-run artifact, validates its identity/checksum and embeds it in a no-source build. All 31 registry dependencies are version/hash locked and installed as wheels; the cloud build performs no source-package builds. The Python image and Google auth/setup actions are pinned by digest/commit.
Review gates
Validation
Final CI
At
c242846a, all checks are terminal: seven successes, five neutral frontend checks and the intentionally skipped liveverifyjob. Offline validation, all CodeQL languages and Sonar pass; Sonar reports zero issues. Validation run, CodeQL run.Related: #1763, #1764, #1853.