Skip to content

[codex] Add APIGW schedule polling resource#133

Merged
dengyh merged 1 commit into
masterfrom
codex/schedule-apigw-auth
Jul 1, 2026
Merged

[codex] Add APIGW schedule polling resource#133
dengyh merged 1 commit into
masterfrom
codex/schedule-apigw-auth

Conversation

@dengyh

@dengyh dengyh commented Jul 1, 2026

Copy link
Copy Markdown
Collaborator

What

  • Add an APIGW resource for GET /bk_plugin/schedule/{id} and route it to the existing backend schedule endpoint.
  • Keep the runtime schedule API behavior unchanged: no new framework setting, no new DRF permission check, and existing direct calls still behave as before.
  • Configure the schedule resource as public/apply-able with app authentication only:
    • isPublic: true
    • allowApplyPermission: true
    • userVerifiedRequired: false
    • appVerifiedRequired: true
    • resourcePermissionRequired: false

Why

External platforms can poll plugin execution status through APIGW using the same schedule interface shape, while the gateway handles application identity verification.

Validation

  • PYTEST_DISABLE_PLUGIN_AUTOLOAD=1 DJANGO_SETTINGS_MODULE=tests.settings PYTHONPATH=. pytest tests/services/bpf_service/test_resources_yaml.py -q
  • black --check tests/services/bpf_service/test_resources_yaml.py
  • flake8 tests/services/bpf_service/test_resources_yaml.py
  • git diff --check origin/master
  • GitHub Actions after e8a69fb: CodeQL, framework unittest, framework lint, runtime lint all passed.

@dengyh dengyh marked this pull request as ready for review July 1, 2026 11:15
@dengyh dengyh force-pushed the codex/schedule-apigw-auth branch from 230de2f to 73b990f Compare July 1, 2026 11:24
@dengyh dengyh marked this pull request as draft July 1, 2026 11:25
@dengyh dengyh force-pushed the codex/schedule-apigw-auth branch from 73b990f to e8a69fb Compare July 1, 2026 11:31
@dengyh dengyh marked this pull request as ready for review July 1, 2026 11:32
@dengyh dengyh merged commit a734e27 into master Jul 1, 2026
9 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.

2 participants