Skip to content

[AZINTS-4618] CAJ migration script - delete old resources - #234

Open
parsons90 wants to merge 7 commits into
rebecca.parsons/AZINTS-4618/caj_migration_script_4from
rebecca.parsons/AZINTS-4618/caj_migration_script_5
Open

[AZINTS-4618] CAJ migration script - delete old resources#234
parsons90 wants to merge 7 commits into
rebecca.parsons/AZINTS-4618/caj_migration_script_4from
rebecca.parsons/AZINTS-4618/caj_migration_script_5

Conversation

@parsons90

@parsons90 parsons90 commented Aug 14, 2026

Copy link
Copy Markdown
Contributor
  1. Add deletion of old resources as the last step in the script. If the deletion fails, we should not rollback the script, but log warnings to customers to delete the resources.
  2. Fix a bug in the pause/restart of the deployer task. Azure has a bug where pausing a scheduled CAJ can sometimes put it into a state where it can't be restarted. We need to modify the cron schedule to pause it.
  3. Fix a bug where the LOG_LEVEL, DD_TELEMETRY, and DD_SITE vars were not being propagated to the new CAJs
  4. Fix a bug in the role assignment rollback where it would delete the resource groups.

testing

Migrated an LFO. The resulting tasks run successfully
Screenshot 2026-08-17 at 12 54 37 PM
Screenshot 2026-08-17 at 12 36 34 PM

@parsons90 parsons90 changed the title Rebecca.parsons/azints 4618/caj migration script 5 [AZINTS-4618] CAJ migration script - delete old resources Aug 17, 2026
@parsons90
parsons90 marked this pull request as ready for review August 17, 2026 16:59
@parsons90
parsons90 requested a review from a team as a code owner August 17, 2026 16:59
@parsons90
parsons90 requested a review from benblaustein August 17, 2026 16:59

@benblaustein benblaustein left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Looks good to me, I think these tests are not getting included in CI though.

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.

2 participants