openshift / openshift/machine-config-operator

move rhel7 worker node updates entirely to openshift-ansible

Open
#1,592 13 comments 2 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

lifecycle/frozen
Dominant language
Go
Stars
269
Forks
529
Avg merge
3d 18h
Merged PRs (30d)
83

Description

It's really confusing today the role and split between openshift-ansible and the MCO (we need a FAQ for this).

Anyways here's an idea: what if we stopped the MCD running on !CoreOS systems, and instead openshift-ansible always needed to be run to apply changes from MachineConfig. It'd be openshift-ansible that'd somehow trigger drain/reboot etc.

The downside of this is e.g. if a user wants to change the kubelet config, they oc edit but then need to bounce out of Kube down to Ansible+ssh.

But the massive benefit is that it's super clear how things work - the MCO can optimize for CoreOS, openshift-ansible optimizes fully for traditional.

And with PRs like https://github.com/openshift/machine-config-operator/pull/1586/ for example - we can rely on /usr/etc for original files, and do things like make better use of OSTree.

Contributor guide

No contributing guide indexed for this repository

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 reviewing the MCD/MCO behavior and the current boundary with openshift-ansible. Read PR 1586 and issue 119 for the related assumptions. Done would require an agreed design and implementation for moving RHEL7 worker updates, including drain and reboot handling, entirely into openshift-ansible.

Written by the indexing model from the issue text.

Assessment

Tech stack
ansible, go
Domain
devops, infrastructure, operating-systems
Issue type
Refactor
Difficulty
5/5
Estimated time
Over a week
Activity status
Stale
Clarity
Needs clarification
Newbie friendliness
20/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.