Skip to content

Update stacks resources to match upstream serializer changes #1220

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
merged 9 commits into from
Sep 23, 2025
Merged

Update stacks resources to match upstream serializer changes #1220

merged 9 commits into from
Sep 23, 2025
+328 −132

Conversation

ctrombley
Copy link
Collaborator

@ctrombley ctrombley commented Sep 15, 2025 •
edited
Loading

Description

This PR updates the stacks domain models to match new serializers being used by the API.

Testing plan

Integration tests should cover it.

Output from tests

❯ envchain local gotestsum --format=testname -- -v -run TestStack
PASS TestStackConfigurationList/with_pagination_options (0.15s)
PASS TestStackConfigurationList (10.11s)
PASS TestStackConfigurationCreateUploadAndRead (3.60s)
PASS TestStackConfigurationSummaryList/Successful_empty_list (0.11s)
PASS TestStackConfigurationSummaryList/Successful_multiple_config_summary_list (0.11s)
PASS TestStackConfigurationSummaryList/Unsuccessful_list (0.00s)
PASS TestStackConfigurationSummaryList (10.88s)
PASS TestStackDeploymentGroupsList/List_with_valid_stack_configuration_ID (0.41s)
PASS TestStackDeploymentGroupsList/List_with_invalid_stack_configuration_ID (0.00s)
PASS TestStackDeploymentGroupsList/List_with_pagination (0.23s)
PASS TestStackDeploymentGroupsList (17.79s)
PASS TestStackDeploymentGroupsRead/Read_with_valid_ID (0.10s)
PASS TestStackDeploymentGroupsRead/Read_with_invalid_ID (0.00s)
PASS TestStackDeploymentGroupsRead (16.04s)
PASS TestStackDeploymentGroupsApproveAllPlans/Approving_all_plans (0.15s)
PASS TestStackDeploymentGroupsApproveAllPlans (17.88s)
PASS TestStackDeploymentGroupsRerun/No_deployments_specified_for_rerun (0.00s)
PASS TestStackDeploymentGroupsRerun/Rerun_with_invalid_ID (0.00s)
PASS TestStackDeploymentGroupsRerun (26.17s)
PASS TestStackDeploymentGroupSummaryList/Successful_multiple_deployment_group_summary_list (0.15s)
PASS TestStackDeploymentGroupSummaryList/Unsuccessful_list (0.00s)
PASS TestStackDeploymentGroupSummaryList (38.00s)
PASS TestStackDeploymentRunsList/List_without_options (0.71s)
PASS TestStackDeploymentRunsList/List_with_pagination (0.71s)
PASS TestStackDeploymentRunsList (18.99s)
PASS TestStackDeploymentRunsRead/Read_with_valid_ID (0.11s)
PASS TestStackDeploymentRunsRead/Read_with_invalid_ID (0.55s)
PASS TestStackDeploymentRunsRead (44.75s)
PASS TestStackDeploymentRunsApproveAllPlans/Approve_all_plans (0.17s)
PASS TestStackDeploymentRunsApproveAllPlans (19.94s)
SKIP TestStackDeploymentRunsCancel/cancel_deployment_run (0.00s)
PASS TestStackDeploymentRunsCancel (23.18s)
PASS TestStackDeploymentStepsList/List_with_invalid_stack_deployment_run_ID (0.20s)
PASS TestStackDeploymentStepsList/List_without_options (0.53s)
PASS TestStackDeploymentStepsList/List_with_pagination (0.56s)
PASS TestStackDeploymentStepsList (22.05s)
PASS TestStackDeploymentStepsRead/Read_with_valid_ID (0.11s)
PASS TestStackDeploymentStepsRead/Read_with_invalid_ID (0.36s)
PASS TestStackDeploymentStepsRead (19.36s)
PASS TestStackDeploymentStepsAdvance/Advance_with_valid_ID (0.49s)
PASS TestStackDeploymentStepsAdvance/Advance_with_invalid_ID (0.34s)
PASS TestStackDeploymentStepsAdvance (26.10s)
PASS TestStackCreateAndList/List_without_options (0.20s)
PASS TestStackCreateAndList/List_with_name_filter (0.33s)
PASS TestStackCreateAndList/List_with_project_filter (0.43s)
PASS TestStackCreateAndList/List_with_pagination (0.45s)
PASS TestStackCreateAndList/List_with_sort_options (0.57s)
PASS TestStackCreateAndList (5.86s)
PASS TestStackReadUpdateDelete (7.32s)
PASS TestStackRemoveVCSBacking (5.79s)
PASS TestStackReadUpdateForceDelete (7.73s)
PASS TestStackConverged (24.08s)
PASS .

=== Skipped
=== SKIP: . TestStackDeploymentRunsCancel/cancel_deployment_run (0.00s)
    stack_deployment_runs_integration_test.go:231:
    --- SKIP: TestStackDeploymentRunsCancel/cancel_deployment_run (0.00s)

DONE 52 tests, 1 skipped in 368.109s

@ctrombley ctrombley self-assigned this Sep 15, 2025
@ctrombley ctrombley requested a review from a team as a code owner September 15, 2025 23:19
@ctrombley ctrombley changed the title Wip cleanup Update stacks resources with upstream serializer changes Sep 15, 2025
@ctrombley ctrombley changed the title Update stacks resources with upstream serializer changes Update stacks resources to match upstream serializer changes Sep 15, 2025
sahar-azizighannad
sahar-azizighannad reviewed Sep 16, 2025
View reviewed changes
sahar-azizighannad
sahar-azizighannad reviewed Sep 16, 2025
View reviewed changes
Maed223
Maed223 reviewed Sep 22, 2025
View reviewed changes
Maed223
Maed223 previously approved these changes Sep 22, 2025
View reviewed changes
@Maed223 Maed223 dismissed their stale review September 22, 2025 19:56

Noticed a quick issue

Maed223
Maed223 previously approved these changes Sep 22, 2025
View reviewed changes
@ctrombley ctrombley merged commit 130528f into main Sep 23, 2025
8 checks passed
@ctrombley ctrombley deleted the wip-cleanup branch September 23, 2025 16:06
@github-actions GitHub Actions
Copy link

Reminder to the contributor that merged this PR: if your changes have added important functionality or fixed a relevant bug, open a follow-up PR to update CHANGELOG.md with a note on your changes.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Maed223 Maed223 Maed223 approved these changes

@sahar-azizighannad sahar-azizighannad Awaiting requested review from sahar-azizighannad

Assignees

@ctrombley ctrombley

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@ctrombley @Maed223 @sahar-azizighannad