Skip to content

fix: open API spec #13551

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
carole-lavillonniere merged 2 commits into from
Jan 8, 2026
Merged

fix: open API spec #13551

carole-lavillonniere merged 2 commits into from
Jan 8, 2026

Conversation

@carole-lavillonniere
Copy link
Contributor

@carole-lavillonniere carole-lavillonniere commented Dec 19, 2025 •
edited
Loading

Motivation

This is a first pass of fixes to the OpenAPI spec. These issues were detected by running schemathesis.

Changes

Issues That Were Fixed:

  • Indentation Problems (some were actual errors, some were not errors but rather consistency issues that the Pycharm yaml linter fixed)
  • Added new response codes to various endpoints
  • Added missing object attribute

@carole-lavillonniere carole-lavillonniere added this to the 4.13 milestone Dec 19, 2025
@carole-lavillonniere carole-lavillonniere added area: documentation Issues related to LocalStack documentation semver: patch Non-breaking changes which can be included in patch releases docs: skip Pull request does not require documentation changes notes: skip Pull request does not have to be mentioned in the release notes labels Dec 19, 2025
@github-actions
Copy link

github-actions bot commented Dec 19, 2025

Test Results - Preflight, Unit

22 982 tests  ±0   21 140 ✅ ±0   6m 24s ⏱️ +23s
     1 suites ±0    1 842 💤 ±0 
     1 files   ±0        0 ❌ ±0 

Results for commit 4ab83eb. ± Comparison against base commit 056da83.

@github-actions
Copy link

github-actions bot commented Dec 19, 2025

Test Results (amd64) - Acceptance

7 tests  ±0   5 ✅ ±0   3m 2s ⏱️ +13s
1 suites ±0   2 💤 ±0 
1 files   ±0   0 ❌ ±0 

Results for commit 4ab83eb. ± Comparison against base commit 056da83.

@github-actions
Copy link

github-actions bot commented Dec 19, 2025

Test Results (amd64) - Integration, Bootstrap

    5 files      5 suites   2h 35m 32s ⏱️
5 559 tests 4 999 ✅ 560 💤 0 ❌
5 565 runs  4 999 ✅ 566 💤 0 ❌

Results for commit 4ab83eb.

@github-actions
Copy link

github-actions bot commented Dec 19, 2025

LocalStack Community integration with Pro

    2 files  ±0      2 suites  ±0   1h 59m 5s ⏱️ + 1m 53s
5 153 tests ±0  4 756 ✅ ±0  397 💤 ±0  0 ❌ ±0 
5 155 runs  ±0  4 756 ✅ ±0  399 💤 ±0  0 ❌ ±0 

Results for commit 4ab83eb. ± Comparison against base commit 056da83.

viren-nadkarni
viren-nadkarni approved these changes Jan 8, 2026
View reviewed changes
Copy link
Member

@viren-nadkarni viren-nadkarni left a comment

Choose a reason for hiding this comment

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

Looks good, thanks for the fixes!

@carole-lavillonniere carole-lavillonniere merged commit c706058 into main Jan 8, 2026
14 checks passed
@carole-lavillonniere carole-lavillonniere deleted the fix-openapi-spec branch January 8, 2026 11:19
@cloutierMat cloutierMat mentioned this pull request Jan 8, 2026
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@viren-nadkarni viren-nadkarni viren-nadkarni approved these changes

@giograno giograno Awaiting requested review from giograno

Assignees

No one assigned

Labels

area: documentation Issues related to LocalStack documentation docs: skip Pull request does not require documentation changes notes: skip Pull request does not have to be mentioned in the release notes semver: patch Non-breaking changes which can be included in patch releases

Projects

None yet

Milestone

4.13

Development

Successfully merging this pull request may close these issues.

3 participants

@carole-lavillonniere @viren-nadkarni