Skip to content

Fix subsystem CI workflow and policy PATCH validation#15

Merged
gciavarrini merged 2 commits into
dcm-project:mainfrom
gciavarrini:fix-ci-subsystem
Jun 22, 2026
Merged

Fix subsystem CI workflow and policy PATCH validation#15
gciavarrini merged 2 commits into
dcm-project:mainfrom
gciavarrini:fix-ci-subsystem

Conversation

@gciavarrini

@gciavarrini gciavarrini commented Jun 22, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Inline container image lists in the subsystem workflow. GitHub rejects env context in with inputs for reusable workflows, which broke the black-box jobs on main.
  • Set SkipSettingDefaults in OpenAPI request validation. kin-openapi rewrites bodies after applying schema defaults but only encodes application/json, so policy PATCH requests (application/merge-patch+json) returned 400 before the handler ran.

GitHub Actions rejects env in with inputs for workflow_call.
Inline pre-pull image lists in each subsystem job.

Assisted-By: Claude (Anthropic)

Signed-off-by: Gloria Ciavarrini <gciavarrini@redhat.com>
@qodo-code-review

Copy link
Copy Markdown

Qodo reviews are paused for this user.

Troubleshooting steps vary by plan Learn more →

On a Teams plan?
Reviews resume once this user has a paid seat and their Git account is linked in Qodo.
Link Git account →

Using GitHub Enterprise Server, GitLab Self-Managed, or Bitbucket Data Center?
These require an Enterprise plan - Contact us
Contact us →

kin-openapi rewrites request bodies after applying schema defaults but
only encodes application/json. PATCH policies use merge-patch+json, so
validation returned 400 before the handler ran.
Assisted-by: Claude (Anthropic)

Signed-off-by: Gloria Ciavarrini <gciavarrini@redhat.com>
@gciavarrini gciavarrini changed the title Fix subsystem workflow env usage in CI Fix subsystem CI workflow and policy PATCH validation Jun 22, 2026
@gciavarrini gciavarrini merged commit c319b2b into dcm-project:main Jun 22, 2026
6 checks passed
@gciavarrini gciavarrini deleted the fix-ci-subsystem branch June 22, 2026 15:21
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.

3 participants