Skip to content

Add auth-fake-executor profile for local development - #5118

Open
nikola-jokic wants to merge 2 commits into
masterfrom
auth-fake-executor-profile
Open

Add auth-fake-executor profile for local development#5118
nikola-jokic wants to merge 2 commits into
masterfrom
auth-fake-executor-profile

Conversation

@nikola-jokic

Copy link
Copy Markdown
Contributor

What type of PR is this?

Dev experience

What this PR does / why we need it

makes it easier to test job transformations without running executor while still having auth logic.

@greptile-apps

greptile-apps Bot commented Aug 13, 2026

Copy link
Copy Markdown
Contributor

Greptile Summary

The PR adds an authenticated fake-executor profile for exercising authenticated local job flows without Kubernetes.

  • Adds the auth-fake-executor Goreman Procfile.
  • Connects the profile to the auth Compose dependencies and Keycloak readiness check.
  • Documents the new profile in the local-development guides.

Confidence Score: 4/5

The PR is not yet safe to merge because its documented auth-fake-executor -dap invocation fails before startup.

The current profile-selection logic appends -dap and requires _local/procfiles/auth-fake-executor-dap.Procfile, but this PR does not add that file, so the previously reported startup failure remains.

Files Needing Attention: magefiles/dev.go, _local/procfiles/auth-fake-executor-dap.Procfile

Important Files Changed

Filename Overview
magefiles/dev.go Adds profile selection, auth dependency startup, and Keycloak readiness handling; the existing DAP selection path still targets a missing profile variant.
_local/procfiles/auth-fake-executor.Procfile Defines the authenticated local service set using the fake executor instead of the Kubernetes executor.
docs/developer_guide.md Documents selection and behavior of the authenticated fake-executor profile.
_local/README.md Adds the new profile to the local Procfile inventory, although its claim that every listed profile has a DAP variant is not yet satisfied.

Reviews (4): Last reviewed commit: "wip" | Re-trigger Greptile

Comment thread magefiles/dev.go
Comment thread magefiles/dev.go Outdated
@nikola-jokic nikola-jokic changed the title Add auth-fake-executor profile for OIDC testing without Kubernetes Add auth-fake-executor profile for local development Aug 14, 2026
@nikola-jokic
nikola-jokic force-pushed the auth-fake-executor-profile branch 2 times, most recently from d98bf12 to a905c3b Compare August 18, 2026 08:04
@nikola-jokic
nikola-jokic force-pushed the auth-fake-executor-profile branch from a905c3b to 4fb6203 Compare August 18, 2026 10:20
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.

1 participant