Skip to content

Add runtime smoke tests for uncovered code samples - #6792

Draft
dprotaso wants to merge 8 commits into
knative:mainfrom
dprotaso:samples-ci-coverage
Draft

dprotaso wants to merge 8 commits into
knative:mainfrom
dprotaso:samples-ci-coverage

Conversation

@dprotaso

@dprotaso dprotaso commented Nov 7, 2026 •

Copy link
Copy Markdown
Member

Summary

Add nine self-contained ci.sh smoke-test scripts for existing samples that currently have none:

  • Parallel helper functions
  • Heartbeat event source
  • Spring and Vert.x CloudEvents
  • Git webhooks
  • gRPC ping
  • Multi-container
  • WebSockets
  • Existing capture logger

Each script follows the existing main → build → deploy → test pattern. Fixtures and client checks are contained in the relevant scripts. GitHub API calls use a local fixture and need no real credentials.

Existing ci.sh scripts and .github/workflows/samples-test.yaml are unchanged. No new application samples, support libraries, or harness regression tests are added.

Bookstore samples are deliberately excluded for now; there are no bookstore changes in this PR. Dependabot configuration changes are also excluded.

Validation

Local runtime testing was stopped at the author's request; GitHub CI will validate the final version. The dedicated local test cluster has been deleted.

@knative-prow knative-prow Bot added the do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. label Nov 7, 2026
@netlify

netlify Bot commented Nov 7, 2026 •

Copy link
Copy Markdown

✅ Deploy Preview for knative ready!

Name Link
🔨 Latest commit 128c87f
🔍 Latest deploy log https://app.netlify.com/projects/knative/deploys/6aefbf7f2720aa528168577a
😎 Deploy Preview https://deploy-preview-6792--knative.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@knative-prow
knative-prow Bot requested review from Cali0707 and skonto November 7, 2026 02:44
@knative-prow

knative-prow Bot commented Nov 7, 2026

Copy link
Copy Markdown

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: dprotaso

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Details Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@knative-prow knative-prow Bot added size/XL Denotes a PR that changes 500-999 lines, ignoring generated files. approved Indicates a PR has been approved by an approver from all required OWNERS files. labels Nov 7, 2026
@knative-prow knative-prow Bot added size/XXL Denotes a PR that changes 1000+ lines, ignoring generated files. and removed size/XL Denotes a PR that changes 500-999 lines, ignoring generated files. labels Nov 7, 2026
@knative-prow knative-prow Bot added size/XL Denotes a PR that changes 500-999 lines, ignoring generated files. and removed size/XXL Denotes a PR that changes 1000+ lines, ignoring generated files. labels Nov 7, 2026
@knative-prow knative-prow Bot added size/XXL Denotes a PR that changes 1000+ lines, ignoring generated files. and removed size/XL Denotes a PR that changes 500-999 lines, ignoring generated files. labels Nov 7, 2026
@knative-prow knative-prow Bot added size/XL Denotes a PR that changes 500-999 lines, ignoring generated files. and removed size/XXL Denotes a PR that changes 1000+ lines, ignoring generated files. labels Nov 7, 2026

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

approved Indicates a PR has been approved by an approver from all required OWNERS files. do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. size/XL Denotes a PR that changes 500-999 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant