This repository was archived by the owner on Sep 15, 2026. It is now read-only.
Feat/everest - #4395
Closed
geoffsee wants to merge 35 commits into
Closed
Feat/everest#4395geoffsee wants to merge 35 commits into
geoffsee wants to merge 35 commits into
Conversation
Widen test-pr.yml triggers beyond main, and add knative-runner and deploy package jobs that skip cleanly when those packages are absent.
Adds an explicit push trigger so stacked branches get CI even when pull_request workflow runs are not delivered on this fork.
push and pull_request both fire on stacked feat/* updates; cancel-in-progress concurrency collapses the duplicates.
Collect Ginkgo/Bun cover percentages as job outputs and roll them into one Summary table so stacked PR runs show coverage without digging through logs.
Replace Result.Requeue and distribution Loglevel with their supported replacements.
@latest was resolving to golangci-lint v1, which cannot load the config or read Go 1.27 export data and breaks make lint on the stack bottom.
Stack-bottom PRs target next, which was excluded from the pull_request and push branch filters.
Feature branches already run via pull_request; push on feat/** duplicated runs and flooded the Actions list.
Keep cf push unchanged: AppWorkload.Spec.RunnerName selects the runtime, and the statefulset-runner now ignores workloads it does not own.
Set RunnerName in statefulset-runner reconcile tests after the runner filter guard, and silence the int->int32 conversion gosec flags on finalize.
Point HTTPRoutes at the Knative public ClusterIP (with Host rewrite) so idle apps can use min-scale 0, and assign synthetic process indexes when pod-index labels are missing or duplicated.
Contour rejects URLRewrite on BackendRef.Filters (only header modifiers). Rule-level URLRewrite is Accepted and still sets Host for the activator.
cf scale -i N sets min-scale and max-scale to N so replicas stay warm and a scale above 1 actually runs that many pods. instances=0 keeps min-scale 0 and max-scale 1 so the Service remains.
Cover the AppWorkload→Knative converter and reconciler, and widen test-pr.yml so feat/* bases (not only main) execute the suite, including knative-runner and deploy package tests when present.
Replace Result.Requeue and distribution Loglevel with their supported replacements.
@latest was resolving to golangci-lint v1.64.x, which cannot read Go 1.27 export data and breaks make lint / CI.
…r error handling. Refactor Knative Service functions for improved testability and error path coverage.
Expose knativeRunner.include (off by default) for RBAC, RunnerInfo, and controller gating so Knative Serving stays an external dependency like Contour.
Vendor the Knative Operator, apply a Kourier ClusterIP KnativeServing CR from install-dependencies.sh, and enable knative-runner on Kind Helm installs.
Add Knative as a common dependency, enable knative-runner in the Helm examples, and allow KNATIVE_DOMAIN overrides when bootstrapping via install-dependencies.sh.
Align go.mod and builder images with the current stable Go release.
Automate the platform install guides with reusable deploy/lib components
and stacked programs under deploy/{kind,eks,gke}.
Hub cloudfoundry/korifi-*:latest has no knative-runner. Kind builds controllers/api/migration from this checkout, kind-loads them, and pins Helm with IfNotPresent. The installer Job skips Knative (SKIP_KNATIVE); KnativeServing owns the operator on kind/EKS/GKE.
KindKorifiImages imports @pulumi/command; typecheck failed without the dependency, and bun tests use import.meta which tsc rejects under nodenext.
Enable-flag layout keeps adding NATS/MinIO (etc.) a small drop-in without reshaping stacks.
Split the slow envtest suite by package tree so PR wall clock tracks the slowest shard instead of the full controllers tree.
Opt-in GraalVM-class native staging (Liberica NIK) via BP_NATIVE_IMAGE, with kind staging memory high enough to compile a Spring Boot native image.
… private registries.
osb-service is a multi-offering Open Service Broker (Postgres first) serving HTTPS on 8443. Pulumi deploys it as a CFServiceBroker and TLS-enables the shared Postgres StatefulSet (sslmode=require). /healthz and /metrics skip basic auth so kubelet probes succeed.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Is there a related GitHub Issue?
What is this change about?
Does this PR introduce a breaking change?
Acceptance Steps
Tag your pair, your PM, and/or team