Skip to content
This repository has been archived by the owner on Mar 24, 2020. It is now read-only.

Add addresses to Machine #49

Closed
russellb opened this issue Apr 3, 2019 · 5 comments
Closed

Add addresses to Machine #49

russellb opened this issue Apr 3, 2019 · 5 comments
Labels
kind/bug Categorizes issue or PR as related to a bug.

Comments

@russellb
Copy link
Member

russellb commented Apr 3, 2019

We need to populate the addresses field of Machines.

The contents must match what shows up in the associated Node, but the information must come from elsewhere, as it's needed sooner.

This has a dependency on the baremetal-operator completing integration of host introspection. The host introspection data will contain IP and hostname information that we can then copy into the Machine.

@russellb
Copy link
Member Author

This depends on metal3-io/baremetal-operator#147

@russellb
Copy link
Member Author

This was mostly implemented by #24

but a Node will also have the hostname, and we don't have that implemented yet.

There's a related RFE about the hostname here: metal3-io/baremetal-operator#190

@russellb
Copy link
Member Author

@bcrochet Are you working on the hostname part of this, too?

@bcrochet
Copy link
Contributor

@bcrochet Are you working on the hostname part of this, too?

I can. I'll take a look at the rfe.

@russellb russellb added the kind/bug Categorizes issue or PR as related to a bug. label May 29, 2019
@bcrochet
Copy link
Contributor

Implemented by metal3-io/baremetal-operator#213 and #91

@russellb russellb closed this as completed Jul 4, 2019
n1r1 pushed a commit to n1r1/cluster-api-provider-baremetal that referenced this issue Feb 23, 2020
Bug 1804408: update vendored copy of baremetal-operator
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
kind/bug Categorizes issue or PR as related to a bug.
Projects
None yet
Development

No branches or pull requests

2 participants