Skip to content

FEAT: Standardizing AIRTInitializer#1578

Merged
ValbuenaVC merged 17 commits intomicrosoft:mainfrom
ValbuenaVC:airt_initializer
Apr 10, 2026
Merged

FEAT: Standardizing AIRTInitializer#1578
ValbuenaVC merged 17 commits intomicrosoft:mainfrom
ValbuenaVC:airt_initializer

Conversation

@ValbuenaVC
Copy link
Copy Markdown
Contributor

@ValbuenaVC ValbuenaVC commented Apr 7, 2026

Description

A very small PR to standardize the AIRT initialization flow.

  • Default storage location is set to an Azure SQL database using AZURE_SQL_DB_CONNECTION_STRING and AZURE_STORAGE_ACCOUNT_DB_DATA_CONTAINER_URL.
  • GLOBAL_MEMORY_LABELS is used to store username and current operation name. For backwards compatibility, these are operator and operation in .pyrit_conf, and username and op_name in the memory labels respectively.
  • Everything propagates through GLOBAL_MEMORY_LABELS. Users seeking to change the operation name can do so by changing the value in .pyrit_conf and re-running the initializer.
  • The initializer requires values for operation and operator in the config to prevent regression.

Tests and Documentation

  • Added basic tests to tests.unit.setup.test_airt_initializer.

Victor Valbuena added 2 commits April 8, 2026 23:19
@ValbuenaVC ValbuenaVC marked this pull request as ready for review April 8, 2026 23:23
@ValbuenaVC ValbuenaVC changed the title [DRAFT] FEAT: Standardizing AIRTInitializer FEAT: Standardizing AIRTInitializer Apr 8, 2026
Victor Valbuena added 2 commits April 8, 2026 23:34
Copy link
Copy Markdown
Contributor

@behnam-o behnam-o left a comment

Choose a reason for hiding this comment

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

some little changes/suggestions

Copy link
Copy Markdown
Contributor

@behnam-o behnam-o left a comment

Choose a reason for hiding this comment

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

LGTM! Thanks Victor!

@ValbuenaVC ValbuenaVC merged commit 30dc0b0 into microsoft:main Apr 10, 2026
30 checks passed
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.

5 participants