Skip to content

Remove the vestigial OpenShift client role#1372

Draft
dholt wants to merge 1 commit into
masterfrom
dholt/remove-openshift-role
Draft

Remove the vestigial OpenShift client role#1372
dholt wants to merge 1 commit into
masterfrom
dholt/remove-openshift-role

Conversation

@dholt

@dholt dholt commented Jul 20, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Remove the vestigial openshift client role, the bootstrap-openshift.yml playbook, its imports in k8s-cluster.yml and the GPU Operator playbook, and the meta-dependencies from the three GPU roles.
  • Nothing consumes it: the GPU Operator, device plugin, and GPU feature discovery roles deploy exclusively through Helm; no playbook or role uses the Ansible k8s modules against the virtualenv this role created; and the roles that need EPEL on Red Hat install it themselves. The role installed the deprecated OpenShift Python client for a module usage that no longer exists in the repository.
  • The legacy RoCE backend role is unaffected — it installs its own client inside its own tasks.

Validation

  • git diff --check; YAML parse of changed files; full role lint (0 failures); playbooks/k8s-cluster.yml syntax check passes.
  • Removal of dead code only: no default deployment path executes any of the removed logic's effects.

The openshift role installed the deprecated OpenShift Python client
into a dedicated virtualenv (plus EPEL on Red Hat) as a meta-dependency
of the GPU Operator, device plugin, and GPU feature discovery roles and
via a bootstrap playbook wired into k8s-cluster.yml. Nothing consumes
it: those roles deploy exclusively through Helm, no playbook or role in
the repository uses the Ansible k8s modules against that virtualenv,
and the roles that need EPEL install it themselves.

Remove the role, the bootstrap playbook and its imports, and the empty
meta-dependencies. The legacy RoCE backend role is unaffected — it
installs its own client inside its own tasks.
@dholt
dholt force-pushed the dholt/remove-openshift-role branch from efaa1bf to 0a9cafb Compare July 20, 2026 21:32
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