-
Notifications
You must be signed in to change notification settings - Fork 39
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
CRD: ManagedOSVersionChannel
#63
Comments
ManagedOSVersion
and ManagedOSVersionChannel`ManagedOSVersion
and ManagedOSVersionChannel
Blocked by #67 |
I just had a quick look at the harvester code base to see how this is implemented specifically in their use case so we can cover their usage too:
|
@mudler Just want to add this might change slightly because we don't want to add custom fields to the responder. More detail about a version will be retrieved from another URL. |
ManagedOSVersion
and ManagedOSVersionChannel
ManagedOSVersionChannel
Included in rancher-sandbox/rancheros-operator#14 |
Follow-ups are tracked in #58 |
Signed-off-by: Ettore Di Giacinto <edigiacinto@suse.com>
Part of #58
Create the following new CRDs and controller for the resources types described below
ManagedOSVersionChannel
and the controllerthe
ManagedOSVersion
resources are populated by a controller, or either a kubernetes mechanism that satisfyManagedOSVersionChannel
source and constraints. The ManagedOSVersionChannel can be defined as such:where a type of channel (e.g. JSON, URL, XML, Github) populates
ManagedOSVersions
by using the specified optionsIt defines with which rules
ManagedOSVersion
should be populated (e.g. github repository tags, releases, a custom json file to parse)The controller make sure to add labels/annotations where appropriately to notify that a ManagedOS version can be applied.
Ping the harvester team about their strategy and if it does cover their use cases too. Note they might have already a json file or something similar for their available versions, in which we might want to sync up within this card as we want to collect as much feedback as possible at this stage.
cc @bk201
The text was updated successfully, but these errors were encountered: