NVIDIA / NVIDIA/OpenShell

test(e2e): move Keycloak provider refresh coverage out of rootless Podman lane

Open
#2,984 1 comment 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

state:stale
Dominant language
Rust
Stars
8.7k
Forks
1.3k
Avg merge
2d 11h
Merged PRs (30d)
253

Description

Description

Re-add the provider_refresh_keycloak e2e coverage outside the rootless Podman replacement path. The old direct-runner rootless Podman matrix included e2e:provider-refresh-keycloak, but the test validates provider OAuth2 refresh-token recovery behavior rather than rootless Podman behavior.

Context

PR #2981 removed the direct-runner rootless Podman job, which also removed CI execution of e2e:provider-refresh-keycloak because that task was part of the old rootless Podman matrix.

PR #2982 replaces the old direct-runner rootless Podman lane with a Nix VM-backed rootless Podman smoke lane. During comparison with the pre-#2981 pipeline, we identified that provider_refresh_keycloak was previously run in the rootless Podman matrix even though it is driver-independent in intent.

The test verifies that a valid Keycloak refresh token can rotate provider credentials, that a revoked refresh token fails with invalid_grant, and that provider refresh status reports reauthorization is required without rendering the parked scheduling sentinel as a date.

Definition of Done

  • provider_refresh_keycloak runs from a driver-independent conformance or provider-refresh e2e location.
  • The coverage no longer depends specifically on the rootless Podman lane.
  • CI invokes the test against an appropriate supported gateway backend.
  • Documentation or task names make the test's provider-refresh purpose clear.

Contributor guide

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

Start by comparing the pre-PR #2981 rootless Podman matrix with the replacement from PR #2982, locating the e2e:provider-refresh-keycloak task and its CI configuration. Move or re-register that task in a driver-independent provider-refresh location, run it against a supported gateway backend, and confirm the task name and documentation describe provider OAuth2 refresh coverage.

Written by the indexing model from the issue text.

Assessment

Tech stack
rust
Domain
authentication, ci-cd, testing-qa
Issue type
Refactor
Difficulty
3/5
Estimated time
1-2 days
Activity status
Active
Clarity
Mostly clear
Newbie friendliness
67/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.