OSAC-1112: Enable GitHub Pages for the fulfillment-service repository#89
Conversation
The `fulfillment-service` repository recently added a GitHub Actions workflow to publish OpenAPI specifications to GitHub Pages (see PR #619). However, the Pages feature was only enabled manually in the repository settings, so it gets reverted every time this Terraform configuration is re-applied, because the `pages` variable defaults to `null`. This adds the `pages` configuration with `build_type = "workflow"` to match the workflow-based deployment used by `publish-openapi.yaml`. Related: https://redhat.atlassian.net/browse/OSAC-1112 Related: osac-project/fulfillment-service#619 Assisted-by: Cursor Signed-off-by: Juan Hernandez <juan.hernandez@redhat.com>
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Repository: osac-project/coderabbit/.coderabbit.yaml Review profile: ASSERTIVE Plan: Enterprise Run ID: 📒 Files selected for processing (1)
WalkthroughUpdated ChangesGitHub Pages and Branch Protection Configuration
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~3 minutes Security & Risk AssessmentRisk Severity: Low-Medium — Configuration-level infrastructure change with deployment implications. Observable Security Impacts:
Review focus: Confirm the workflow build process for Pages includes proper access controls and that status checks remain sufficient to prevent unreviewed code from merging. Poem
🚥 Pre-merge checks | ✅ 11✅ Passed checks (11 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
Comment |
|
FYI: @tzvatot you will need something like this if you are manually enabling the pages feature and you want it to survive, otherwise it will be automatically disabled when this configuration is re-applied. |
Summary
The
fulfillment-servicerepository recently added a GitHub Actions workflow to publishOpenAPI specifications to GitHub Pages (see osac-project/fulfillment-service#619). However,
the Pages feature was only enabled manually in the repository settings, so it gets reverted
every time this Terraform configuration is re-applied, because the
pagesvariable defaultsto
null.This adds the
pagesconfiguration withbuild_type = "workflow"to match the workflow-baseddeployment used by
publish-openapi.yaml.Related: https://redhat.atlassian.net/browse/OSAC-1112
Related: osac-project/fulfillment-service#619
Test plan
terraform planshows no unexpected changes beyond enabling Pages.fulfillment-servicerepository withbuild_type = "workflow".https://osac-project.github.io/fulfillment-service/openapi/.Summary by CodeRabbit