Skip to main content

iri-mcd

Describes the acceptance criteria and BDD tests to be used for new/existing features of the InternalReleaseImage MachineConfigDaemon manager. The manager implements the NoRegistryClusterInstall main feature for the part related to manage/monitor a single control plane node

Source facts

Repository
openshift/machine-config-operator
Last source activity
September 29, 2026 at 19:06
Detected SKILL.md language
English
Stars
270
Forks
533

Install options

The review-first prompt is selected by default. You can switch to a direct command or download a local copy.

Review the source files

Read SKILL.md and any companion files shown by SkillsMP before deciding whether to install.

File Explorer
4 files

Showing SKILL.md

SKILL.md
Source instructions · Read-only preview
name
iri-mcd
description
Describes the acceptance criteria and BDD tests to be used for new/existing features of the InternalReleaseImage MachineConfigDaemon manager. The manager implements the NoRegistryClusterInstall main feature for the part related to manage/monitor a single control plane node
user-invocable
false
# InternalReleaseImage daemon manager BDD workflow When working on InternalReleaseImage daemon manager changes: 1. Identify the specific behavior being changed. 2. Read only the relevant acceptance file under `acceptance/`. 3. If the change touches multiple behaviors, read all matching files. 4. Do not invent behavior that is not covered by the acceptance criteria. 5. When behavior changes, update the relevant acceptance file before implementing code. # General implementation notes When defining the implementation: - Ensure that any new portion of code is covered at least by one or more unit tests. - Keep production changes minimal. - Avoid duplications, prefer a coding style that improves the readability and maintenance. - Do not perform broad refactors unless needed to make the behavior testable. - If a behavior is not covered by acceptance criteria, stop and ask before implementing it. - When reporting completion, map each changed test back to the acceptance scenario it covers. - Minimize comments, and keep them short. # Specific IRI MCD manager implementation notes - Keep the `syncInternalReleaseImage` method short and readable. Prefer to refactor included tasks into private type methods. - Do not instantiate the IRI registry within `syncInternalReleaseImage` method more than once per loop: keep it simple stupid. # Test implementation notes - When testing different cases for the same scenario, use the cases := []struct{} to capture the relevant key fields for the test. Add always a speaking name field to represent the current case. - Reuse the existing test methods if possible. - Keep the test focused on the behaviors, avoid testing unnecessary technical details. - Do not add too many comments to the tests # Functional testing notes The NoRegistryClusterInstall feature is also tested functionally in `/test/e2e-iri` folder. ## Supporting files - `acceptance/storage-reclaim.md`: IRI deletion workflow, how to reclaim the disk node space previously used. - `acceptance/feature-disabled.md`: the expected behavior when the feature is not enabled.
View on GitHub