Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Inventory #10

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Inventory #10

wants to merge 2 commits into from

Conversation

nibalizer
Copy link

This adds a dynamic inventory to the collection

This provides a dynamic inventory for IBM Cloud vpc instances to ansible
users.
@svennam92 svennam92 requested a review from Anil-CM April 22, 2020 16:12
@nibalizer
Copy link
Author

Any update? @svennam92 @Anil-CM ?

@svennam92
Copy link
Contributor

@nibalizer we haven't forgot about this -- we're working on how we can support contributions like this as this repo is generated from the terraform provider.

@Anil-CM Anil-CM force-pushed the master branch 3 times, most recently from bd47d88 to 09c183e Compare May 20, 2020 06:34
@nibalizer
Copy link
Author

@svennam92 Ok! I'm happy to maintain this in a repo that you can slurp in automatically if that makes it easier.

@svennam92
Copy link
Contributor

@nibalizer last I spoke with the team, they said they have a mechanism for managing these "non-generated" assets, but let me ask @albee-jhoney to weigh in here.

@jjasghar
Copy link

Getting dynamic inventories upstreamed will make life hella easier.

@Anil-CM
Copy link
Contributor

Anil-CM commented Aug 24, 2020

Hi @nibalizer @jjasghar,

This PR listing out the VPC vm instances using the python vpc-go-sdk. To avoid the python sdk dependency, we have come up with a module - ibm_is_instances_info which lists all the VPC vm instances.

@adamsilkcm
Copy link

While using the sdk for what could be done with an API call may be overkill on the dependencies, that's what's shown in the python documentation for the API at https://cloud.ibm.com/apidocs/vpc?code=python

And given that this collection is essentially a wrapper for terraform, and terraform would have to have resources defined already to import state and do the same thing, why is the sdk dependency viewed as a worse solution than having to create and upkeep all the fake terraform (or the terraform dependency for that matter)? Ansible is python-based; why not make use of IBM's python sdk to bypass terraform?

@muchafel
Copy link

muchafel commented Apr 6, 2022

Is this already available when I make use of the collection ibm.cloudcollection ?
I have integrated the collection via custom EE in AWX (auto-plugin enabled), however the inventory plgin seem to be unknown:

Failed to parse /runner/project/inventories/cattle/ibmcloud.yml
with auto plugin: inventory config
'/runner/project/inventories/cattle/ibmcloud.yml' specifies unknown plugin
'ibmcloud.ibmcollection.vpcinventory'

@levivic
Copy link

levivic commented Jul 27, 2023

When can this be reviewed and merged? @Anil-CM

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.

7 participants