Skip to content

Add support for IBMCloud/PowerVS images#16

Merged
bgilbert merged 4 commits intocoreos:mainfrom
Prashanth684:powervs
Nov 4, 2021
Merged

Add support for IBMCloud/PowerVS images#16
bgilbert merged 4 commits intocoreos:mainfrom
Prashanth684:powervs

Conversation

@Prashanth684
Copy link
Copy Markdown
Contributor

Adding support for IBMCloud/PowerVS objects to be added to the stream.

similar to coreos/stream-metadata-go#35

@bgilbert
Copy link
Copy Markdown
Contributor

I know we haven't generalized the AWS image struct in the Rust bindings, but we should probably avoid compounding the mistake, and go with ReplicatedObject/RegionObject again here.

@Prashanth684
Copy link
Copy Markdown
Contributor Author

I know we haven't generalized the AWS image struct in the Rust bindings, but we should probably avoid compounding the mistake, and go with ReplicatedObject/RegionObject again here.

yes that makes sense. done.

Adding support for IBMCloud/PowerVS objects to be added to the stream.

similar to coreos/stream-metadata-go#35
Copy link
Copy Markdown
Contributor

@bgilbert bgilbert left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM; just some wording suggestions.

Prashanth684 and others added 3 commits November 4, 2021 14:35
Co-authored-by: Benjamin Gilbert <bgilbert@backtick.net>
Co-authored-by: Benjamin Gilbert <bgilbert@backtick.net>
Co-authored-by: Benjamin Gilbert <bgilbert@backtick.net>
Copy link
Copy Markdown
Contributor

@bgilbert bgilbert left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants