Skip to content

Conversation

@Jaapis
Copy link
Contributor

@Jaapis Jaapis commented Nov 25, 2025

Jira: https://issues.redhat.com/browse/AAP-59447

SUMMARY

Update schema upload workflows to organize S3 files by product name:

  • Upload schemas to s3://awx-public-ci-files/{product}/{branch}/schema.json
  • Update Makefile to download from product-specific paths for schema diff
  • Update feature branch deletion to clean up from correct product path

This separates AWX and Tower schemas into distinct S3 folders.

🤖 Generated with Claude Code

ISSUE TYPE
  • New or Enhanced Feature
COMPONENT NAME
  • API
ADDITIONAL INFORMATION


Note

Organizes API schema S3 paths by product/repo and updates upload, deletion, and schema diff to use product-scoped locations.

  • CI/Workflows:
    • Upload schema: Change S3 destination to s3://awx-public-ci-files/${{ github.event.repository.name }}/${{ github.ref_name }}/schema.json in .github/workflows/upload_schema.yml.
    • Feature branch deletion: Update S3 delete path to awx-public-ci-files/${{ github.event.repository.name }}/${GITHUB_REF##*/}/schema.json in .github/workflows/feature_branch_deletion.yml.
  • Makefile:
    • Add SCHEMA_DIFF_BASE_FOLDER (default awx).
    • Update detect-schema-change to fetch reference schema from https://s3.amazonaws.com/awx-public-ci-files/$(SCHEMA_DIFF_BASE_FOLDER)/$(SCHEMA_DIFF_BASE_BRANCH)/schema.json.

Written by Cursor Bugbot for commit 53908d7. This will update automatically on new commits. Configure here.

Update schema upload workflows to organize S3 files by product name:
- Upload schemas to s3://awx-public-ci-files/{product}/{branch}/schema.json
- Update Makefile to download from product-specific paths for schema diff
- Update feature branch deletion to clean up from correct product path

This separates AWX and Tower schemas into distinct S3 folders.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <[email protected]>
@sonarqubecloud
Copy link

@Jaapis Jaapis marked this pull request as ready for review November 26, 2025 12:48
@Jaapis Jaapis merged commit 0d86874 into ansible:devel Nov 27, 2025
23 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants