-
Notifications
You must be signed in to change notification settings - Fork 0
Absorb ImageVersionOS into ImageVersion #596
Copy link
Copy link
Open
Labels
cvp:0Necessary projects we are undertaking that don’t directly deliver value to the customerNecessary projects we are undertaking that don’t directly deliver value to the customerdockerRelated to container images we produceRelated to container images we producetdp:1The individual working on it notices.The individual working on it notices.
Description
Metadata
Metadata
Assignees
Labels
cvp:0Necessary projects we are undertaking that don’t directly deliver value to the customerNecessary projects we are undertaking that don’t directly deliver value to the customerdockerRelated to container images we produceRelated to container images we producetdp:1The individual working on it notices.The individual working on it notices.
Type
Fields
Give feedbackNo fields configured for issues without a type.
`ImageVersionOS` (`version_os.py`, 133 LOC) is a shallow wrapper: 8 fields, 4 methods, 1 validator. It handles `BuildOS` lookup and adds `primary`, `extension`, and `tagDisplayName` fields. The indirection adds a parent-pointer contract without hiding meaningful complexity.
Absorb the fields and methods directly into `ImageVersion`. `BuildOS` lookup logic stays separate. Delete `version_os.py`. Reduces the number of files needed to trace image version resolution from ~11 to ~9.
Files: `config/image/version_os.py`, `config/image/version.py`, `config/image/build_os.py`