Enhance baremetal sub command to expose hosts defined in the documents

[aschiefe]

The UI will need to operate on the baremetal as defined in the manifests. However the UI is blind to the manifests and needs to know which hosts are available for the baremetal actions to be run against.

In order to complete the UI integration there will need to be a way to expose the hosts to the UI / command line to show which baremetal targets can be operated on. It would be preferable that this information is produced as a structure that can be referenced by the UI and printed in a human readable format for the CLI

The UI will be unable to complete the baremetal integration without some level of host information exposed to it for display / operational purposes.

[airshipbot]

Related Change #761991

Subject: Move remote client interface to separate package
Link: https://review.opendev.org/761991
Status: NEW
Owner: Kostyantyn Kalynovskyi ([email protected])

Approvals

Code-Review
! None
Verified
+1 Zuul
Workflow
! None

Last Updated: 2020-11-11 11:30:31 CST

[airshipbot]

Related Change #759329

Subject: Add baremetal inventory implementation.
Link: https://review.opendev.org/759329
Status: NEW
Owner: Kostyantyn Kalynovskyi ([email protected])

Approvals

Code-Review
! None
Verified
+1 Zuul
Workflow
! None

Last Updated: 2020-11-11 12:00:32 CST

[airshipbot]

Related Change #762213

Subject: Add baremetal inventory implementation.
Link: https://review.opendev.org/762213
Status: NEW
Owner: Kostyantyn Kalynovskyi ([email protected])

Approvals

Code-Review
! None
Verified
+1 Zuul
Workflow
! None

Last Updated: 2020-11-10 17:45:35 CST

[airshipbot]

Related Change #762215

Subject: Implement RunAction method of the baremetal interface
Link: https://review.opendev.org/762215
Status: NEW
Owner: Kostyantyn Kalynovskyi ([email protected])

Approvals

Code-Review
! None
Verified
+1 Zuul
Workflow
! None

Last Updated: 2020-11-10 17:45:19 CST

[airshipbot]

A [Related Change](https://review.opendev.org/761991 was merged. This issue may be ready to close.

[airshipbot]

A [Related Change](https://review.opendev.org/762213 was merged. This issue may be ready to close.

[airshipbot]

A [Related Change](https://review.opendev.org/762215 was merged. This issue may be ready to close.

[sirajyasin]

@ak3216 @teoyaomiqui
I assume all the core logic to fetch inventory is already implemented, however the cli command 'airshipctl bare metal list' is still pending. If no one has started working on that yet, I can look into adding the new cli command. Please suggest

[teoyaomiqui]

@sirajyasin that is partially correct. First would be to develop an underlying extension to remote interface to be printed as YAML: https://github.com/airshipit/airshipctl/blob/master/pkg/remote/ifc/ifc.go#L25-L38

this maybe a little tricky, because when client is initialized, no instructions are given to it about the BMH object here: https://github.com/airshipit/airshipctl/blob/master/pkg/remote/ifc/ifc.go#L41-L45

so i think we may need to redefine the constructor, to encompass this information.

WHen this is done, we can implement ListCommand that would do essentially do a Select() from here https://github.com/airshipit/airshipctl/blob/master/pkg/inventory/ifc/interface.go#L30 and print each host

[teoyaomiqui]

sorry clicked close accidentally :)

[teoyaomiqui]

@sirajyasin do u still want to work on it?

[PratikDhanave]

@eak13 please assign to me.

[PratikDhanave]

work in progress https://review.opendev.org/c/airship/airshipctl/+/808750

[pallavgupta]

@eak13 since Pratik is out for some time, pls assign this issue to me. I already started working into it.