Skip to content

WIP: implement host discovery#545

Closed
dhellmann wants to merge 6 commits intometal3-io:masterfrom
dhellmann:discovery
Closed

WIP: implement host discovery#545
dhellmann wants to merge 6 commits intometal3-io:masterfrom
dhellmann:discovery

Conversation

@dhellmann
Copy link
Copy Markdown
Member

@dhellmann dhellmann commented Jun 2, 2020

Addresses #41

See metal3-io/metal3-docs#98 as well

dhellmann added 6 commits June 2, 2020 14:20
Signed-off-by: Doug Hellmann <dhellmann@redhat.com>
Moving the code for managing the ironic clients to its own package
makes it easier to reuse.

Signed-off-by: Doug Hellmann <dhellmann@redhat.com>
Signed-off-by: Doug Hellmann <dhellmann@redhat.com>
Signed-off-by: Doug Hellmann <dhellmann@redhat.com>
Signed-off-by: Doug Hellmann <dhellmann@redhat.com>
Signed-off-by: Doug Hellmann <dhellmann@redhat.com>
@metal3-io-bot metal3-io-bot added the do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. label Jun 2, 2020
@metal3-io-bot
Copy link
Copy Markdown
Contributor

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: dhellmann

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Details Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@metal3-io-bot metal3-io-bot added approved Indicates a PR has been approved by an approver from all required OWNERS files. size/L Denotes a PR that changes 100-499 lines, ignoring generated files. labels Jun 2, 2020
@dhellmann
Copy link
Copy Markdown
Member Author

The design is going in a different direction, so we aren't going to want this.

@dhellmann dhellmann closed this Jun 26, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

approved Indicates a PR has been approved by an approver from all required OWNERS files. do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. size/L Denotes a PR that changes 100-499 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants