Consider splitting pkg/provisioner/* into golang plugins.

[s3rj1k]

User Story

It would be nice to have plugin support for BYO provisioner by using golang plugins approach.

Detailed Description

Moving pkg/provisioner/* into golang plugins would require making go mod init on them, updating build process, adding code that will load plugins on start.

Anything else you would like to add:

Moving out stock provisioner code would also help with making statemachine part be properly isolated from provisioners.

/kind feature

[dtantsur]

It's not a bad idea, although

1. at this point BMO makes so many Ironic-specific assumptions that developing an alternative provisioner is hardly possible,
2. the Provisioner interface keeps evolving and won't be stable for now.

[dtantsur]

As an aside, I'd really like to extract a reusable Ironic-centric library out of pkg/provisioner/ironic and move it to a separate reusable module.

[Rozzii]

/triage accepted
/help
@s3rj1k could you please describe a bit more your custom provisioner use case ? Is it an opensource provisioner, is the provisioner already compatible with the BMO state machine (which is derived from the Ironic state machine) ?

[metal3-io-bot]

@Rozzii:
This request has been marked as needing help from a contributor.

Please ensure the request meets the requirements listed here.

If this request no longer meets these requirements, the label can be removed
by commenting with the /remove-help command.

In response to this:

/triage accepted
/help
@s3rj1k could you please describe a bit more your custom provisioner use case ? Is it an opensource provisioner, is the provisioner already compatible with the BMO state machine (which is derived from the Ironic state machine) ?

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

[s3rj1k]

could you please describe a bit more your custom provisioner use case

One that I used before is basically very patched Ironic provisioner and by having this plugin system I would hope to see more hard separation between logic so that any kind off-stream patched could be applied exclusively to the plugin code itself.

But really any kind of other BM provisioners could be done if plugin system would reduce the scope of implementation.

[s3rj1k]

As an aside, I'd really like to extract a reusable Ironic-centric library out of pkg/provisioner/ironic and move it to a separate reusable module.

This would be a first step, yes, this will somewhat ensure that provisioner could be more isolated, it always feels like BMO is missing this part of isolation of implementations.

[s3rj1k]

Ironic-specific assumptions

IMO, state machine can be reused, some states can be just No-Op in case of other than Ironic provisioner is used.

[s3rj1k]

I would like to have the ability to use any kind of cloud provider API (meaning BM offerings, not VMs that they offer) with proven to work state-machine and have all of this interchangeable, openstack <-> somethingother just by switching plugin.

So that ironic-specific things in this case can be of use.

[metal3-io-bot]

Issues go stale after 90d of inactivity.
Mark the issue as fresh with /remove-lifecycle stale.
Stale issues will close after an additional 30d of inactivity.

If this issue is safe to close now please do so with /close.

/lifecycle stale

[Rozzii]

/remove-lifecycle stale
/lifecycle-frozen

[Rozzii]

/lifecycle-frozen

[Rozzii]

/lifecycle frozen