Skip to content

feat: Add --aviator-app-mapping option to ssc bulkaudit action#939

Open
kireetivar wants to merge 1 commit intofortify:feat/v3.x/aviator/26.2from
kireetivar:p/cdatla/bulk_audit_mapping
Open

feat: Add --aviator-app-mapping option to ssc bulkaudit action#939
kireetivar wants to merge 1 commit intofortify:feat/v3.x/aviator/26.2from
kireetivar:p/cdatla/bulk_audit_mapping

Conversation

@kireetivar
Copy link
Collaborator

@kireetivar kireetivar commented Mar 4, 2026

This PR enhances the SSC bulkaudit action to support varying Aviator application mapping strategies and improves the visibility of dry-run operations.

Key Changes

  • Configurable Mapping: Replaced the previous mapping logic with a formal --aviator-app-mapping option.
    • app (default): Maps one Aviator app per SSC Project.
    • version: Maps one Aviator app per SSC Version (Project__Version).
  • Logging Improvements:
    • Moved dry-run logs (e.g., "Would create app...", "Would audit...") from log.progress to log.info.
    • Reason: log.progress writes to console and is often filtered out or not captured in log files(previously these are available in the log file). Since dry-runs are intended to provide an audit trail of planned actions, log.info (writing to stdout) ensures these messages stay visible in terminal.

output:

> java -jar .\fcli.jar ssc action run bulkaudit --add-aviator-tags -n --log-file log1.txt --aviator-app-mapping version -f 293aa1ce-6897-462c-b96f-bb76e01cab83:b0189a87-93cf-4cbe-9251-7fe395ada831

START: aviator_apps
SUCCESS: aviator_apps
Would prepare tags for avtest:1.0
Would audit avtest:1.0
Dry-run complete (mapping: version) - would process 1 versions and create 0 apps
> java -jar .\fcli.jar ssc action run bulkaudit --add-aviator-tags --log-file log1.txt --aviator-app-mapping version -f 293aa1ce-6897-462c-b96f-bb76e01cab83:b0189a87-93cf-4cbe-9251-7fe395ada831   
START: aviator_apps
SUCCESS: aviator_apps
START: prepare_tags
 Status    Entity                Details                                                Action   
 VERIFIED  Custom Tag            'Aviator prediction' is already configured correctly.  PREPARED
 VERIFIED  Custom Tag            'Aviator status' is already configured correctly.      PREPARED
 SKIPPED   Application Versions  All 1 targeted version(s) were already configured.     PREPARED


SUCCESS: prepare_tags
START: run_audit
 Id      Application name  Name    Artifact id  Action  
 38      avtest            1.0     380          AUDITED

@kireetivar kireetivar self-assigned this Mar 4, 2026
@kireetivar kireetivar marked this pull request as ready for review March 5, 2026 05:24
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.

1 participant